Не получается добавление элемента в список

@Lepricon228 · Регистрация: 15.10.2018

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

Вот код:

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
#include <iostream>
#include <conio.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <locale.h>
using namespace std;
 
 
template <typename T>
class CList
{
    struct list
    {
        T value;
        struct list *next;
    };
    struct list *head, *tail;
public:
    CList(){head=NULL;}
    void Addlist();
    void print();
    void removeall();
    bool DoubleEveryN();
    //~Clist(){removeall();}
};
 
 
template <typename T>
void CList<T>::print()
{
    struct list *p=head;
    while (p!=NULL)
    {
        cout<<p->value<<"->";
        p=p->next;
    }
    cout<<("NULL\n");
}
 
template <typename T>
void CList<T>::removeall()
{
    if (head!=NULL)
    {
        struct list *p;
        while (head!=NULL)
        {
            p=head;
            head=head->next;
            delete p;
        }
        head=NULL;
    }
}
 
template <typename T>
bool CList<T>::DoubleEveryN()
{
    int x;
    cout << "Введите число, элемент которого по счету будет дублироватся:> ";
    while(!(cin >> x) || x<=0)
    {
        fflush(stdin);
        cin.clear();
        cout<<"Введите положительное число:> ";
    }
    int count=0;
    struct list *newlist;
    newlist = new struct list;
    struct list *ptr=head;
    struct list *curr=head;
    while(curr!=NULL)
    {
        count++;
        if(count%x==0)
        {
            newlist->value=curr->value;
            //////////////////////////
            while(ptr->next!=curr)
            {
                ptr=ptr->next;
            }
            newlist->next=ptr->next;
            ptr->next=newlist;
            cout<<newlist->value<<"->";
        }
        cout<<curr->value<<"->";
        curr=curr->next;
    }
    cout<<"NULL\n";
    return true;
}
 
template <typename T>
void CList<T>::Addlist()
{
    int a;
    while(!(cin >> a))
    {
        fflush(stdin);
        cin.clear();
        cout<<"Введите ЧИСЛО:>";
    }
    struct list *newlist;;
    newlist = new struct list;
    newlist->value=a;
    if (head==NULL)
    {
        tail=head=newlist;
        head->next = NULL;
    }
    else
    {
        tail->next=newlist;
        newlist->next = NULL;
        tail = newlist;
 
    }
}
 
int main()
{
    
    system("color f0");
    CList <int>list; //создаётся объект класса, при этом автоматически вызывается конструктор
    setlocale(0,"RUS"); // обеспесиваем вывод в консольное окно символов кириллицы
    int key, k;
    do
    {
        cout <<"\n1 - добавление элемента\n";
        cout <<"2 - вывод списка на экран\n";
        cout <<"3 - Задача варианта №8\n";
        cout <<"4 - очистка списка\n";
        cout <<"ESC - выход\n";
        cout <<"Ваш выбор:> ";
        key=getch();            
        switch(key)
        {
        case '1': 
            printf("Введите численный элемент списка: ");
            list.Addlist();
            break;
        case '2': 
            list.print();
            break;
        case '3':
            list.DoubleEveryN(); 
            break;
        case '4':
            list.removeall();
            break;
        }
    }
    while(key!=27);
    return 0; //объект класса уничтожается, автоматически вызывается деструктор
}

Проблема в том, что функция на дублирование, почему-то криво работает, при том что вывод который в ней работает хорошо, а общий плохо. В чем проблема?

@SuperKir · 17.02.2019, 13:26

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

cout << "Введите число, элемент которого по счету будет дублироватся:> ";

Переведи. Что конкретно нужно сделать?

@Lepricon228 · 17.02.2019, 13:28 **[ТС]**

Пользователь вводит индекс элемента, который надо удалить, то есть пользователь вводит 2 , и удаляется каждый второй элемент списка.

@SuperKir · 17.02.2019, 13:34

Lepricon228,

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

который надо удалить

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

Введите число, элемент которого по счету будет дублироватся

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

и удаляется каждый второй элемент списка.

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

будет дублироватся

Вот как?! Ну как?! Вот сам когда писал, понимал значения слова "дублироваться"?

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

и удаляется каждый второй элемент списка.

А теперь вопрос. Допустим, что список из 10 элементов. Мы удаляем 2й элемент. На его место встает 3й.
Дальше мы какой удаляем? 3й, который был 4м? Или 4й, который был 5? Или вообще 3й, который стал на 2 место? Относительно чего идет удаление?

@Lepricon228 · 17.02.2019, 13:43 **[ТС]**

Простите, опечаточка))) Не удаление а дублирование, понимаю я это так: создаем доп. узел, вот:

C++
1
newlist = new struct list;

после, чего я создаю цикл, который идет по всему списку, и когда проходит условие

C++
1
cout%x==0

, я запускаю новый цикл, чтобы получить указатель на curr элемент

C++
1
ptr->next

, после его получения, я присваиваю новому элементу указатель на курр

C++
1
ptr->next

, и прежний указатель указывает на элемент newlist;

@ReDoX · 17.02.2019, 17:18

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

создаем доп. узел

доп узел нужно создавать при каждом добавлении, а не один раз

У меня есть нужные функции, адаптируйте под свой код:

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 insert_before(List*& head, List*& before, int what) {
  // Предполагается, что before существует в списке.
  if (before == head) {
    List* new_node = new List();
 
    new_node->inf = what;
    new_node->next = head;
 
    head = new_node;
  } else {
    List* tmp = head;
 
    while (tmp->next != before)
      tmp = tmp->next;
 
    List* new_node = new List();
 
    new_node->inf = what;
    new_node->next = before;
 
    tmp->next = new_node;
  }
}
 
void insert_after(List*& head, List*& after, int what) {
  // Предполагается, что after существует в списке.
  List* new_node = new List();
 
  new_node->inf = what;
  new_node->next = after->next;
 
  after->next = new_node;
}

@SuperKir · 17.02.2019, 17:23

Lepricon228, Хорошо, когда добавится элемент в X позицию, откуда начинается отсчет следующей X позиции?
Для добавления необходимо знать текущий и предыдущий элемент.
Текущий - для подсчета позиции. Предыдущий, чтобы вставить.

C++
1
2
newNode->next = predNode -> next;
predNode->next = newNode;

А дальше возникает вопрос - откуда считать count, чтобы вставить следующий, ведь количество уже изменилось

P.s. узел создается каждый раз при добавлении

@Lepricon228 · 17.02.2019, 20:07 **[ТС]**

Ответ прост, нужно функцию на создание узла и выделение на него памяти просто поместить в цикл, а именно перед строчкой с слешами. Вот так:

C++
1
2
3
newlist = new struct list;
            newlist->value=curr->value;
            //////////////////////////

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .	Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»	SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .

@Lepricon228 1 / 1 / 0 Регистрация: 15.10.2018 Сообщений: 89
	17.02.2019, 13:28 [ТС]
	Пользователь вводит индекс элемента, который надо удалить, то есть пользователь вводит 2 , и удаляется каждый второй элемент списка. 0

Не получается добавление элемента в список

Решение