Primero, las funciones <funcion_s> como <gets_s> no son estándar. En compiladores GNU/Linux no funcionan. En su lugar te recomiendo usar <fgets> que permite pasar como parámetro el tamaño máximo.
char nombre[20];
fgets(nombre
, 20, stdin
);
Segundo, las llamadas al sistema <system> son muy pesadas. La mejor alternativa del <system("pause")> es <getchar()> en C y cin.get() en C++.
Tercero, no se puede usar <fflush> con la entrada estándar, su comportamiento no está definido.
Cuarto, haces 3 veces el mismo bucle el cual no está muy bien implementado para contar las vocales. Haz una función que haga eso y pásale distintos parámetros en cada llamada.
int contarVocales(char *palabra, int size){
int contadorVocales = 0;
for(int i = 0; i < size; i++)
if(palabra[i] == 'a' || palabra[i] == 'e' || palabra[i] == 'i' || palabra[i] == 'o' || palabra[i] == 'u')
contadorVocales++;
return contadorVocales;
}
Tus bucles comparan con cada una de las vocales por separado y aunque coincida la 'a' siguen comparando con todas las demás vocales (lo cual es absurdo, si una letra es 'a' no va a ser 'e' también). Usando el operador || en cuanto una coincide se incrementa el contador y se salta el resto de comprobaciones.
Quinto, entiendo que quieres juntar el nombre con sus apellidos con un espacio entre medias. Luego lo imprimes todo junto pero al después imprimes cada caracter en orden inverso, por qué?
Sexto, entiendo también que quieras vaciar las cadenas de caracteres pero no uses 4 variables distintas si todas son iguales (una cadena vacía), usa la misma para todo.
Séptimo, no hagas una asignación de <w> con <if else>. Ese tipo de códigos se implementa así:
int w = 0;
if(/*lo que sea*/)
w = 1;
Y no se utiliza el <else>.
Edit: Y en tu caso ni siquiera es necesario usar esa variable. Mete directamente la comparación de <strcmp> como condición para el <do while>. Además soy el único que se ha dado cuenta de que según el valor de la comparación le asignas el opuesto a <w>...
. Es como si yo hiciese
int a = 0, b;
if(a == 0)
b = 1;
else
b = 0;
Es decir, un poco absurdo.
Octavo, te recomiendo usar las funciones en las que indicas el número de caracteres a tratar, por seguridad. En vez de <strcmp>, usar <strncmp>; en vez de <strcpy>, usar <strncpy>, etc.
Noveno, ese código es muy mejorable. Pero tampoco voy a matarme a darte un montón de consejos cuando si has llegado a leer hasta aquí es porque sólo te interesaba el décimo punto (por eso lo he dejado para el final, para asegurarme de que no has ignorado todas mis recomendaciones
)
Décimo, quieres reiniciar el número de vocales? Pues pon los contadores a 0 en cada repetición del programa, es decir, dentro del <do while>. Tan simple como eso. Suerte.
Mostrar Mensajes
):