Skip to main content
2 de 3
se añadieron 2 caracteres en el cuerpo
PaperBirdMaster
  • 48.1k
  • 6
  • 45
  • 86

Antes de proponerte una solución, prefiero responder por qué no te funciona tu código:

size_t sz = sizeof (tableroEP);
songsList.write(reinterpret_cast<const char*>(&sz), sizeof(sz));
songsList.write(reinterpret_cast<const char*>(&tableroEP[0]), sz * sizeof(tableroEP[0]));

De momento la puedo guardar así, pero como guardo el objeto, el archivo no tiene algo legible.

En ese código tienes una cosa bien y una cosa mal. ###Datos y contenedor no son lo mismo. Cuando llamas al operador de indexado (operator[]) de tu clase plantilla Tablero estás devolviendo una referencia a un vector interno y pretendes escribir este vector en el archivo. Para hacerte una idea de cuán equivocada es la idea, es como si para saber lo cargado que va un camión de mudanzas pesaras el camión entero cuando quieres pesar sólo el contenido del remolque.

Cometes el error mencionado dos veces, la primera al considerar que el tamaño de tus datos es el tamaño del tablero (el tamaño del camión no es el tamaño del contenido de su remolque) y la segunda al considerar el vector como el dato a almacenar.

Si guardas el vector como en tu código estarás guardando la representación binaria del mismo, no los datos que maneja. Suponiendo que tu vector sea el std::vector de la STL, puedes acceder al puntero a los datos almacenados mediante la función std::vector::data ; eso es lo que quieres guardar:

auto tamanyo = tableroEP[0].size() * sizeof(T);
songsList.write(reinterpret_cast<const char*>(tableroEP[0].data()), tamanyo);

###La función write escribe datos en crudo. La función std::basic_ostream::write te permite escribir datos en binario (en realidad guarda caracteres, pero puede usarse para guardar la representación binaria de un objeto), así que no esperes abrir el objeto y ver algo legible a no ser que uses un visor hexadecimal. ###Propuesta. Para empezar, olvida la memoria dinámica en tu clase Tablero, además de no necesitarla la pides en tus constructores pero no la liberas en ningún destructor. Crea los vectores internos usando el segundo constructor de std::vector :

template<typename T>
struct Tablero
{
    using fila_t = std::vector<T>;
    using tabla_t = std::vector<fila_t>;

    Tablero() : Tablero(3, 3) {}
    //          ~~~~~~~~~~~~~ <--- Constructor delegado
    Tablero(int filas, int columas) :
        filas_(filas), columnas_(columnas),
        matriz(filas, fila_t(columnas))
    {}
    Tablero(int filas, int columas, const T &dato) :
        filas_(filas), columnas_(columnas),
        matriz(filas, fila_t(columnas, dato))
    {}

    tabla_t &getMatriz();
    fila_t &operator[](int fila);

private:
    tabla matriz;
    int filas_;
    int columnas_;
};

Con esta aproximación evitarás el tedio de manejar la memoria manualmente y también evitarás el problema de generar fugas de memoria como estabas generando.

En cuanto a escritura y lectura en archivo, te aconsejo crear unas funciones save y load:

template<typename T>
struct Tablero
{
    void save(const std::string file_name)
    {
        if (std::ofstream file{file_name})
        {
            // Guardar filas
            file.write(reinterpret_cast<const char *>(&filas_), sizeof(filas_));
            // Guardar columnas
            file.write(reinterpret_cast<const char *>(&columnas_), sizeof(columnas_));
            // Guardar datos
            for (const auto &fila : matriz)
                file.write(reinterpret_cast<const char *>(fila.data()), fila.size() * sizeof(T));
        }
    }
    void load(const std::string file_name)
    {
        if (std::ifstream file{file_name})
        {
            // Leer filas
            file.read(reinterpret_cast<char *>(&filas_), sizeof(filas_));
            // Leer columnas
            file.read(reinterpret_cast<char *>(&columnas_), sizeof(columnas_));
            // Reajustar tamaños
            matriz.resize(filas, fila_t(columnas_));
            for (auto &fila : matriz) fila.resize(columnas_);
            // Leer datos
            for (auto &fila : matriz)
                for (auto &dato : fila)
                    file.read(reinterpert_cast<char *>(&dato, sizeof(T));
        }
    }
}
PaperBirdMaster
  • 48.1k
  • 6
  • 45
  • 86