no me ordena bien 3 numeros

[ost]

Hola,

estoy dándole vueltas y vueltas al ejercicio y no se el motivo por el cual no me ordena bien los números:

Escribir un programa que lea tres números enteros
en tres variables n1, n2, n3 reasigne los valores de las variables para
que n1 sea la más pequeña, y n3 la más grande Finalmente que escriba los
tres valores ordenados.

Código:

#include <stdio.h>
#include <conio.h> //getch

main()
{
      int n1, n2, n3; 
      printf("introduce numero n1 \n"); 
      scanf("%d",&n1); 
      printf("introduce numero n2 \n"); 
      scanf("%d",&n2);
      printf("introduce numero n3 \n"); 
      scanf("%d",&n3);
      
      if(n1>n2){
         if(n2>n3){
                                        n1=n3;n2=n2;n3=n1;//n1>n2 n2>n3
         }else if(n1>n3){
                                        n1=n2;n2=n3;n3=n1;//n1>n2 n2<n3 n1>n3
         }else{
                                        n1=n2;n2=n1;n3=n3;//n1>n2 n2<n3 n1<n3
         }              
      }else if(n1>n3){
                                        n1=n3;n2=n1;n2=n2;//n1<n2 n1>n3
         }else if(n2>n3){
                                        n1=n1;n2=n3;n3=n2;//n1<n2 n1<n3  n2>n3
         }else{
                                        n1=n1;n2=n2;n3=n3;//n1<n2 n1<n3  n2<n3
         }
      printf("El orden es el siguiente: \n\t %d \t %d \t %d", n1,n2,n3 ); 
      getch();
}

Muchas gracias.
ost.

[Slackin]

Fijate bien en líneas como:

Código:

if(n1>n2){
         if(n2>n3){
                                        n1=n3;n2=n2;n3=n1;//n1>n2 n2>n3

El valor inicial de n1 lo pierdes, y luego lo asignas a n3. Necesitas tener variables temporales para guardes valores.

Saludos

[ost]

muchisimas gracias, era eso.

dejo el codigo por si alguien le pasaba lo mismo, si hubiese otra forma mas facil o mejor de hacerlo, por favor que alguien la ponga.

aunque no tenía nada que ver con el problema, tambien tenía una errata en el codigo, la pongo ya corregida tambien;

Código:

      }else if(n1>n3){
                                        n1=n3;n2=n1;n2=n2;//n1<n2 n1>n3

Código:

#include <stdio.h>
#include <conio.h> //getch

int main()
{
      int n1, n2, n3;
	  int nt1, nt2, nt3;
      printf("introduce numero n1 \n"); 
      scanf("%d",&n1); 
      printf("introduce numero n2 \n"); 
      scanf("%d",&n2);
      printf("introduce numero n3 \n"); 
      scanf("%d",&n3);
      nt1=n1;nt2=n2;nt3=n3;
      if(n1>n2){
         if(n2>n3){
                                        n1=nt3;n2=nt2;n3=nt1;//n1>n2 n2>n3
         }else if(n1>n3){
                                        n1=nt2;n2=nt3;n3=nt1;//n1>n2 n2<n3 n1>n3
         }else{
                                        n1=nt2;n2=nt1;n3=nt3;//n1>n2 n2<n3 n1<n3
         }              
      }else if(n1>n3){
                                        n1=nt3;n2=nt1;n3=nt2;//n1<n2 n1>n3
         }else if(n2>n3){
                                        n1=nt1;n2=nt3;n3=nt2;//n1<n2 n1<n3  n2>n3
         }else{
                                        n1=nt1;n2=nt2;n3=nt3;//n1<n2 n1<n3  n2<n3
         }
      printf("El orden es el siguiente: \n\t %d \t %d \t %d", n1,n2,n3 ); 
      getch();
}

ost.

[joticajulian]

Otra forma de ordenar es por "ordenamiento de burbuja", porque dado el caso que se quisiera ordenar 10 números o 100 números por el método que haces sería casi que imposible por lo largo.
Pongo un ejemplo de ordenamiento burbuja:

Código c:

#include <stdio.h>
 
int main(){
   int n[3]; 
   int temp;
 
   //ingreso de datos
   for(int i=0;i<3;i++){
      printf("introduce numero n%d: ",i+1); 
      scanf("%d",&n[i]);
   } 
   
   //ordenamiento burbuja
   for(int i=3;i>0;i--){
      for(int j=0; j <i-1; j++){
         if(n[j] > n[j+1]){
            temp = n[j];
            n[j] = n[j+1];
            n[j+1] = temp;
         }
      }
   }
 
   //impresión de datos ordenados
   printf("El orden es el siguiente: ");
   for(int i=0;i<3;i++){
      printf("%d ",n[i]);
   }
   getchar();
   return 0;
}
 

[ivdrako]

q tal, pues un metodo mejor para ordenar n cantidad de numeros es con un algoritmo que se llama mergeSort, lo que se hace es meter tus n numeros en un arreglo e ir dividiendo en mitades el arreglo y ordenar por parejas los numeros, al final se juntan todas las partes y queda ordenado, deben disculpar que no les pase el codigo, pero nunca lo he hecho en lenguaje c solo en java jeje, aun asi el algoritmo es facil de encontrar en internet y mejor explicado q el mio, la ventaja de mergeSort sobre el metodo de la burbuja (bubbleSort) es que tarda en resolverse tiempo de n*log (n) y el de la burbuja es de tiempo de n2 (n cuadrada), sirve para cuando se quieren ordenar arreglos grandes, ojala les sirva esta info.


saludos