Покритикуйте пожалуйста программу

MrGluck · Регистрация: 29.11.2010

Студворк — интернет-сервис помощи студентам

Student.h

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
#ifndef _STUDENT_H
#define _STUDENT_H
 
class Student
{
      public:
          Student();
          void del();
          void add_surname(char* y);
          void add_name(char* y);
          void add_otchestvo(char* y);
          void add_date(char* y);
          void add_adress(char* y);
          void add_stip(int x);
          void change_free();
          char* get_surname();
          char* get_name();
          char* get_otchestvo();
          char* get_date();
          char* get_adress();
          int get_stip();
          bool get_free();
      private:
          char *surname, *name, *otchestvo, *date, *adress;
          int stip;
          bool free;
};
 
#endif

Student.cpp

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
#include <iostream>
#include "Student.h"
 
Student::Student()
{
    free = 1;
}
 
void Student::del()
{
    free = 1;
}
 
void Student::add_surname(char *y)
{
     surname = new char (strlen(y) - 1);
     strcpy(surname, y);
}
 
void Student::add_name(char *y)
{
     name = new char (strlen(y) - 1);
     strcpy(name, y);
}
 
void Student::add_otchestvo(char *y)
{
     otchestvo = new char (strlen(y) - 1);
     strcpy(otchestvo, y);
}
 
void Student::add_date(char* y)
{
     date = new char (strlen(y) - 1);
     strcpy(date, y);
}
 
void Student::add_adress(char* y)
{
     adress = new char (strlen(y) - 1);
     strcpy(adress, y);
}
 
void Student::add_stip(int x)
{
     stip = x;
}
 
void Student::change_free()
{
     free = 0;
}
 
char* Student::get_surname()
{
      return surname;
}
 
char* Student::get_name()
{
      return name;
}
 
char* Student::get_otchestvo()
{
      return otchestvo;
}
 
char* Student::get_date()
{
      return date;
}
 
char* Student::get_adress()
{
      return adress;
}
 
int Student::get_stip()
{
     return stip;
}
 
bool Student::get_free()
{
     return free;
}

main.cpp

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
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
#include <iostream>
#include <fstream>
#include <conio.h>
#define N 20
#include "Student.h"
using namespace std;
 
Student s[N];
 
// ïîèñê ñâîáîäíîãî ýëåìåíòà
int poiskfree(); // return index
// ïîèñê ïî ÷èñëîâîìó çíà÷åíèþ
int poiski(int x);
// ïîèñê ïî ñèìâîëüíîìó çíà÷åíèþ
int poiskc(char* y, int choose);
// ïîèñê ýëåìåíòà ñ ìèíèìàëüíûì çíà÷åíèåì ïîëÿ
int poiskmin();
// ñîðòèðîâêà ïî âîçðàñòàíèþ
void sortup_surname();
void sortup_name();
void sortup_otchestvo();
void sortup_adress();
void sortup_stip();
// ñîðòèðîâêà ïî óáûâàíèþ
void sortdown_surname();
void sortdown_name();
void sortdown_otchestvo();
void sortdown_adress();
void sortdown_stip();
// ÷òåíèå ôàéëà
bool read_from_file(char* filename);
// çàïèñü â ôàéë
bool write_in_file(char* filename);
 
int main()
{
    
    setlocale(LC_ALL,"Russian");
    int number = 0, index, choose;
    do
    {
         system("cls");
         cout<< "Êîëè÷åñòâî ýëåìåíòîâ: "<< number
             << "\n×òî æåëàåòå ñäåëàòü: \n"
             << "1.  Óäàëèòü ýëåìåíò\n"
             << "2.  Íàéòè ñâîáîäíûé ýëåìåíò\n"
             << "3.  Ââåñòè íîâûé ýëåìåíò\n"
             << "4.  Âûâåñòè ýëåìåíò\n"
             << "5.  Âûâåñòè âñå ýëåìåíòû\n"
             << "6.  Ïîèñê ïî çíà÷åíèþ\n"
             << "7.  Ïîèñê ìèíèìàëüíîãî çíà÷åíèÿ\n"
             << "8.  Îòñîðòèðîâàòü ïî âîçðàñòàíèþ\n"
             << "9.  Îòñîðòèðîâàòü ïî óáûâàíèþ\n"
             << "10. Ñ÷èòàòü ñ ôàéëà\n"
             << "11. Çàïèñàòü â ôàéë\n\n"
             << "Âàø âûáîð: ";
         cin>> choose;
         switch(choose)
         {
             case 1:
             {
                  cout<< "Ââåäèòå èíäåêñ: ";
                  cin>> index;
                  if (s[index].get_free() == 0)
                      number--;
                  s[index].del();
                  break;
             }
             case 2:
             {
                  cout<< "Ñâîáîäíûé ýëåìåíò: "<< poiskfree();
                  break;
             }
             case 3:
             {
                  char surname[20], name[20], otchestvo[20], date[10], adress[50];
                  int stip;
                  do
                  {
                      cout<< "Ââåäèòå '-1' äëÿ âîçâðàòà â ãëàâíîå ìåíþ. \nÂâåäèòå èíäåêñ: ";
                      cin>> index;
                      if (index < -1 || index > 20)
                          cout<< "Íåâåðíûé èíäåêñ, ïîïðîáóéòå åùå ðàç\n";
                  } while (index < -1 || index > 20);
                  if (index == -1)
                      break;
                  cout<< "Ââåäèòå èìÿ: ";
                  cin>> name;
                  cout<< "Ââåäèòå ôàìèëèþ: ";
                  cin>> surname;
                  cout<< "Ââåäèòå îò÷åñòâî: ";
                  cin>> otchestvo;
                  cout<< "Ââåäèòå äàòó ðîæäåíèÿ â ôîðìàòå dd.mm.yyyy: ";
                  cin>> date;
                  cout<< "Ââåäèòå àäðåñ: ";
                  cin>> adress;
                  cout<< "Ââåäèòå ñòèïåíäèþ: ";
                  cin>> stip;
                  s[index].add_name (name);
                  s[index].add_surname (surname);
                  s[index].add_otchestvo (otchestvo);
                  s[index].add_date (date);
                  s[index].add_adress (adress);
                  s[index].add_stip (stip);
                  s[index].change_free();
                  number++;
                  break;
             }
             case 4:
             {
                  do
                  {
                      cout<< "Ââåäèòå '-1' äëÿ âîçâðàòà â ãëàâíîå ìåíþ. \nÂâåäèòå èíäåêñ: ";
                      cin>> index;
                      if (index < -1 || index > 20)
                          cout<< "Íåâåðíûé èíäåêñ, ïîïðîáóéòå åùå ðàç\n";
                      // åñëè ýëåìåíò ïóñò
                      else if (s[index].get_free() == 1)
                          cout<< "Ýëåìåíò ïóñò, ïîïðîáóéòå åùå ðàç\n";
                  } while (index < -1 || index > 20 || s[index].get_free() == 1);
                  if (index == -1)
                      break;
                  cout<< "Ñòóäåíò ïîä èíäåêñîì "<< index<< ":\n"
                      << "Ô.È.Î. : "<< s[index].get_surname()<< " "<< s[index].get_name()<< " "<< s[index].get_otchestvo()
                      << "\nÄàòà ðîæäåíèÿ: "<< s[index].get_date()
                      << "\nÀäðåñ: "<< s[index].get_adress()
                      << "\nÑòèïåíäèÿ: "<< s[index].get_stip()<< "\n";
                  break;
             }
             case 5:
             {
                  for (int i=0; i < N; i++)
                      if (s[i].get_free() == 0)
                          cout<< "Ñòóäåíò ïîä èíäåêñîì "<< i<< ":\n"
                          << "Ô.È.Î. : "<< s[i].get_surname()<< " "<< s[i].get_name()<< " "<< s[i].get_otchestvo()
                          << "\nÄàòà ðîæäåíèÿ: "<< s[i].get_date()
                          << "\nÀäðåñ: "<< s[i].get_adress()
                          << "\nÑòèïåíäèÿ: "<< s[i].get_stip()<< endl<< endl;
                  break;
             }
             case 6:
             {
                  int poisk_choose;
                  do
                  {
                      system("cls");
                      cout<< "Ââåäèòå '-1' äëÿ âîçâðàòà â ãëàâíîå ìåíþ\n"
                          << "Ïî êàêîìó êðèòåðèþ áóäåì èñêàòü:\n"
                          << "1. ñòèïåíäèè\n"
                          << "2. èìåíè\n"
                          << "3. ôàìèëèè\n"
                          << "4. îò÷åñòâó\n"
                          << "5. äàòå ðîæäåíèÿ\n"
                          << "6. àäðåñó\n"
                          << "Âàø âûáîð: ";
                      cin>> poisk_choose;
                      if ((poisk_choose < 1 || poisk_choose > 6) && poisk_choose != -1)
                          cout<< "Âû ââåëè íåâåðíîå çíà÷åíèå, ïîïðîáóéòå åùå ðàç\n";
                  } while ((poisk_choose < 1 || poisk_choose > 6) && poisk_choose != -1);
                  if (poisk_choose == -1)
                      break;
                  if (poisk_choose == 1)
                  {
                      int poiskstip;
                      cout<< "Ââåäèòå ñòèïåíäèþ: ";
                      cin>> poiskstip;
                      if (poiski(poiskstip) == -1)
                          cout<< "Ýëåìåíò íå íàéäåí\n";
                      else
                          cout<< "Ýëåìåíò ïîä íîìåðîì "<< poiski(poiskstip) << " óäîâëåòâîðÿåò ïîèñê\n";
                  }
                  else
                  {
                      char poiskchar[20];
                      cout<< "Ââåäèòå ñëîâî: ";
                      cin>> poiskchar;
                      if (poiskc(poiskchar, poisk_choose) == -1)
                          cout<< "Ýëåìåíò íå íàéäåí\n";
                      else
                          cout<< "Ýëåìåíò ïîä íîìåðîì "<< poiskc(poiskchar, poisk_choose) << " óäîâëåòâîðÿåò ïîèñê\n";
                  }
                  break;
             }
             case 7:
             {
                  if (poiskmin() == -1)
                      cout<< "Ýëåìåíò íå íàéäåí\n";
                  else
                      cout<< "Ýëåìåíò ïîä íîìåðîì "<< poiskmin() << " óäîâëåòâîðÿåò ïîèñê\n";
                  break;
             }
             case 8:
             {
                  int sortup_choose;
                  do
                  {
                      cout<< "Ââåäèòå '-1' äëÿ âîçâðàòà â ãëàâíîå ìåíþ\n"
                          << "Ïî êàêîìó êðèòåðèþ áóäåì ñîðòèðîâàòü:\n"
                          << "1. Ïî èìåíè\n"
                          << "2. Ïî ôàìèëèè\n"
                          << "3. Ïî îò÷åñòâó\n"
                          << "4. Ïî ñòèïåíäèè\n"
                          << "Âàø âûáîð: ";
                      cin>> sortup_choose;
                      if (sortup_choose > 5 || sortup_choose < 1 && sortup_choose != -1)
                          cout<< "Âû ââåëè íåâåðíîå çíà÷åíèå. ïîïðîáóéòå åùå ðàç\n";
                  } while (sortup_choose > 5 || sortup_choose < 1 && sortup_choose != -1);
                  if (sortup_choose == -1)
                      break;
                  if (sortup_choose == 1)
                      sortup_name();
                  if (sortup_choose == 2)
                      sortup_surname();
                  if (sortup_choose == 3)
                      sortup_otchestvo();
                  if (sortup_choose == 4)
                      sortup_stip();                      
                  break;
             }
             case 9:
             {
                  int sortdown_choose;
                  do
                  {
                      cout<< "Ââåäèòå '-1' äëÿ âîçâðàòà â ãëàâíîå ìåíþ\n"
                          << "Ïî êàêîìó êðèòåðèþ áóäåì ñîðòèðîâàòü:\n"
                          << "1. Ïî èìåíè\n"
                          << "2. Ïî ôàìèëèè\n"
                          << "3. Ïî îò÷åñòâó\n"
                          << "4. Ïî ñòèïåíäèè\n"
                          << "Âàø âûáîð: ";
                      cin>> sortdown_choose;
                      if (sortdown_choose > 5 || sortdown_choose < 1 && sortdown_choose != -1)
                          cout<< "Âû ââåëè íåâåðíîå çíà÷åíèå. ïîïðîáóéòå åùå ðàç\n";
                  } while (sortdown_choose > 5 || sortdown_choose < 1 && sortdown_choose != -1);
                  if (sortdown_choose == -1)
                      break;
                  if (sortdown_choose == 1)
                      sortdown_name();
                  if (sortdown_choose == 2)
                      sortdown_surname();
                  if (sortdown_choose == 3)
                      sortdown_otchestvo();
                  if (sortdown_choose == 4)
                      sortdown_stip();                      
                  break;
             }
             case 10:
             {
                  char filename[80];
                  do
                  {                       
                       cout<< "Ââåäèòå èìÿ ôàéëà: ";
                       cin>> filename;
                       if (ifstream(filename) == false)
                           cerr<< "Íåâîçìîæíî îòêðûòü "<< filename<< endl;
                  } while (ifstream(filename) == false);
                  if (read_from_file (filename) == 0)
                      cerr<< "Îøèáêà ïðè ÷òåíèè ôàéëà\n";
                  break;
             }
             case 11:
             {
                  char _filename[80];
                  cout<< "Ââåäèòå èìÿ ôàéëà: ";
                  cin>> _filename;
                  if (write_in_file (_filename) == 0)
                      cerr<< "Îøèáêà ïðè çàïèñè â ôàéë\n";
                  break;
             }
             default:
                 cout<< "Âû ââåëè íåâåðíîå çíà÷åíèå";
         }
         cout<< "\nÆåëàåòå ïðîäîëæèòü y/n?\n";
    } while (getch()!= 'n');
    return 0;
}
 
int poiskfree()
{
    for (int i=0; i < N; i++)
        if (s[i].get_free() == true)
            return i;
}
 
int poiski (int x)
{
    for (int i=0; i < N; i++)
    {
        if (s[i].get_stip() == x)
            return i;
        if (i == N-1)
            return -1;
    }
}
 
int poiskc (char *y, int choose)
{
    for (int i=0; i < N; i++)
    {
        if ((choose == 2 && stricmp(s[i].get_name(), y) == 0) || 
            (choose == 3 && stricmp(s[i].get_surname(), y) == 0) ||
            (choose == 4 && stricmp(s[i].get_otchestvo(), y) == 0) ||
            (choose == 5 && stricmp(s[i].get_date(), y) == 0 ||
            (choose == 6 && stricmp(s[i].get_otchestvo(), y) == 0)))
            return i;
        if (i == N-1)
            return -1;
    }
}
 
int poiskmin()
{
    int min = 65535, minindex = -1;
    for (int i=0; i < N; i++)
        if (s[i].get_free() == 0 && s[i].get_stip() < min)
        {
            min = s[i].get_stip();
            minindex = i;
        }
    return minindex;
}
 
void sortup_surname()
{
     for (int i = N-1; i > 0; i--)
         for (int j=0; j < i; j++)
             if (s[j].get_surname() > s[j+1].get_surname())
                 swap(s[j], s[j+1]);
}
 
void sortup_name()
{
     for (int i = N-1; i > 0; i--)
         for (int j=0; j < i; j++)
             if (s[j].get_name() > s[j+1].get_name())
                 swap(s[j], s[j+1]);
}
 
void sortup_otchestvo()
{
     for (int i = N-1; i > 0; i--)
         for (int j=0; j < i; j++)
             if (s[j].get_otchestvo() > s[j+1].get_otchestvo())
                 swap(s[j], s[j+1]);
}
 
void sortup_adress()
{
     for (int i = N-1; i > 0; i--)
         for (int j=0; j < i; j++)
             if (s[j].get_adress() > s[j+1].get_adress())
                 swap(s[j], s[j+1]);
}
 
void sortup_stip()
{
     for (int i = N-1; i > 0; i--)
         for (int j=0; j < i; j++)
             if (s[j].get_stip() > s[j+1].get_stip())
                 swap(s[j], s[j+1]);
}
 
void sortdown_surname()
{
     for (int i = N-1; i > 0; i--)
         for (int j=0; j < i; j++)
             if (s[j].get_surname() < s[j+1].get_surname())
                 swap(s[j], s[j+1]);
}
 
void sortdown_name()
{
     for (int i = N-1; i > 0; i--)
         for (int j=0; j < i; j++)
             if (s[j].get_name() < s[j+1].get_name())
                 swap(s[j], s[j+1]);
}
 
void sortdown_otchestvo()
{
     for (int i = N-1; i > 0; i--)
         for (int j=0; j < i; j++)
             if (s[j].get_otchestvo() < s[j+1].get_otchestvo())
                 swap(s[j], s[j+1]);
}
 
void sortdown_adress()
{
     for (int i = N-1; i > 0; i--)
         for (int j=0; j < i; j++)
             if (s[j].get_adress() < s[j+1].get_adress())
                 swap(s[j], s[j+1]);
}
 
void sortdown_stip()
{
     for (int i = N-1; i > 0; i--)
         for (int j=0; j < i; j++)
             if (s[j].get_stip() < s[j+1].get_stip())
                 swap(s[j], s[j+1]);
}
 
bool write_in_file(char* filename)
{
     ofstream o(filename, ios::binary);
     if (o.bad())
         return 0;
     for (int i=0; i < N && !o.eof(); i++)
     {
             o.write((char*)&s[i], sizeof(s[i]));
             if (o.bad())
                 return 0;
     }
     o.close();
     return 1;
}
 
bool read_from_file(char* filename)
{
     ifstream i(filename, ios::binary);
     if (i.bad())
         return 0;
     for (int j=0; j < N && !i.eof(); j++)
     {
         i.read((char*)&s[j], sizeof(s[j]));
         if (i.bad())
             return 0;
     }
     i.close();
     return 1;
}

Скажите, где что написано некорректно. Если кто поможет - буду очень благодарен. Если понадобится - могу скинуть само задание.

@Deviaphan · 08.10.2011, 11:01

Нет деструктора.
Читать не буду, многабукаф.

@aeshes · 08.10.2011, 11:38

Еще нужен, наверное, конструктор копирования
И в выражениях

C++
1
otchestvo = new char (strlen(y) - 1);

разве не квадратные скобки должны быть, вот так:

C++
1
otchestvo = new char [strlen(y) - 1];

И зачем вы делаете на 1 меньше длину строки не понятно

А если будет более точное задание, то можно будет и на остальное посмотреть более пристально )

Jupiter · 08.10.2011, 12:31

Сообщение от MrGluck

surname = new char (strlen(y) - 1);

все new переделать вот так

C++
1
surname = new char [strlen(y) + 1];

@rangerx · 08.10.2011, 13:05

То, что касается класса:
1) Все сеттеры не мешало бы переименовать из add_x в set_x.
2) Все геттеры должны быть константными методами т.е.

C++
1
char* get_x() const;

и раз уж поля класса, значения которых они возвращают, имеют тип char* - возвращать const char*.

C++
1
const char* get_x() const;

3) Не может существовать объект класса Student без установленных полей. У данного класса должен быть конструктор с параметрами.
4) Во всех сеттерах перед использованием new память, что была выделена ранее, не освобождается. Как результат, утечка.
5) Отсутствует деструктор, в котором должны освобождаться выделенные ресурсы. При уничтожении объекта, опять же, утечка памяти.
6)

Сообщение от MrGluck

surname = new char (strlen(y) - 1);

это выделение памяти под 1 элемент типа char и инициализация его числом strlen(y) - 1. Правильный вариант должен выглядеть так

C++
1
surname = new char[ strlen(y) + 1];

7) Дублирование кода в виде

C++
1
2
surname = new char (strlen(y) - 1);
strcpy(surname, y);

8) Нет реализации конструктора копий и оператора присваивания.
9) Все сеттеры в качестве параметра должны принимать const char*, а не char*.
10) Булевым переменным принято передавать значения true\false, а не 1\0.
11) Не подключён <cstring> хотя используется функция strcpy.

@#pragma · 08.10.2011, 15:32

Лучше не использовать имена для переменных, которые совпадают с названиями библиотечных функций C и C++ (в данном случае это функция free() )

C++
1
2
3
bool free;
bool get_free();
change_free()

Также get_free немного неудачное название для функции (как и change_free в контексте написанного выше), оно не отражает её предназначение. К тому же функция возвращает значение типа bool, наверное лучше использовать приставку is_ ,а не get_, хотя я где-то встречал споры на этот счёт, нужны ли приставки вообще при правильной организации класса. Знающие люди советуют избавляться от set-методов, и всю работу передавать конструкторам класса.

@Gepar · 08.10.2011, 16:42

Можно я по-придираюсь к именованию?

C++
1
surname, *name, *[B]otchestvo[/B],

Ну смешно же. Чатается так же как как "Hello, my Имя is ..."

Сообщение от MrGluck

int get_stip();

И что оно делает? раздевает (strip) студента ?
Возможно тут буду я не прав так как весь код не смотрел, но что Вы делаете здесь:

C++
1
2
3
4
5
void Student::add_surname(char *y)
{
     surname = new char (strlen(y) - 1);
     strcpy(surname, y);
}

? Отбрасываете нуль-символ чтобы создать новую строку без него? Не знаю тогда как это у Вас в итоге работает, хотя если работает то я что-то неправильно понял.
Ну а так код и правда большой и всё в основном строится на том чтобы заполнять правильно поля и делать нужные проверки так что лучше задайте конкретный вопрос если у Вас что-то не получается.

Jupiter · 08.10.2011, 18:11

Сообщение от Gepar

Возможно тут буду я не прав так как весь код не смотрел, но что Вы делаете здесь: Отбрасываете нуль-символ чтобы создать новую строку без него?

создает один символ)

@Gepar · 08.10.2011, 19:31

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

Jupiter · 08.10.2011, 19:35

Сообщение от Gepar

ну тогда назначение мне тем более не понятно

круглые скобки это инициализация
new char (strlen(y) - 1) - создаем один символ значение которого будет strlen(y) - 1 по таблице ASCII

MrGluck · 09.10.2011, 00:28 **[ТС]**

Всем большое спасибо. Внес коррективы в программу. Дописал деструктор, удалил все сеты и решил вместо них сделать конструктор с аргументами. Возникли проблемы в объявлении массива экземпляров класса, написании копирующего конструктора и оператора присваивания. Подскажите, как это можно реализовать?

Student.h

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
#ifndef _STUDENT_H
#define _STUDENT_H
 
class Student
{
      public:
          Student(char *n, char *s, char *p, char *d, char *a, int st);
          Student (Student& s);
          void operator = (Student& s);
          ~Student();
          void del();
          const char* get_surname() const;
          const char* get_name() const;
          const char* get_patronymic() const;
          const char* get_date() const;
          const char* get_adress() const;
          const int get_stip() const;
          const bool get_free() const;
      private:
          char *surname, *name, *patronymic, *date, *adress;
          int stip;
          bool is_free;
};
 
#endif

Student.cpp

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
#include <iostream>
#include "Student.h"
 
Student::Student(char *n, char *s, char *p, char *d, char *a, int st)
{
    name = new char [strlen(n) + 1];
    strcpy(name, n);
    surname = new char [strlen(s) + 1];
    strcpy(surname, s);
    patronymic = new char [strlen(p) + 1];
    strcpy(patronymic, p);
    date = new char [strlen(d) + 1];
    strcpy(date, d);
    adress = new char [strlen(a) + 1];
    strcpy(adress, a);
    stip = st;
    is_free = true;
}
 
Student::Student(Student& s)
{
    name = new char [strlen(s.name) + 1];
    strcpy(name, s.name);
    surname = new char [strlen(s.surname) + 1];
    strcpy(surname, s.surname);
    patronymic = new char [strlen(s.patronymic) + 1];
    strcpy(patronymic, s.patronymic);
    date = new char [strlen(s.date) + 1];
    strcpy(date, s.date);
    adress = new char [strlen(s.adress) + 1];
    strcpy(adress, s.adress);
    stip = s.stip;
    is_free = s.is_free;
}
 
void Student::operator = (Student& s)
{
    name = new char [strlen(s.name) + 1];
    strcpy(name, s.name);
    surname = new char [strlen(s.surname) + 1];
    strcpy(surname, s.surname);
    patronymic = new char [strlen(s.patronymic) + 1];
    strcpy(patronymic, s.patronymic);
    date = new char [strlen(s.date) + 1];
    strcpy(date, s.date);
    adress = new char [strlen(s.adress) + 1];
    strcpy(adress, s.adress);
    stip = s.stip;
    is_free = s.is_free;
}
    
Student::~Student()
{
    delete[] name;
    delete[] surname;
    delete[] patronymic;
    delete[] date;
    delete[] adress;
}
 
void Student::del()
{
    is_free = true;
}
 
const char* Student::get_surname() const
{
      return surname;
}
 
const char* Student::get_name() const
{
      return name;
}
 
const char* Student::get_patronymic() const
{
      return patronymic;
}
 
const char* Student::get_date() const
{
      return date;
}
 
const char* Student::get_adress() const
{
      return adress;
}
 
const int Student::get_stip() const
{
     return stip;
}
 
const bool Student::get_free() const
{
     return is_free;
}

main.cpp

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
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
#include <iostream>
#include <cstring>
#include <fstream>
#include <conio.h>
#define N 20
#include "Student.h"
using namespace std;
 
Student s[N];
 
// ïîèñê ñâîáîäíîãî ýëåìåíòà
int poiskfree(); // return index
// ïîèñê ïî ÷èñëîâîìó çíà÷åíèþ
int poiski(int x);
// ïîèñê ïî ñèìâîëüíîìó çíà÷åíèþ
int poiskc(char* y, int choose);
// ïîèñê ýëåìåíòà ñ ìèíèìàëüíûì çíà÷åíèåì ïîëÿ
int poiskmin();
// ñîðòèðîâêà ïî âîçðàñòàíèþ
void sortup_surname();
void sortup_name();
void sortup_patronymic();
void sortup_adress();
void sortup_stip();
// ñîðòèðîâêà ïî óáûâàíèþ
void sortdown_surname();
void sortdown_name();
void sortdown_patronymic();
void sortdown_adress();
void sortdown_stip();
// ÷òåíèå ôàéëà
bool read_from_file(char* filename);
// çàïèñü â ôàéë
bool write_in_file(char* filename);
 
int main()
{
    
    setlocale(LC_ALL,"Russian");
    int number = 0, index, choose;
    do
    {
         system("cls");
         cout<< "Êîëè÷åñòâî ýëåìåíòîâ: "<< number
             << "\n×òî æåëàåòå ñäåëàòü: \n"
             << "1.  Óäàëèòü ýëåìåíò\n"
             << "2.  Íàéòè ñâîáîäíûé ýëåìåíò\n"
             << "3.  Ââåñòè íîâûé ýëåìåíò\n"
             << "4.  Âûâåñòè ýëåìåíò\n"
             << "5.  Âûâåñòè âñå ýëåìåíòû\n"
             << "6.  Ïîèñê ïî çíà÷åíèþ\n"
             << "7.  Ïîèñê ìèíèìàëüíîãî çíà÷åíèÿ\n"
             << "8.  Îòñîðòèðîâàòü ïî âîçðàñòàíèþ\n"
             << "9.  Îòñîðòèðîâàòü ïî óáûâàíèþ\n"
             << "10. Ñ÷èòàòü ñ ôàéëà\n"
             << "11. Çàïèñàòü â ôàéë\n\n"
             << "Âàø âûáîð: ";
         cin>> choose;
         switch(choose)
         {
             case 1:
             {
                  cout<< "Ââåäèòå èíäåêñ: ";
                  cin>> index;
                  if (s[index].get_free() == false)
                      number--;
                  s[index].del();
                  break;
             }
             case 2:
             {
                  cout<< "Ñâîáîäíûé ýëåìåíò: "<< poiskfree();
                  break;
             }
             case 3:
             {
                  char surname[20], name[20], patronymic[20], date[10], adress[50];
                  int stip;
                  do
                  {
                      cout<< "Ââåäèòå '-1' äëÿ âîçâðàòà â ãëàâíîå ìåíþ. \nÂâåäèòå èíäåêñ: ";
                      cin>> index;
                      if (index < -1 || index > 20)
                          cout<< "Íåâåðíûé èíäåêñ, ïîïðîáóéòå åùå ðàç\n";
                  } while (index < -1 || index > 20);
                  if (index == -1)
                      break;
                  cout<< "Ââåäèòå èìÿ: ";
                  cin>> name;
                  cout<< "Ââåäèòå ôàìèëèþ: ";
                  cin>> surname;
                  cout<< "Ââåäèòå îò÷åñòâî: ";
                  cin>> patronymic;
                  cout<< "Ââåäèòå äàòó ðîæäåíèÿ â ôîðìàòå dd.mm.yyyy: ";
                  cin>> date;
                  cout<< "Ââåäèòå àäðåñ: ";
                  cin>> adress;
                  cout<< "Ââåäèòå ñòèïåíäèþ: ";
                  cin>> stip;
                  s[index](name, surname, patronymic, date, adress, stip);
                  number++;
                  break;
             }
             case 4:
             {
                  do
                  {
                      cout<< "Ââåäèòå '-1' äëÿ âîçâðàòà â ãëàâíîå ìåíþ. \nÂâåäèòå èíäåêñ: ";
                      cin>> index;
                      if (index < -1 || index > 20)
                          cout<< "Íåâåðíûé èíäåêñ, ïîïðîáóéòå åùå ðàç\n";
                      // åñëè ýëåìåíò ïóñò
                      else if (s[index].get_free() == true)
                          cout<< "Ýëåìåíò ïóñò, ïîïðîáóéòå åùå ðàç\n";
                  } while (index < -1 || index > 20 || s[index].get_free() == true);
                  if (index == -1)
                      break;
                  cout<< "Ñòóäåíò ïîä èíäåêñîì "<< index<< ":\n"
                      << "Ô.È.Î. : "<< s[index].get_surname()<< " "<< s[index].get_name()<< " "<< s[index].get_patronymic()
                      << "\nÄàòà ðîæäåíèÿ: "<< s[index].get_date()
                      << "\nÀäðåñ: "<< s[index].get_adress()
                      << "\nÑòèïåíäèÿ: "<< s[index].get_stip()<< "\n";
                  break;
             }
             case 5:
             {
                  for (int i=0; i < N; i++)
                      if (s[i].get_free() == false)
                          cout<< "Ñòóäåíò ïîä èíäåêñîì "<< i<< ":\n"
                          << "Ô.È.Î. : "<< s[i].get_surname()<< " "<< s[i].get_name()<< " "<< s[i].get_patronymic()
                          << "\nÄàòà ðîæäåíèÿ: "<< s[i].get_date()
                          << "\nÀäðåñ: "<< s[i].get_adress()
                          << "\nÑòèïåíäèÿ: "<< s[i].get_stip()<< endl<< endl;
                  break;
             }
             case 6:
             {
                  int poisk_choose;
                  do
                  {
                      system("cls");
                      cout<< "Ââåäèòå '-1' äëÿ âîçâðàòà â ãëàâíîå ìåíþ\n"
                          << "Ïî êàêîìó êðèòåðèþ áóäåì èñêàòü:\n"
                          << "1. ñòèïåíäèè\n"
                          << "2. èìåíè\n"
                          << "3. ôàìèëèè\n"
                          << "4. îò÷åñòâó\n"
                          << "5. äàòå ðîæäåíèÿ\n"
                          << "6. àäðåñó\n"
                          << "Âàø âûáîð: ";
                      cin>> poisk_choose;
                      if ((poisk_choose < 1 || poisk_choose > 6) && poisk_choose != -1)
                          cout<< "Âû ââåëè íåâåðíîå çíà÷åíèå, ïîïðîáóéòå åùå ðàç\n";
                  } while ((poisk_choose < 1 || poisk_choose > 6) && poisk_choose != -1);
                  if (poisk_choose == -1)
                      break;
                  if (poisk_choose == 1)
                  {
                      int poiskstip;
                      cout<< "Ââåäèòå ñòèïåíäèþ: ";
                      cin>> poiskstip;
                      if (poiski(poiskstip) == -1)
                          cout<< "Ýëåìåíò íå íàéäåí\n";
                      else
                          cout<< "Ýëåìåíò ïîä íîìåðîì "<< poiski(poiskstip) << " óäîâëåòâîðÿåò ïîèñê\n";
                  }
                  else
                  {
                      char poiskchar[20];
                      cout<< "Ââåäèòå ñëîâî: ";
                      cin>> poiskchar;
                      if (poiskc(poiskchar, poisk_choose) == -1)
                          cout<< "Ýëåìåíò íå íàéäåí\n";
                      else
                          cout<< "Ýëåìåíò ïîä íîìåðîì "<< poiskc(poiskchar, poisk_choose) << " óäîâëåòâîðÿåò ïîèñê\n";
                  }
                  break;
             }
             case 7:
             {
                  if (poiskmin() == -1)
                      cout<< "Ýëåìåíò íå íàéäåí\n";
                  else
                      cout<< "Ýëåìåíò ïîä íîìåðîì "<< poiskmin() << " óäîâëåòâîðÿåò ïîèñê\n";
                  break;
             }
             case 8:
             {
                  int sortup_choose;
                  do
                  {
                      cout<< "Ââåäèòå '-1' äëÿ âîçâðàòà â ãëàâíîå ìåíþ\n"
                          << "Ïî êàêîìó êðèòåðèþ áóäåì ñîðòèðîâàòü:\n"
                          << "1. Ïî èìåíè\n"
                          << "2. Ïî ôàìèëèè\n"
                          << "3. Ïî îò÷åñòâó\n"
                          << "4. Ïî ñòèïåíäèè\n"
                          << "Âàø âûáîð: ";
                      cin>> sortup_choose;
                      if (sortup_choose > 5 || sortup_choose < 1 && sortup_choose != -1)
                          cout<< "Âû ââåëè íåâåðíîå çíà÷åíèå. ïîïðîáóéòå åùå ðàç\n";
                  } while (sortup_choose > 5 || sortup_choose < 1 && sortup_choose != -1);
                  if (sortup_choose == -1)
                      break;
                  if (sortup_choose == 1)
                      sortup_name();
                  if (sortup_choose == 2)
                      sortup_surname();
                  if (sortup_choose == 3)
                      sortup_patronymic();
                  if (sortup_choose == 4)
                      sortup_stip();                      
                  break;
             }
             case 9:
             {
                  int sortdown_choose;
                  do
                  {
                      cout<< "Ââåäèòå '-1' äëÿ âîçâðàòà â ãëàâíîå ìåíþ\n"
                          << "Ïî êàêîìó êðèòåðèþ áóäåì ñîðòèðîâàòü:\n"
                          << "1. Ïî èìåíè\n"
                          << "2. Ïî ôàìèëèè\n"
                          << "3. Ïî îò÷åñòâó\n"
                          << "4. Ïî ñòèïåíäèè\n"
                          << "Âàø âûáîð: ";
                      cin>> sortdown_choose;
                      if (sortdown_choose > 5 || sortdown_choose < 1 && sortdown_choose != -1)
                          cout<< "Âû ââåëè íåâåðíîå çíà÷åíèå. ïîïðîáóéòå åùå ðàç\n";
                  } while (sortdown_choose > 5 || sortdown_choose < 1 && sortdown_choose != -1);
                  if (sortdown_choose == -1)
                      break;
                  if (sortdown_choose == 1)
                      sortdown_name();
                  if (sortdown_choose == 2)
                      sortdown_surname();
                  if (sortdown_choose == 3)
                      sortdown_patronymic();
                  if (sortdown_choose == 4)
                      sortdown_stip();                      
                  break;
             }
             case 10:
             {
                  char filename[80];
                  do
                  {                       
                       cout<< "Ââåäèòå èìÿ ôàéëà: ";
                       cin>> filename;
                       if (ifstream(filename) == false)
                           cerr<< "Íåâîçìîæíî îòêðûòü "<< filename<< endl;
                  } while (ifstream(filename) == false);
                  if (read_from_file (filename) == false)
                      cerr<< "Îøèáêà ïðè ÷òåíèè ôàéëà\n";
                  break;
             }
             case 11:
             {
                  char _filename[80];
                  cout<< "Ââåäèòå èìÿ ôàéëà: ";
                  cin>> _filename;
                  if (write_in_file (_filename) == false)
                      cerr<< "Îøèáêà ïðè çàïèñè â ôàéë\n";
                  break;
             }
             default:
                 cout<< "Âû ââåëè íåâåðíîå çíà÷åíèå";
         }
         cout<< "\nÆåëàåòå ïðîäîëæèòü y/n?\n";
    } while (getch()!= 'n');
    return 0;
}
 
int poiskfree()
{
    for (int i=0; i < N; i++)
        if (s[i].get_free() == true)
            return i;
}
 
int poiski (int x)
{
    for (int i=0; i < N; i++)
    {
        if (s[i].get_stip() == x)
            return i;
        if (i == N-1)
            return -1;
    }
}
 
int poiskc (char *y, int choose)
{
    for (int i=0; i < N; i++)
    {
        if ((choose == 2 && stricmp(s[i].get_name(), y) == 0) || 
            (choose == 3 && stricmp(s[i].get_surname(), y) == 0) ||
            (choose == 4 && stricmp(s[i].get_patronymic(), y) == 0) ||
            (choose == 5 && stricmp(s[i].get_date(), y) == 0 ||
            (choose == 6 && stricmp(s[i].get_patronymic(), y) == 0)))
            return i;
        if (i == N-1)
            return -1;
    }
}
 
int poiskmin()
{
    int min = 65535, minindex = -1;
    for (int i=0; i < N; i++)
        if (s[i].get_free() == false && s[i].get_stip() < min)
        {
            min = s[i].get_stip();
            minindex = i;
        }
    return minindex;
}
 
void sortup_surname()
{
     for (int i = N-1; i > 0; i--)
         for (int j=0; j < i; j++)
             if (s[j].get_surname() > s[j+1].get_surname())
                 swap(s[j], s[j+1]);
}
 
void sortup_name()
{
     for (int i = N-1; i > 0; i--)
         for (int j=0; j < i; j++)
             if (s[j].get_name() > s[j+1].get_name())
                 swap(s[j], s[j+1]);
}
 
void sortup_patronymic()
{
     for (int i = N-1; i > 0; i--)
         for (int j=0; j < i; j++)
             if (s[j].get_patronymic() > s[j+1].get_patronymic())
                 swap(s[j], s[j+1]);
}
 
void sortup_adress()
{
     for (int i = N-1; i > 0; i--)
         for (int j=0; j < i; j++)
             if (s[j].get_adress() > s[j+1].get_adress())
                 swap(s[j], s[j+1]);
}
 
void sortup_stip()
{
     for (int i = N-1; i > 0; i--)
         for (int j=0; j < i; j++)
             if (s[j].get_stip() > s[j+1].get_stip())
                 swap(s[j], s[j+1]);
}
 
void sortdown_surname()
{
     for (int i = N-1; i > 0; i--)
         for (int j=0; j < i; j++)
             if (s[j].get_surname() < s[j+1].get_surname())
                 swap(s[j], s[j+1]);
}
 
void sortdown_name()
{
     for (int i = N-1; i > 0; i--)
         for (int j=0; j < i; j++)
             if (s[j].get_name() < s[j+1].get_name())
                 swap(s[j], s[j+1]);
}
 
void sortdown_patronymic()
{
     for (int i = N-1; i > 0; i--)
         for (int j=0; j < i; j++)
             if (s[j].get_patronymic() < s[j+1].get_patronymic())
                 swap(s[j], s[j+1]);
}
 
void sortdown_adress()
{
     for (int i = N-1; i > 0; i--)
         for (int j=0; j < i; j++)
             if (s[j].get_adress() < s[j+1].get_adress())
                 swap(s[j], s[j+1]);
}
 
void sortdown_stip()
{
     for (int i = N-1; i > 0; i--)
         for (int j=0; j < i; j++)
             if (s[j].get_stip() < s[j+1].get_stip())
                 swap(s[j], s[j+1]);
}
 
bool write_in_file(char* filename)
{
     ofstream o(filename, ios::binary);
     if (o.bad())
         return 0;
     for (int i=0; i < N && !o.eof(); i++)
     {
             o.write((char*)&s[i], sizeof(s[i]));
             if (o.bad())
                 return 0;
     }
     o.close();
     return 1;
}
 
bool read_from_file(char* filename)
{
     ifstream i(filename, ios::binary);
     if (i.bad())
         return 0;
     for (int j=0; j < N && !i.eof(); j++)
     {
         i.read((char*)&s[j], sizeof(s[j]));
         if (i.bad())
             return 0;
     }
     i.close();
     return 1;
}

@aeshes · 09.10.2011, 01:44

1) оператор присваивания не может быть void. Он должен возвращать ссылку на объект, для которого вызван

C++
1
2
3
4
5
const Student& Student::operator=(const Student& z)
{
...
return *this;
}

2) вот эта конструкция мне не нравится

C++
1
s[index](name, surname, patronymic, date, adress, stip);

Или сделать общий метод set, который принимает все эти параметры

C++
1
s[index].set(name, surname, patronymic, date, adress, stip);

или так

C++
1
s[index]=Student(name, surname, patronymic, date, adress, stip);

3) в конструкторе копий и в операторе= желательно очищать память, выделенную под поля объекта до этого, а только потом выделять новую

C++
1
2
3
4
5
6
7
8
9
if(name)
    delete [] name;
if(z.name)
{
    name=new char[strlen(z.name)+1];
    strcpy(name,z.name);
}
else
    name=0;

4) в присваивании можно сделать проверку на самоприсваивание

C++
1
 if(&z==this) return *this;

@rangerx · 09.10.2011, 01:54

Сообщение от MrGluck

Student(char *n, char *s, char *p, char *d, char *a, int st);

Все параметры типа char* предназначенные только для чтения должны быть объявлены c модификатором const.

C++
1
2
Student(const char *n, const char *s, const char *p, 
        const char *d, const char *a, int st);

Сообщение от MrGluck

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
Student::Student(char *n, char *s, char *p, char *d, char *a, int st)
{
    name = new char [strlen(n) + 1];
    strcpy(name, n);
    surname = new char [strlen(s) + 1];
    strcpy(surname, s);
    patronymic = new char [strlen(p) + 1];
    strcpy(patronymic, p);
    date = new char [strlen(d) + 1];
    strcpy(date, d);
    adress = new char [strlen(a) + 1];
    strcpy(adress, a);
    stip = st;
    is_free = true;
}

дублирование кода так и осталось... Всё это следовало бы переписать вот так

C++
1
2
3
4
5
char* cstrdup(const char *src)
{
    char *dest = new char[ strlen(src) + 1 ];
    return strcpy(dest, src);
}

C++
1
2
3
4
5
6
7
8
9
10
11
Student(const char *n, const char *s, const char *p, 
        const char *d, const char *a, int st)
{
    name = cstrdup(n);
    surname = cstrdup(s);
    patronymic = cstrdup(p);
    date = cstrdup(d);
    adress = cstrdup(a);
    stip = st;
    is_free = true;
}

конструтор копий

C++
1
2
3
4
5
6
7
8
9
10
Student::Student(const Student &s)
{
    name = cstrdup(s.name);
    surname = cstrdup(s.surname);
    patronymic = cstrdup(s.patronymic);
    date = cstrdup(s.date);
    adress = cstrdup(s.adress);
    stip = s.stip;
    is_free = s.is_free;
}

оператор присваивания

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
Student &Student::operator = (const Student &s)
{    
    // проверка на присваивание объекта самому себе
    if(&s == this) 
        return *this;
 
    // очищаем выделенную ранее память(можно назвать этот метод как-то более удачно...)
    cleanUp(); 
 
    name = cstrdup(s.name);
    surname = cstrdup(s.surname);
    patronymic = cstrdup(s.patronymic);
    date = cstrdup(s.date);
    adress = cstrdup(s.adress);
    stip = s.stip;
    is_free = s.is_free;
    
    return *this;
}

private метод cleanUp

C++
1
2
3
4
5
6
7
8
void Student::cleanUp()
{
    delete[] name;
    delete[] surname;
    delete[] patronymic;
    delete[] date;
    delete[] adress;
}

деструктор

C++
1
2
3
4
Student::~Student()
{
    cleanUp();
}

@Gepar · 09.10.2011, 02:31

Сообщение от Jupiter

new char (strlen(y) - 1) - создаем один символ значение которого будет strlen(y) - 1 по таблице ASCII

Когда я увидел что скобки круглые (а не квадратные) то я прекрасно понял что оно делает, но я не понял зачем. Ну есть у нас студент с именем "Вася", зачем нам хранить символьную переменную с кодом по таблице 4? Оно ведь никак не характеризует нашу строку Вася так как строку Петя и Гадя оно так же характеризует кодом 4.

Добавлено через 3 минуты
Сстоп, но ведь по коду там дальше идёт копирование в этот участок памяти отведённый под один символ всей полученной строки в аргументах функции и никакими проверками насчёт размера строки в виде указателя на char там и не пахнет ...хотя может это автор хотел передать через указатель на char этой функции один символ ... в общем я окончательно запутался, нужно слово автора по поводу назначения этой функции и что же она должна делать.

MrGluck · 09.10.2011, 04:20 **[ТС]**

Я хотел выделить память под символьный массив, просто ошибся

Добавлено через 1 час 23 минуты
rangerx,
aeshes, спасибо за помощь.

Остался последний вопрос: вот у меня есть функции записи в файл и чтения с файла в бинарном режиме. Если записать и считать с файла не закрывая программу, то все нормально, а если записать в файл, закрыть программу, снова запустить и попытаться считать с файла, то ничего не выводит и более того, программа аварийно завершается. Как устранить эту ошибку и с чем она может быть связанна?

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
bool write_in_file(char* filename)
{
     ofstream o(filename, ios::binary);
     if (o.bad())
         return false;
     for (int i=0; i < N && !o.eof(); i++)
     {
             o.write((char*)&s[i], sizeof(s[i]));
             if (o.bad())
                 return false;
     }
     o.close();
     return true;
}
 
bool read_from_file(char* filename)
{
     ifstream i(filename, ios::binary);
     if (i.bad())
         return false;
     for (int j=0; j < N && !i.eof(); j++)
     {
         i.read((char*)&s[j], sizeof(s[j]));
         if (i.bad())
             return false;
     }
     i.close();
     return true;
}

@Gepar · 09.10.2011, 13:41

Сообщение от MrGluck

Я хотел выделить память под символьный массив, просто ошибся

ага, ну вот, значит мои мысли в начале по поводу квадратных скобок верные, ну тогда надо выделять памяти столько же, length возвращает длину строки вместе с нуль символом, если я не ошибаюсь )

MrGluck, попробуйте перед чтением файла устанавливать положение в начало файла через seekg, это по поводу вашего последнего вопроса, хотя не ясно что у вас там за s[j], это глобальная какая-то переменная чтоли, в общем попробуйте пока вручную устанавливать положение для чтения в начало файла, а если не поможет то будем искать другую возможную проблему )

MrGluck · 09.10.2011, 15:59 **[ТС]**

Сообщение от Gepar

MrGluck, попробуйте перед чтением файла устанавливать положение в начало файла через seekg,

Рано я обрадовался, не помогло(

@Gepar · 09.10.2011, 16:45

MrGluck, ну тогда наверняка у Вас где-то ошибка не в этой функции, смотрите своих студентов - глобальный массив на n элементов, может вы задаёт его не достаточным по размерам для того чтобы всех студентов считывать, попробуйте задавать его гораздо большим (в main) чем студентов в файле перед считыванием, а вообще глобальные переменные - зло, теперь сложно разобраться читая Ваш код.

@rangerx · 10.10.2011, 13:51

MrGluck, так сохранять\читать объекты класса нельзя(в данном случае именно "нельзя", а не "не рекомендуется")... Вообще, я бы советовал тебе сохранить всё в текстовом виде(просто, надёжно, и переносимо

).
P.S. весь код в main.cpp не мешало бы переписать. Или хотя бы исправить явные ошибки вроде

C++
1
if (s[j].get_name() > s[j+1].get_name()) // сравнение указателей, а не строк

MrGluck · 10.10.2011, 18:44 **[ТС]**

Сообщение от rangerx

MrGluck, так сохранять\читать объекты класса нельзя(в данном случае именно "нельзя", а не "не рекомендуется")... Вообще, я бы советовал тебе сохранить всё в текстовом виде(просто, надёжно, и переносимо

).

А как можно? Записывать их не как элемент, а почленно? По заданию необходимо сохранять/считывать в бинарном формате.

P.S. весь код в main.cpp не мешало бы переписать. Или хотя бы исправить явные ошибки вроде

C++
1
if (s[j].get_name() > s[j+1].get_name()) // сравнение указателей, а не строк

Ну а как иначе сравнить name, не объявляя их public?

Новые блоги и статьи Все статьи Все блоги /
Вот уже год прошел, как у меня домен в reg.ru ... Etyuhibosecyu 16.04.2026 И ничего они мне не сделали. Если отвязать карту, никакие услуги они не навяжут. Я бы с радостью продлил еще на два года, чтобы не мучиться с временным доменом и меня уже знали по red-star-soft. com,. . .	Знаешь почему 90% людей редко бывают счастливыми? kumehtar 14.04.2026 Потому что они ждут. Ждут выходных, ждут отпуска, ждут удачного момента. . . а удачный момент так и не приходит.	Фиксация колонок в отчете СКД Maks 14.04.2026 Фиксация колонок в СКД отчета типа Таблица. Задача: зафиксировать три левых колонки в отчете. Процедура ПриКомпоновкеРезультата(ДокументРезультат, ДанныеРасшифровки, СтандартнаяОбработка) / / . . .	Настройки VS Code Loafer 13.04.2026 { "cmake. configureOnOpen": false, "diffEditor. ignoreTrimWhitespace": true, "editor. guides. bracketPairs": "active", "extensions. ignoreRecommendations": true, . . .
Оптимизация кода на разграничение прав доступа к элементам формы Maks 13.04.2026 Алгоритм из решения ниже реализован на нетиповом документе, разработанного в конфигурации КА2. Задачи, как таковой, поставлено не было, проделанное ниже исключительно моя инициатива. Было так:. . .	Контроль заполнения и очистка дат в зависимости от значения перечислений Maks 12.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "ПланированиеПерсонала", разработанного в конфигурации КА2. Задача: реализовать контроль корректности заполнения дат назначения. . .	Архитектура слоя интернета для сервера-слоя. Hrethgir 11.04.2026 В продолжение https:/ / www. cyberforum. ru/ blogs/ 223907/ 10860. html Знаешь что я подумал? Раз мы все источники пишем в голове ветки, то ничего не мешает добавить в голову такой источник, который сам. . .	Подстановка значения реквизита справочника в табличную часть документа Maks 10.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "ПланированиеПерсонала", разработанного в конфигурации КА2. Задача №1: при указании работ (справочник РаботыПоРемонтуСпецтехники),. . .

@Deviaphan Делаю внезапно и красиво 1313 / 1228 / 72 Регистрация: 22.03.2011 Сообщений: 3,744
	08.10.2011, 11:01
	Нет деструктора. Читать не буду, многабукаф. 1

@Gepar 1186 / 543 / 78 Регистрация: 01.07.2009 Сообщений: 3,517
	08.10.2011, 19:31
	Jupiter, а, там же не квадратные скобки, ну тогда назначение мне тем более не понятно, но видать это какая-то хитрая задумка автора. 1

@Gepar 1186 / 543 / 78 Регистрация: 01.07.2009 Сообщений: 3,517
	09.10.2011, 16:45
	MrGluck, ну тогда наверняка у Вас где-то ошибка не в этой функции, смотрите своих студентов - глобальный массив на n элементов, может вы задаёт его не достаточным по размерам для того чтобы всех студентов считывать, попробуйте задавать его гораздо большим (в main) чем студентов в файле перед считыванием, а вообще глобальные переменные - зло, теперь сложно разобраться читая Ваш код. 0