el codigo no imprime una variable por que?

[sonrasiel]

hola tengo un problema tengo un código y quiero que ordene los valores de cada posición de arreglo me de los impares y haga la sumatoria de cada numero pero la variable que debería hacer la sumatoria no la imprime y tampoco imprime un texto que escribí que dice "la suma es". por que pasa esto?

Edito: el programa no compila y el dev me estaba compilando uno anterior no se por que aun estoy trabajando en arreglarlo asi que no se gasten en revisar el daño

aquí esta el código

Código C++:

#include <iostream>
#include <cstdlib>
using namespace std;
 
int main()
{
    system("color 70");
    int array[5],x=0,j=0,temp;
    for (x=0;x<=4;x++)
    {
        cout << "digite el "<<x<<" numero "<<endl;
        cin >> array[x];
    }
    for (x=0;x<=4;x++)
        for(j=0;j<3;j++)
    {
            if(array[x] > array[x+1])
            {
                temp = array[x];
                array[x] = array[x+1];
                array[x+1] = temp;
            }
        
    }
    cout <<"Ordenamiento: " <<endl;
    for (x=0;x<=4;x++)
    {
        cout << array[x] << endl;
    }
    cout << "IMPARES:"<<endl;
    for (x=0;x<=4;x++)
    {
        if(array[x] % 2 != 0){
            if(array[x] >1)
            {
                int i,suma;
                cout<<"los impares menores a\t"<<array[x]<<"son\t"<<endl;
                for(i=1;i<array[x];i=i+2){
                    suma=suma+i;
                    cout<<i<<",";
                }
                cout<<endl;
                cout<<"la suma de los impares es \t"<<i<<"el bueno"<<suma<<endl;
            }
            
            cout << "Valor-->\t" << array[x]<<endl<<endl;
        }
    }
    system("pause");
    return 0;
}
}
 

[cronopiomx]

la variable suma tienes que inicializarla en 0, prueba a ver.

saludos
cronos

[cronopiomx]

el algoritmo de ordenamiento que usas es Bubble Sort, pero creo que lo estas usando mal, si lo que realizas en tu codigo es un simple ordenamiento de 1 arreglo de int, entonces debes hacerlo asi:
for (x=0; x<=4 ;x++)
for(j=x+1; j<=4 ; j++)
{
if(array[x] > array[j])
{
temp = array[x];
array[x] = array[j];
array[j] = temp;
}

}

s2
cronos

[OscarMariano]

En la linea 11 no deberia ser x sino un numero (int num)

[OscarMariano]

Iniciado por OscarMariano

En la linea 11 no deberia ser x sino un numero (int num)

Me confundi ya entendi que querias hacer ahora veo que paso

[OscarMariano]

Para empezar acumuladores y contadores se declaran antes de la estructura repetitiva en tu caso el for afuera del for debiste poner int suma=0;

[OscarMariano]

Asi esta mas bonito lo que quieres haces fijate bien en tus bucles como los haces ahi has tenido mas problemas, por algo dicen manten el codigo simple :

#include <iostream>
#include <cstdlib>
using namespace std;

int main()
{
system("color 70");
int array[5],x=0,j=0,temp;
for (x=0;x<=4;x++)
{
cout << "digite el numero que ira en la posicion "<<x<<" del arreglo =>";
cin >> array[x];
}
for (x=0; x<=4 ;x++)
for(j=x+1; j<=4 ; j++)
{
if(array[x] > array[j])
{
temp = array[x];
array[x] = array[j];
array[j] = temp;
}
}
cout <<"Ordenamiento: " <<endl;
for (x=0;x<=4;x++)
{
cout << array[x] << endl;
}


int suma=0,suma2=0;
for (x=0;x<=4;x++)
{
if(array[x] % 2 != 0){
suma=suma+array[x];
}

else{
suma2=suma2+array[x];
}

}
cout<<"la suma de los impares es =>"<<suma<<endl;
cout<<"la suma de los pares es =>"<<suma2<<endl;
cout<<"la suma total es =>"<<suma+suma2<<endl;
system("pause");
return 0;
}