Foro de elhacker.net

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

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