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

Двусвязный список - C++

Восстановить пароль Регистрация
 
The_bolT
70 / 70 / 1
Регистрация: 22.01.2013
Сообщений: 231
23.02.2013, 12:51     Двусвязный список #1
Вот решил написать двусвязный список. Написаны только две функции решил попробовать работает или нет, оказалось, что нет. Компилятор почему-то здесь выдает ошибку, но я не понимаю почему.
C++
1
p->next = p1;
list.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
#ifndef list_h
#define list_h
 
class list
{
public:
    list(){head = new (node); head->next = NULL;}
    ~list(){delete head;}
    void push_back(int);
    void push_front(int);
    void pop_back();
    void pop_front();
    void print();
private:
    struct node
    {
        int data;
        node* next;
        node* prev;
    };
    node* head;
};
 
#endif

list.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
#include <iostream>
#include "list.h"
 
using namespace std;
 
void list::push_back(int el)
{
    node* p,*p1;
    p = head->next;  // p указывает на первый элемент в списке
    while(p != NULL)  // После цикла p будет указывать на последний элемент списка
    {p = p->next;}
 
    p1 = new (node); //Создаем новую ячейку
    p1->data = el;  //заносим элемент
    p1->next = NULL;  //Указывает на конец списка
 
    p->next = p1; //Здесь ошибка, но по идее после этой команды наш последний элемент должен указывать на созданный
    p1->prev = p;  //и создан должен указывать на наш последний
}
 
void list::print()
{
    node* p;
    p = head->next;
    while(p != NULL)
    {
        cout<<(p->data)<<" ";
        p = p->next;
    }
}

main.cpp
Кликните здесь для просмотра всего текста
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
#include <iostream>
#include "list.h"
 
using namespace std;
 
int main()
{
    list L;
    for(int i=0;i<10;++i)
        L.push_back(i+1);
    L.print();
    system("pause>0");
    return 0;
}
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
23.02.2013, 12:51     Двусвязный список
Посмотрите здесь:

Двусвязный список C++
двусвязный список C++
Двусвязный список C++
Переделать двусвязный список в двусвязный кольцевой C++
C++ двусвязный список
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Buckstabue
 Аватар для Buckstabue
175 / 124 / 6
Регистрация: 12.01.2012
Сообщений: 624
23.02.2013, 13:05     Двусвязный список #2
Разыменование нулевого указателя в том месте ведь
C++
1
2
3
4
5
6
7
8
9
while(p != NULL)  // После цикла p будет указывать на последний элемент списка
    {p = p->next;}
    // теперь p = NULL!!! 
 
    p1 = new (node); //Создаем новую ячейку
    p1->data = el;  //заносим элемент
    p1->next = NULL;  //Указывает на конец списка
 
    p->next = p1;
Тебе наверное надо в цикле сделать while (p->next != NULL)

Добавлено через 3 минуты
Вообще зачем весь список пробегать, когда можно создать указатель на хвост. Ну и деструктор у тебя удаляет только самый первый узел, а остальные элементы списка висят в памяти
The_bolT
70 / 70 / 1
Регистрация: 22.01.2013
Сообщений: 231
23.02.2013, 14:29  [ТС]     Двусвязный список #3
Я еще напишу отдельную функцию для удаления всего списка. Указатель на хвост не плохая идея) Написал while (p-> next! = NULL) теперь там ошибку выдает.

Добавлено через 51 минуту
С функцией разобрался push_back. Нужно было только две строки местами поменять.
C++
1
2
p1->prev = p; 
    p->next = p1;
Но возникло еще пару вопросов до трех функций: правильно написана функция push_front, и почему не работают функции print_reverse и destroy?
Кликните здесь для просмотра всего текста
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
#include <iostream>
#include "list.h"
 
using namespace std;
 
void list::push_back(int el)
{
    node* p,*p1;
    p = head;  
    while(p->next != NULL)  
    {p = p->next;}
 
    p1 = new (node); 
    p1->data = el;  
    p1->next = NULL; 
 
    p1->prev = p; 
    p->next = p1; 
    
}
 
void list::print()
{
    node* p;
    p = head->next;
    while(p != NULL)
    {
        cout<<(p->data)<<" ";
        p = p->next;
    }
}
 
void list::push_front(int el)
{
    node* p;
    p = new (node);
    p->data = el;
    if(head->next == NULL)
    {
        p->next = head->next;
        head->next = p;
        p->prev = head;
    }
    else
    {
        p->next = head->next;
        head->next->prev = p;
        head->next = p;
        p->prev = head;
    }
}
 
void list::print_reverse()
{
    node* p;
    p = head;
    
    while(p->next != NULL)
    {p = p->next;}
 
    while(p->prev != head)
    {
        cout<<(p->data)<<" ";
        p = p->prev;
    }
}
 
void list::destroy()
{
    node* p,* p1;
    p = head;
    p1 = p->next;
    while(p1 != NULL)
    {
        p = p1;
        p1 = p1->next;
        delete p;
    }
}


Добавлено через 15 минут
И еще, первый элемент списка должен указывать на голову?

Добавлено через 14 минут
С теми функциями разобрался, осталась print_reverse.
Такая реализация не работает.
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
void list::print_reverse()
{
    node* p;
    p = head;
    
    while(p != NULL)
    {p = p->next;}
 
    while(p->prev != head)
    {
        cout<<(p->data)<<" ";
        p = p->prev;
    }
}
А вот такая работает, только теряет первый символ.
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
void list::print_reverse()
{
    node* p;
    p = head;
    
    while(p->next != NULL)
    {p = p->next;}
 
    while(p->prev != head)
    {
        cout<<(p->data)<<" ";
        p = p->prev;
    }
}
Я не понимаю в чем проблема. Вот код полностью:
list.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
#ifndef list_h
#define list_h
 
class list
{
public:
    list(){head = new (node); head->next = NULL;}
    ~list(){destroy(); delete head;}
    void push_back(int);
    void push_front(int);
    void pop_back();
    void pop_front();
    void print();
    void print_reverse();
    void destroy();
private:
    struct node
    {
        int data;
        node* next;
        node* prev;
    };
    node* head;
    node* last;
};
 
#endif

list.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
#include <iostream>
#include "list.h"
 
using namespace std;
 
void list::push_back(int el)
{
    node* p,*p1;
    p = head;  
    while(p->next != NULL)  
    {p = p->next;}
 
    p1 = new (node); 
    p1->data = el;  
    p1->next = NULL; 
 
    p1->prev = p; 
    p->next = p1; 
    
}
 
void list::print()
{
    node* p;
    p = head->next;
    while(p != NULL)
    {
        cout<<(p->data)<<" ";
        p = p->next;
    }
}
 
void list::push_front(int el)
{
    node* p;
    p = new (node);
    p->data = el;
    if(head->next == NULL)
    {
        p->next = head->next;
        head->next = p;
        p->prev = head;
    }
    else
    {
        p->next = head->next;
        head->next->prev = p;
        head->next = p;
        p->prev = head;
    }
}
 
void list::print_reverse()
{
    node* p;
    p = head;
    
    while(p->next != NULL)
    {p = p->next;}
 
    while(p->prev != head)
    {
        cout<<(p->data)<<" ";
        p = p->prev;
    }
}
 
void list::destroy()
{
    node* p,* p1;
    p = head;
    p1 = p->next;
    while(p1 != NULL)
    {
        p = p1;
        p1 = p1->next;
        delete p;
    }
}

main.cpp
Кликните здесь для просмотра всего текста
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
#include <iostream>
#include "list.h"
 
using namespace std;
 
int main()
{
    {
    list L;
    for(int i=0;i<10;++i)
        L.push_front(i+1);
    L.print();
    cout<<endl;
    L.print_reverse();
    }
    system("pause>0");
    return 0;
}
Schizorb
 Аватар для Schizorb
508 / 460 / 16
Регистрация: 07.04.2012
Сообщений: 865
Записей в блоге: 1
Завершенные тесты: 1
23.02.2013, 14:31     Двусвязный список #4
C++
1
2
3
4
5
while(p != head)
{
    cout<<(p->data)<<" ";
    p = p->prev;
}
Или нет?
The_bolT
70 / 70 / 1
Регистрация: 22.01.2013
Сообщений: 231
23.02.2013, 16:22  [ТС]     Двусвязный список #5
Вот пишу функцию для удаления последнего элемента из списка, но почему-то не удаляет.
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
void list::pop_back()
{
    node* p,* p1;
    p = head;
    p1 = p->next;
    //После цикла p1 указывает на последний элемент, а p на предпоследней.
    while(p1 != NULL)
    {
        p = p1;
        p1 = p1->next;
    }
    p->next = NULL; //Это чтобы предпоследний элемент указывал на конец списка
    delete p1; //Удаляем последний элемент
}
Schizorb
 Аватар для Schizorb
508 / 460 / 16
Регистрация: 07.04.2012
Сообщений: 865
Записей в блоге: 1
Завершенные тесты: 1
23.02.2013, 18:20     Двусвязный список #6
C++
1
2
3
4
5
6
7
while(p1 != NULL)
{
    p = p1;
    p1 = p1->next;
}
p->next = NULL; //Это чтобы предпоследний элемент указывал на конец списка
delete p1; //Удаляем последний элемент
Получилось, что после цикла p содержит адрес последнего элемента, а p1 - NULL. Попробуй так:

C++
1
while(p1->next != NULL)
Yandex
Объявления
23.02.2013, 18:20     Двусвязный список
Ответ Создать тему
Опции темы

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