jperez2016 haznos un favor a todos y respira hondo, por favor.
No puedes ir en plan metralleta bombardeando con fallos diferentes que no tienen ninguna relación entre sí sin tan siquiera saber si los errores anteriores están corregidos. No digo que haya que regalar bombones... pero con dar de vez en cuando las gracias cuando alguien te aporta una solución es de agradecer.
Para empezar tienes que saber que para que te demos una solución puede no ser suficiente poner las 30 líneas de un bucle para encontrar un error. Por ejemplo, en el primer código que has puesto, el que no tiene las etiquetas GeSHi, no has puesto de qué tipo son las variables que se usan... simbolos1 es una matriz de tipo int... dos vectores anidados... una clase propia... esta información es importante de cara a evaluar un código.
Y bueno, te voy a dar algunas sugerencias más, es tu decisión adoptarlas o ignorarlas:
Usa el tipo boolPara variables que sean "si/no" o "verdadero/falso", es conveniente usar el tipo bool. 'encontrado', en tu caso, tiene dos posibles valores: 's' y 'n'. La solución que has adoptado no es la más conveniente porque... qué sucede si te equivocas y pones 'S' ??? tu código dejará de funcionar. Lo bueno de usar el tipo bool es que si pasas valores que no sean true o false el compilador te avisará de ello... digamos que te apoyas en el compilador para detectar errores.
Reduce el ámbito de las variablesEn C++ puedes declarar la variable que controla el bucle dentro del propio for, con esto consigues que la vida de esa variable esté limitada al propio bucle. Te permite evitar errores tontos al reutilizar variables:
for( int i=0; i<5; i++ )
cout << i << " ";
// Esta linea da error... i no existe
cout << i << endl;
// Aqui puedes redeclarar i sin problemas
for ( int i=2; i<10; i++ )
cout << i << " ";
Haz uso de los contenedores de C++Para controlar listas de elementos (hablando de tu primera duda) lo mejor es usar los contenedores de C++:
- vector: los elementos se almacenan en el orden que tu les dictes. Admite valores duplicados.
- set: los elementos se almacenan ordenados de menor a mayor. No admite duplicados.
- multiset: los elementos se almacenan ordenados de menor a mayor. Admite duplicados.
- map: almacena un par "clave - valor". No admite claves duplicadas pero sí valores duplicados.
- multimap: similar a map, admite claves duplicadas.
A continuación te presento dos formas (hay más) de usar contenedores para comprobar números repetidos:
* Opción 1. Es la mejor si los elementos tienen que estar desordenados
#include <algorithm>
#include <iostream>
#include <vector>
int main( )
{
std::vector< int > elementos;
elementos.push_back( 3 );
for ( int i=0; i<10; i++ )
{
// find busca 'i' en el vector y, si se encuentra, devuelve su posicion
if ( std::find( elementos.begin( ), elementos.end( ), i ) == elementos.end( ) )
elementos.push_back( i );
}
// Imprime en pantalla: 3 0 1 2 4 5 6 7 8 9
for ( int i=0; i<elementos.size( ); i++ )
std::cout << i << " ";
* Opción 2: ideal si no importa el orden de los elementos:
#include <iostream>
#include <set>
int main( )
{
std::set< int > elementos;
for ( int i=0; i<10; i++ )
elementos.insert( i );
for ( int i=0; i<10; i++ )
elementos.insert( i );
// set no funciona con indices, hay que recurrir a los iteradores
// Imprime por pantalla: 0 1 2 3 4 5 6 7 8 9
// No hay elementos repetidos
for ( auto it = elementos.begin( ); it != elementos.end( ); ++it )
cout << *it << " ";
}
No uses conio.hconio.h no es portable, es mejor no usar este tipo de librerías y recurrir a opciones que se ajusten más al estándar. Una posibilidad para sustituir a getch es std::cin.ignore( ), aunque no es la única.
Aprende a usar enumeradosHay que evitar en la medida de lo posible poner valores "a pelo" en el código, ya que son propensos a provocar errores.
En tu caso, por ejemplo, tienes 2 jugadores. Esto lo puedes expresar en el código de la siguiente forma:
enum Jugador
{
NoJugador,
Jugador1,
Jugador2
};
int main( )
{
Jugador jug = NoJugador;
int respuesta;
cout << "Introduce el jugador activo: ";
cin >> respuesta;
switch ( respuesta )
{
case 1:
jug = Jugador1;
break;
case 2:
jug = Jugador2;
break;
}
if ( jug == NoJugador )
cout << "Jugador " << jug << " elegido" << endl;
else
cout << "No se ha seleccionado un jugador valido" << endl;
}
Parejas de datosSi en algún momento necesitas manejar parejas de datos, puedes usar la clase "pair". Incluso puedes combinar pair con contenedores:
#include <iostream>
#include <vector>
int main( )
{
std::vector< std::pair< int, float > > datos;
datos.push_back( std::make_pair( 1, 3.4 ) );
datos.push_back( std::make_pair( 2, 5.7 ) );
for ( int i = 0; i < datos.size( ); i++ )
{
int valorEntero = datos[ i ].first; // Recuperamos el primer elemento del pair
float valorDecimal = datos[ i ].second; // Recuperamos el segundo elemento del pair
std::cout << valorEntero << " - " << valorDecimal << std::endl;
}
}
Mostrar Mensajes
. Eso sí, esta palabra es mejor usarla con cuidado porque puede complicar la lectura del código y dar problemas si se usa de forma indiscriminada.