Buenas, resulta que tengo el siguiente ejercicio:


Y me resulto este código:

#include <stdio.h>
#include <stdlib.h>
 
# define L 4;
 
void datos(char *ln);
 
int main()
{
    int rep;
    char linea[80];
    char *l;
    l = &linea[0];
    for(rep=0; rep < 4; rep++)
        datos(l);
    return 0;
}
 
void datos(char *ln){
    int i;
    int cont=0;
 
    printf("Escriba las linea\n");
    fgets(ln,80,stdin);
    for(i=0; i<strlen(ln); i++){
        if(*(ln+i)=='a' || *(ln+i)=='e' || *(ln+i)=='i' || *(ln+i)=='o' || *(ln+i)=='u')
            cont++;
    }
    printf("numero de vocales en la frase:  %d \n\n", cont);
}
 

Pero algo no me termina de convencer, no se muy bien como usar punteros, y algo me parece que esta mal en el programa, uds que piensan, no necesito rehacer el ejercicio o ayuda con la tarea, solo lo hago para pasar el tiempo, y solo quiero saber opiniones, porque este código, pues, seguro que se puede mejorar, aún sigo practicando el uso de apuntadores.

Hola! para mi gusto lo que se puede agregar es que la funcion retorne algo, en este caso la cantidad de vocales osea cont, asi no se ejecuta todo en la funcion(pero esto es una cuestion de gustos)


Saludos

Solo un comentario:

fgets(ln,79,stdin); // 79 para dejar espacio al NULL del final

Cuando se llama a la función sus argumentos indican donde se localiza el contenedor, la capacidad de este en caracteres y el stream donde se extraerán estos, respectivamente.

La función extrae caracteres del stream y los almacena en el contenedor hasta que se cumpla uno de dos escenarios:

A) El numero de caracteres almacenados es igual a N-1.
B) El carácter almacenado es '\n'.

Cuando eso sucede la función almacena el carácter '\0' en la siguiente posicion y termina retornando la dirección del contenedor (o NULL en caso de error o fin de archivo). Por ende no es necesario reservar manualmente un carácter. En resumen la forma politicamente correcta es:

#define NUM_CHARS  128 /* Solo un ejemplo */
 
/* ... */
 
char linea[NUM_CHARS];
 
/* ... */
 
fgets(linea, NUM_CHARS, stdin);

----

Pero cuando se utiliza la función scanf (familia incluida) con el especificador "%Ns" si se debe tener esa precaución ya que ella almacena como máximo el numero de caracteres indicados y a continuación almacena el '\0'. Con ella debemos utilizar:

#define NUM_CHARS  128 /* Solo un ejemplo */
 
/* ... */
 
char linea[NUM_CHARS];
 
/* ... */
 
scanf("%127s", linea);

Para automatizar la generación de la cadena de formato "%127s" se puede utilizar (por supuesto previamente a la llamada) la función sprintf con un array auxiliar.

Un saludo