Duda programa C++

[Intael]

Buenos días!

Estoy atascado diseñando un programa en C++. Consiste en diseñar un programa que al introducir 3 números enteros, te diga cuál de ellos es el más grande. Me compila sin problemas, sin embargo se supone que tengo que subir el archivo *cpp a un "juez" que me verifique que el programa funciona correctamente. Aunque el *exe me va bien (es decir, me da el máximo de 3 números), el juez me dice que el programa está mal... Alguien me sabe decir en qué me he equivocado? Aquí os dejo el programa:

#include <iostream>

using namespace std;

int main ()

{
int a, b ,c;
cin>>a;
cin>>b;
cin>>c;
if (a>b and a>c){cout<<a<<endl;}
if (b>a and b>c){cout<<b<<endl;}
if (c>a and c>b){cout<<c<<endl;}
}

[sam90]

Posiblemente el juez no usa esa forma de leer los datos... muchos jueces leen los datos desde un archivo...tambien puede ser que un int no sea suficientemente grande, prueba usar long...

[Intael]

Iniciado por sam90

Posiblemente el juez no usa esa forma de leer los datos... muchos jueces leen los datos desde un archivo...tambien puede ser que un int no sea suficientemente grande, prueba usar long...

Acabo de darme cuenta de q lo estaba subiendo al ejercicio equivocado xD

Ahora el problema es cómo hacer que ese programa sea válido para "el máximo de 3 números enteros incluso cuando los 3 son iguales".

Alguna idea?

[sam90]

Utiliza el >= en vez de usar el >.

[Intael]

He probado con esto, pero no furula:

#include <iostream>

using namespace std;

int main ()

{
int a, b ,c;
cin>>a;
cin>>b;
cin>>c;
if (a>=b and a>c){cout<<a<<endl;}
if (b>=a and b>c){cout<<b<<endl;}
if (c>=a and c>b){cout<<c<<endl;}
}


Alguna idea?

[pacoman]

Prueba con esto:

Código C++:

#include <iostream>
 
using namespace std;
 
int main ()
 
{
int a, b ,c;
cin>>a;
cin>>b;
cin>>c;
if (a>=b and a>=c){cout<<a<<endl;}
if (b>=a and b>=c){cout<<b<<endl;}
if (c>=a and c>=b){cout<<c<<endl;}
}
 

[Intael]

Iniciado por pacoman

Prueba con esto:

Código C++:

#include <iostream>
 
using namespace std;
 
int main ()
 
{
int a, b ,c;
cin>>a;
cin>>b;
cin>>c;
if (a>=b and a>=c){cout<<a<<endl;}
if (b>=a and b>=c){cout<<b<<endl;}
if (c>=a and c>=b){cout<<c<<endl;}
}
 

Poniendo eso, el programa me da 3 veces un número si introduzco ese número 3 veces. Es decir, si introduzco 1 1 1 el programa me da 1 1 1. Se supone que tiene que darme unasola vez 1.

[latinpower]

ahí tendrías que comparar también los datos entre ellos para ver que no se repitan. Yo lo haría al ingresar.

[sam90]

Yo lo que haria es una funcion max:

Código C++:

int max(int a , int b) 
{
    if (a < b) return b;
    else return a;
}
 
int main()
{
    int a, b ,c; 
    cin>>a;
    cin>>b;
    cin>>c;
    cout << "El numero maximo es " << max(a,max(b,c)) << endl;
}
 

[vladimirgude]

Aquí, otra solución. Saludos...

Código:

#include <iostream>

using namespace std;

int main ()
{
int a, b ,c;
cin>>a;
cin>>b;
cin>>c;
if (a>=b)
   if (a>=c)
      {cout<<a<<endl;}
   else
      {cout<<c<<endl;}
else
   if (b>c)
      {cout<<b<<endl;}
   else
      {cout<<c<<endl;}
}

[Intael]

Iniciado por vladimirgude

Aquí, otra solución. Saludos...

Código:

#include <iostream>

using namespace std;

int main ()
{
int a, b ,c;
cin>>a;
cin>>b;
cin>>c;
if (a>=b)
   if (a>=c)
      {cout<<a<<endl;}
   else
      {cout<<c<<endl;}
else
   if (b>c)
      {cout<<b<<endl;}
   else
      {cout<<c<<endl;}
}

Esa es la respuetsa correcta, el Juez me dice que está bien :) Muchísimas gracias!

[Intael]

¿Cómo sería si lo que busco es el máximo de 4 números (incluyendo la posibilidad de que se repitan)? Pensé que el procedimiento sería análogo a #10 pero con 4 números, pero... Sinceramente, no me aclaro.

[sam90]

Deberias usar algo mas dinamico...usar un arreglo y un for para ir recorriendolo...
No puedes ir modificando todo cada vez que te agregan un numero.

[Intael]

Buenas,

He probado con otra combinación de If's y el programa me ejecuta perfecta y coherentemente todo lo que le introduzco. Sin embargo, el juez me dice que hay un error de compilación, a pesar de que me compila perfectamente... A ver si me podéis decir en qué me he equivocado o sugerirme una solución alternativa:

#include <iostream>

using namespace std;

int main ()

{
int a, b, c, d;
cin>>a>>b>>c>>d;
if (a>=b)
{if (a>=c) {if (a>=d) {cout<<a;} else {cout<<d;}} else {if (c>=d) {cout<<c;} else {cout<<d;}}}
else
{if (b>c) {if (b>d) {cout<<b;} else {cout<<d;}} else {if (c>d) {cout<<c;} else {cout<<d;}}
}}

[vladimirgude]

No veo el error de compilación, sin embargo aquí va una solución muy parecida a tu código. Espero ayude...

Código:

#include <iostream>

using namespace std;

int main ()
{
int a, b ,c, d;
cin>>a;
cin>>b;
cin>>c;
cin>>d;
if (a>=b)
   if (a>=c)
      if (a>=d)
         {cout<<a<<endl;}
      else
         {cout<<d<<endl;}
   else
      if (c>=d)
         {cout<<c<<endl;}
      else
         {cout<<d<<endl;}
else
   if (b>=c)
      if (b>=d)
         {cout<<b<<endl;}
      else
         {cout<<d<<endl;}
   else
      if (c>=d)
         {cout<<c<<endl;}
      else
         {cout<<d<<endl;}
}

[s00rk]

Hay algo que aun veo que no has agregado, al final del programa

return 0;

El main lo tienes que regresara un valor y jamas lo estas haciendo no se ni porque no te marca error en eso al compilar ... :/

P.D. Perdon por responder no vi la fecha del tema, solo vi el del ultimo que comento mas no el del primero, aun asi dejo mi respuesta por ver que nadie puso lo mas obvio que pudo haber sido que el "juez" no lo haya evaluado bien.