[SOLUCIONADO] problema con bucle for

[icm9]

tengo que ingresar una cadena de caracteres y devolver que vocales y en que cantidad hay.

hice esto_

Código C++:

#include <iostream>
#include <stdio.h>
using namespace std;
int main(){
    int a,e,i,o,u;
    string cadena;
    cin >> cadena;
    for(int x;x<=cadena.length();x++){
        if(cadena[x]=='a'){
            a++;
        }
        else if(cadena[x]=='e'){
            e++;
        }
        else if(cadena[x]=='i'){
            i++;
        }
        else if(cadena[x]=='o'){
            o++;
        }
        else if(cadena[x]=='u'){
            u++;
        }
    }
    cout << a << endl;
    cout << e << endl;
    cout << i << endl;
    cout << o << endl;
    cout << u << endl;
    return 0;
}
 

ponga la cadena que ponga lo que sale en la consola es:

Código:

0
0
0
2272144
0

pero si le saco esto:

Código C++:

else if(cadena[x]=='i'){
            i++;
        }
        else if(cadena[x]=='o'){
            o++;
        }
        else if(cadena[x]=='u'){
            u++;
        }
 

ya si me reconoce por lo menos la A y E, pero cuando tiene q mostrar el valor en pantalla de I , U
muestra esto:
i = 2272144
u = 2280688

no entiendo q pasa

[yeraypg]

Para usar la función length(); tendrías que incluir la libreria string en c++. Esto lo puedes hacer poniendo debajo de los otros "include de tu código:
#include <string>

[icm9]

pero si puedo usar length() igual, me devuelve la longitud de la cadena, igual puse #include <string> y no pasa nada distinto

[leosansan]

Iniciado por icm9

pero si puedo usar length() igual, me devuelve la longitud de la cadena, igual puse #include <string> y no pasa nada distinto

Tienes que inicializar todas las variables a cero, a=0,e=0,i=0,o=0,u=0, "incluyendo el x = 0" inicial del for. Y te sobra la librería stdio.h.

¡¡¡Saluditos!!!

[eferion]

Iniciado por yeraypg

Para usar la función length(); tendrías que incluir la libreria string en c++. Esto lo puedes hacer poniendo debajo de los otros "include de tu código:
#include <string>

Está programando en C++. La clase "string" de C++ dispone de un método llamado "length" que devuelve la longitud de la cadena. No hace falta incluir la librería "string.h" que está pensada para trabajar con arrays de caracteres.

Por lo demás, coincido con leosansan

Un saludo.

[yeraypg]

Lo siento

[eferion]

Iniciado por yeraypg

Lo siento

No hay que sentir nada... un despiste lo tiene cualquiera.

Lo he comentado sobretodo para evitar dudas.

[icm9]

Iniciado por leosansan

Tienes que inicializar todas las variables a cero, a=0,e=0,i=0,o=0,u=0, "incluyendo el x = 0" inicial del for. Y te sobra la librería stdio.h.

¡¡¡Saluditos!!!

Es eso gracias.