¿Qué esta mal con este código en c?

No uses la función gets, usa fgets en su lugar.

Uff pero no uses tantos `else`. Úsa un `else` por cada `if`.

Tienes que colocar operadores lógicos en los `if`, no lo haces bien (con paréntesis claramente definidos). Por ejemplo:

// Para detectar vocales:

if ((c[n]=='a') || (c[n]=='e') || (c[n]=='i') || (c[n]=='o') || (c[n]=='u')){

 //Se cuenta una vocal:

  voc++; }

// Para detectar consonantes:

if ((c[n]!='a') || (c[n]!='e') || (c[n]!='i') || (c[n]!='o') || (c[n]!='u')){

 //Se cuenta una consonante:

  cons++; } // Yo ésto no lo haría como un `else`, lo dejaría como `if`.

`gets` no es un inconveniente mientras estés practicando como aprendiz. Úsa lo que necesites para capturar cadenas.

Es un programa en C no en C++, utiliza la extension '.c' no '.cpp'.

Ni idea apenas termine informática basica

Tienes muchos else acumulados, intenta cambiarlos a alternativas simples, osea, todos esos else if que estan dentro del else if cambialos solo por if.

Lo que no comprendo es que si es C, porque la extensión del archivo es .cpp .

Es C o C++??

Edit: No será mejor utilizar un bucle for para contar variables?? Deberías de intentar para ver como te va.

Saludos

.................