juego memoria en c++

[jperez2016]

hola, estoy tratando de declarar un jugador como inactivo y no puedo. pueden ayudarme¿?

Código (cpp) [Seleccionar] Expandir

clrscr();
                       cout<<"\nIngrese el alias del jugador que desea declarar como inactivo ";

                       gets(desactivar);
                       encontrado='n';
                       igual=0;
                       for(j=0;j<100;j++)
                         {
                           for(k=0;k<20;k++)
                             {
                               if(jugadores[j].alias[k]==desactivar[k])
                                 igual++;
                             }
                           if(igual==20)
                             {
                               jugadores[j].activo='n';
                               encontrado='s';
                             }
                         }

[eferion]

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 bool
Para 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 variables
En 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:

Código (cpp) [Seleccionar] Expandir


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

Código (c) [Seleccionar] Expandir


#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:

Código (cpp) [Seleccionar] Expandir


#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.h

conio.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 enumerados

Hay 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:

Código (cpp) [Seleccionar] Expandir


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 datos

Si en algún momento necesitas manejar parejas de datos, puedes usar la clase "pair". Incluso puedes combinar pair con contenedores:

Código (cpp) [Seleccionar] Expandir


#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;
  }
}