c++ - sirve - La función de lectura de Ifstream no se carga en el vector

manejo de archivos en c++ fstream

Question

Soy un poco nuevo en la programación, así que no estoy seguro de cómo buscar este problema, y ​​sé que hice otras 2 preguntas sobre esto, pero parece que no puedo hacer que funcione.

Tengo un problema, donde tengo un vector:

vector<Device*> Devicelist_;

Donde intento cargar Dispositivos para usar esta función (ya hice una función Guardar, que funciona):

bool Devicelist::LoadFromFile() //Opdaterer vector<Device> fra fil
{
    ifstream LoadFile("Devices.dat", ios::in | ios::binary);

    if (!LoadFile)
    {
        cerr << "File could not be opened." << endl;
        return false;
    }

    LoadFile.seekg(0, ios::end);
    int numberOfDevices = LoadFile.tellg() / sizeof(Device);

    for (int i = 0; i < numberOfDevices; i++)
    {   
        Devicelist_.push_back(new Device);
        LoadFile.read(reinterpret_cast<char *>(Devicelist_[i]), sizeof(Device));
    }

    cout << Devicelist_[0]->getName() << endl;

    LoadFile.close();
    return true;
}

El problema es que LoadFile.read() no carga ningún dispositivo en la lista de proveedores.

¿Puedes ver cuál es mi problema? Gracias por adelantado.

Answer 1

Esta línea

LoadFile.seekg(0, ios::end);

pone el archivo al final del archivo. Tienes que volver a ponerlo al principio

LoadFile.seekg(0, ios::beg);

Actualizar

Puede simplificar el código guardando la cantidad de Device s en la parte superior del archivo. Entonces, puedes usar:

int numberOfDevices = 0;
LoadFile.read(&numberOfDevies, sizeof(int));

for (int i = 0; i < numberOfDevices; i++)

No será necesario obtener el tamaño del archivo para deducir el número de Device .

La otra alternativa es la sugerida por @ phyrrus9 donde continúa leyendo los Device hasta que no haya más Device para leer del archivo.

Answer 2

Tu problema es realmente muy simple. Olvidó restablecer su posición de obtención:

LoadFile.seekg(0, ios::end);
int numberOfDevices = LoadFile.tellg() / sizeof(Device);

for (int i = 0; i < numberOfDevices; i++)

debiera ser

LoadFile.seekg(0, ios::end);
int numberOfDevices = LoadFile.tellg() / sizeof(Device);
LoadFile.seekg(0L, ios::beg);
for (int i = 0; i < numberOfDevices; i++)

Una alternativa para encontrar el número es usar estadísticas:

#include <sys/stat.h>
int getNumberOfDevices(char *filename)
{
    struct stat st;
    return st.st_size / sizeof(Device);
}

o, si desea evitar la estadística, podría hacer algo como esto:

bool Devicelist::LoadFromFile() //Opdaterer vector<Device> fra fil
{
    ifstream LoadFile("Devices.dat", ios::in | ios::binary);
    if (!LoadFile)
    {
        cerr << "File could not be opened." << endl;
        return false;
    }
    int numberOfDevices = 0;
    while (true)
    {   
        Device *tmp = new device; 
        LoadFile.read(reinterpret_cast<char *>(tmp), sizeof(Device));
        if (LoadFile.good()) //we successfully read one
        {
            ++numberOfDevices;
            Devicelist_.push_back(tmp);
        }
        else break; //get out of the infinite loop
    }
    cout << Devicelist_[0]->getName() << endl;
    LoadFile.close();
    return true;
}

De esta forma, los lee a todos, sin perder posiciones, y lleva un conteo cuando termina.