"Удаление" элемента списка

@Etete · Регистрация: 27.12.2010

Студворк — интернет-сервис помощи студентам

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
#include <iostream>
 
using namespace std;
 
class List
{
private:
    struct Node
    {
        int data;
        Node *next;
    };
    Node *first;
public:
    List()
    {
        first = NULL;
    }
    void add(int a)
    {
        Node *temp = new Node;
        if(first == NULL)
        {
            temp->data = a;
            temp->next = NULL;
            first = temp;
        }
        else
        {
            temp = first;
            while(temp->next != NULL)
                temp = temp->next;
            temp->next = new Node;
            temp = temp->next;
            temp->data = a;
            temp->next = NULL;
        }
    }
 
    void print()
    {
        Node *temp = first;
        while(temp != NULL)
        {
            cout << temp->data << ' ';
            temp = temp->next;
        }
    }
 
    void remove(int a)
    {
        Node *temp = first;
        Node *sFirst = first;
        do
        {
            if(temp == first && a == temp->data)
            {
                first = temp->next;
                return;
            }
            else if(temp->next == NULL && a == temp->data)
            {
                temp->next = temp;
                return;
            }
            else if(temp->data == a)
            {
                while(1)
                {
                    if(sFirst->next == temp)
                    {
                        sFirst = sFirst->next;
                        break;
                    }
                }
                sFirst->next = temp->next;
                return;
            }
            temp = temp->next;
 
        } while(temp->data != a);
    }
};
 
int main()
{
    List a;
    a.add(1);
    a.add(2);
    a.add(3);
    a.add(4);
    a.add(5);
    a.remove(1);
    a.print();
    return 0;
}

Функция remove работает только для первого контейнера. В чём проблема?

@3BEPb_TyTa · 13.01.2013, 01:26

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
void remove(int a)
    {
        Node* tmp = first;
        if(tmp->data == a){
            first = tmp->next;
            delete tmp;
            return;
        }
        while(true){
            if(tmp->next != 0){
                if(tmp->next->data == a)
                    break;
                tmp = tmp->next;
            }
            else
                return;
        }
        Node* forDelete = tmp->next;
        tmp->next = forDelete->next;
        delete forDelete;
    }

@Etete · 13.01.2013, 02:13 **[ТС]**

Разве мы имеем право юзать тут delete, если не выделяли память с помощью new?

@OhMyGodSoLong · 13.01.2013, 02:20

Но ведь когда-то её выделили с помощью new?

@UserAK · 13.01.2013, 02:39

Сообщение от Etete

Разве мы имеем право юзать тут delete, если не выделяли память с помощью new?

как это не выделяли? в функции void add(int) выделили, в функции void remove(int) освободили, всё в пределах класса красиво и аккуратно

@Etete · 13.01.2013, 14:34 **[ТС]**

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
void remove(int a)
    {
        Node *temp = first;
        Node *tempF = first;
        while(temp != NULL)
        {
            if(first->data == a)
            {
                delete first;
                first = temp->next;
                return;
            }
            if(temp->data == a)
            {
                while(1)
                {
                    if(tempF->next == temp)
                    {
                        if(temp->next == NULL)
                        {
                            delete temp;
                            tempF->next = NULL;
                            return;
                        }
                        else
                        {
                            tempF->next = temp->next;
                            delete temp;
                            return;
                        }
                    }
                    tempF = tempF->next;
                }
            }
            temp = temp->next;
       }
    }
};

Получилось вот это. Как научиться писать такой компактный код, как пишет его
3BEPb_TyTa ? Или это только с опытом придёт?

Croessmah · 13.01.2013, 14:39

Не по теме:

Сообщение от Etete

Получилось вот это. Как научиться писать такой компактный код, как пишет его
3BEPb_TyTa ? Или это только с опытом придёт?

Ну если Вы отформатируете свой код, так же как он свой, то у Вас будет выглядеть не многим больше

Новые блоги и статьи Все статьи Все блоги /
Новый CodeBlocs. Версия 25.03 palva 04.01.2026 Оказывается, недавно вышла новая версия CodeBlocks за номером 25. 03. Когда-то давно я возился с только что вышедшей тогда версией 20. 03. С тех пор я давно снёс всё с компьютера и забыл. Теперь. . .	Модель микоризы: классовый агентный подход anaschu 02.01.2026 Раньше это было два гриба и бактерия. Теперь три гриба, растение. И на уровне агентов добавится между грибами или бактериями взаимодействий. До того я пробовал подход через многомерные массивы,. . .	Учёным и волонтёрам проекта «Einstein@home» удалось обнаружить четыре гамма-лучевых пульсара в джете Млечного Пути Programma_Boinc 01.01.2026 Учёным и волонтёрам проекта «Einstein@home» удалось обнаружить четыре гамма-лучевых пульсара в джете Млечного Пути Сочетание глобально распределённой вычислительной мощности и инновационных. . .	Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Programma_Boinc 28.12.2025 Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / ********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / ********/ gallery/ bKBkQFf Пост отсюда. . .	Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Programma_Boinc 26.12.2025 Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .
Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .	PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .

@Etete 0 / 0 / 1 Регистрация: 27.12.2010 Сообщений: 40
	13.01.2013, 02:13 [ТС]
	Разве мы имеем право юзать тут delete, если не выделяли память с помощью new? 0

@OhMyGodSoLong ~ Эврика! ~ 1258 / 1007 / 74 Регистрация: 24.07.2012 Сообщений: 2,002
	13.01.2013, 02:20
	Но ведь когда-то её выделили с помощью new? 0