No me ejecuta como quiero ;c necesito que me cuente las vocales ingresadas y no me sale:CCCCCCC
Que tengo mal??
#include <stdlib.h>
#include <conio.h>
#include <stdio.h>
char *pidetexto();
void contar_vocal(char *p, int num[5]);
void imprime(int num[5]);
int main()
{
char *texto;
int num[5];
texto=pidetexto();
contar_vocal(texto,num);
imprime(num);
getch();
}
char* pidetexto()
{
char *q,text[50];
printf("\ningrese el texto: ");
scanf("%s", &text);
q=text;
return q;
}
void contar_vocal(char *p , int num[5])
{
int num1[5];
num1[0]=0;
num1[1]=1;
num1[2]=2;
num1[3]=3;
num1[4]=4;
while (*p)
{
if(*p =='a' || *p== 'A')
{num1[0]=num1[0]+1;}
if(*p =='e' || *p== 'E')
{num1[1]=num1[1]+1;}
if(*p =='i' || *p== 'I')
{num1[1]=num1[1]+1;}
if(*p =='o' || *p== 'O')
{num1[1]=num1[1]+1;}
if(*p =='u' || *p== 'U')
{num1[1]=num1[1]+1;}
p++;
}
*(&num[0])=num1[0];
*(&num[1])=num1[1];
*(&num[2])=num1[2];
*(&num[3])=num1[3];
*(&num[4])=num1[4];
//muercielago
}
void imprime(int num[5])
{
printf("\nla letra a se repite %d", num[0]);
printf("\nla letra e se repite %d", num[1]);
printf("\nla letra i se repite %d", num[2]);
printf("\nla letra o se repite %d", num[3]);
printf("\nla letra u se repite %d", num[4]);
}
no te debería salir "CCCCCCC" te sale la cantidad de apariciones... pero el numero es incorrecto por lo que hay en las lineas de la 33 a la 36, porque le das valores diferentes a 0... los if deberías hacerlos con if else, es inutil que recorra todo, ya que si es "a" no puede ser ningúna de las otras
si arreglas lo de esos valores, estás listo
le cambie los valores a 0 y lo de las condiciones de if else y no me sale:C ayuda xfa
if(*p =='a' || *p== 'A')
{num1[0]=num1[0]+1;} //mejor - > {num[0]+=1;}
if(*p =='e' || *p== 'E')
{num1[1]=num1[1]+1;}
if(*p =='i' || *p== 'I')
{num1[1]=num1[1]+1;} //mejor -> {num[2]+=1;}
if(*p =='o' || *p== 'O')
{num1[1]=num1[1]+1;} //mejor -> {num[3]+=1;}
if(*p =='u' || *p== 'U')
{num1[1]=num1[1]+1;} //mejor -> {num[4]+=1;}
Puedes iniciar todos los valores de num a 0 con memset(num,0,sizeof(num));
No se si con eso ya funcione, no tengo un compilador para probar
Un saludo
gracias ya me salio :DDD
limpiando un poco el codigo
#include <stdlib.h>
#include <conio.h>
#include <stdio.h>
char *pidetexto();
void contar_vocal(char *p, int num[5]);
void imprime(int num[5]);
int main()
{
char *texto;
int num[5];
texto=pidetexto();
contar_vocal(texto,num);
imprime(num);
getch();
}
char* pidetexto()
{
char *q,text[50];
printf("\ningrese el texto: ");
scanf("%s", &text);
q=text;
return q;
}
void contar_vocal(char *p , int num[5])
{
num[0]=0;
num[1]=0;
num[2]=0;
num[3]=0;
num[4]=0;
while (*p){
if(*p =='a' || *p== 'A')
{num[0]++;}
else if(*p =='e' || *p== 'E')
{num[1]++;}
else if(*p =='i' || *p== 'I')
{num[2]++;}
else if(*p =='o' || *p== 'O')
{num[3]++;}
else if(*p =='u' || *p== 'U')
{num[4]++;}
p++;
}
}
void imprime(int num[5]){
printf("\nla letra a se repite %d", num[0]);
printf("\nla letra e se repite %d", num[1]);
printf("\nla letra i se repite %d", num[2]);
printf("\nla letra o se repite %d", num[3]);
printf("\nla letra u se repite %d", num[4]);
}
Cita de: Jonathanxd en 28 Mayo 2015, 05:26 AMNo me ejecuta como quiero ;c necesito que me cuente las vocales ingresadas y no me sale
Ademas de lo ya comentado por
engel lex y
0xFer deberías evitar el uso de la biblioteca conio de Borland (la explicación en |Lo que no hay que hacer en C/C++. Nivel basico| (http://foro.elhacker.net/programacion_cc/lo_que_no_hay_que_hacer_en_cc_nivel_basico-t277729.0.html)) y un error importante se encuentra en la función de lectura:
char* pidetexto()
{
char *q,text[50];
printf("\ningrese el texto: ");
scanf("%s", &text);
q=text;
return q;
}El valor que retorna "pidetexto" es la dirección de "text" pero al ser esta una variable local se destruye (su memoria se libera) al terminar la función con lo cual el puntero resultante (de la llamada) no es valido.
Para solucionarlo debes reservar memoria con malloc y utilizar la dirección en memoria del bloque como valor de retorno de la función, por ejemplo:
char *pidetexto(void)
{
char *q;
if ((q = malloc(50)) != NULL){
puts("Ingrese el texto:");
scanf("%s", q); /* Falta validar el resultado de scanf */
}
return q;
}Y después del uso del bloque hay que liberar la memoria mediante la función free.
Un saludo