Programa busca segundo mayor c++

[Payaso73]

Hola, me gustaria que me digan cual es el error de este programa que busca los dos mejores resultados... el profesor nos explico otra forma con variables auxiliares, pero yo lo habia hecho asi al principio y "de vista" me gustaba, compila sin error, pero cuando lo ejecuto no anda... yo se que hay una forma mejor de buscar el primero y segundo mejor, pero me gustaria saber en donde esta mi error

Código:

int main(int argc, char *argv[]) {
	
	int puntaje[5];
	int ganador;
	int sub;
	
	for(int i=0; i<5; i++)
	{
		cout<<"ingrese el puntaje: ";
		cin>>puntaje[i];
	}
	
	for(int i=0; i<4; i++)
	{
		if(puntaje[i]>puntaje[i+1])
			ganador=i;
	}
	for(int i=0; i<4; i++)
	{
		if(puntaje[i]>puntaje[i+1])
		{
			if(i!=ganador)
				sub=i;
		}
	}
	
	cout<<"el ganador es el participante "<<ganador+1<<" con "<<puntaje[ganador]<<" puntos";
	cout<<endl<<"el segundo es el participante "<<sub+1<<" con "<<puntaje[sub]<<" puntos";
	
	return 0;
}

Gracias por el tiempo, saludos

[ahrion]

Con mejor puntaje te referís a mayor puntaje?

[ahrion]

Te marco algunos errores :

*Estás definiendo la variable "i" tres veces (una en cada for). Defini una al principio de main.
*Al no poner ningúna pausa, el programa se va a ejecutar tan rapido que ni lo vas a ver se cierra enseguida. Poné :
system("pause");
ó
system("pause > nul");

Igual ahora en un ratito trato de arreglarlo

[Annoying]

Para estas cosas tienes que usar auxiliares en mi opinión.

Código:

for(int i=0; i<4; i++)
	{
		if(puntaje[i]>puntaje[i+1])
			ganador=i;
	}

Aquí nunca registrarás el 4to número del array como ganador, aparte de que, pongo un ejemplo:

Numeros: 4 3 5 2 9
FOR 1: 4 > 3? ganador=0(4);
FOR 2: 3 > 5? ganador=0(4);
FOR 3: 5 > 2? ganador=3(5);
FOR 4: 2 > 9? ganador=3(5);

En todo caso tendría que ser algo asi:

Código:

    numero=0;
                numero2=0;
                for(int i=0; i<5; i++)
	{
	       if(numero<puntaje[i])
                       {
                             if(numero2<numero)
                             {
                                  sub=ganador;
                                  numero2=numero;
                              }
                               numero=puntaje[i];
                               ganador=i;
                        }
                        else
                         {
                                if(numero2<puntaje[i])
                                {
                                      sub=i;
                                      numero2=puntaje[i];
                                 }
                          }
	}

[Payaso73]

Gracias ahrion ya solucione la declaración del i y en cuanto a la pausa no me hace falta porque lo veo igual... si con mejor puntaje me refiero al mayor.

Si Annoying, ahí tenes razon, se me habia pasado eso

Aquí nunca registrarás el 4to número del array como ganador, aparte de que, pongo un ejemplo:

Numeros: 4 3 5 2 9
FOR 1: 4 > 3? ganador=0(4);
FOR 2: 3 > 5? ganador=0(4);
FOR 3: 5 > 2? ganador=3(5);
FOR 4: 2 > 9? ganador=3(5);

y tambien podría comparar el "ganador" además de los dos puntajes... algo asi:

Código:

for(i=0; i<4; i++)
	{
		if((puntaje[i]>puntaje[i+1])&&(puntaje[i]>puntaje[ganador]))
			ganador=i;
	}

¿no?

Aclaro que sé que se puede hacer mejor con varias variables auxiliares... pero queria encontrar mi error.

Gracias a los dos.
Saludos

[Annoying]

El problema sigue en que el último elemento nunca será el número ganador(mayor) y que al tener un número que sea mayor que el actual ganador, este ganador pasa casi automáticamente a ser el segundo mayor.

Código:

for(i=0; i<4; i++)
	{
		if((puntaje[i]>puntaje[i+1])&&(puntaje[i]>puntaje[ganador]))
			ganador=i;
	}

[Payaso73]

Claro... entiendo... voy a seguir buscando otra forma que no sea con variables auxiliares... solo de terco! jaja

Gracias a los dos...
Saludos

[Annoying]

Iniciado por Payaso73

Claro... entiendo... voy a seguir buscando otra forma que no sea con variables auxiliares... solo de terco! jaja

Gracias a los dos...
Saludos

El persistir es una virtud, suerte :P

[Payaso73]

Si encuentro una nueva forma te mando un MP, llamamos a amigos y hacemos una fiesta para festejar jaja (por las dudas ya compre el fernet)

Saludos