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

Односвязанный список - C++

Восстановить пароль Регистрация
 
Stelgi
0 / 0 / 0
Регистрация: 14.09.2012
Сообщений: 55
05.06.2016, 12:53     Односвязанный список #1
Здравствуйте. Вот задача:Структура содержит фамилию и 4 оценки. Удалить из списка имеющих 2, 3.
Создаю список( структура List). Есть проблема с удалением элементов. За удаление отвечает Delete.
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
void Delete(List **begin){
    
    if(*begin==NULL)
    return;
    
    List *t = *begin;
    
    for(int i(0);i<4;i++){
        if( ((t->a.mark[i]) == 2) || ((t->a.mark[i]) == 3) ){
            *begin = t->next;
            delete t;
            i=12;
 
        }
    }
    
    
    List *t1 = t->next;
    bool data=true;
    int i=0;
    while(t1){
        for(int i(0);i<4;i++){
            if( ((t1->a.mark[i]) == 2) || ((t1->a.mark[i]) == 3) ){
            t->next = t1->next;
            delete t1;
            goto l;
            }
        }
        l:
        t=t1;
        t1 = t1->next; 
        
    }
    system("pause");    
    
}
В маин есть строчка
C++
1
List *begin = NULL;
и обращение к функции
C++
1
Delete(&begin);
При запуске программы происходит создание списка, а затем я могу просмотреть его, с помощью функции output
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
void output(List *begin){
    
    system("cls");
    List *print = begin;
    
    cout << "Г’ГҐГЄГіГ№ГЁГҐ îöåГ*ГЄГЁ" << endl;
 
    
    while(print){
        cout << endl;
        cout << print->a.lastName << " -> \t";
        for(int i(0);i<4;i++){
            cout << i+1 << "-Гї îöåГ*ГЄГ* - " << print->a.mark[i] << "\t";
        }
        print=print->next;
    }
    
    cout << endl;
    system("Pause");
}
Затем, я использую функцию удаление, вроде ошибок нет, после хочу просмотреть результат функцией output и происходит следующее, бывает все нормально выводится, а бывает бесконечный цикл непонятно чего.
Миниатюры
Односвязанный список   Односвязанный список  
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
05.06.2016, 12:53     Односвязанный список
Посмотрите здесь:

C++ Сформировать список из вещественных чисел. Упорядочить список по возрастанию.
C++ Преобразовать список рёбер в список смежностей
Описать функцию, которая формирует список Common, включив в него элементы, ко-торые входят одновременно в список М1 и М2 C++
C++ Односвязанный список, удаление узла из списка
C++ Реализовать кольцевой список. Как закольцевать список обычный?
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Stitch Igorek
 Аватар для Stitch Igorek
44 / 44 / 17
Регистрация: 02.04.2016
Сообщений: 300
Завершенные тесты: 1
05.06.2016, 13:52     Односвязанный список #2
а у тебя точно никаких ошибок не выскакивает?
а то как то странно, что это вообще работает. 11 строка - delete t;, 18 строка - List *t1 = t->next;

Не по теме:

зачем оформлять половину кода в нечитабельный красный цвет...

Stelgi
0 / 0 / 0
Регистрация: 14.09.2012
Сообщений: 55
05.06.2016, 14:07  [ТС]     Односвязанный список #3
Stitch Igorek,
а у тебя точно никаких ошибок не выскакивает?
а то как то странно, что это вообще работает. 11 строка - delete t;, 18 строка - List *t1 = t->next;
При компиляции ошибок никаких нет
Stitch Igorek
 Аватар для Stitch Igorek
44 / 44 / 17
Регистрация: 02.04.2016
Сообщений: 300
Завершенные тесты: 1
05.06.2016, 14:18     Односвязанный список #4
Цитата Сообщение от Stelgi Посмотреть сообщение
При компиляции ошибок никаких нет
на скриншоте ясно видно, что у тебя выводит куски памяти вообще не относящиеся к твоей программе. пересмотри повнимательнее такие ньюансы как 11 и 18 строки, так же **begin в функции Delete - при проходе списка у тебя begin смещается и больше не указывает на начало списка. даже если функция Delete ничего не удалит - результат работы программы после ее вызова не предскажет даже Ванга.
Stelgi
0 / 0 / 0
Регистрация: 14.09.2012
Сообщений: 55
05.06.2016, 20:46  [ТС]     Односвязанный список #5
Все равно не нашел, но появился вопрос. После работы этой части кода
C++
1
2
3
4
5
6
7
8
9
10
    List *t = *begin;
    
    for(int i(0);i<4;i++){
        if( ((t->a.mark[i]) == 2) || ((t->a.mark[i]) == 3) ){
            *begin = t->next;
            delete t;
            i=12;
 
        }
    }
Чему равен t, на что ссылается
Yandex
Объявления
05.06.2016, 20:46     Односвязанный список
Ответ Создать тему
Опции темы

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