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

С++ для начинающих

Войти
Регистрация
Восстановить пароль
 
Рейтинг: Рейтинг темы: голосов - 9, средняя оценка - 4.89
!Андрей!
6 / 6 / 0
Регистрация: 31.01.2012
Сообщений: 134
#1

Удаление последнего элемента из списка - C++

28.10.2012, 14:40. Просмотров 1532. Ответов 10
Метки нет (Все метки)

Есть функция
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
void List::DeleteElement(void)
{
    Element *vsp;
    if (!begin) 
    {
        cout << "Is empty!\n";
        return;
    }
    else if(begin->next==NULL) 
    {
        delete begin;
        cout << "Element deleted!\n";
        return;
    }
    else
    {
        vsp = begin;
        while(vsp->next!=NULL)
            vsp = vsp->next;
        delete vsp;//сваливается здесь
 
        while (vsp->next!=NULL) vsp=vsp->next;
        end=vsp;
        end->next=NULL;
 
        cout << "Element deleted!\n";
    
        if(begin)
            NumberElements();
    }
    system("pause");
    return;
}
Подскажите пожалуйста что не так?
0
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
28.10.2012, 14:40
Здравствуйте! Я подобрал для вас темы с ответами на вопрос Удаление последнего элемента из списка (C++):

Удаление последнего элемента из списка - C++
Добрый день! Проблема такая, удаляя последний элемент из списка, выдает ошибку Помогите разобраться! Спасибо Код программы ...

Удаление последнего элемента списка - C++
А вот когда удаляешь последний элемент списка, обязательно в предпоследнем присваивать указатель на NULL?

Удаление последнего элемента односвязного списка - C++
как сделать в списке pop_back и полное очищение? struct Node { int value; Node* next; }; struct List { Node...

Как сделать удаление последнего элемента списка ? - C++
Помогите написать функцию удаления последнего элемента списка.

Удаление последнего введенного элемента из списка (стека) - C++
Напишите пожалуйста программу для удаления последнего введенного элемента из списка (стеку)

Удаление последнего элемента Дека - C++
Все работает, кроме удаления последнего элемента. Подскажите пожалуйста, что не так? #include &lt;cstdlib&gt; #include &lt;iostream&gt; using...

Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
Kuzia domovenok
1891 / 1746 / 118
Регистрация: 25.03.2012
Сообщений: 5,925
Записей в блоге: 1
28.10.2012, 14:50 #2
Цитата Сообщение от !Андрей! Посмотреть сообщение
C++
1
2
3
4
5
6
7
8
9
10
11
if (!begin) 
   {
     cout << "Is empty!\n";
    return;
  }
  else if(begin->next==NULL) 
  {
    delete begin;
     cout << "Element deleted!\n";
    return;
  }
Первое, что бросилось в глаза - при удалении первого и единственного элемента begin не обнуляется. При этом, при следующем вызове функции (для полученного списка без элементов) проверка if (!begin) может не сработать и возникнет ошибочная ситуация.
Цитата Сообщение от !Андрей! Посмотреть сообщение
delete vsp;//сваливается здесь
while (vsp->next!=NULL) vsp=vsp->next;
далее здесь. Не знаю, что ты хотел сделать. Но ты только что удалил элемент по указателю, а затем как ни в чём не бывало пытаешься продолжить с ним работу.
0
BRcr
4008 / 2297 / 155
Регистрация: 03.02.2011
Сообщений: 5,064
Записей в блоге: 10
28.10.2012, 14:52 #3
Цитата Сообщение от !Андрей! Посмотреть сообщение
C++
1
2
delete vsp;//сваливается здесь
while (vsp->next!=NULL) vsp=vsp->next;
Ты только что удалил объект и тут же обращаешься к нему, будто он еще существует.
0
Kuzia domovenok
1891 / 1746 / 118
Регистрация: 25.03.2012
Сообщений: 5,925
Записей в блоге: 1
28.10.2012, 14:52 #4
Цитата Сообщение от !Андрей! Посмотреть сообщение
delete vsp;//сваливается здесь
while (vsp->next!=NULL) vsp=vsp->next;
далее здесь. Не знаю, что ты хотел сделать. Но ты только что удалил элемент по указателю, а затем как ни в чём не бывало пытаешься продолжить с ним работу.
0
!Андрей!
6 / 6 / 0
Регистрация: 31.01.2012
Сообщений: 134
28.10.2012, 17:46  [ТС] #5
Kuzia domovenok, ломается именно удаление

Добавлено через 18 секунд
BRcr, ломается именно удаление

Добавлено через 15 минут
Так - не вылетает, но и ничего не удаляется

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
void List::DeleteElement(void)
{
    if (!begin) 
    {
        cout << "Is empty!\n";
        return;
    }
    else if(begin->next==NULL) 
    {
        delete begin;
        begin=NULL;
        cout << "Element deleted!\n";
        return;
    }
    else
    {
        Element* toDelete = begin;
 
        for(int i = 0; i < (NumberElements() - 1);i++)
            toDelete = toDelete -> next;
 
        delete toDelete -> next;
 
        toDelete -> next = NULL;
        end = toDelete;
 
        system("pause");
        return;
    }
}
0
BRcr
4008 / 2297 / 155
Регистрация: 03.02.2011
Сообщений: 5,064
Записей в блоге: 10
28.10.2012, 17:51 #6
Ерунда какая-то. Выкладывай весь код.
0
!Андрей!
6 / 6 / 0
Регистрация: 31.01.2012
Сообщений: 134
28.10.2012, 18:00  [ТС] #7
BRcr,
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
// spisok.cpp: определяет точку входа для консольного приложения.
//
 
#include "stdafx.h"
#include <conio.h>
#include <iostream>
#include <string>
using namespace std;
 
class List
{
public:
    int numsCount; // количество элементов
 
    class Data
    {
    public:
        string language_program;
        int length_program;
        int kol_oper;
        int time_program;
    };
 
    struct Element
    {
        Data data;
        Element *next;
 
    }*begin, *end;
 
    List();
    ~List();
 
    void writeToFile();
    bool readFromFile();
 
    void AddElement(void);
    void DeleteElement(void);
    void ShowElements(void);
    int NumberElements(void);
 
}l;
 
int _tmain(int argc, _TCHAR* argv[])
{
    l.readFromFile();
    l.NumberElements();
 
    int key = 0;
    do
    {
        system("cls");
        cout<<"LIST:\n"<<"\t1- Add element\n"
            <<"\t2- Delete\n"
            <<"\t3- Show elements\n"
            <<"\nFor exit press ESC\n\n";
 
        key = _getch();
 
        switch(key)
        {
        case '1': l.AddElement(); break;     
        case '2': l.DeleteElement(); break;
        case '3': l.ShowElements(); break;
        }
    }
    while (key!=27);//ESCAPE
    l.writeToFile();
 
    return 0;
}
 
void List::writeToFile()
{
 
    FILE *file;
    file = fopen( "data.bin", "wb" );
    Element* temp;
    temp=begin;
    for(int i = 0; i < NumberElements();i++)
    {
        fwrite(temp,sizeof(Data),1,file);
        temp = temp->next;
    };
    fclose(file);
    return;
}
 
List::List()
{
    begin = end = NULL;
    numsCount = 0;
}
 
List::~List()
{
    Element *vsp;
    while(begin)
    {
        vsp = begin;
        begin = begin -> next;
        delete vsp;
    }
}
 
bool List::readFromFile()
{
    Element *temp;
    FILE *file;
    file=fopen("data.bin","rb");
    begin=NULL;
    while(temp=new Element,fread(temp,sizeof(Data),1,file))
    {                 
        temp->next=NULL;      
        if(begin==NULL) {
            begin=temp;
            end=begin;
            end->next=NULL;
        }
        else {
            end->next=temp;
            end=end->next;
            end->next=NULL;
        };           
    };         
    fclose(file);
    return true;
}     
void List::AddElement(void)
{
    Element *temp;
    temp = new Element;
    cout << endl << "\tEnter language program: ";
    _flushall();
    getline(cin,temp->data.language_program);
    cout << "\tEnter length program: ";
    cin >> temp->data.length_program;
    cout << "\tEnter kol-vo operatorov: ";
    cin >> temp->data.kol_oper;            
    cout << "\tTime program: ";
    cin >> temp->data.time_program;
 
    if (begin == NULL)
        begin = temp;
    else
        end -> next = temp;
 
    end = temp;
    end -> next = NULL;
    return;
}
 
void List::ShowElements(void)
{
    Element *temp;
    if(NumberElements())
    {
        temp = begin;
        for(int i = 0; i < NumberElements();i++)
        {
            cout << "\n" << (i+1) << ".\tlanguage program: " << temp->data.language_program
                << "\n\tlength program: " << temp->data.length_program
                << "\n\tkol-vo operatorov: " << temp->data.kol_oper
                << "\n\ttime program: " << temp->data.time_program
                << endl << endl;
            temp = temp->next;
        }
    }
    else
        cout << "Empty!";
    system("pause");
    return;
}
 
void List::DeleteElement(void)
{
    if (!begin) 
    {
        cout << "Is empty!\n";
 
    }
    else if(begin->next==NULL) 
    {
        delete begin;
        begin = NULL;
        begin -> next = NULL;
        cout << "Element deleted!\n";
    }
    else
    {
        Element* toDelete = begin;
 
        for(int i = 0; i < (NumberElements() - 1);i++)
        {
            cout << toDelete->data.language_program <<endl;
            toDelete = toDelete -> next;
        }
        
        end = toDelete;
 
        delete end -> next;
        end -> next = NULL;
 
    }
    system("pause");
    return;
}
 
int List::NumberElements(void)
{
    Element *vsp;
    numsCount = 0;
 
    if(begin!=NULL)
    {
        vsp = begin;
        while(vsp)
        {
            numsCount ++;
            vsp = vsp -> next;
        }
        return numsCount;
    }
    else
        return 0;
}
Добавлено через 5 минут
BRcr, по хорошему - я даже на бумажке расписал - он доходит до предпоследнего э-та (т.к. (NumberElements() - 1)), потом конец приравнивается к предпоследнему э-ту(end = toDelete и удаляется следующий э-т (delete end -> next, а потом - end -> next = NULL;

я хз что не так
0
Kuzia domovenok
1891 / 1746 / 118
Регистрация: 25.03.2012
Сообщений: 5,925
Записей в блоге: 1
28.10.2012, 18:37 #8
Цитата Сообщение от !Андрей! Посмотреть сообщение
C++
1
2
3
4
5
6
7
8
9
Element* toDelete = begin;
for(int i = 0; i < (NumberElements() - 1);i++)
        {
                  cout << toDelete->data.language_program <<endl;
                  toDelete = toDelete -> next;
         }
end = toDelete;
delete end -> next;
end -> next = NULL;
По-моему, тут ты пролистываешь в цикле на один больше элемент, чем хотелось. В итоге toDelete это не бывший предпоследний, а бывший последний элемент. (наверное)
0
BRcr
4008 / 2297 / 155
Регистрация: 03.02.2011
Сообщений: 5,064
Записей в блоге: 10
28.10.2012, 18:49 #9
Так работает:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
void List::DeleteElement( void ) {
    if ( !begin ) {
        cout << "Is empty!\n";
    }
    else if ( begin->next == NULL ) {
        delete begin;
        begin = NULL;
        cout << "Element deleted!\n";
    }
    else {
        Element *toDelete = begin;
        while ( toDelete->next->next ) {
            toDelete = toDelete->next;
        }
        end = toDelete;
        delete end->next;
        end->next = NULL;
    }
    system( "pause" );
    return;
}
Функция readFromFile(), кстати, как-то кривовата. Что это за строчка вообще?
C++
1
while(temp=new Element,fread(temp,sizeof(Data),1,file))
0
!Андрей!
6 / 6 / 0
Регистрация: 31.01.2012
Сообщений: 134
28.10.2012, 18:58  [ТС] #10
BRcr, при нажатии двойки(удаление)
0
Миниатюры
Удаление последнего элемента из списка  
BRcr
4008 / 2297 / 155
Регистрация: 03.02.2011
Сообщений: 5,064
Записей в блоге: 10
28.10.2012, 19:03 #11
У меня работает себе спокойненько. Отследи строчку.
0
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
28.10.2012, 19:03
Привет! Вот еще темы с ответами:

Удаление последнего элемента в строке - C++
Добрый вечер. Не могу решить задачу со строками. Нужно построить односвязный список, удалить последний элемент и вывести на экран список...

Удаление последнего чётного элемента массива - C++
Здравствуйте,помогите дописать программу,пожалуйста. Нужно,чтобы программа удаляла последний чётный элемент массива.Что-то написал,но...

Реализация списка.Удаление хвостового элемента.Поиск элемента - C++
Всем привет,мое задание выглядит так : В класс List&lt;T&gt; из классной работы добавить следующие методы: void addHead(T...

Найти сумму первого и последнего элемента списка - C++
Двунаправленный список.Найти сумму первого и последнего элемента. спасибо


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

Или воспользуйтесь поиском по форуму:
Yandex
Объявления
28.10.2012, 19:03
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru