Duda .

[K-YreX]

Tienes varias opciones para hacerlo:
- Opción 1: Usar contadores independientes para cada vector. En ese caso te quedaría algo así:

Código (cpp) [Seleccionar] Expandir


int contadorPar = 0, contadorImpar = 0;
for(i = 0; i < 10; i++)
    if(numeros[i] % 2 == 0){
        pares[contadorPar] = numeros[i];
        contadorPar++;
    }
    else{
        impares[contadorImpar] = numeros[i];
        contadorImpar++;
    }
De esta manera cada vector <pares> e <impares> tendrán los elementos en sus 5 primeras posiciones (si imprimes hasta 10 verás que las 5 últimas ya no tienen nada). Me he evitado todo lo que tienes tú en cada <if>; ya que tú lo que haces es sustituir el elemento de <numeros> por 0 una vez lo guardas en el otro vector, algo que no es necesario para que el programa funcione correctamente, pero que si te lo piden así está bien hecho.

- Opción 2: No mostrar los elementos que sean 0. En este caso no puedes usar tu código:

Código (cpp) [Seleccionar] Expandir

while(pares[i]!=0){
cout<<pares[i]<<" ";
}

Eso dentro del primer for para imprimir los numeros pares pero no se como.

En este caso muestras los elementos MIENTRAS un elemento no sea 0, entonces en cuanto encuentra un 0 va a dejar de imprimir. Además el tema de que se repita el 2 continuamente es porque al usar un <while> el incremento <i++> del <for> tienes que hacerlo explícitamente dentro del <while>.

Para que esta segunda opción funcione tendrías que hacerlo con un <for> para recorrer el vector como antes y un <if> dentro para no imprimir los valores que sean 0. El código sería de este estilo:

Código (cpp) [Seleccionar] Expandir


for(i = 0; i < 10; i++)
    if(pares[i] != 0)
        cout << pares[i] << "  ";

La opción más correcta es la primera pero te muestro la segunda para que veas que es posible plantear tu idea.

[Estudiante000000]

Tienes varias opciones para hacerlo:
- Opción 1: Usar contadores independientes para cada vector. En ese caso te quedaría algo así:

Código (cpp) [Seleccionar] Expandir


int contadorPar = 0, contadorImpar = 0;
for(i = 0; i < 10; i++)
    if(numeros[i] % 2 == 0){
        pares[contadorPar] = numeros[i];
        contadorPar++;
    }
    else{
        impares[contadorImpar] = numeros[i];
        contadorImpar++;
    }
De esta manera cada vector <pares> e <impares> tendrán los elementos en sus 5 primeras posiciones (si imprimes hasta 10 verás que las 5 últimas ya no tienen nada). Me he evitado todo lo que tienes tú en cada <if>; ya que tú lo que haces es sustituir el elemento de <numeros> por 0 una vez lo guardas en el otro vector, algo que no es necesario para que el programa funcione correctamente, pero que si te lo piden así está bien hecho.

- Opción 2: No mostrar los elementos que sean 0. En este caso no puedes usar tu código:En este caso muestras los elementos MIENTRAS un elemento no sea 0, entonces en cuanto encuentra un 0 va a dejar de imprimir. Además el tema de que se repita el 2 continuamente es porque al usar un <while> el incremento <i++> del <for> tienes que hacerlo explícitamente dentro del <while>.

Para que esta segunda opción funcione tendrías que hacerlo con un <for> para recorrer el vector como antes y un <if> dentro para no imprimir los valores que sean 0. El código sería de este estilo:

Código (cpp) [Seleccionar] Expandir


for(i = 0; i < 10; i++)
    if(pares[i] != 0)
        cout << pares[i] << "  ";

La opción más correcta es la primera pero te muestro la segunda para que veas que es posible plantear tu idea.

Listo logre hacerlo , muchas gracias por tu ayuda y tus explicaciones estoy aprendiendo mucho gracias a ti de verdad mil gracias , asi me quedo el codigo

Código (cpp) [Seleccionar] Expandir


#include<iostream>
#include<cstdlib>

using namespace std;

int main(){

int numeros[10] = {1,2,3,4,5,6,7,8,9,10};
int pares[10]={0};
int impares[10]={0};
int i,j,aux1,aux2;

for(i=0;i<10;i++){
if(numeros[i]%2==0){
aux1=numeros[i];
    numeros[i]=pares[i];
    pares[i]=aux1;
}
else {
aux2=numeros[i];
numeros[i]=impares[i];
impares[i]=aux2;
}
}
     
      cout<<"Numeros Pares ";
      cout<<"\n";
      cout<<"\n";
      for(i=1;i<10;i=i+2){
      cout<<pares[i]<<" ";
      }
     
     
     
      cout<<"\n";
      cout<<"\n";
      cout<<"Numeros Impares ";
      cout<<"\n";
      cout<<"\n";
      for(i=0;i<10;i=i+2){
      cout<<impares[i]<<" ";
      }


cin.get();
return 0;
}





[K-YreX]

Cierto, no me había dado cuenta que podías imprimirlos de dos en dos 

Ya para terminar de pulir ese código:
- En vez de ir poniendo un <cout> para cada salto de línea puedes poner por ejemplo:

Código (cpp) [Seleccionar] Expandir


cout << "Numeros Impares " << endl << endl;
Y ya tienes ahí dos saltos de línea.
- O declarar una constante para el tamaño de los vectores por si más adelante quieres guardar 20 números, sólo tienes que modificarlo en un sitio:

Código (cpp) [Seleccionar] Expandir


const int TAM = 10;
Y usar la variable <TAM> en el programa donde has usado el <10> directamente. Así consigues programas más genéricos.

[Estudiante000000]

Cierto, no me había dado cuenta que podías imprimirlos de dos en dos  

Ya para terminar de pulir ese código:
- En vez de ir poniendo un <cout> para cada salto de línea puedes poner por ejemplo:

Código (cpp) [Seleccionar] Expandir


cout << "Numeros Impares " << endl << endl;
Y ya tienes ahí dos saltos de línea.
- O declarar una constante para el tamaño de los vectores por si más adelante quieres guardar 20 números, sólo tienes que modificarlo en un sitio:

Código (cpp) [Seleccionar] Expandir


const int TAM = 10;
Y usar la variable <TAM> en el programa donde has usado el <10> directamente. Así consigues programas más genéricos.

Ahora que lo pienso ahora si , me piden un programa que me pida 10 numeros y  yo los ingrese , ahi si deberia utilizar la opcion 1 que me diste ? , porque ya no podria colocar como hice imprimirlos de dos en dos porque ya no estaran  ordenados asi como los puse en el vector numeros correcto ?

[K-YreX]

Exacto, no puedes dar por hecho que van a estar ordenados.
Además para no desperdiciar posiciones del vector, es más correcto introducirlos seguidos que no introducir un valor cada 2 o cada n posiciones.

[Estudiante000000]

Exacto, no puedes dar por hecho que van a estar ordenados.
Además para no desperdiciar posiciones del vector, es más correcto introducirlos seguidos que no introducir un valor cada 2 o cada n posiciones.

- Opción 1: Usar contadores independientes para cada vector. En ese caso te quedaría algo así:
Código

   int contadorPar = 0, contadorImpar = 0;
   for(i = 0; i < 10; i++)
       if(numeros % 2 == 0){
           pares[contadorPar] = numeros;
           contadorPar++;
       }
       else{
           impares[contadorImpar] = numeros;
           contadorImpar++;
       }

De esta manera cada vector <pares> e <impares> tendrán los elementos en sus 5 primeras posiciones (si imprimes hasta 10 verás que las 5 últimas ya no tienen nada). Me he evitado todo lo que tienes tú en cada <if>; ya que tú lo que haces es sustituir el elemento de <numeros> por 0 una vez lo guardas en el otro vector, algo que no es necesario para que el programa funcione correctamente, pero que si te lo piden así está bien hecho.


Utilize esta opcion y me parece mucho mejor ,  gracias  a esto no es necesario usar variables auxiliares ¿Porque? , y Bueno el codigo me quedo asi

Código (cpp) [Seleccionar] Expandir

#include<iostream>
#include<cstdlib>

using namespace std;

int main(){


int numeros[10];
int pares[10]={0};
int impares[10]={0};
int i;
int contadorPar=0 , contadorImpar=0;


   for(i=0;i<10;i++){
    cout<<i+1<<"-Digite un numero = ";
    cin>>numeros[i];
   }


 
for(i=0;i<10;i++){
if(numeros[i]%2==0){
 if(numeros[i] % 2 == 0){
             pares[contadorPar] = numeros[i];
             contadorPar++;
   }
   
}
else {
impares[contadorImpar] = numeros[i];
           contadorImpar++;
}
}
     
     cout<<"Numeros Pares ";
     cout<<"\n";
     cout<<"\n";
     
     for(i=0;i<10;i=i++){
      cout<<pares[i]<<" ";
     }
     
     
     
     cout<<"\n";
     cout<<"\n";
     cout<<"Numeros Impares ";
     cout<<"\n";
     cout<<"\n";
     for(i=0;i<10;i=i+2){
      cout<<impares[i]<<" ";
     }


cin.get();
return 0;
}


Me parece bien queda asi , si ingreso 10 numeros  ejemplo.

Numero 1 = 5
Numero 2 = 7
Numero 3 = 9
Numero 4 = 10
Numero 5 = 3
Numero 6 = 5
Numero 7 = 19
Numero 8 = 20
Numero 9 = 12
Numero 10 = 7

Me saldria en la consola asi :

Numeros Pares

10 20 12 0 0 0 0 0 0 0

Numeros Impares

5 7 9 3 5 19 7 0 0 0

¿Como podria no imprimir esos ceros?

[K-YreX]

Primero:

Utilize esta opcion y me parece mucho mejor ,  gracias  a esto no es necesario usar variables auxiliares ¿Porque? , y Bueno el codigo me quedo asi

Porque lo que hacías antes era intercambiar el valor de la posición <i> de <pares> (o <impares>) por la de <numeros> pero con guardar el valor de <numeros> en <pares> o <impares> es suficiente.
Tú al usar una variable auxiliar estabas por decirlo de alguna manera borrando cada elemento una vez que trabajabas con él, pero no era necesario.

Segundo:

Código (cpp) [Seleccionar] Expandir


for(i=0;i<10;i++){
if(numeros[i]%2==0){
 if(numeros[i] % 2 == 0)
Aquí estás repitiendo dos veces el mismo <if>, borra uno de ellos.

Y no imprimir 0 te doy una pista: al finalizar el <for> donde clasificas los números en <pares> e <impares>; <contadorPar> vale la cantidad de dígitos pares y <contadorImpar> de la misma manera vale la cantidad de dígitos impares. Úsalos en los últimos bucles <for>.

[Estudiante000000]

Primero:
Porque lo que hacías antes era intercambiar el valor de la posición <i> de <pares> (o <impares>) por la de <numeros> pero con guardar el valor de <numeros> en <pares> o <impares> es suficiente.
Tú al usar una variable auxiliar estabas por decirlo de alguna manera borrando cada elemento una vez que trabajabas con él, pero no era necesario.

Segundo:

Código (cpp) [Seleccionar] Expandir


for(i=0;i<10;i++){
if(numeros[i]%2==0){
 if(numeros[i] % 2 == 0)
Aquí estás repitiendo dos veces el mismo <if>, borra uno de ellos.

Y no imprimir 0 te doy una pista: al finalizar el <for> donde clasificas los números en <pares> e <impares>; <contadorPar> vale la cantidad de dígitos pares y <contadorImpar> de la misma manera vale la cantidad de dígitos impares. Úsalos en los últimos bucles <for>.

Muchisimas gracias por fin todas las dudas de este programa estan aclaradas , me quedo asi el codigo.

Código (cpp) [Seleccionar] Expandir

#include<iostream>
#include<cstdlib>

using namespace std;

int main(){

    const int TAM =10;
int numeros[TAM];
int pares[TAM]={0};
int impares[TAM]={0};
int i;
int contadorPar=0 , contadorImpar=0;


    for(i=0;i<TAM;i++){
    cout<<i+1<<"-Digite un numero = ";
    cin>>numeros[i];
    }


 
for(i=0;i<TAM;i++){
if(numeros[i]%2==0){
              pares[contadorPar] = numeros[i];
              contadorPar++;
    }
   
else {
impares[contadorImpar] = numeros[i];
            contadorImpar++;
}
}
      cout<<"\n";
      cout<<"Numeros Pares ";
      cout<<"\n";
      cout<<"\n";
     
      for(i=0;i<contadorPar;i=i++){
      cout<<pares[i]<<" ";
      }
     
     
     
      cout<<"\n";
      cout<<"\n";
      cout<<"Numeros Impares ";
      cout<<"\n";
      cout<<"\n";
      for(i=0;i<contadorImpar;i=i++){
      cout<<impares[i]<<" ";
      }


cin.get();
return 0;
}



Y para finalizar en los for entonces hacer un for de esta manera esta mal cierto

Código (cpp) [Seleccionar] Expandir

for(i=0;i=9;i++){
}

[K-YreX]

Y para finalizar en los for entonces hacer un for de esta manera esta mal cierto

Código (cpp) [Seleccionar] Expandir

for(i=0;i=9;i++){
}

Sí, sería incorrecto, en primer lugar porque el segundo campo del <for> que debería ser una condición, no lo es, es una asignación. El funcionamiento de ese <for> es el siguiente: entras en el <for> y se asigna el valor inicial <i = 0>. Se llega a la condición que como no es una condición sino una asignación se ejecuta entonces ahora <i = 9>. Se ejecuta el interior del bucle. Termina una iteración y se produce el incremento <i = 10>, se llega a la condición de nuevo y... <i = 9>. Se vuelve a ejecutar el bucle.
Conclusión: tenemos un bucle infinito en el que i siempre vale 9.

Resumen: <a = b> asignas a <a> el valor de <b>
                 <a == b> comparas que el valor de <a> y el de <b> sean iguales

Quieres probarlo? Simple, ejecuta esto (no hace falta que modifiques nada) y podrás ver los valores de i por pantalla (no esperes que acabe):

Código (cpp) [Seleccionar] Expandir


#include <iostream>
using namespace std;
int main(){
for(int i = 0; i = 9; i++)
cout << i << endl;
}

En todo caso la otra opción (haciendo correctamente una condición y no una asignación) sería usando dos iguales, tal que así:

Código (cpp) [Seleccionar] Expandir


#include <iostream>
using namespace std;
int main(){
for(int i = 0; i == 9; i++)
cout << "El bucle se ha ejecutado" << endl;
}
Resultado de este código: NINGUNO. Empieza el bucle <i = 0>, llegamos a la condición... ¿<i == 9>? No, <i = 0>; entonces sale del bucle. Nunca llegará a ejecutarse lo que haya dentro del bucle.

[Estudiante000000]

Sí, sería incorrecto, en primer lugar porque el segundo campo del <for> que debería ser una condición, no lo es, es una asignación. El funcionamiento de ese <for> es el siguiente: entras en el <for> y se asigna el valor inicial <i = 0>. Se llega a la condición que como no es una condición sino una asignación se ejecuta entonces ahora <i = 9>. Se ejecuta el interior del bucle. Termina una iteración y se produce el incremento <i = 10>, se llega a la condición de nuevo y... <i = 9>. Se vuelve a ejecutar el bucle.
Conclusión: tenemos un bucle infinito en el que i siempre vale 9.

Resumen: <a = b> asignas a <a> el valor de <b>
                 <a == b> comparas que el valor de <a> y el de <b> sean iguales

Quieres probarlo? Simple, ejecuta esto (no hace falta que modifiques nada) y podrás ver los valores de i por pantalla (no esperes que acabe):

Código (cpp) [Seleccionar] Expandir


#include <iostream>
using namespace std;
int main(){
for(int i = 0; i = 9; i++)
cout << i << endl;
}

En todo caso la otra opción (haciendo correctamente una condición y no una asignación) sería usando dos iguales, tal que así:

Código (cpp) [Seleccionar] Expandir


#include <iostream>
using namespace std;
int main(){
for(int i = 0; i == 9; i++)
cout << "El bucle se ha ejecutado" << endl;
}
Resultado de este código: NINGUNO. Empieza el bucle <i = 0>, llegamos a la condición... ¿<i == 9>? No, <i = 0>; entonces sale del bucle. Nunca llegará a ejecutarse lo que haya dentro del bucle.

Mil gracias entendi perfectamente todo lo que me has explicado estoy aprendiendo mucho gracias a tus explicaciones.