2 respuestas en este tema

[m1kevil]

cpp

// Funcion para definir nombres de los jugadores y quien empieza primero //
int TurnoInicial (int Jugadores)
{
    char vec[Jugadores];
    int i;   
    for (i=0; i<Jugadores; i++)
    {
            printf("Ingrese el nombre del jugador %d: ", i+1);
            scanf ("%s", &vec[i]);
    }
    for (i=0; i<Jugadores; i++)
    {
            printf("El jugador %d se llama: %s\n", i+1, vec[i]); 
    }
   
}

Buenas! Estoy desarrollando un juego simple en C "puro" y necesito que luego de determinar la cantidad de jugadores que participaran del juego, una funcion sea capaz de almacenar los nombres de los jugadores (ingresados por teclado), y guardalos en un vector de la cantidad de posiciones de la variable 'Jugadores'.
Este vector será utilizado luego para desplegar el nombre del jugador en el momento que le toque su turno de tirar los dados por ejemplo.

Con el código expuesto arriba, funciona bien hasta que ingreso el ultimo nombre por teclado. Luego el programa se cuelga. Pienso que el problema radica en la funcion scanf y en el %s . Quizas otro método de entrada para los strings funcione mejor en este caso, pero es la primera vez que manejo cadenas de caracteres en C.

Por favor agradecería que me dieran una mano!
Desde ya muchas gracias!

[escafandra]

Bienvenido a delphiacces  m1kevil.

Dudo que el código que expones compile tal como esta publicado pues no puedes definir un array estático de un tamaño definido por una variable.

Una primera aproximación a tu problema sería definir un array de dos dimensiones, una para el número máximo de jugadores admitidos y otra para el tamaño de los nombres. Si ese array lo defines como variable global lo conocerán todas las funciones de tu juego.

Este sería el ejemplo descrito:

cpp

char Nombres[5][41];  // Cinco jugadores 40 caracteres por nombre (mas el nulo final)
 
int TurnoInicial (int Jugadores)
{
  int i;
  char Buffer[1024];  // Buffer para la entrada por teclado.
 
  for(i = 0; i < Jugadores; i++){
    printf("Ingrese el nombre del jugador %d: ", i+1);
    strncpy(Nombres[i], gets(Buffer), 40);  // aseguramos que no entran mas de 40 caracteres
  }
  for(i = 0; i < Jugadores; i++){
    printf("El jugador %d se llama: %s\n", i+1, Nombres[i]);
  }
}

El siguiente paso sería definir un array dinámico en lugar de estático, esto complica un poquito mas el código y en este caso no debería ser necesario.


Saludos.

[escafandra]

Una variante de la función anterior, mas segura para el desbordamiento de buffer:

cpp

// Funcion para definir nombres de los jugadores //
void ingresarNombres (int Jugadores)
{
  int i, n;
  for(i = 0; i < Jugadores; i++){
    printf("Ingrese el nombre del jugador %d: ", i+1);
    n = strlen(fgets(Nombres[i], 40, stdin))-1;
    if(Nombres[i][n] == '\n') Nombres[i][n] = 0;
  }
  for(i = 0; i < Jugadores; i++){
    printf("El jugador %d se llama: %s\n", i+1, Nombres[i]);
  }
}

Saludos.