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

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

Восстановить пароль Регистрация
 
Керра
Модератор
 Аватар для Керра
1270 / 438 / 45
Регистрация: 24.08.2011
Сообщений: 2,123
08.06.2014, 18:33     Ошибка при передаче результата функции #1
Оператор + возвращает правильное значение (по крайней мере в конце его тела я вывожу полученный список, и все верно, он возвращает его по значению), затем это значение по ссылке передается в функцию вывода, и там оказывается, что первый элемент списка испорчен - неправильный указатель. Как это может быть?

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
5282 / 4801 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
08.06.2014, 18:48     Ошибка при передаче результата функции #2
Цитата Сообщение от Керра Посмотреть сообщение
затем это значение по ссылке передается в функцию вывода
Ссылка на локальный объект? Передавайте по значению.
Керра
Модератор
 Аватар для Керра
1270 / 438 / 45
Регистрация: 24.08.2011
Сообщений: 2,123
08.06.2014, 18:54  [ТС]     Ошибка при передаче результата функции #3
alsav22, не помогло
BRcr
 Аватар для BRcr
4003 / 2292 / 155
Регистрация: 03.02.2011
Сообщений: 5,064
Записей в блоге: 10
08.06.2014, 18:58     Ошибка при передаче результата функции #4
Что-то с временными объектами в стэке происходит... но в чем корень зла - как-то я не уверен.
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;
}
Керра
Модератор
 Аватар для Керра
1270 / 438 / 45
Регистрация: 24.08.2011
Сообщений: 2,123
08.06.2014, 19:00  [ТС]     Ошибка при передаче результата функции #5
BRcr, пробовала - не помогло
BRcr
 Аватар для BRcr
4003 / 2292 / 155
Регистрация: 03.02.2011
Сообщений: 5,064
Записей в блоге: 10
08.06.2014, 19:05     Ошибка при передаче результата функции #6
Ну, тогда единственный напрашивающийся виновник - оператор копирования.
Керра
Модератор
 Аватар для Керра
1270 / 438 / 45
Регистрация: 24.08.2011
Сообщений: 2,123
08.06.2014, 19:05  [ТС]     Ошибка при передаче результата функции #7
BRcr, тоже как раз об этом подумала, сейчас буду пробовать
Tulosba
:)
Эксперт С++
4378 / 3221 / 297
Регистрация: 19.02.2013
Сообщений: 9,044
08.06.2014, 19:07     Ошибка при передаче результата функции #8
Керра, а использовать std::string вместо char[] и std::list вместо самопальной реализации нельзя? В глубины кода вникать не хочется, но интуиция подсказывает, что дело в копирующем конструкторе PlanEducation. Точнее в отсутствии его реализации.
Керра
Модератор
 Аватар для Керра
1270 / 438 / 45
Регистрация: 24.08.2011
Сообщений: 2,123
08.06.2014, 19:30  [ТС]     Ошибка при передаче результата функции #9
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
:)
Эксперт С++
4378 / 3221 / 297
Регистрация: 19.02.2013
Сообщений: 9,044
08.06.2014, 19:36     Ошибка при передаче результата функции #10
Керра, во-первых, если всё таки можно использовать какие-то стандартные типы, я бы посоветовал их использовать (т.е. для начала заменить строки на std::string). Во-вторых, если список нужно реализовать самому, то сделать это стоит отдельно от всех остальных классов, т.е. чтобы он вёл себя так же как std::list, по крайней мере, в той части, которая необходима в программе. Это позволит избежать всяческих сложностей как в функциях Include/Exclude. Объекты и функции, которые используют списки, ничего не должны знать о их внутреннем устройстве. Разделяй и властвуй. В-третьих, если аргумент не предполагается изменять, передавать его следует по константной ссылке. И в-четвертых, при реализации своих классов не забывайте про правило трёх.
Керра
Модератор
 Аватар для Керра
1270 / 438 / 45
Регистрация: 24.08.2011
Сообщений: 2,123
08.06.2014, 19:52  [ТС]     Ошибка при передаче результата функции #11
Tulosba, ну, за исключением того, что список состоит конкретно из объектов класса Discipline (в данной ситуации нет смысла делать шаблон), то все ок. Насчет Include/Exclude не поняла - там вроде никаких сложностей нет, в Include происходит сортировка по полю subject.semester, а в Exclude все вроде очевидно. Правилу трех тоже уже соответствует. С константной ссылкой тоже уже шлифовка.
Стандартные классы и функции конечно хотелось бы использовать, но по заданию надо все вручную делать. Даже насчет String не уверена что можно.
Tulosba
:)
Эксперт С++
4378 / 3221 / 297
Регистрация: 19.02.2013
Сообщений: 9,044
08.06.2014, 20:24     Ошибка при передаче результата функции #12
Цитата Сообщение от Керра Посмотреть сообщение
Насчет Include/Exclude не поняла - там вроде никаких сложностей нет
Сложность из-за открытой реализации списка. Следует сделать нормальный класс, с приватной секций для данных и публичными методами доступа к ним. Чтобы код, который использует списки, оперировал функциями типа Add, Remove и пр., а не шаманил с указателями prev,next напрямую. Так же у списка должны быть свои конструктор/деструктор. Всё это уменьшает связность, а значит упрощает понимание кода и его поддержку.
Керра
Модератор
 Аватар для Керра
1270 / 438 / 45
Регистрация: 24.08.2011
Сообщений: 2,123
08.06.2014, 20:28  [ТС]     Ошибка при передаче результата функции #13
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, а, поняла. Но то ж структура, по заданию именно структура должна быть.
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
08.06.2014, 20:37     Ошибка при передаче результата функции
Еще ссылки по теме:

Ошибка при передаче двумерного массива в функцию C++
Remove_if - при передаче функции класса выдает ошибки C++
C++ Ошибка при передаче в функцию

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

Или воспользуйтесь поиском по форуму:
BRcr
 Аватар для BRcr
4003 / 2292 / 155
Регистрация: 03.02.2011
Сообщений: 5,064
Записей в блоге: 10
08.06.2014, 20:37     Ошибка при передаче результата функции #14
Цитата Сообщение от Керра Посмотреть сообщение
Но то ж структура, по заданию именно структура должна быть.
Ну, пусть и будет себе структура. Только сделай для нее обертку в виде класса, как Tulosba советует. И все довольны.
Yandex
Объявления
08.06.2014, 20:37     Ошибка при передаче результата функции
Ответ Создать тему
Опции темы

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