Foro de elhacker.net

buenas noches gente
como puedo hacer para cargar una matriz de 10x10 con 50 pares de numeros o caracteres aleatorios sin repetir en C++?
ayuda por favor

:)

Qué sabes de c++ y que llevas hecho? Donde tienes la duda de como continuar

while(aleatorio1<50)
    {
      randomize();
      num1=random(10);
      num2=random(10);
      num3=random(10);
      num4=random(10);
      num5=random(50);
      if(num1==num3&&num2==num4)
        eselmismo='s';
      if (eselmismo='n')
        {
          for(i=0;i<TAM;i++)
            {
              for(j=0;j<TAM;j++)
                {
                  if(simbolos1[j]==num5)
                    repite++;
                }
            }
        }
      if (repite<2)
        {
          simbolos1[num1][num2]=num5;
          simbolos1[num3][num4]=num5;
          aleatorio1++;
        }
      repite=0;
    }

  for(i=0;i<TAM;i++)
    {
      for(j=0;j<TAM;j++)
        {
          cout<<"     "<<simbolos1[j];
        }
      cout<<"\n";
    }
  /*while(aleatorio1<50)
    {
      while(num<60||num>255)
        {
          srand(time(0));
          num=rand();
          letra=(char)num;
        }
      while(num1<0||num1>TAM||num2<0||num2>TAM||num3<0||num3>TAM||num4<0||num4>TAM||(num1==num3 && num2==num4))
        {
          srand(time(0));
          num1=rand();

          num2=rand();

          num3=rand();

          num4=rand();
        }
      for(i=0;i<50;i++)
        {
          if(cargados==letra)
            {
              serepite='s';
            }
        }
      if(serepite=='n')
        {
          simbolos1[num1][num2]=letra;
          simbolos1[num3][num4]=letra;
          num5=num-60;
          cargados[num5]=letra;
          aleatorio1++;
        }
    }

esto queda en un loop infinito :(

Pusiste al principio if(eselmismo='n') cuando deberias poner ==

srand(time(0)); se coloca solo 1 vez al principio del programa, y nada más.

Bueno son un par de cosas que vi. Tienes varias cosas que revisar en el código :o


PD: La próxima vez, coloca las etiquetas GeSHi del foro para colocar códigos:

int main(){
    return 0;
}

como colocos las etiquetas GeSHI esas? :huh:

Donde los botones de negrita, cursiva, etc tienes a la derecha del todo GeSHi.

aún poniendo srand(time(0)) no funciona :(

Te voy a dejar un truquito para no tener que poner ese tremendo while() con todas las condiciones de los aleatorios. Eso podría dejar el programa ahí indefinidamente.

int num = rand()%(MAX-MIN+1)+MIN;

Eso genera un valor entre MAX y MIN, ambos incluídos.

El operador % devuelve el resto de una división. Poner rand()%num va a devolver un número que será el resto de rand()/num. El resto va desde 0 hasta num-1, eso ya son matemáticas.

Lo de mínimo ya solo es un arreglo matemático de fácil comprensión.

Suerte. Si implementas eso, y revisas el código, quizás te vaya.

Tengo un problema, cuando ingreso una letra en el jugador entro en un loop infinito. Me pueden ayudar?

do //ciclo hacer mientras el jugador ingresado sea incorrecto.
  {
 
    cout<<"\n";
    cout<<"Que jugador comienza? ";   //pregunta que jugador comienza.
    cin>>jug;   //guardo el jugador.
    getch();
    if (jug!=1&&jug!=2) //me fijo si ingreso un jugador que es incorrecto.
      {
        cout<<"No existe ese jugador. ";  //muestro que no es un jugador válido.
        cin>>jug;
      }
    else   //si el jugador es correcto.
	   jug_correcto='s';
  }while(jug_correcto=='n');

Cuando tengas una duda o problema con uno de tus programas por favor publica el código fuente completo.

En el fragmento que publicas la segunda lectura (cuando el valor de "jug" es diferente de 1 y 2) es innecesaria y deberías evitar el uso de la biblioteca conio de Borland, mas información en la pagina |Lo que no hay que hacer en C/C++. Nivel basico| (http://foro.elhacker.net/programacion_cc/lo_que_no_hay_que_hacer_en_cc_nivel_basico-t277729.0.html)

Un saludo

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

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

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:

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

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:

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