VPF::Неразбериха с файлами - Форум программистов

NoNo

Дата 28.6.2007, 16:07 (ссылка)

(нет голосов)

Загрузка ...

Шустрый

Профиль
Группа: Участник
Сообщений: 109
Регистрация: 14.11.2006

Репутация: нет
Всего: нет

Вот код:

Код


#include <iostream>
#include <conio.h>
#include <fstream>
using namespace std;
class person
{
    private:
        char name[40];
        int age;
    public:
        void getdata()
        {
            cout << "\nВведите имя: ";
            cin >> name;
            cout << "Введите возраст: ";
            cin >> age;
        }
        void showdata()
        {
            cout << "\nИмя: " <<name;
            cout << "\nВозраст: " << age;
        }
        void getin(int pn)
        {
            ifstream infile;
            infile.open("person.dat", ios :: binary);
            infile.seekg(pn * sizeof(*this));
            infile.read((char*)this, sizeof(*this));
        }
        void getout()
        {
            ofstream outfile;
            outfile.open("person.dag", ios :: binary);
            outfile.write((char*)this, sizeof(*this));
        }
};
int main()
{
    person p;
    for (int i = 0; i < 3; i++) {
        p.getdata();
        p.getout();
    }
    for (int i = 0; i < 3; i++) {
        p.getin(i);
        p.showdata();
    }
    getch();
    return 0;
}

мне кажется что то с person :: getin(int pn) намудрил smile

в результате выводит три раза последнюю запись

jonie

Дата 28.6.2007, 16:33 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 5613
Регистрация: 21.8.2005
Где: Владимир

Репутация: 15
Всего: 118

скорее с getout().
пишешь поверх всегда (т.е. записываешь только "НА" первую запись).
надоб смещаться для дозаписи.
и вобще не очень корректно писать (*this) - там мб выравнивания и таблица вирт. функций - получишь ошибки в runTime (ну 2-ое при условии что дальше развите пояивтся с виртуальностью ессно)...
ну а еще файлы называй одинаково - а то пишешь в один, читаешь с другого...

да и на ошибки тоже проверять стоит.

Это сообщение отредактировал(а) jonie - 28.6.2007, 16:57

--------------------

Что-то не поняли? -> Напейтесь до зеленых человечков... эта сверхцивилизация Вам поможет...

apook

Дата 28.6.2007, 17:15 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 794
Регистрация: 12.7.2006

Репутация: 9
Всего: 23

в твоем случае просто нет файла с которого надо читать
и читаются данные которые в данный момент содержат переменные name и age,
а точнее данные введенне последними!

Пока выяснял код слегка изменил:

Код


#include <iostream.h>
#include <conio.h>
#include <fstream.h>

using namespace std;



creatf( char *name )
{
ofstream f( name, ios::out );
f.close();
}


class person
{
    private:
        struct
        {
            char name[ 40 ];
            int age;
            } data;

    public:
        person();
        ~person();
        void getdata()
        {
            cout << "\nВведите имя: ";
            cin >> this->data.name;
            cout << "Введите возраст: ";
            cin >> this->data.age;
        }

        void showdata()
        {
            cout << "\nИмя: " << this->data.name;
            cout << "\nВозраст: " << this->data.age;
        }

        void getin( int pn )
        {
            file.seekg( pn*sizeof(this->data), ios::beg );
            file.read( (char *)&this->data, sizeof(this->data) );
        }

        void getout()
        {
            file.seekp( 0, ios::end );
            file.write((char *)&this->data, sizeof(this->data));
        }

protected:
    fstream file;

    };

person::person()
{
creatf( ".\\person.dat" );
file.open( ".\\person.dat", ios::in | ios::out | ios::binary );
}

person::~person()
{
file.close();
}


int main()
{
int i;
person p;

    for ( i = 0; i < 3; i++ ) {
        p.getdata();
        p.getout();
    }
    for (int i = 0; i < 3; i++)
    {
        p.getin( i );
        p.showdata();
        }


    getch();
    return 0;
}

--------------------

Мои руки из дуба, голова из свинца ну и пусть ...

jonie

Дата 28.6.2007, 18:39 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 5613
Регистрация: 21.8.2005
Где: Владимир

Репутация: 15
Всего: 118

и даже обернув в структуру проблема записи байт выравнивающих не решается 8)
можно принудительно заставить большинство компиляторов сделать выравнивание каким надо (может так статься, что данные будут записанные скажем с выравниванием 8 байт (умолчание), а считыватся с выравниваеним 1 байт).
для MSVCpp и gcc выравнивание ставится как

Код


#pragma pack(push)
#pragma pack(2)  //например 2 байта
/* 
 *тут структура ваша 
 */
#pragma pack(pop)

не надо ржать - это реальная проблема, и баг искать будете долго случись такое (из своего опыта)...

--------------------

Что-то не поняли? -> Напейтесь до зеленых человечков... эта сверхцивилизация Вам поможет...

NoNo

Дата 29.6.2007, 09:48 (ссылка)

(нет голосов)

Загрузка ...

Шустрый

Профиль
Группа: Участник
Сообщений: 109
Регистрация: 14.11.2006

Репутация: нет
Всего: нет

Цитата

ну а еще файлы называй одинаково - а то пишешь в один, читаешь с другого...

насчёт файла я тут ошибся когда сообщения писал.....просто листинг программы сократил, выкинул лишнии методы и процедуру.....
на ошибки прога моя тоже проверяет всё нормально....
а насчёт функции getout() ты прав, в ней ошибка была....
спасибо) smile

Добавлено через 9 минут и 16 секунд

Цитата

Пока выяснял код слегка изменил:

ок, списибо) всё нормально работает.....

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| C/C++: Общие вопросы \| Следующая тема »