Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
 
Рейтинг 4.58/40: Рейтинг темы: голосов - 40, средняя оценка - 4.58
6 / 6 / 1
Регистрация: 31.01.2012
Сообщений: 134
1

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

28.10.2012, 14:40. Показов 7419. Ответов 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

Помощь в написании контрольных, курсовых и дипломных работ здесь.

Programming
Эксперт
94731 / 64177 / 26122
Регистрация: 12.04.2006
Сообщений: 116,782
28.10.2012, 14:40
Ответы с готовыми решениями:

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

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

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

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

10
3413 / 2772 / 751
Регистрация: 25.03.2012
Сообщений: 10,083
Записей в блоге: 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
4033 / 2323 / 292
Регистрация: 03.02.2011
Сообщений: 5,066
Записей в блоге: 10
28.10.2012, 14:52 3
Цитата Сообщение от !Андрей! Посмотреть сообщение
C++
1
2
delete vsp;//сваливается здесь
while (vsp->next!=NULL) vsp=vsp->next;
Ты только что удалил объект и тут же обращаешься к нему, будто он еще существует.
0
3413 / 2772 / 751
Регистрация: 25.03.2012
Сообщений: 10,083
Записей в блоге: 1
28.10.2012, 14:52 4
Цитата Сообщение от !Андрей! Посмотреть сообщение
delete vsp;//сваливается здесь
while (vsp->next!=NULL) vsp=vsp->next;
далее здесь. Не знаю, что ты хотел сделать. Но ты только что удалил элемент по указателю, а затем как ни в чём не бывало пытаешься продолжить с ним работу.
0
6 / 6 / 1
Регистрация: 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
4033 / 2323 / 292
Регистрация: 03.02.2011
Сообщений: 5,066
Записей в блоге: 10
28.10.2012, 17:51 6
Ерунда какая-то. Выкладывай весь код.
0
6 / 6 / 1
Регистрация: 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
3413 / 2772 / 751
Регистрация: 25.03.2012
Сообщений: 10,083
Записей в блоге: 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
4033 / 2323 / 292
Регистрация: 03.02.2011
Сообщений: 5,066
Записей в блоге: 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))
1
6 / 6 / 1
Регистрация: 31.01.2012
Сообщений: 134
28.10.2012, 18:58  [ТС] 10
BRcr, при нажатии двойки(удаление)
0
Миниатюры
Удаление последнего элемента из списка  
4033 / 2323 / 292
Регистрация: 03.02.2011
Сообщений: 5,066
Записей в блоге: 10
28.10.2012, 19:03 11
У меня работает себе спокойненько. Отследи строчку.
0
IT_Exp
Эксперт
87844 / 49110 / 22898
Регистрация: 17.06.2006
Сообщений: 92,604
28.10.2012, 19:03

Удаление последнего элемента из однонаправленного линейного списка
Здравствуйте, удаляю значится элемент: (x - порядковый номер удаляемого элемента, начиная с 1....

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

Удаление первого и последнего элемента двусвязного списка с++
Удаление первого и последнего элемента двусвязного списка с++ namespace vlados { ...

Удаление последнего элемента массива
Функция для удаления последнего элемента в массиве. укажите на ошибку пожалуйста, функция выдает...


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

Или воспользуйтесь поиском по форуму:
11
Ответ Создать тему
Опции темы

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