Форум программистов, компьютерный форум CyberForum.ru

Не могу найти ошибку. Запись/чтение из файла - C++

Восстановить пароль Регистрация
 
 
murtukov
 Аватар для murtukov
8 / 8 / 3
Регистрация: 30.01.2013
Сообщений: 99
06.10.2013, 00:45     Не могу найти ошибку. Запись/чтение из файла #1
Простите за довольно таки большой код, но обратиться больше некуда. Урезать не получилось, постараюсь на словах дать общую концепцию:

Есть абстрактный базовый класс Person и два его наследника: Student и Laborer.
Также есть класс Database, который хранит в себе эти объекты с помощью контейнера std::list<Person const *>.

Вот пример кода, работающий без ошибок:

Student s1; - создать объект
s1.askData(); - запросить данные у пользователя

Laborer l1; - создать объект
l1.askData(); - запросить данные у пользователя

Database db; - создать базу объектов
db.add(&s1); - добавить s1 в базу
db.add(&l1); - добавить l1 в базу

db.printList(); - вывести все данные всех объектов из базы

Так же в базу можно добавлять объекты сразу, без предварительного создания объектов:

Объект определяется количеством аргументов
db.add("Martin", 311017); - добавляет в базу объект Student с именем Martin и ID 311017
db.add("Klark", "4943334325", "14B"); - добавляет в базу объект Laborer с именем Klark, номером телефона и бюро.

Вы наверное спросите себя, зачем я это все здесь пишу, если все работает? Проблема в том, что надо реализовать сохранение всей базы db в файл и считывание из этого файла. Код мною уже написан, но не работает.

Метод сохраняющий базу называется void Database::saveToFile();
Для считывания есть метод void Database::readFromFile();

Собственно ошибка кроется где-то в них. Файл создается и информация вроде как сохраняется, но вот считать не получается. Прошу помочь разобраться.

Полный код программы:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
#include <iostream>
#include <list>
#include <typeinfo>
#include <stdlib.h>
#include <fstream>
using namespace std;
 
class Person;
class Student;
class Laborer;
typedef list<Person const *> Perslist;
 
enum PersType { tstudent, tLaborer };
////////////////////////////////////////////////////////////////
class Person
{
protected:
    string name;
 
public:
    Person()
    {}
 
    Person(const char *n)
    { name = n; }
 
    virtual ~Person()
    {}
 
    string retName() const
    { return name; }
 
    void getName(const char *n)
    { name = n; }
 
    void askData()
    { cout << "  Enter a name: "; cin >> name; }
 
    void printStream(ostream& os) const
    { os << name << endl; }
 
    virtual void print() const
    { cout << "  Name: " << name << endl; }
 
    bool operator == (string s)
    {
        if(name == s)
            return 1;
        else
            return 0;
    }
 
    virtual PersType getType() const;
 
    virtual void pure() = 0; // pure virtual func
};
////////////////////////////////////////////////////////////////
class Student : public Person
{
    unsigned int matricul;
 
public:
    Student() {}
 
    Student(string const n, unsigned int m)
    {
        Person::name = n;
        matricul = m;
    }
 
    void askData()
    {
        Person::askData();
        cout << "  Enter a matriculation: ";
        cin >> matricul;
    }
 
    void printStream(ostream& os) const
    {
        Person::printStream(os);
        cout << matricul << endl << endl;
    }
 
    void print() const
    {
        Person::print();
        cout << "  Matriculation: " << matricul << endl << endl;
    }
    void pure() {}
};
////////////////////////////////////////////////////////////////
class Laborer : public Person
{
    string tel;
    string buro;
public:
    Laborer() {}
 
    Laborer(string const n, string const t,
                string const b)
    {
        Person::name = n;
        tel = t;
        buro = b;
    }
 
    void askData()
    {
        Person::askData();
        cout << "  Enter a tel: ";
        cin >> tel;
        cout << "  Enter a buro: ";
        cin >> buro;
    }
 
    void printStream(ostream& os)
    {
        Person::printStream(os);
        os << "  Telefon: " << tel << endl
           << "  Buro: " << buro << endl << endl;
    }
 
    void print() const
    {
        Person::print();
        cout << "  Telefon: " << tel << endl
             << "  Buro: " << buro << endl << endl;
    }
 
    void pure() {}
};
////////////////////////////////////////////////////////////////
class Database
{
    Perslist plist;
 
public:
    ~Database()
    {
        plist.clear();
    }
 
    void add(Person const *p)
    {
        plist.push_back(p);
    }
 
    void add(string const name, unsigned const matr)
    {
        Student *s = new Student(name, matr);
        plist.push_back(s);
    }
 
    void add(string const name, string const tel,
             string const buro)
    {
        Laborer *m = new Laborer(name, tel, buro);
        plist.push_back(m);
    }
 
    void printList()
    {
        Perslist::iterator it = plist.begin();
        for (int i=0; i < plist.size(); ++i, ++it)
        {
            (*it)->print();
        }
    }
 
    const Person* find(string const n)
    {
 
        Perslist::iterator it = plist.begin();
        for (int i=0; i < plist.size(); ++i, ++it)
        {
            if( (*it)->retName() == n )
            {
                return *it;
            }
        }
    }
 
    int howMuchS()
    {
        int n = 0;
        string stud = "7Student";
        Perslist::iterator it = plist.begin();
        for(int i=0; i < plist.size(); ++i, ++it)
        {
            if(typeid(**it).name() == stud)
                ++n;
        }
        return n;
    }
 
    void saveToFile(const char *filename)
    {
        PersType ptype;
        int size;
        fstream ofile;
        ofile.open(filename, ios::app | ios::out |
                                        ios::binary);
 
        Perslist::iterator it = plist.begin();
 
        for(int i=0; i < plist.size(); ++i, ++it)
        {
            ptype = (*it)->getType();
            ofile.write((char*)&ptype, sizeof(ptype));
 
            switch(ptype)           // найти его размер
            {
                case tstudent:   size  = sizeof(Student); break;
                case tLaborer: size  = sizeof(Laborer); break;
            }
            ofile.write((char*)*it, size);
        }
        ofile.close();
 
    }
 
    void readFromFile(const char *filename)
    {
        PersType ptype;
        int size;
        fstream ifile;
        Person *p;
 
        ifile.open(filename, ios::binary);
 
        Perslist::iterator it = plist.begin();
        Perslist::iterator endit = plist.end();
        while(it != endit)
        {
            ifile.read((char*)&ptype, sizeof(ptype));
 
            switch(ptype)
            {
            case tstudent:
                p = new Student;
                size = sizeof(Student);
                ifile.read((char*)p, size);
                plist.push_back(p);
                break;
            case tLaborer:
                p = new Laborer;
                size = sizeof(Laborer);
                ifile.read((char*)p, size);
                plist.push_back(p);
                break;
            default:
                cout << "Unknown format" << endl;
            }
        }
        ifile.close();
    }
};
////////////////////////////////////////////////////////////////
 PersType Person::getType() const
 {
        if(typeid(*this) == typeid(Student))
            return tstudent;
        else if(typeid(*this) == typeid(Laborer))
            return tLaborer;
        else
        {
            cerr << "\nWrong type of Person";
            exit(1);
        }
}
//--------------------------------------------------------------
bool operator == (string& s, Person& p)
{
    if(s == p.retName())
        return 1;
    else
        return 0;
}
//--------------------------------------------------------------
void operator << (ostream& os, Person& p)
{
    os << p.retName();
}
////////////////////////////////////////////////////////////////
int main()
{
    Database db;
 
    db.add("Kristina", 241089);
    db.add("Martina", 311017);
    db.add("Kirill", 1111);
    db.add("Harold", "0170-4081883", "Vertrieb");
 
    db.saveToFile("Proverka.dat");
 
    db.printList();
    return 0;
}
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
06.10.2013, 00:45     Не могу найти ошибку. Запись/чтение из файла
Посмотрите здесь:

Запись и чтение с файла C++
C++ запись\чтение из файла
Чтение/запись файла C++
C++ НЕ могу найти ошибку.Чтение из файла
Запись и чтение из файла C++
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
alsav22
5282 / 4801 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
06.10.2013, 00:50     Не могу найти ошибку. Запись/чтение из файла #2
Если в объектах, которые пишите в файл в бинарном режиме, есть объекты (типа string), то ничего не получится. Объект string и строка, с которой он связан - не одно и тоже. Пишите поля в текстовом режие.
murtukov
 Аватар для murtukov
8 / 8 / 3
Регистрация: 30.01.2013
Сообщений: 99
06.10.2013, 00:57  [ТС]     Не могу найти ошибку. Запись/чтение из файла #3
alsav22,
Как тогда сохранить объект с полем типа string в файл?
Что вы подразумеваете под "Пишите поля в текстовом режиме"?
alsav22
5282 / 4801 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
06.10.2013, 00:59     Не могу найти ошибку. Запись/чтение из файла #4
Цитата Сообщение от murtukov Посмотреть сообщение
Что вы подразумеваете под "Пишите поля в текстовом режиме"?
Не бинарные методы чтения\записи: <<, >>, getline() и пр. Какой выбрать - сами смотрите, в код не вникал.
murtukov
 Аватар для murtukov
8 / 8 / 3
Регистрация: 30.01.2013
Сообщений: 99
06.10.2013, 01:22  [ТС]     Не могу найти ошибку. Запись/чтение из файла #5
alsav22, я заменил все строки типа string на char, ничего не изменилось, копирование не работает. Дело в ошибочном коде
alsav22
5282 / 4801 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
06.10.2013, 01:32     Не могу найти ошибку. Запись/чтение из файла #6
Цитата Сообщение от murtukov Посмотреть сообщение
Дело в ошибочном коде
Понятно, что в коде (в чём же ещё?). Ошибка может быть и не одна. Я же написал, что в код не вникал. Покажите как сделали.
murtukov
 Аватар для murtukov
8 / 8 / 3
Регистрация: 30.01.2013
Сообщений: 99
06.10.2013, 01:40  [ТС]     Не могу найти ошибку. Запись/чтение из файла #7
alsav22,

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
#include <iostream>
#include <list>
#include <typeinfo>
#include <stdlib.h>
#include <fstream>
#include <string.h>
using namespace std;
 
class Person;
class Student;
class Mitarbeiter;
typedef list<Person const *> Perslist;
 
enum PersType { tstudent, tmitarbeiter };
////////////////////////////////////////////////////////////////
class Person
{
protected:
    char name[100];
 
public:
    Person()
    {}
 
    Person(const char *n)
    {
        strcpy(name, n);
    }
 
    virtual ~Person()
    {}
 
    const char* retName() const
    {
        return name;
    }
 
    void getName(const char *n)
    {
        strcpy(name, n);
    }
 
    void askData()
    {
        cout << "  Enter a name: ";
        cin >> name;
    }
 
    void printStream(ostream& os) const
    {
        os << name << endl;
    }
 
    virtual void print() const
    {
        cout << "  Name: " << name << endl;
    }
 
    bool operator == (string s)
    {
        if(name == s)
            return 1;
        else
            return 0;
    }
 
    virtual PersType getType() const;
 
    virtual void pure() = 0; // pure virtual func
};
////////////////////////////////////////////////////////////////
class Student : public Person
{
    unsigned int matrikel;
 
public:
    Student() {}
 
    Student(char const *n, unsigned int m)
    {
        strcpy(Person::name, n);
        matrikel = m;
    }
 
    void askData()
    {
        Person::askData();
        cout << "  Enter a matriculation: ";
        cin >> matrikel;
    }
 
    void printStream(ostream& os) const
    {
        Person::printStream(os);
        cout << matrikel << endl << endl;
    }
 
    void print() const
    {
        Person::print();
        cout << "  Matriculation: " << matrikel << endl << endl;
    }
    void pure() {}
};
////////////////////////////////////////////////////////////////
class Mitarbeiter : public Person
{
    char tel[100];
    char buro[100];
public:
    Mitarbeiter() {}
 
    Mitarbeiter(char const *n, char const *t,
                char const *b)
    {
        strcpy(Person::name, n);
        strcpy(tel, t);
        strcpy(buro, b);
    }
 
    void askData()
    {
        Person::askData();
        cout << "  Enter a tel: ";
        cin >> tel;
        cout << "  Enter a buro: ";
        cin >> buro;
    }
 
    void printStream(ostream& os)
    {
        Person::printStream(os);
        os << "  Telefon: " << tel << endl
           << "  Buro: " << buro << endl << endl;
    }
    void print() const
    {
        Person::print();
        cout << "  Telefon: " << tel << endl
             << "  Buro: " << buro << endl << endl;
    }
 
    void pure() {}
};
////////////////////////////////////////////////////////////////
class Database
{
    Perslist plist;
 
public:
    ~Database()
    {
        plist.clear();
    }
 
    void add(Person const *p)
    {
        plist.push_back(p);
    }
 
    void add(char const *name, unsigned const matr)
    {
        Student *s = new Student(name, matr);
        plist.push_back(s);
    }
 
    void add(char const *name, char const *tel,
             char const *buro)
    {
        Mitarbeiter *m = new Mitarbeiter(name, tel, buro);
        plist.push_back(m);
    }
 
    void printList()
    {
        Perslist::iterator it = plist.begin();
        for (int i=0; i < plist.size(); ++i, ++it)
        {
            (*it)->print();
        }
    }
 
    const Person* find(string const n)
    {
 
        Perslist::iterator it = plist.begin();
        for (int i=0; i < plist.size(); ++i, ++it)
        {
            if( (*it)->retName() == n )
            {
                return *it;
            }
        }
    }
 
    int howMuchS()
    {
        int n = 0;
        string stud = "7Student";
        Perslist::iterator it = plist.begin();
        for(int i=0; i < plist.size(); ++i, ++it)
        {
            if(typeid(**it).name() == stud)
                ++n;
        }
        return n;
    }
 
    void saveToFile(const char *filename)
    {
        PersType ptype;
        int size;
        fstream ofile;
        ofile.open(filename, ios::app | ios::out |
                                        ios::binary);
 
        Perslist::iterator it = plist.begin();
 
        for(int i=0; i < plist.size(); ++i, ++it)
        {
            ptype = (*it)->getType();
            ofile.write((char*)&ptype, sizeof(ptype));
 
            switch(ptype)           // найти его размер
            {
                case tstudent:   size  = sizeof(Student); break;
                case tmitarbeiter: size  = sizeof(Mitarbeiter); break;
            }
            ofile.write((char*)*it, size);
        }
        ofile.close();
 
    }
 
    void readFromFile(const char *filename)
    {
        PersType ptype;
        int size;
        fstream ifile;
        Person *p;
 
        ifile.open(filename, ios::binary);
 
        Perslist::iterator it = plist.begin();
        Perslist::iterator endit = plist.end();
        while(it != endit)
        {
            ifile.read((char*)&ptype, sizeof(ptype));
 
            switch(ptype)
            {
            case tstudent:
                p = new Student;
                size = sizeof(Student);
                ifile.read((char*)p, size);
                plist.push_back(p);
                break;
            case tmitarbeiter:
                p = new Mitarbeiter;
                size = sizeof(Mitarbeiter);
                ifile.read((char*)p, size);
                plist.push_back(p);
                break;
            default:
                cout << "Unknown format" << endl;
            }
        }
        ifile.close();
    }
};
////////////////////////////////////////////////////////////////
 PersType Person::getType() const
 {
        if(typeid(*this) == typeid(Student))
            return tstudent;
        else if(typeid(*this) == typeid(Mitarbeiter))
            return tmitarbeiter;
        else
        {
            cerr << "\nWrong type of Person";
            exit(1);
        }
}
//--------------------------------------------------------------
bool operator == (string& s, Person& p)
{
    if(s == p.retName())
        return 1;
    else
        return 0;
}
//--------------------------------------------------------------
void operator << (ostream& os, Person& p)
{
    os << p.retName();
}
////////////////////////////////////////////////////////////////
int main()
{
 
    Database db;
 
    db.add("Kristina", 241089);
    db.add("Martina", 311017);
    db.add("Kirill", 1111);
    db.add("Harold", "0170-4081883", "Vertrieb");
 
    db.saveToFile("Proverka.dat");
 
//    db.readFromFile("Proverka.dat");
    db.printList();
    return 0;
}
novi4ok
549 / 502 / 8
Регистрация: 23.07.2009
Сообщений: 2,359
Записей в блоге: 1
06.10.2013, 01:51     Не могу найти ошибку. Запись/чтение из файла #8
а ты провто для интереса возьми и выведи куда-нибудь себе значение sizeof(Student). будешь очень удивлен.

Добавлено через 58 секунд
нужно нормально сериализировать и десериализировать объекты. они "от рождения" этого делать не умеют
alsav22
5282 / 4801 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
06.10.2013, 02:01     Не могу найти ошибку. Запись/чтение из файла #9
murtukov, моё предлоожение было: не использоавть бинарное чтение\запись.

Добавлено через 1 минуту
А если использовать, то тут нужно подумать.
murtukov
 Аватар для murtukov
8 / 8 / 3
Регистрация: 30.01.2013
Сообщений: 99
06.10.2013, 02:01  [ТС]     Не могу найти ошибку. Запись/чтение из файла #10
alsav22, я понял, но это не решение, я уже сохранял объекты со строковыми полями (char*) в файл и все прекрасно работало. Могу показать рабочий пример
novi4ok
549 / 502 / 8
Регистрация: 23.07.2009
Сообщений: 2,359
Записей в блоге: 1
06.10.2013, 02:03     Не могу найти ошибку. Запись/чтение из файла #11
Цитата Сообщение от alsav22 Посмотреть сообщение
murtukov, моё предлоожение было: не использоавть бинарное чтение\запись.
это куда? просто понимать нужно, что именно ты пишешь.
murtukov
 Аватар для murtukov
8 / 8 / 3
Регистрация: 30.01.2013
Сообщений: 99
06.10.2013, 02:03  [ТС]     Не могу найти ошибку. Запись/чтение из файла #12
novi4ok, novi4ok, видимо мне надо почитать о сериализации и десериализации объектов. Никогда об этом не слышал прежде. Есть хорошие источники?
alsav22
5282 / 4801 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
06.10.2013, 02:03     Не могу найти ошибку. Запись/чтение из файла #13
Я бы вообще так не делал. Где-то работает, а где-то не будет. Но это дело хозяйское...
http://habrahabr.ru/post/142662/
http://www.kalinin.ru/programming/cpp/31_07_00.shtml
Использование #pragma pack(push,1)... pack(pop)
murtukov
 Аватар для murtukov
8 / 8 / 3
Регистрация: 30.01.2013
Сообщений: 99
06.10.2013, 02:07  [ТС]     Не могу найти ошибку. Запись/чтение из файла #14
Пожалуйста, если лень вникать в весь код (а это и не обязательно), то вникните хотя бы в методы saveToFile() и readFromFile()

Добавлено через 1 минуту
alsav22, спасибо за ссылочки, обязательно прочту
alsav22
5282 / 4801 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
06.10.2013, 02:09     Не могу найти ошибку. Запись/чтение из файла #15
murtukov, ваш код не компилируется. А у вас? Вы там строки С сравниваете, как string.
novi4ok
549 / 502 / 8
Регистрация: 23.07.2009
Сообщений: 2,359
Записей в блоге: 1
06.10.2013, 02:12     Не могу найти ошибку. Запись/чтение из файла #16
Цитата Сообщение от murtukov Посмотреть сообщение
novi4ok, novi4ok, видимо мне надо почитать о сериализации и десериализации объектов. Никогда об этом не слышал прежде. Есть хорошие источники?
гугль - один из неплохих.
принцип всех сериализаций - объеат должен уметь преобразовать себя в строку и восстановать свое состояние из нее.
твоя ошибка в том, что ты предположил, что sizeof (что-нибудь) тебе вернет отпечаток памяти, хранящий всю информацию об объекте. это предположение - ошибочно. например, твой объект имеет элемент std::string, и там - какое-то значение. оно может быть пустым, а может быть очень длинным. но sizeof (твойобъект) известен уже на этапе компиляции. не закрадывается сомнение, что значения этого поля в этой длине не предусмотрено?
murtukov
 Аватар для murtukov
8 / 8 / 3
Регистрация: 30.01.2013
Сообщений: 99
06.10.2013, 02:27  [ТС]     Не могу найти ошибку. Запись/чтение из файла #17
alsav22, это из-за того, что изначально вместо всех строк char* были string, я изменил код полчаса назад, эту строку видимо забыл, да и не важно это, я все равно не вызываю метод howMuchS();
Мне бы разобраться с методами записи и чтения из файла

Добавлено через 1 минуту
Цитата Сообщение от novi4ok Посмотреть сообщение
принцип всех сериализаций - объеат должен уметь преобразовать себя в строку и восстановать свое состояние из нее.
твоя ошибка в том, что ты предположил, что sizeof (что-нибудь) тебе вернет отпечаток памяти, хранящий всю информацию об объекте. это предположение - ошибочно. например, твой объект имеет элемент std::string, и там - какое-то значение. оно может быть пустым, а может быть очень длинным. но sizeof (твойобъект) известен уже на этапе компиляции. не закрадывается сомнение, что значения этого поля в этой длине не предусмотрено?
Спасибо, мысль уловил)
alsav22
5282 / 4801 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
06.10.2013, 03:58     Не могу найти ошибку. Запись/чтение из файла #18
Цитата Сообщение от murtukov Посмотреть сообщение
да и не важно это, я все равно не вызываю метод howMuchS()
Вам не важно, а тем кто будет разбираться с этим кодом? Ваши ошибки исправлять? Если уж код выкладываете, то потрудитесь проверить: компилируется он или нет.
У вас в классах виртуальные функции присутствуют, поэтому бинарное чтение\запись не подходят, даже если поля не объекты.

Добавлено через 6 минут
Вот тут темы похожие:
Запись и чтение объектов разных классов в один файл
Запись и извлечение данных в несколько запусков программы
murtukov
 Аватар для murtukov
8 / 8 / 3
Регистрация: 30.01.2013
Сообщений: 99
06.10.2013, 16:53  [ТС]     Не могу найти ошибку. Запись/чтение из файла #19
Цитата Сообщение от alsav22 Посмотреть сообщение
Если уж код выкладываете, то потрудитесь проверить: компилируется он или нет.
вообще-то я потрудился и проверил, код компилируется.

Вот рабочий код программы, где есть виртуальные методы и все прекрасно записывается/считывается бинарным методом (взято из книги Лафоре) :

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
#include <fstream>      // для потоковых файловых функций
#include <iostream>
#include <typeinfo>     // для typeid()
using namespace std;
#include <stdlib.h>    // для exit()
 
const int LEN = 32;     // Максимальная длина фамилий
const int MAXEM = 100;  // максимальное число работников
 
enum employee_type { tmanager, tscientist, tlaborer };
///////////////////////////////////////////////////////////
class employee                // класс employee 
{
private:
    char name[LEN];           // фамилия работника
    unsigned long number;     // номер работника
    static int n;             // текущее число работников
    static employee* arrap[]; // массив указателей на класс работников
public:
    virtual void getdata()
    {
        cin.ignore(10, '\n');
        cout << "  Введите фамилию: "; cin >> name;
        cout << "  Введите номер: ";    cin >> number;
    }
    virtual void putdata()
    {
        cout << "\n  Фамилия: " << name;
        cout << "\n  Номер: " << number;
    }
    virtual employee_type get_type(); // получить тип
    static void add();                // добавить работника
    static void display();            // вывести данные обо всех
    static void read();               // чтение из файла
    static void write();              // запись в файл
};
//---------------------------------------------------------
// статические переменные
int employee::n;                  // текущее число работников
employee* employee::arrap[MAXEM]; // массив указателей на класс работников
///////////////////////////////////////////////////////////
// класс manager (менеджеры)
class manager : public employee
{
private:
    char title[LEN];  // титул ("вице-президент" и т. п.)
    double  dues;     // Налоги гольф-клуба
public:
    void getdata()
    {
        employee::getdata();
        cout << "  Введите титул: ";       cin >> title;
        cout << "  Введите налоги: "; cin >> dues;
    }
    void putdata()
    {
        employee::putdata();
        cout << "\n  Титул: " << title;
        cout << "\n  Налоги гольф-клуба: " << dues;
    }
};
///////////////////////////////////////////////////////////
// класс scientist (ученые)
class scientist : public employee
{
private:
    int pubs;         // число публикаций
public:
    void getdata()
    {
        employee::getdata();
        cout << "  Введите число публикаций: "; cin >> pubs;
    }
    void putdata()
    {
        employee::putdata();
        cout << "\n  Число публикаций: " << pubs;
    }
};
///////////////////////////////////////////////////////////
// класс laborer (рабочие)
class laborer : public employee
{
};
///////////////////////////////////////////////////////////
// добавить работника в список (хранится в ОП)
void employee::add()
{
    char ch;
    cout << "'m' для добавления менеджера"
        "\n's' для добавления ученого"
        "\n'l' для добавления рабочего"
        "\nВаш выбор: ";
    cin >> ch;
    switch(ch)
    {   // создать объект указанного типа
    case 'm': arrap[n] = new manager;  break;
    case 's': arrap[n] = new scientist; break;
    case 'l': arrap[n] = new laborer;  break;
    default: cout << "\nНеизвестный тип работника\n"; return;
    }
    arrap[n++]->getdata();      // Получить данные от пользователя
}
//---------------------------------------------------------
// Вывести данные обо всех работниках
void employee::display()
{
    for(int j = 0; j < n; j++)
    {
        cout  << (j + 1);       // вывести номер
        switch(arrap[j]->get_type()) // вывести тип 
        {
        case tmanager:  cout << ". Тип: Менеджер";  break;
        case tscientist: cout << ". Тип: Ученый"; break;
        case tlaborer:   cout << ". Тип: Рабочий";  break;
        default: cout << ". Неизвестный тип";
        }
        arrap[j]->putdata();    // Вывод данных
        cout << endl;
    }
}
//---------------------------------------------------------
// Возврат типа объекта
employee_type employee::get_type()
{
    if(typeid(*this) == typeid(manager))
        return tmanager;
    else if(typeid(*this) == typeid(scientist))
        return tscientist;
    else if(typeid(*this) == typeid(laborer))
        return tlaborer;
    else
    { cerr << "\nНеправильный тип работника"; exit(1); }
    return tmanager;
}
//---------------------------------------------------------
// Записать все объекты, хранящиеся в памяти, в файл
void employee::write()
{
    int size;
    cout << "Идет запись " << n << " работников.\n";
    ofstream ouf;               // открыть ofstream в двоичном виде
    employee_type etype;        // тип каждого объекта employee
 
    ouf.open("EMPLOY.DAT", ios::trunc | ios::binary);
    if(!ouf)
    { cout << "\nНевозможно открыть файл\n"; return; }
    for(int j = 0; j < n; j++)  // Для каждого объекта
    {                           // получить его тип
        etype = arrap[j]->get_type();
 
        // записать данные в файл
        ouf.write((char*)&etype, sizeof(etype));
        switch(etype)           // найти его размер
        {
        case tmanager:   size  = sizeof(manager); break;
        case tscientist: size  = sizeof(scientist); break;
        case tlaborer:   size  = sizeof(laborer); break;
        }  
 
        // запись объекта employee в файл   
        ouf.write((char*)(arrap[j]), size);
        if(!ouf)
        { cout << "\nЗапись в файл невозможна\n"; return; }
    }
}
//---------------------------------------------------------
// чтение всех данных из файла в память
void employee::read()
{
    int size;               // размер объекта employee 
    employee_type etype;    // тип работника
    ifstream inf;           // открыть ifstream в двоичном виде
    inf.open("EMPLOY.DAT", ios::binary);
    if(!inf)
    { cout << "\nНевозможно открыть файл\n"; return; }
    n = 0;                  // В памяти работников нет
    while(true)
    {                       // чтение типа следующего работника
        inf.read((char*)&etype, sizeof(etype));
        if(inf.eof())   // выход из цикла по EOF
            break;
        if(!inf)        // ошибка чтения типа
        { cout << "\nНевозможно чтение типа\n"; return; }
        switch(etype)
        {               // создать нового работника
        case tmanager:  // корректного типа
            arrap[n] = new manager;
            size = sizeof(manager);
            break;
        case tscientist:
            arrap[n] = new scientist;
            size = sizeof(scientist);
            break;
        case tlaborer:
            arrap[n] = new laborer;
            size = sizeof(laborer);
            break;
        default: cout << "\nНеизвестный тип в файле\n"; return;
        }                     // чтение данных из файла
        inf.read((char*)arrap[n], size);
        if(!inf)              // ошибка, но не EOF
        { cout << "\nЧтение данных из файла невозможно\n"; return; }
        n++;                  // счетчик работников увеличить
    }                         // end while
    cout << "Идет чтение " << n << " работников\n";
}
///////////////////////////////////////////////////////////
int main()
{
    system("chcp 1251 > nul");
 
    char ch;
    while(true)
    {
        cout << "'a' - добавление сведений о работнике"
            "\n'd' - вывести сведения обо всех работниках"
            "\n'w' - записать все данные в файл"
            "\n'r' - прочитать все данные из файла"
            "\n'x' - выход"
            "\nВаш выбор: ";
        cin >> ch;
        switch(ch)
        {
        case 'a':           // добавить работника
            employee::add(); break;
        case 'd':           // вывести все сведения
            employee::display(); break;
        case 'w':           // запись в файл
            employee::write(); break;
        case 'r':           // чтение всех данных из файла
            employee::read(); break;
        case 'x': exit(0);  // выход
        default: cout << "\nНеизвестная команда";
        }                   // end switch
    }                       // end while
 
    return 0;
}
Добавлено через 1 час 35 минут
Урезал код в два раза. Создам новую тему и выложу его туда, а иначе эту уже никто не будет читать.
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
06.10.2013, 19:27     Не могу найти ошибку. Запись/чтение из файла
Еще ссылки по теме:

Чтение и запись из файла C++
C++ Запись и чтение с файла
C++ Запись и чтение из файла

Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
alsav22
5282 / 4801 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
06.10.2013, 19:27     Не могу найти ошибку. Запись/чтение из файла #20
Цитата Сообщение от murtukov Посмотреть сообщение
вообще-то я потрудился и проверил, код компилируется.
Код из 7 поста? Сказки не рассказывайте. Вот результат его компиляции в студии:
1>------ Построение начато: проект: LAB, Конфигурация: Debug Win32 ------
1> main.cpp
1>main.cpp(392): warning C4996: 'strcpy': This function or variable may be unsafe. Consider using strcpy_s instead. To disable deprecation, use _CRT_SECURE_NO_WARNINGS. See online help for details.
1> C:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\include\string.h(105): см. объявление "strcpy"
1>main.cpp(405): warning C4996: 'strcpy': This function or variable may be unsafe. Consider using strcpy_s instead. To disable deprecation, use _CRT_SECURE_NO_WARNINGS. See online help for details.
1> C:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\include\string.h(105): см. объявление "strcpy"
1>main.cpp(426): error C2679: бинарный "==": не найден оператор, принимающий правый операнд типа "std::string" (или приемлемое преобразование отсутствует)
1> может быть "встроенный оператор C++==(char [100], char [100])"
1> C:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\include\exception(470): или "bool std::operator ==(const std::_Exception_ptr &,const std::_Exception_ptr &)"
1> C:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\include\exception(475): или "bool std::operator ==(std::_Null_type,const std::_Exception_ptr &)"
1> C:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\include\exception(481): или "bool std::operator ==(const std::_Exception_ptr &,std::_Null_type)"
1> C:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\include\system_error(408): или "bool std::operator ==(const std::error_code &,const std::error_condition &)"
1> C:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\include\system_error(416): или "bool std::operator ==(const std::error_condition &,const std::error_code &)"
1> при попытке сопоставить список аргументов "(char [100], std::string)"
1>main.cpp(542): warning C4018: <: несоответствие типов со знаком и без знака
1>main.cpp(552): warning C4018: <: несоответствие типов со знаком и без знака
1>main.cpp(554): error C2679: бинарный "==": не найден оператор, принимающий правый операнд типа "const std::string" (или приемлемое преобразование отсутствует)
1> может быть "встроенный оператор C++==(const char *, const char *)"
1> C:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\include\exception(470): или "bool std::operator ==(const std::_Exception_ptr &,const std::_Exception_ptr &)"
1> C:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\include\exception(475): или "bool std::operator ==(std::_Null_type,const std::_Exception_ptr &)"
1> C:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\include\exception(481): или "bool std::operator ==(const std::_Exception_ptr &,std::_Null_type)"
1> C:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\include\system_error(408): или "bool std::operator ==(const std::error_code &,const std::error_condition &)"
1> C:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\include\system_error(416): или "bool std::operator ==(const std::error_condition &,const std::error_code &)"
1> при попытке сопоставить список аргументов "(const char *, const std::string)"
1>main.cpp(566): warning C4018: <: несоответствие типов со знаком и без знака
1>main.cpp(568): error C2679: бинарный "==": не найден оператор, принимающий правый операнд типа "std::string" (или приемлемое преобразование отсутствует)
1> может быть "встроенный оператор C++==(const char *, const char *)"
1> C:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\include\exception(470): или "bool std::operator ==(const std::_Exception_ptr &,const std::_Exception_ptr &)"
1> C:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\include\exception(475): или "bool std::operator ==(std::_Null_type,const std::_Exception_ptr &)"
1> C:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\include\exception(481): или "bool std::operator ==(const std::_Exception_ptr &,std::_Null_type)"
1> C:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\include\system_error(408): или "bool std::operator ==(const std::error_code &,const std::error_condition &)"
1> C:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\include\system_error(416): или "bool std::operator ==(const std::error_condition &,const std::error_code &)"
1> при попытке сопоставить список аргументов "(const char *, std::string)"
1>main.cpp(584): warning C4018: <: несоответствие типов со знаком и без знака
1>main.cpp(652): error C2678: бинарный "==": не найден оператор, принимающий левый операнд типа "std::string" (или приемлемое преобразование отсутствует)
1> может быть "встроенный оператор C++==(const char *, const char *)"
1> C:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\include\exception(470): или "bool std::operator ==(const std::_Exception_ptr &,const std::_Exception_ptr &)"
1> C:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\include\exception(475): или "bool std::operator ==(std::_Null_type,const std::_Exception_ptr &)"
1> C:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\include\exception(481): или "bool std::operator ==(const std::_Exception_ptr &,std::_Null_type)"
1> C:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\include\system_error(408): или "bool std::operator ==(const std::error_code &,const std::error_condition &)"
1> C:\Program Files (x86)\Microsoft Visual Studio 10.0\VC\include\system_error(416): или "bool std::operator ==(const std::error_condition &,const std::error_code &)"
1> main.cpp(650): или "bool operator ==(std::string &,Person &)"
1> при попытке сопоставить список аргументов "(std::string, const char *)"
========== Построение: успешно: 0, с ошибками: 1, без изменений: 0, пропущено: 0 ==========
Цитата Сообщение от murtukov Посмотреть сообщение
Вот рабочий код программы, где есть виртуальные методы и все прекрасно записывается/считывается бинарным методом (взято из книги Лафоре) :
Всё, что знаю по этому поводу, я написал. Остальное - ваше дело.
Yandex
Объявления
06.10.2013, 19:27     Не могу найти ошибку. Запись/чтение из файла
Ответ Создать тему
Опции темы

Текущее время: 07:27. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru