[SOLUCIONADO] Problema con tellg() - Archivos

[Ch3Ch0]

Hola, como estan? aqui yo molestando con un nuevo problema jaja, a ver si me pueden ayudar. Bueno, resulta que quiero sacar el tamaño del archivo con tellg y siempre me da -1 (con tellp me da lo mismo), y no le encuentro la vuelta, no se en que puedo estar fallando.
Aca les dejo dejo el codigo

Código C++:

#include <iostream>
#include <fstream>
 
using namespace std;
class prueba {
    int p;
public:
    void escribir();
    void leer();
};
 
void prueba::escribir() {
    cin>>p;
}
void prueba::leer() {
    cout<<"Numero "<<p<<endl;
}
 
 
int main(int argc, char *argv[]) {
    prueba P[3];
    fstream archivo;
    
    archivo.open("probar", ios::in | ios::out | ios::binary);
    archivo.seekg(0);
    
    for(int i=0; i<3;i++){
    cout<<"Numero "<<i+1<<": ";
    P[i].escribir();
    archivo.write((char *) &P, sizeof P);
    }
    archivo.seekg(0);
    cout<<"\n";
    
    for(int i=0; i<3; i++) {
    archivo.read((char *) &P, sizeof P);
    P[i].leer();
    }
    
    int L = sizeof(P);
    
    cout<<"\n";
    
    cout<<"Longitud archivo: "<<L;
    cout<<"\n";
    
    int m;
    cout<<"Ingresar numero de objeto: ";
    cin>>m;
    int pos = (m-1) * L;
    
    //cout.setf(ios::hex, ios::basefield);
    cout<<"Posicion Objeto: "<<pos<<"\n";
    
    
    archivo.seekg(0);
    
    int start = archivo.tellg ();
    cout<<"Start: "<<start<<endl;
    archivo.seekg (0, ios :: end);
    
    int end = archivo.tellg ();
    cout<<"End: "<<end<<endl;
    int size = (end - start);
    cout<<"Size: "<<size<<endl;
    
    
    
    archivo.seekg(0, ios::end);
    long TamTotal = archivo.tellg();
    cout<<"Tamanio Total: "<<TamTotal;
    cout<<"\n";
    int CantObj = TamTotal/L;
    cout<<"Cantidad de Objetos: "<<CantObj;
    
    archivo.close();
    
    return 0;
}
 

Por ahi tambien puede ser que algunas de las sentencias tengan un error logico y no me estoy dando cuenta, lo raro es que siempre me da como resultado -1

[dehm]

Hola:
A ver si alguien te puede dar una explicación más técnica de como se comportan los objetos fstream.
Mientras, lo único que te puedo decir es que seekp() y tellp() son métodos de ostream.
Así que te puede funcionar esto. Es crear un objeto ifstream para la creación y llenado del archivo, y un objeto ofstream para su lectura:

Código C++:

int main(int argc, char *argv[])
{
    prueba P[3];
    ofstream archivo ("probar", ios::out | ios::binary);
 
    for(int i=0; i<3; i++)
        {
            cout<<"Numero "<<i+1<<": ";
            P[i].escribir();
            archivo.write((char *) &P, sizeof P);
        }
    archivo.close();
 
    ifstream archivo2 ("probar", ios::in | ios::binary);
 
    for(int i=0; i<3; i++)
        {
            archivo2.read((char *) &P, sizeof P);
            P[i].leer();
        }
 
    int L = sizeof(P);
 
    cout<<"\n";
 
    cout<<"Longitud archivo: "<<L;
    cout<<"\n";
 
    int m;
    cout<<"Ingresar numero de objeto: ";
    cin>>m;
    int pos = (m-1) * L;
 
    //cout.setf(ios::hex, ios::basefield);
    cout<<"Posicion Objeto: "<<pos<<"\n";
 
 
    archivo2.seekg(0);
 
    int start = archivo2.tellg ();
    cout<<"Start: "<<start<<endl;
    archivo2.seekg (0, ios :: end);
 
    int end = archivo2.tellg ();
    cout<<"End: "<<end<<endl;
    int size = (end - start);
    cout<<"Size: "<<size<<endl;
 
 
 
    archivo2.seekg(0, ios::end);
    long TamTotal = archivo2.tellg();
    cout<<"Tamanio Total: "<<TamTotal;
    cout<<"\n";
    int CantObj = TamTotal/L;
    cout<<"Cantidad de Objetos: "<<CantObj;
 
    archivo2.close();
 
    return 0;
}
 

Siento no ser de más ayuda. Bueno, también decir que hay que acostumbrase a los casting de C++. En este caso, donde pones:

Código C++:

archivo.write((char *) &P, sizeof P);
 

Código C++:

archivo2.read((char *) &P, sizeof P);
 

habría que hacer:

Código C++:

archivo2.write(reinterpret_cast<char *>(&P), sizeof(P));
 

Código C++:

archivo2.read(reinterpret_cast<char *>(&P), sizeof(P));
 

[Ch3Ch0]

Iniciado por dehm

Hola:
A ver si alguien te puede dar una explicación más técnica de como se comportan los objetos fstream.
Mientras, lo único que te puedo decir es que seekp() y tellp() son métodos de ostream.
Así que te puede funcionar esto. Es crear un objeto ifstream para la creación y llenado del archivo, y un objeto ofstream para su lectura:

Código C++:

int main(int argc, char *argv[])
{
    prueba P[3];
    ofstream archivo ("probar", ios::out | ios::binary);
 
    for(int i=0; i<3; i++)
        {
            cout<<"Numero "<<i+1<<": ";
            P[i].escribir();
            archivo.write((char *) &P, sizeof P);
        }
    archivo.close();
 
    ifstream archivo2 ("probar", ios::in | ios::binary);
 
    for(int i=0; i<3; i++)
        {
            archivo2.read((char *) &P, sizeof P);
            P[i].leer();
        }
 
    int L = sizeof(P);
 
    cout<<"\n";
 
    cout<<"Longitud archivo: "<<L;
    cout<<"\n";
 
    int m;
    cout<<"Ingresar numero de objeto: ";
    cin>>m;
    int pos = (m-1) * L;
 
    //cout.setf(ios::hex, ios::basefield);
    cout<<"Posicion Objeto: "<<pos<<"\n";
 
 
    archivo2.seekg(0);
 
    int start = archivo2.tellg ();
    cout<<"Start: "<<start<<endl;
    archivo2.seekg (0, ios :: end);
 
    int end = archivo2.tellg ();
    cout<<"End: "<<end<<endl;
    int size = (end - start);
    cout<<"Size: "<<size<<endl;
 
 
 
    archivo2.seekg(0, ios::end);
    long TamTotal = archivo2.tellg();
    cout<<"Tamanio Total: "<<TamTotal;
    cout<<"\n";
    int CantObj = TamTotal/L;
    cout<<"Cantidad de Objetos: "<<CantObj;
 
    archivo2.close();
 
    return 0;
}
 

Siento no ser de más ayuda. Bueno, también decir que hay que acostumbrase a los casting de C++. En este caso, donde pones:

Código C++:

archivo.write((char *) &P, sizeof P);
 

Código C++:

archivo2.read((char *) &P, sizeof P);
 

habría que hacer:

Código C++:

archivo2.write(reinterpret_cast<char *>(&P), sizeof(P));
 

Código C++:

archivo2.read(reinterpret_cast<char *>(&P), sizeof(P));
 

Hola, Dehm..pense que ya nadie responderia a esta problema que me surgio jaja. Gracias por contestar y por tu ayuda, mañana pruebo con esto que me decis y si me da los resultados deseados o no lo hare saber. Un abrazo

[Ch3Ch0]

Bueno, ahora si me funciono, gracias. Lo que no entiendo bien es porque si tengo un arreglo de, por ej en este caso, tres enteros (4 bytes) el tamaño no deberia darme de 12 bytes en vez de 36? que es el valor que me da el metodo tellg().

[dehm]

Hola de nuevo:

Bueno, es la segunda vez que edito el post, ya que no atinaba con la respuesta correcta.
Aprovecho para corregir también la primera respuesta, ya que intuía que no era correcta.
Bueno, el caso es que creo que el motivo por el que no funcionaba seekg()/tellg() era porque no estaba bien el metodo de escritura, y se activaba el flag de error.
Prueba con estas líneas en tu código:

Código C++:

if (archivo.fail())
    {
        cout<<"Limpiar flags"<<endl;
    }
    archivo.clear();
 

Si hay error, te mostrará el mensaje y se limpiará el flag de error.

Sobre la duda, el problema es que al hacer la escritura, reservas el tamaño de P (12 bytes) y al hacerlo dentro del bucle, reservas un total de 3*12 bytes, que son los 36 bytes.
Creo que éste código corrige los errores, y de paso hace uso de un archivo para lectura/escritura, que es lo que querías y que yo equivocadamente te indiqué que no era posible:

Código C++:

#include <iostream>
#include <fstream>
 
using namespace std;
class prueba
{
    int p;
public:
    void escribir();
    void leer();
};
 
void prueba::escribir()
{
    cin>>p;
}
void prueba::leer()
{
    cout<<"Numero: "<<p<<endl;
}
 
 
int main(int argc, char *argv[])
{
    prueba P[3];
    prueba P2[3];
    
    fstream archivo ("probar", ios::in | ios::out | ios::trunc | ios::binary);
 
    for(int i=0; i<3; i++)
        {
            cout<<"Numero "<<i+1<<": ";
            P[i].escribir();
            archivo.write(reinterpret_cast<char *>(&P[i]), sizeof(prueba));
        }
    archivo.seekg(0);
    archivo.read(reinterpret_cast<char *>(&P2), sizeof(prueba)*3); //archivo.read(reinterpret_cast<char *>(&P2), sizeof(P));
    for(int i=0; i<3; i++)
        {
            P2[i].leer();
        }
 
    int L = sizeof(P);
 
    cout<<"\n";
 
    cout<<"Longitud archivo: "<<L;
    cout<<"\n";
 
    int m;
    cout<<"Ingresar numero de objeto: ";
    cin>>m;
    int pos = (m-1) * sizeof(P[0]);
 
    //cout.setf(ios::hex, ios::basefield);
    cout<<"Posicion Objeto: "<<pos<<"\n";
    if (archivo.fail())
    {
        cout<<"Limpiar flags"<<endl;
    }
    archivo.clear();
 
    archivo.seekg(0L, ios::beg);
    int start = archivo.tellg ();
    cout<<"Start: "<<start<<endl;
 
    archivo.seekg (0L, ios :: end);
    int end = archivo.tellg ();
    cout<<"End: "<<end<<endl;
    int size = (end - start);
    cout<<"Size: "<<size<<endl;
 
    archivo.seekg(0, ios::end);
    long TamTotal = archivo.tellg();
    cout<<"Tamanio Total: "<<TamTotal;
    cout<<"\n";
    int CantObj = TamTotal/L;
    cout<<"Cantidad de Objetos: "<<CantObj;
 
    archivo.close();
 
    return 0;
}
 

[Ch3Ch0]

Iniciado por dehm

Hola de nuevo:

Bueno, es la segunda vez que edito el post, ya que no atinaba con la respuesta correcta.
Aprovecho para corregir también la primera respuesta, ya que intuía que no era correcta.
Bueno, el caso es que creo que el motivo por el que no funcionaba seekg()/tellg() era porque no estaba bien el metodo de escritura, y se activaba el flag de error.
Prueba con estas líneas en tu código:

Código C++:

if (archivo.fail())
    {
        cout<<"Limpiar flags"<<endl;
    }
    archivo.clear();
 

Si hay error, te mostrará el mensaje y se limpiará el flag de error.

Sobre la duda, el problema es que al hacer la escritura, reservas el tamaño de P (12 bytes) y al hacerlo dentro del bucle, reservas un total de 3*12 bytes, que son los 36 bytes.
Creo que éste código corrige los errores, y de paso hace uso de un archivo para lectura/escritura, que es lo que querías y que yo equivocadamente te indiqué que no era posible:

Código C++:

#include <iostream>
#include <fstream>
 
using namespace std;
class prueba
{
    int p;
public:
    void escribir();
    void leer();
};
 
void prueba::escribir()
{
    cin>>p;
}
void prueba::leer()
{
    cout<<"Numero: "<<p<<endl;
}
 
 
int main(int argc, char *argv[])
{
    prueba P[3];
    prueba P2[3];
    
    fstream archivo ("probar", ios::in | ios::out | ios::trunc | ios::binary);
 
    for(int i=0; i<3; i++)
        {
            cout<<"Numero "<<i+1<<": ";
            P[i].escribir();
            archivo.write(reinterpret_cast<char *>(&P[i]), sizeof(prueba));
        }
    archivo.seekg(0);
    archivo.read(reinterpret_cast<char *>(&P2), sizeof(prueba)*3); //archivo.read(reinterpret_cast<char *>(&P2), sizeof(P));
    for(int i=0; i<3; i++)
        {
            P2[i].leer();
        }
 
    int L = sizeof(P);
 
    cout<<"\n";
 
    cout<<"Longitud archivo: "<<L;
    cout<<"\n";
 
    int m;
    cout<<"Ingresar numero de objeto: ";
    cin>>m;
    int pos = (m-1) * sizeof(P[0]);
 
    //cout.setf(ios::hex, ios::basefield);
    cout<<"Posicion Objeto: "<<pos<<"\n";
    if (archivo.fail())
    {
        cout<<"Limpiar flags"<<endl;
    }
    archivo.clear();
 
    archivo.seekg(0L, ios::beg);
    int start = archivo.tellg ();
    cout<<"Start: "<<start<<endl;
 
    archivo.seekg (0L, ios :: end);
    int end = archivo.tellg ();
    cout<<"End: "<<end<<endl;
    int size = (end - start);
    cout<<"Size: "<<size<<endl;
 
    archivo.seekg(0, ios::end);
    long TamTotal = archivo.tellg();
    cout<<"Tamanio Total: "<<TamTotal;
    cout<<"\n";
    int CantObj = TamTotal/L;
    cout<<"Cantidad de Objetos: "<<CantObj;
 
    archivo.close();
 
    return 0;
}
 

Hola, dehm, primero que nada perdon que no haya respondido antes pero gracias por responder y darme la solucion correcta. Lo unico que me quede pensando es en porque declaraste dos arreglos,
yo probe el codigo con un solo arreglo y metiendo archivo.read(reinterpret_cast<char *>(&P[i]), sizeof(prueba)*3); de esta forma dentro del bucle for adonde se encuentra .leer() y me funciono igual,
lo que no se es si lo que hice es lo correcto pero en si mi pregunta es porque dos arreglos y no uno.
Por lo demas ninguna duda y por cierto si no se puede volver a responder a un post despues de un tiempo(me ausente un par de meses) pido disculpas y haganmelo saber. Saludos

[dehm]

Hola:

No es necesario declarar un arreglo aparte para volcar los datos.
Lo que pasa es que si usas el mismo arreglo para introducir datos, volcarlos al fichero y después recuperarlos del fichero, puede ocurrir que no sepamos si los datos que lees después de esas operaciones son los originales o realmente se ha producido una transferencia efectiva de información.
Volcando los datos del fichero a un arreglo diferente tienes la seguridad de que los datos que estas leyendo en P2 son los del fichero y no los que estaban originalmente.

Pero ese es el único motivo, no hay ninguna obligación ni tema técnico.
Saludos!