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

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

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

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

28.10.2012, 14:40. Просмотров 1449. Ответов 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;
}
Подскажите пожалуйста что не так?
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
28.10.2012, 14:40     Удаление последнего элемента из списка
Посмотрите здесь:
C++ Удаление последнего элемента списка
Удаление последнего элемента из списка C++
C++ Удаление последнего элемента односвязного списка
Удаление последнего введенного элемента из списка (стека) C++
Как сделать удаление последнего элемента списка ? C++
Удаление последнего элемента в строке C++
C++ Удаление последнего элемента Дека
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Kuzia domovenok
1889 / 1744 / 117
Регистрация: 25.03.2012
Сообщений: 5,922
Записей в блоге: 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;
далее здесь. Не знаю, что ты хотел сделать. Но ты только что удалил элемент по указателю, а затем как ни в чём не бывало пытаешься продолжить с ним работу.
BRcr
4006 / 2295 / 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;
Ты только что удалил объект и тут же обращаешься к нему, будто он еще существует.
Kuzia domovenok
1889 / 1744 / 117
Регистрация: 25.03.2012
Сообщений: 5,922
Записей в блоге: 1
28.10.2012, 14:52     Удаление последнего элемента из списка #4
Цитата Сообщение от !Андрей! Посмотреть сообщение
delete vsp;//сваливается здесь
while (vsp->next!=NULL) vsp=vsp->next;
далее здесь. Не знаю, что ты хотел сделать. Но ты только что удалил элемент по указателю, а затем как ни в чём не бывало пытаешься продолжить с ним работу.
!Андрей!
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;
    }
}
BRcr
4006 / 2295 / 155
Регистрация: 03.02.2011
Сообщений: 5,064
Записей в блоге: 10
28.10.2012, 17:51     Удаление последнего элемента из списка #6
Ерунда какая-то. Выкладывай весь код.
!Андрей!
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;

я хз что не так
Kuzia domovenok
1889 / 1744 / 117
Регистрация: 25.03.2012
Сообщений: 5,922
Записей в блоге: 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 это не бывший предпоследний, а бывший последний элемент. (наверное)
BRcr
4006 / 2295 / 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))
!Андрей!
6 / 6 / 0
Регистрация: 31.01.2012
Сообщений: 134
28.10.2012, 18:58  [ТС]     Удаление последнего элемента из списка #10
BRcr, при нажатии двойки(удаление)
Миниатюры
Удаление последнего элемента из списка  
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
28.10.2012, 19:03     Удаление последнего элемента из списка
Еще ссылки по теме:
Удаление последнего чётного элемента массива C++
Найти сумму первого и последнего элемента списка C++
Поправить функцию удаления последнего элемента односвязного списка C++
Удаление элемента списка и всего списка C++
В программе не работает 1)удаление элемента, последнего в списке-зависает и C++

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

Или воспользуйтесь поиском по форуму:
BRcr
4006 / 2295 / 155
Регистрация: 03.02.2011
Сообщений: 5,064
Записей в блоге: 10
28.10.2012, 19:03     Удаление последнего элемента из списка #11
У меня работает себе спокойненько. Отследи строчку.
Yandex
Объявления
28.10.2012, 19:03     Удаление последнего элемента из списка
Ответ Создать тему
Опции темы

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