Ejecuta doble

[boronix]

Quiero que me ordene tres numeros de mayor a menor. Me ejecuta bien, pero el problema me viene cuando dos de los tres numeros son iguales. no se porque cuando meto (3,3,1) me lo ordena de una sola vez (3,3,1) y en cambio cuando meto (1,3,3) me repite el orden, es decir, me lo hace saber dos veces. Alguien me puede decir la manera de que me lo diga de una sola vez?

#include<stdlib.h>
#include<stdio.h>
int main()
{
int n1,n2,n3 ;
printf("Dime el primer numero: ");
scanf("%d",&n1);
printf("Dime el segundo numero: ");
scanf("%d",&n2);
printf("Dime el tercer numero: ");
scanf("%d",&n3);
if(n1>=n2&&n2>=n3)
{
printf("Los numeros en orden descendente son: %d,%d,%d\n",n1,n2,n3);
}
else
{
if(n1>=n3&&n3>=n2)
{
printf("Los numero en orden descendente son: %d,%d,%d\n",n1,n3,n2);
}
if(n2>=n1&&n1>=n3)
{
printf("Los numero en orden descendente son: %d,%d,%d\n",n1,n2,n3);
}
if(n2>=n3&&n3>=n1)
{
printf("Los numero en orden descendente son: %d,%d,%d\n",n1,n2,n3);
}
if(n3>=n1&&n1>=n2)
{
printf("Los numero en orden descendente son: %d,%d,%d\n",n1,n2,n3);
}
if(n3>=n2&&n2>=n1)
{
printf("Los numeros en orden descendente son: %d,%d,%d\n",n1,n2,n3);
}
}
system("PAUSE");
}

Muchisimas gracias a tod@s los que me habeis ayudado hasta ahora. No me gusta rendirme a la primera, pero llevo 2 dias y nada y antes de qedarme con la duda, prefiero tragarme mi orgullo. Al fin y al cabo el programa esta bien, pero es mejorable en este aspecto

[amchacon]

El error tiene que ver con que dos números sean iguales, lo que hace que sea posible que se cumplan dos condiciones... Tienes que quitar la condición de igualdad de algunos sitios (o poner elses en cada if).

En la programación siempre hay que intentar usar algoritmos genéricos, usar un montón de ifs no es adecuado (que pasa cuando quieras ordenador 4 números?). ¿Has dado for y vectores?

[dontexplain]

No necesitas usar tantos if ni hacerlo tan farragoso.

En primer lugar usa una directiva de preprocesador que imprima una función para concluri el número mayor.

Código C++:

#define gmax(a,b) ((a)>(b)?(a):(b))
#define gmin(a,b) ((a)<(b)?(a):(b))
using namespace std;
int main(){
int n1,n2,n3, max1, max2, min1,min2;
// scanfs
 
max2 = gmax(n1,n2); // intermedio mayor
max1 = gmax(n3,max2); // mayor de los 3
min2  = gmin(n1,n2); // intermedio menor
min1  = gmin(n3,min2); // menor de los 3
 
printf("Los numeros en orden descendente son: %d,%d,%d\n",max1,max2, min1);
return 0;
}
 

[amchacon]

Segun dicen, en c++ no se debería usar los preprocesadores. Es preferible las funciones inline (más fácil detectar los errores).