1

Estoy haciendo un programa que, basado en un número aleatorio, seleccione la línea correspondiente a ese número y la imprima en pantalla, pero necesito que me imprima, la asociada al número aleatorio, y las 4 siguientes, ya que se trata de una pregunta y sus cuatro opciones. Este proceso se ha de repetir "x" veces (en este caso 20 veces) y necesito que cada vez imprima una pregunta distinta, pero no pasa nada si esporádicamente alguna pregunta se repite.

Tan sólo tengo esto, alguien me podría ayudar? Muchas gracias de antemano. (El while lo he puesto porque necesito que la primera línea que imprima sea múltiplo de 5)

void preguntasyrespuestastrivial() {
  system("cls");

  int i;
  i = 0;
  for (i = 0; i < 20; i++) {

    cout << "Pregunta " << i + 1 << ": " << endl;

    ifstream archivo_entrada("preguntas.txt");;
    string linea;
    int contador = 0;
    int aleatorio;

    srand(time(NULL));
    int DESDE = 0, HASTA = 56;

    aleatorio = rand() % (HASTA - DESDE + 1) + DESDE;

    while ((aleatorio != 5) && ((aleatorio != 10) && (aleatorio != 15) && (aleatorio != 20) && (aleatorio != 25) && (aleatorio != 30) && ((aleatorio != 35) && (aleatorio != 40) && (aleatorio != 45) && (aleatorio != 50)))) {
      aleatorio = rand() % (HASTA - DESDE + 1) + DESDE;
    }
    cin >> aleatorio;

    while (getline(archivo_entrada, linea)) {

      if (aleatorio == contador) {
        cout << linea << endl;
      }
      contador++;
    }
    
    system("pause");
  }
}
2
  • Puedes usar el operador % para simplificar el ciclo while. Algo como while(aleatorio % 5 != 0 and aleatorio <= 5 and aleatorio >= 50) Commented el 17 ene. 2021 a las 14:54
  • Fantástico, muchas gracias!! Commented el 17 ene. 2021 a las 16:22

1 respuesta 1

0

El uso de las funciones rand y srand está desaconsejado en , primero porque son utilidades de cabeceras de y segundo porque son unas funciones inseguras, son propensas a fallos y son ineficientes (hasta el punto de que se ha propuesto deprecarlas).

Dado que programas en C++, debes usar las utilidades de generación de números pseudo-aleatorios de dicho lenguaje, la cabecera <random>. Asi pues, si necesitas un número aleatorio entre A y B, puedes hacer lo siguiente:

// Habiendo incluido <random>:

std::random_device dispositivo;                     // Se usa para obtener la raíz aleatoria
std::mt19937 generador(dispositivo());              // Usa el algoritmo mersenne_twister_engine
std::uniform_int_distribution<> distribucion(A, B); // Los números generados estarán entre A y B


auto numero = distribucion(generador); // Generamos un número pseudo-aleatorio

Si necesitas que los números se generen como múltiplos de cinco, no generes números al tuntún y descartes los que no te valgan, eso es terriblemente ineficiente pues tu generación falla el 80% de las veces. Simplemente genera un número y multiplícalo por cinco.

También es terriblemente ineficiente abrir el archivo de preguntas cada vez que quieres acceder a un quinteto de ellas, lo mejor es leer en memoria todas las preguntas y trabajar los datos desde ahí. Con estos cambios, tu código podría parecerse a:

int main()
{
    const auto preguntas = leer_preguntas();

    std::random_device dispositivo;
    std::mt19937 generador(dispositivo());
    std::uniform_int_distribution<> distribucion(0, 4);

    for (int pregunta = distribucion(generador) * 5, fin = pregunta + 5; pregunta != fin; ++pregunta)
        std::cout << preguntas[pregunta] << '\n';

    return 0;
}

Puedes ver el código funcionando en Try it online!.

1
  • Muchísimas gracias por tanta info, usaré todo lo que pueda, la verdad que me has ayudado un montón, gracias!!!! Commented el 18 ene. 2021 a las 15:19

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.