Ошибка при передаче результата функции

@MayaNash · Регистрация: 24.08.2011

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

Оператор + возвращает правильное значение (по крайней мере в конце его тела я вывожу полученный список, и все верно, он возвращает его по значению), затем это значение по ссылке передается в функцию вывода, и там оказывается, что первый элемент списка испорчен - неправильный указатель. Как это может быть?

discipline.h:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
#pragma once
 
struct Discipline
{
    int number, semester, totalHours, classroomHours;
    char type; // f - федеральная, r - региональная, v - по выбору
    char name[50];
    bool finalControl; // false - зачет, true - экзамен
};
 
struct ListOfDisciplines
{
    Discipline subject;
    int selfStudyHours;
    bool courseWork;
    ListOfDisciplines *prev, *next;
};

planeducation.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
30
31
32
33
34
35
#pragma once
#include "discipline.h"
#include <iostream>
using std::ostream;
 
class PlanEducation
{
    friend ostream& operator<<(ostream &, PlanEducation &);
public:
    PlanEducation(); // читает с клавиатуры необходимые поля
    PlanEducation(bool); // независимо от значения аргумента, ничего не читает, только создает
    ~PlanEducation(); // корректно удаляет список
    void Include(); // читает с клавиатуры поля дисциплины, вставляет на нужное место (сортировка по семестру)
    void Exclude(); // читает с клавиатуры название дисциплины, ищет, удаляет
    void FindSemester(int sem);
    void FindType(char t);
    void FindFinalControl(bool t);
    int TotalHours();
    void CalculateExamsCredits();
    PlanEducation operator+(PlanEducation &r);
    PlanEducation operator-(PlanEducation &r);
    PlanEducation operator*(PlanEducation &r);
    ListOfDisciplines* GroupSemester(int sem);
    ListOfDisciplines* GroupType(char t);
    ListOfDisciplines* GroupFinalControl(bool t);
    ListOfDisciplines* GroupCourseWork(bool c);
private:
    char code[15], name[50], approvalDate[10];
    int totalHoursStd;
    ListOfDisciplines *list;
    bool NotFound(ListOfDisciplines &);
    void Include(ListOfDisciplines &);
    void Exclude(ListOfDisciplines &);
    int nExams(int sem);
};

planeducation.cpp (operator+ - строка 304, operator << - в самом конце, PrintList в 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
#include "planeducation.h"
#include <iostream>
using std::cin;
using std::cout;
using std::endl;
 
PlanEducation::PlanEducation()
{
    fflush(stdin);
    cout << "Код специальности: ";
    gets(code);
    cout << "Название специальности: ";
    gets(name);
    cout << "Дата утверждения (дд.мм.гггг): ";
    gets(approvalDate);
    cout << "Общее количество часов специальности по стандарту: ";
    cin >> totalHoursStd;
 
    int n;
    do
    {
        cout << "Количество дисциплин: ";
        cin >> n;
    }
    while (n <= 0);
    list = NULL;
    for (int i = 0; i < n; i++)
    {
        cout << "\nДисциплина №" << i+1 << endl;
        Include();
    }
    cout << endl;
    
    for (ListOfDisciplines *tek = list; tek != NULL; tek = tek->next)
        if (tek->prev == NULL || tek->subject.semester != tek->prev->subject.semester)
            while (nExams(tek->subject.semester) < 3)
            {
                cout << "Количество экзаменов в " << tek->subject.semester << "-м семестре меньше 3!\nВведите еще одну дисциплину с экзаменом!\n";
                Include();
                cout << endl;
            }
}
 
PlanEducation::PlanEducation(bool b): totalHoursStd(0), list(NULL)
{
    strcpy(code,"");
    strcpy(name,"");
    strcpy(approvalDate,"");
}
 
PlanEducation::~PlanEducation()
{
    ListOfDisciplines *tmp;
    while (list != NULL)
    {
        tmp = list;
        list = list->next;
        delete tmp;
    }
}
 
void PlanEducation::Include()
{
    ListOfDisciplines *newD = new ListOfDisciplines;
    cout << "Название дисциплины: ";
    fflush(stdin);
    gets(newD->subject.name);
    cout << "Семестр: ";
    cin >> newD->subject.semester;
    cout << "Тип (f - федеральная, r - региональная, v - по выбору): ";
    cin >> newD->subject.type;
    fflush(stdin);
    do
    {
        cout << "Общее количество часов: ";
        cin >> newD->subject.totalHours;
        if (TotalHours() + newD->subject.totalHours > totalHoursStd)
            cout << "Общее количество часов превышает стандарт! Введите меньшее количество часов!\n";
    }
    while (TotalHours() + newD->subject.totalHours > totalHoursStd);
    cout << "Количество аудиторных часов: ";
    cin >> newD->subject.classroomHours;
    newD->selfStudyHours = newD->subject.totalHours - newD->subject.classroomHours;
    char ch;
    do
    {
        cout << "Вид итогового контроля (e - экзамен, z - зачет): ";
        cin >> ch;
        fflush(stdin);
    }
    while (ch != 'e' && ch != 'z');
    if (ch == 'e')
        if (nExams(newD->subject.semester) < 5)
            newD->subject.finalControl = true;
        else
        {
            cout << "В семестре дожно быть не более пяти экзаменов! В качестве итогового контроля будет зачет\n";
            newD->subject.finalControl = false;
        }
    else
        newD->subject.finalControl = false;
    do
    {
        cout << "Наличие курсовой работы (d/n): ";
        cin >> ch;
    }
    while (ch != 'd' && ch != 'n');
    if (ch == 'd')
        newD->courseWork = true;
    else
        newD->courseWork = false;
 
    if (list == NULL)
    {
        list = newD;
        list->subject.number = 1;
        list->prev = NULL;
        list->next = NULL;
    }
    else
    {
        ListOfDisciplines *tek;
        for (tek = list; tek->subject.semester < newD->subject.semester && tek->next != NULL; tek = tek->next);
        newD->subject.number = tek->subject.number + 1;
        newD->next = tek->next;
        tek->next = newD;
        newD->prev = tek;
        if (newD->next != NULL)
            newD->next->prev = newD;
        for (tek = newD->next; tek != NULL; tek = tek->next)
            tek->subject.number++;
    }
}
 
void PlanEducation::Exclude()
{
    char name[50];
    cout << "Название дисциплины: ";
    gets(name);
    ListOfDisciplines *toExclude;
    for (toExclude = list; toExclude != NULL && strcmp(toExclude->subject.name,name) != 0; toExclude = toExclude->next);
    if (toExclude != NULL)
    {
        if (toExclude->prev != NULL)
            toExclude->prev->next = toExclude->next;
        else
            list = list->next;
        if (toExclude->next != NULL)
            toExclude->next->prev = toExclude->prev;
        for (ListOfDisciplines *tek = toExclude->next; tek != NULL; tek = tek->next)
            tek->subject.number--;
        delete toExclude;
    }
}
 
void PlanEducation::FindSemester(int sem)
{
    bool found = false;
    for (ListOfDisciplines *tek = list; tek != NULL; tek = tek->next)
        if (tek->subject.semester == sem)
        {
            cout << tek->subject.name << endl;
            found = true;
        };
    if (!found)
        cout << "Не найдено\n";
    system("pause");
}
 
void PlanEducation::FindType(char t)
{
    bool found = false;
    for (ListOfDisciplines *tek = list; tek != NULL; tek = tek->next)
        if (tek->subject.type == t)
        {
            cout << tek->subject.name << " - " << tek->subject.semester << "-й семестр\n";
            found = true;
        };
    if (!found)
        cout << "Не найдено\n";
    system("pause");
}
    
void PlanEducation::FindFinalControl(bool t)
{
    bool found = false;
    for (ListOfDisciplines *tek = list; tek != NULL; tek = tek->next)
        if (tek->subject.finalControl == t)
        {
            cout << tek->subject.name << " - " << tek->subject.semester << "-й семестр\n";
            found = true;
        };
    if (!found)
        cout << "Не найдено\n";
    system("pause");
}
 
int PlanEducation::TotalHours()
{
    int h = 0;
    for (ListOfDisciplines *tek = list; tek != NULL; tek = tek->next)
        h += tek->subject.totalHours;
    return h;
}
 
void PlanEducation::CalculateExamsCredits()
{
    int exams = 0, credits = 0;
    if (list->subject.finalControl == true)
        exams++;
    else
        credits++;
 
    for (ListOfDisciplines *tek = list->next; tek != NULL; tek = tek->next)
    {
        if (tek->subject.semester != tek->prev->subject.semester)
        {
            cout << "Семестр " << tek->prev->subject.semester << ": " << exams << " экзамен(-а/-ов), "
                << credits << " зачет(-а/-ов)\n";
            if (tek->subject.finalControl == 'e')
                exams = 1, credits = 0;
            else
                exams = 0, credits = 1;
        }
        else
            if (tek->subject.finalControl == true)
                exams++;
            else
                credits++;
        if (tek->next == NULL)
            cout << "Семестр " << tek->subject.semester << ": " << exams << " экзамен(-а/-ов), "
                << credits << " зачет(-а/-ов)\n";
    }
    system("pause");
}
 
bool PlanEducation::NotFound(ListOfDisciplines &l)
{
    for (ListOfDisciplines *tek = list; tek != NULL; tek = tek->next)
        if (strcmp(tek->subject.name, l.subject.name) == 0)
            return false;
    return true;
}
 
void PlanEducation::Include(ListOfDisciplines &l)
{
    if (!NotFound(l))
        return;
 
    ListOfDisciplines *newD = new ListOfDisciplines(l);
    strcpy(newD->subject.name,l.subject.name);
 
    if (list == NULL)
    {
        list = newD;
        list->subject.number = 1;
        list->prev = NULL;
        list->next = NULL;
    }
    else
    {
        ListOfDisciplines *tek;
        for (tek = list; tek->subject.semester < newD->subject.semester && tek->next != NULL; tek = tek->next);
        if (tek->subject.semester < newD->subject.semester)
        {
            newD->next = list;
            newD->prev = NULL;
            list = newD;
        }
        else
        {
            newD->subject.number = tek->subject.number + 1;
            newD->next = tek->next;
            tek->next = newD;
            newD->prev = tek;
            if (newD->next != NULL)
                newD->next->prev = newD;
            for (tek = newD->next; tek != NULL; tek = tek->next)
                tek->subject.number++;
        }
    }
}
 
void PlanEducation::Exclude(ListOfDisciplines &l)
{
    ListOfDisciplines *toExclude;
    for (toExclude = list; toExclude != NULL && strcmp(toExclude->subject.name, l.subject.name) != NULL;
        toExclude = toExclude->next);
 
    if (toExclude != NULL)
    {
        if (toExclude->prev != NULL)
            toExclude->prev->next = toExclude->next;
        else
            list = list->next;
        if (toExclude->next != NULL)
            toExclude->next->prev = toExclude->prev;
        for (ListOfDisciplines *tek = toExclude->next; tek != NULL; tek = tek->next)
            tek->subject.number--;
        delete toExclude;
    }
}
 
PlanEducation PlanEducation::operator+(PlanEducation &r)
{
    PlanEducation newPlan(true);
    ListOfDisciplines *tek;
 
    for (tek = list; tek != NULL; tek = tek->next)
        newPlan.Include(*tek);
    for (tek = r.list; tek != NULL; tek = tek->next)
        newPlan.Include(*tek);
 
    extern void PrintList(ListOfDisciplines *l);
    PrintList(newPlan.list);
    system("pause");
    return newPlan;
}
 
PlanEducation PlanEducation::operator-(PlanEducation &r)
{
    PlanEducation newPlan(true);
    ListOfDisciplines *tek;
 
    for (tek = list; tek != NULL; tek = tek->next)
        newPlan.Include(*tek);
    for (tek = r.list; tek != NULL; tek = tek->next)
        newPlan.Exclude(*tek);
 
    return newPlan;
}
 
PlanEducation PlanEducation::operator*(PlanEducation &r)
{
    PlanEducation newPlan(*this + r);
    extern void PrintList(ListOfDisciplines *l);
    cout << "*this + r: ";
    PrintList(newPlan.list);
    ListOfDisciplines *tek;
 
    for (tek = newPlan.list; tek != NULL; tek = tek->next)
        if (NotFound(*tek) || r.NotFound(*tek))
            newPlan.Exclude(*tek);
 
    return newPlan;
}
 
int PlanEducation::nExams(int sem)
{
    int n = 0;
    for (ListOfDisciplines *tek = list; tek != NULL; tek = tek->next)
        if (tek->subject.semester == sem)
            n++;
        else if (tek->prev != NULL && tek->prev->subject.semester == sem)
            break;
    return n;
}
 
ListOfDisciplines* PlanEducation::GroupSemester(int sem)
{
    ListOfDisciplines *newGroup = NULL, *newElement;
    for (ListOfDisciplines *tek = list; tek != NULL; tek = tek->next)
        if (tek->subject.semester == sem)
        {
            newElement = new ListOfDisciplines(*tek);
            strcpy(newElement->subject.name, tek->subject.name);
            newElement->prev = NULL;
            newElement->next = newGroup;
            newGroup = newElement;
        };
    return newGroup;
}
 
ListOfDisciplines* PlanEducation::GroupType(char t)
{
    ListOfDisciplines *newGroup = NULL, *newElement;
    for (ListOfDisciplines *tek = list; tek != NULL; tek = tek->next)
        if (tek->subject.type == t)
        {
            newElement = new ListOfDisciplines(*tek);
            strcpy(newElement->subject.name, tek->subject.name);
            newElement->prev = NULL;
            newElement->next = newGroup;
            newGroup = newElement;
        };
    return newGroup;
}
 
ListOfDisciplines* PlanEducation::GroupFinalControl(bool t)
{
    ListOfDisciplines *newGroup = NULL, *newElement;
    for (ListOfDisciplines *tek = list; tek != NULL; tek = tek->next)
        if (tek->subject.finalControl == t)
        {
            newElement = new ListOfDisciplines(*tek);
            strcpy(newElement->subject.name, tek->subject.name);
            newElement->prev = NULL;
            newElement->next = newGroup;
            newGroup = newElement;
        };
    return newGroup;
}
 
ListOfDisciplines* PlanEducation::GroupCourseWork(bool c)
{
    ListOfDisciplines *newGroup = NULL, *newElement;
    for (ListOfDisciplines *tek = list; tek != NULL; tek = tek->next)
        if (tek->courseWork == c)
        {
            newElement = new ListOfDisciplines(*tek);
            strcpy(newElement->subject.name, tek->subject.name);
            newElement->prev = NULL;
            newElement->next = newGroup;
            newGroup = newElement;
        };
    return newGroup;
}
 
ostream& operator<<(ostream &out, PlanEducation &p)
{
    extern void PrintList(ListOfDisciplines *l);
    PrintList(p.list);
    return out;
}

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
#include "planeducation.h"
#include <iostream>
using std::cin;
using std::cout;
using std::endl;
 
void Delete(ListOfDisciplines *l);
extern void PrintList(ListOfDisciplines *l);
 
int main()
{
    setlocale(LC_ALL,"rus");
 
    cout << "Ввод исходных данных\n";
    PlanEducation plan;
    /*cout << "\nДисциплины какого семестра найти? ";
    int sem;
    cin >> sem;
    plan.FindSemester(sem);
    cout << "\nКакой тип дисциплин найти? (f - федеральная, r - региональная, v - по выбору) ";
    char ch;
    cin >> ch;
    plan.FindType(ch);
    cout << "\nДисциплины с каким видом итогового контроля найти? (e - экзамен, z - зачет) ";
    cin >> ch;
    if (ch == 'e')
        plan.FindFinalControl(true);
    else
        plan.FindFinalControl(false);
    cout << "\nОбщее количество часов на специальности: " << plan.TotalHours()
        << "\n\nКоличество зачетов и экзаменов по семестрам:\n";
    plan.CalculateExamsCredits();
 
    cout << "\nВыбрать группу, содержащую список дисциплин семестра № ";
    cin >> sem;
    ListOfDisciplines *Group = plan.GroupSemester(sem);
    cout << sem << "-й семестр: ";
    PrintList(Group);
    Delete(Group);
 
    cout << "\nВыбрать группу, содержащую список дисциплин типа (f - федеральная, r - региональная, v - по выбору): ";
    cin >> ch;
    Group = plan.GroupType(ch);
    if (ch == 'f')
        cout << "Дисциплины федерального типа: ";
    else if (ch == 'r')
        cout << "Дисциплины регионального типа: ";
    else
        cout << "Дисциплины по выбору: ";
    PrintList(Group);
    Delete(Group);
 
    do
    {
        cout << "\nВыбрать группу, содержащую список дисциплин с видом итогового контроля (e - экзамен, z - зачет): ";
        cin >> ch;
    }
    while (ch != 'e' && ch != 'z');
    Group = (ch == 'e'? plan.GroupFinalControl(true) : plan.GroupFinalControl(false));
    PrintList(Group);
    Delete(Group);
 
    do
    {
        cout << "\nВыбрать группу, содержащую список дисциплин с наличием курсовой работы (d/n): ";
        cin >> ch;
    }
    while (ch != 'd' && ch != 'n');
    Group = (ch == 'd'? plan.GroupCourseWork(true) : plan.GroupCourseWork(false));
    PrintList(Group);
    Delete(Group);*/
 
    cout << "\nВвод исходных данных для второго учебного плана\n";
    PlanEducation plan2;
    cout << "\nПлан №1 + план №2: " << plan + plan2;
    cout << "План №1 - план №2: " << plan - plan2;
    cout << "План №1 U план №2: " << plan * plan2;
 
    system("pause");
    return 0;
}
 
void Delete(ListOfDisciplines *l)
{
    ListOfDisciplines *tmp;
    while (l != NULL)
    {
        tmp = l;
        l = l->next;
        delete tmp;
    }
}
 
void PrintList(ListOfDisciplines *l)
{
    for (ListOfDisciplines *tek = l; tek != NULL; tek = tek->next)
        cout << tek->subject.name << (tek->next != NULL? ", " : "\n");
}

Добавлено через 33 минуты
актуально

alsav22 · 08.06.2014, 18:48

Сообщение от Керра

затем это значение по ссылке передается в функцию вывода

Ссылка на локальный объект? Передавайте по значению.

@MayaNash · 08.06.2014, 18:54 **[ТС]**

alsav22, не помогло

@BRcr · 08.06.2014, 18:58

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

C++
1
2
3
4
5
6
7
    PlanEducation plan2;
    PlanEducation res( plan + plan2 );
    cout << "\nПлан №1 + план №2: " << res;
    res = plan - plan2;
    cout << "План №1 - план №2: " << res;
    res = plan * plan2;
    cout << "План №1 U план №2: " << res;

Или, как вариант, пусть operator << принимает не по ссылке, тогда откопируется нормально.

C++
1
2
3
4
5
6
ostream& operator<<(ostream &out, PlanEducation p)
{
    extern void PrintList(ListOfDisciplines *l);
    PrintList(p.list);
    return out;
}

@MayaNash · 08.06.2014, 19:00 **[ТС]**

BRcr, пробовала - не помогло

@BRcr · 08.06.2014, 19:05

Ну, тогда единственный напрашивающийся виновник - оператор копирования.

@MayaNash · 08.06.2014, 19:05 **[ТС]**

BRcr, тоже как раз об этом подумала, сейчас буду пробовать

Tulosba · 08.06.2014, 19:07

Керра, а использовать std::string вместо char[] и std::list вместо самопальной реализации нельзя? В глубины кода вникать не хочется, но интуиция подсказывает, что дело в копирующем конструкторе PlanEducation. Точнее в отсутствии его реализации.

@MayaNash · 08.06.2014, 19:30 **[ТС]**

Tulosba, string надо бы, а список по заданию надо вручную писать

Добавлено через 3 минуты
Реализовала копировщик. Теперь ошибка в strcmp на 125 строчке...

Это правильно вообще?

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
PlanEducation::PlanEducation(PlanEducation &r)
{
    *this = r;
}
 
PlanEducation PlanEducation::operator=(PlanEducation &r)
{
    totalHoursStd = r.totalHoursStd;
    strcpy(this->code,r.code);
    strcpy(this->name,r.name);
    strcpy(this->approvalDate,r.approvalDate);
    for (ListOfDisciplines *tek = r.list; tek != NULL; tek = tek->next)
        Include(*tek);
    return *this;
}

Добавлено через 10 минут
Сделала так:

C++
1
2
3
4
5
6
7
8
cout << "\nВвод исходных данных для второго учебного плана\n";
    PlanEducation plan2;
    PlanEducation res(plan + plan2);
    cout << "\nПлан №1 + план №2: " << res;
    res = plan - plan2;
    cout << "План №1 - план №2: " << res;
    res = plan * plan2;
    cout << "План №1 U план №2: " << res;

Прокрутила пошагово, почему-то оказалось, что эта ошибка в strcmp.asm происходит при выходе из operator+.

Tulosba · 08.06.2014, 19:36

Керра, во-первых, если всё таки можно использовать какие-то стандартные типы, я бы посоветовал их использовать (т.е. для начала заменить строки на std::string). Во-вторых, если список нужно реализовать самому, то сделать это стоит отдельно от всех остальных классов, т.е. чтобы он вёл себя так же как std::list, по крайней мере, в той части, которая необходима в программе. Это позволит избежать всяческих сложностей как в функциях Include/Exclude. Объекты и функции, которые используют списки, ничего не должны знать о их внутреннем устройстве. Разделяй и властвуй. В-третьих, если аргумент не предполагается изменять, передавать его следует по константной ссылке. И в-четвертых, при реализации своих классов не забывайте про правило трёх.

@MayaNash · 08.06.2014, 19:52 **[ТС]**

Tulosba, ну, за исключением того, что список состоит конкретно из объектов класса Discipline (в данной ситуации нет смысла делать шаблон), то все ок. Насчет Include/Exclude не поняла - там вроде никаких сложностей нет, в Include происходит сортировка по полю subject.semester, а в Exclude все вроде очевидно. Правилу трех тоже уже соответствует. С константной ссылкой тоже уже шлифовка.
Стандартные классы и функции конечно хотелось бы использовать, но по заданию надо все вручную делать. Даже насчет String не уверена что можно.

Tulosba · 08.06.2014, 20:24

Сообщение от Керра

Насчет Include/Exclude не поняла - там вроде никаких сложностей нет

Сложность из-за открытой реализации списка. Следует сделать нормальный класс, с приватной секций для данных и публичными методами доступа к ним. Чтобы код, который использует списки, оперировал функциями типа Add, Remove и пр., а не шаманил с указателями prev,next напрямую. Так же у списка должны быть свои конструктор/деструктор. Всё это уменьшает связность, а значит упрощает понимание кода и его поддержку.

@MayaNash · 08.06.2014, 20:28 **[ТС]**

Tulosba, закрытая ж секция

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
#pragma once
#include "discipline.h"
#include <iostream>
using std::ostream;
 
class PlanEducation
{
    friend ostream& operator<<(ostream &, PlanEducation);
public:
    PlanEducation(); // читает с клавиатуры необходимые поля
    PlanEducation(bool); // независимо от значения аргумента, ничего не читает, только создает
    PlanEducation(PlanEducation &);
    ~PlanEducation(); // корректно удаляет список
    void Include(); // читает с клавиатуры поля дисциплины, вставляет на нужное место (сортировка по семестру)
    void Exclude(); // читает с клавиатуры название дисциплины, ищет, удаляет
    void FindSemester(int sem);
    void FindType(char t);
    void FindFinalControl(bool t);
    int TotalHours();
    void CalculateExamsCredits();
    PlanEducation* operator+(PlanEducation &r);
    PlanEducation* operator-(PlanEducation &r);
    PlanEducation* operator*(PlanEducation &r);
    PlanEducation operator=(PlanEducation const &r);
    ListOfDisciplines* GroupSemester(int sem);
    ListOfDisciplines* GroupType(char t);
    ListOfDisciplines* GroupFinalControl(bool t);
    ListOfDisciplines* GroupCourseWork(bool c);
private:
    char code[15], name[50], approvalDate[10];
    int totalHoursStd;
    ListOfDisciplines *list;
    bool NotFound(ListOfDisciplines &);
    void Include(ListOfDisciplines &);
    void Exclude(ListOfDisciplines &);
    int nExams(int sem);
};

Добавлено через 39 секунд
Tulosba, а, поняла. Но то ж структура, по заданию именно структура должна быть.

@BRcr · 08.06.2014, 20:37

Сообщение от Керра

Но то ж структура, по заданию именно структура должна быть.

Ну, пусть и будет себе структура. Только сделай для нее обертку в виде класса, как Tulosba советует. И все довольны.

Новые блоги и статьи Все статьи Все блоги /
PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .
Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .	Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .	Мысли в слух kumehtar 18.11.2025 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .	Создание Single Page Application на фреймах krapotkin 16.11.2025 Статья исключительно для начинающих. Подходы оригинальностью не блещут. В век Веб все очень привыкли к дизайну Single-Page-Application . Быстренько разберем подход "на фреймах". Мы делаем одну. . .

@MayaNash 1296 / 470 / 151 Регистрация: 24.08.2011 Сообщений: 2,249
	08.06.2014, 18:54 [ТС]
	alsav22, не помогло 0

@MayaNash 1296 / 470 / 151 Регистрация: 24.08.2011 Сообщений: 2,249
	08.06.2014, 19:00 [ТС]
	BRcr, пробовала - не помогло 0

@BRcr 4043 / 2333 / 292 Регистрация: 03.02.2011 Сообщений: 5,066 Записей в блоге: 10
	08.06.2014, 19:05
	Ну, тогда единственный напрашивающийся виновник - оператор копирования. 0

@MayaNash 1296 / 470 / 151 Регистрация: 24.08.2011 Сообщений: 2,249
	08.06.2014, 19:05 [ТС]
	BRcr, тоже как раз об этом подумала, сейчас буду пробовать 0

Tulosba :) 4773 / 3267 / 497 Регистрация: 19.02.2013 Сообщений: 9,046
	08.06.2014, 19:07
	Керра, а использовать std::string вместо char[] и std::list вместо самопальной реализации нельзя? В глубины кода вникать не хочется, но интуиция подсказывает, что дело в копирующем конструкторе PlanEducation. Точнее в отсутствии его реализации. 0

Tulosba :) 4773 / 3267 / 497 Регистрация: 19.02.2013 Сообщений: 9,046
	08.06.2014, 19:36
	Керра, во-первых, если всё таки можно использовать какие-то стандартные типы, я бы посоветовал их использовать (т.е. для начала заменить строки на std::string). Во-вторых, если список нужно реализовать самому, то сделать это стоит отдельно от всех остальных классов, т.е. чтобы он вёл себя так же как std::list, по крайней мере, в той части, которая необходима в программе. Это позволит избежать всяческих сложностей как в функциях Include/Exclude. Объекты и функции, которые используют списки, ничего не должны знать о их внутреннем устройстве. Разделяй и властвуй. В-третьих, если аргумент не предполагается изменять, передавать его следует по константной ссылке. И в-четвертых, при реализации своих классов не забывайте про правило трёх. 0

@MayaNash 1296 / 470 / 151 Регистрация: 24.08.2011 Сообщений: 2,249
	08.06.2014, 19:52 [ТС]
	Tulosba, ну, за исключением того, что список состоит конкретно из объектов класса Discipline (в данной ситуации нет смысла делать шаблон), то все ок. Насчет Include/Exclude не поняла - там вроде никаких сложностей нет, в Include происходит сортировка по полю subject.semester, а в Exclude все вроде очевидно. Правилу трех тоже уже соответствует. С константной ссылкой тоже уже шлифовка. Стандартные классы и функции конечно хотелось бы использовать, но по заданию надо все вручную делать. Даже насчет String не уверена что можно. 0