0

Estoy intentando hacer un programa en C++ que mande 100 números aleatorio a un archivo, después los recoja y obtenga el número mayor y el número menor. He intentado hacerlo introduciendo los cien números en un vector y sacando el máximo y el mínimo. El archivo lo crea bien y manda los números correctamente, pero a la hora de recogerlos genera números muy extraños. He intentado pasar la parte en la que el vector lo ordena al main, pero eso me ha generado más problemas aun, por eso decidí dejarlo ahí.

#include <iostream>
#include <stdlib.h>
#include <fstream>
#include <time.h>
using namespace std;

void escritura();
void lectura();

int main()
{
    escritura();
    lectura();
    system("pause");
    return 0;
}

void escritura()
{
    int num, c;
    ofstream archivo;
    archivo.open("numeros.txt", ios::out);

    if (archivo.fail()) {
        cout << "No se puede abrir el archivo";
        exit(1);
    }
    srand(time(NULL));
    for (c = 0; c <= 99; c++) {
        num = 0 + rand() % (201 - 1);
        archivo << num << endl;
    }
    archivo.close();
}

void lectura()
{
    int i = 0, numero[100];
    ifstream archivo;

    archivo.open("numeros.txt", ios::in);

    if (archivo.fail()) {
        cout << "No se puede abrir el archivo";
        exit(1);
    }

    i = 0;
    archivo >> numero[i];
    while (!archivo.eof()) { //Mientras no sea el final del archivo
        {
            i++;
            archivo >> numero[i];
        }
        archivo.close();

        int mayor, menor;
        mayor = numero[0]; //Le asignamos el primer elemento del array
        menor = numero[0]; //Así empezamos a comparar

        for (i = 0; i < 100; i++) {
            if (numero[i] > mayor) {
                mayor = numero[i];
            }
            if (numero[i] < menor) {
                menor = numero[i];
            }
        }
        printf("El mayor es %d\n", mayor);
        printf("El menor es %d\n", menor);
    }
}
0

2 respuestas 2

3

No has inicializado la formación int numero[100]; por lo que tendrá valores residuales de memoria, si añades unas llaves vacías a la formación, todos sus elementos se inicializarán a cero:

int numero[100]{}; // Cien elementos a cero.

Otras cosas a tener en cuenta.

  • Las cabeceras <stdlib.h> y <time.h> son cabeceras del lenguaje C; si realmente necesitas usarlas (que no es el caso) deberías usar la versión adaptada a C++ que es <cstdlib> y <ctime> respectivamente. Lee este hilo para saber más del tema.
  • La herramienta de generación de números aleatorios rand está considerada como ineficiente e insegura, hasta el punto que se valora deprecarla. Lee este hilo para saber más del tema.
    • Por otro lado, dicha herramienta genera números distribuidos uniformemente en el rango de cero a RAND_MAX si haces módulo (%) sobre un valor que no sea divisor de RAND_MAX, falsearás la distribución. Usa la cabecera C++ <random> para generar números pseudoaleatorios adecuadamente.
  • No abuses de la cláusula using namespace std; si realmente quieres usarla, hazlo en el ámbito más pequeño posible.
  • Los flujos de salida de datos a archivo (std::ofstream) ya son de salida, no necesitas especificar al abrirlos que pretendes sacar datos (ios::out).
  • Los flujos de entrada de datos a archivo (std::ifstream) ya son de entrada, no necesitas especificar al abrirlos que pretendes leer datos (ios::in).
  • No debes comprobar en el bucle while si has alcanzado el final del archivo ya que leerás un dato de más. Lee este hilo para saber más del tema.
  • Favorece el preincremento frente al postincremento. Lee este artículo para saber más del tema.
  • En C++ la salida por consola se hace mediante std::cout, la función printf es de C.

Propuesta.

Con los consejos anteriores, tu código podría parecerse a:

#include <iostream>
#include <random>
#include <fstream>

void escritura()
{
    /* Los flujos de datos con implícitamente convertibles a booleano,
    si el flujo no se puede abrir, la expresión será 'false'. */
    if (std::ofstream archivo{"numeros.txt"})
    {
        std::random_device device;
        std::mt19937 generador(device());
        /* Generador de números aleatorios distribuidos
        uniformemente entre 0 y 99. */
        std::uniform_int_distribution<> distribucion(0, 99);

        for (int i = 0; i != 100; ++i)
            archivo << distribucion(generador) << '\n';
    }
    else
        std::cout << "No se puede abrir el archivo.\n";
}

void lectura()
{
    // Inicializamos todos los números de la formación a cero.
    int numero[100]{};

    if (std::ifstream archivo{"numeros.txt"})
    {
        for (int i = 0; archivo && i != 100; ++i)
            archivo >> numero[i];
    }
    else
        std::cout << "No se puede abrir el archivo.\n";

    int mayor = numero[0], menor = numero[0];

    for (int i = 0; i != 100; ++i)
    {
        mayor = mayor < numero[i] ? numero[i] : mayor;
        menor = menor > numero[i] ? numero[i] : menor;
    }

    std::cout << "El mayor es " << mayor << '\n'
        << "El menor es " <<  menor << '\n';
}

int main()
{
    escritura();
    lectura();
    return 0;
}
2

Estas ubicando mal archivo.close(); en la función lectura, está dentro del bloque while, esto hace que no termine de leer completamente el archivo ya que se estaría cerrando apenas leído el segundo elemento, entonces archivo.close(); debería estar fuera de este bloque, además, el bloque while debería terminar antes de las declaraciones int mayor, menor;, incluso, si te das cuenta el bloque for está anidado dentro del while y no debería estar así, la función lectura corregida sería:

void lectura() {
    int i = 0, numero[100];
    ifstream archivo;

    archivo.open("numeros.txt", ios::in);

    if (archivo.fail()) {
        cout << "No se puede abrir el archivo";
        exit(1);
    }

    i = 0;
    archivo >> numero[i];
    while (!archivo.eof()) //Mientras no sea el final del archivo
    {
        i++;
        archivo >> numero[i];
    }//aquí termina el bloque while
    archivo.close();

    int mayor, menor;
    mayor = numero[0]; //Le asignamos el primer elemento del array
    menor = numero[0]; //Así empezamos a comparar

    for (i = 0; i < 100; i++) {
        if (numero[i] > mayor) {
            mayor = numero[i];
        }
        if (numero[i] < menor) {
            menor = numero[i];
        }
    }
    printf("El mayor es %d\n", mayor);
    printf("El menor es %d\n", menor);
}

Bueno, no soy un experto en eso de las buenas prácticas de programación pero creo deberías evitar mezclar cabeceras de C y C++, en este caso puedes obviar #include <stdlib.h> y cambiar time.h por ctime, puedes encontrar más detalles aquí.

6
  • No, el problema no es el que describes si no que no ha inicializado la formación. Commented el 25 feb. 2020 a las 23:52
  • Así como está compiló y funcionó, he usado Visual C++ 2017
    – 4lrdyD
    Commented el 26 feb. 2020 a las 0:05
  • El compilador que usaste no tiene nada que ver con que el problema sea diferente al que mencionas. Commented el 26 feb. 2020 a las 0:08
  • Pues el hecho de que archivo.close(); debía estar fuera del bloque while así como el bloque for en la función lectura, creo que no me equivoque con eso, ¿o a que te refieres?.
    – 4lrdyD
    Commented el 26 feb. 2020 a las 0:13
  • Me refiero a que no inicializó la formación, por lo tanto la formación obtuvo valores residuales de memoria. Ese es el origen de los "números muy extraños" que menciona Edurex. El cerrar el archivo en el bucle no provoca la aparición de esos números, provoca que no se lea el archivo al completo. Commented el 26 feb. 2020 a las 0:17

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.