Programa que cuenta vocales

[Supermanxd]

No me ejecuta como quiero ;c necesito que me cuente las vocales ingresadas y no me sale:CCCCCCC
Que tengo mal??

Código (cpp) [Seleccionar] Expandir

#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]);
    }

[engel lex]

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

[Supermanxd]

le cambie los valores a 0 y lo de las condiciones de if else y no me sale:C ayuda xfa

[0xFer]

Código (cpp) [Seleccionar] Expandir


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

[Supermanxd]

gracias ya me salio :DDD

[engel lex]

limpiando un poco el codigo

Código (cpp) [Seleccionar] Expandir

    #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]);
       }

[rir3760]

No 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|) y un error importante se encuentra en la función de lectura:

Código (c) [Seleccionar] Expandir

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:

Código (c) [Seleccionar] Expandir

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