При возвращении объекта из функции данные затираются

@maybecsharp · Регистрация: 02.03.2020

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

Когда возвращаю объект на 88 строке, в нём вызывается конструктор копирования и затем деструктор, но данные при этом затираются, что делать? Как можно это исправить?

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
// clab.h
struct SLabNode
{
    int data;
    SLabNode* next;
};
 
class CLab
{
    public:
        CLab();
        CLab(const CLab& obj);
        ~CLab();
    public:
        void add(const int data);
        void clear();
    public:
        // операторы
    private:
        SLabNode* deleteNode(SLabNode* obj);
    private:
        SLabNode* m_head, * m_tail;
};
 
// clab.cpp
CLab::CLab()
{
    this->m_head = this->m_tail = nullptr;
}
 
CLab::CLab(const CLab& obj) // конструктор копирования
{
    this->m_head = std::move(obj.m_head);
    this->m_tail = std::move(obj.m_tail);
}
 
CLab::~CLab()
{
    if (nullptr != this->m_head) clear();
}
 
void CLab::add(const int data)
{
    // выдление SLabNode и добавление в список с data
}
 
void CLab::clear()
{
    // очистка списка
}
 
SLabNode* CLab::deleteNode(SLabNode* list)
{
    // удаление list и возвращение следующего элемента после list
}
 
// main.cpp
// ...
int getMenuItem(const std::string& allowSymbols)
{
    int inputSymbol = 0;
    do { inputSymbol = _getch(); }
    while (std::string::npos == allowSymbols.find(inputSymbol));
    putchar(inputSymbol);
 
    return tolower(inputSymbol);
}
 
template<>
inline CLab getValue(std::istream& stream, const std::string& message)
{
    CLab buffer;
    bool isContinue = true;
 
    while (true == isContinue)
    {
        std::cout << "Хотите ввести элемент? (Y)es / (N)o"
            << std::endl << "> ";
 
        if ('y' == getMenuItem("YyNn"))
        {
            std::cout << std::endl;
            buffer.add(getValue<int>(std::cin, message));
        }
        else isContinue = false;
    }
 
    return buffer; // <- здесь вызываеся конструктор копирования, после чего деструктор, из-за чего данные затираются и возвращается белеберда
}
 
// ...
CList<CLab> obj; // список из целочисленных списков (элементы которого - целые числа)
obj.add(getValue<CLab>(std::cin, "..."));
// ...

CList - шаблонный класс, CList::add(const T& data)

@zayats80888 · 02.03.2020, 01:00

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

CLab::CLab(const CLab& obj) // конструктор копирования
{
this->m_head = std::move(obj.m_head);
this->m_tail = std::move(obj.m_tail);
}

А что тут происходит по вашему?

@maybecsharp · 02.03.2020, 01:03 **[ТС]**

Копируются указатели. Я предполагал, что ломается именно из-за этого. Но как это исправить?

@zayats80888 · 02.03.2020, 01:08

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

Но как это исправить?

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
class CLab
{
public:
    //...
    CLab(const CLab& obj): CLab()
    {
        auto ptr = obj.m_head;
        while (ptr)
        {
            add(ptr->data);
            ptr = ptr->next;
        }
    }
    CLab(CLab&& obj)
        :
        m_head(std::exchange(obj.m_head, nullptr)),
        m_tail(std::exchange(obj.m_tail, nullptr))
    {}
    //...
};

@maybecsharp · 02.03.2020, 01:13 **[ТС]**

А для чего нужен конструктор CLab(CLab&& obj)?

@zayats80888 · 02.03.2020, 01:16

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

А для чего нужен конструктор CLab(CLab&& obj)?

для NRVO

Да, и еще нужно определить

C++
1
2
CLab& operator=(const CLab& obj);
CLab& operator=(CLab&& obj);

@maybecsharp · 02.03.2020, 01:33 **[ТС]**

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
CLab& CLab::operator=(const CLab& obj)
{
    this->m_head = std::move(obj.m_head);
    this->m_tail = std::move(obj.m_tail);
    return *this;
}
 
CLab& CLab::operator=(CLab&& obj)
{
    this->m_head = std::move(obj.m_head);
    this->m_tail = std::move(obj.m_tail);
    return *this;
}

Но всё равно, выполнив obj.add на 93 строке, он переходит в деструктор и удаляет список, ломая все данные.

Вот, как очищаю список:

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
void CLab::clear()
{
    this->m_tail = nullptr; // сразу удаляем конец
 
    while (this->m_head) // пока список не пуст
    {
        if (this->m_head)
        {
            this->m_head = deleteNode(this->m_head);
        }
        else // иначе список пуст
        {
            delete this->m_head;
            this->m_head = nullptr;
        }
    }
 
SLabNode* CLab::deleteNode(SLabNode* list)
{
    SLabNode* tempList = list->next;
    delete list;
    list = nullptr;
    return tempList; // возвращаем следующий элемент
}
}

@zayats80888 · 02.03.2020, 01:36

присваивание добавил?

Добавлено через 42 секунды

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
class CLab
{
public:
    //...
    CLab(const CLab& obj) : CLab()
    {
        auto ptr = obj.m_head;
        while (ptr)
        {
            add(ptr->data);
            ptr = ptr->next;
        }
    }
    CLab& operator=(const CLab& obj)
    {
        if (this != &obj)
        {
            *this = CLab(obj);
        }
        return *this;
    }
    CLab(CLab&& obj) noexcept
        :
        m_head(std::exchange(obj.m_head, nullptr)),
        m_tail(std::exchange(obj.m_tail, nullptr))
    {}
    CLab& operator=(CLab&& obj) noexcept
    {
        std::swap(m_head, obj.m_head);
        std::swap(m_tail, obj.m_tail);
        return *this;
    }
    //...

Новые блоги и статьи Все статьи Все блоги /
Кому нужен AOT? DevAlt 26.03.2026 Решил сделать простой ланчер Написал заготовку: dotnet new console --aot -o UrlHandler var items = args. Split(":"); var tag = items; var id = items; var executable = args;. . .	Отправка уведомления на почту при изменении наименования справочника Maks 24.03.2026 Программная отправка письма электронной почты на примере изменения наименования типового справочника "Склады" в конфигурации БП3. Перед реализацией необходимо выполнить настройку системной учетной. . .	модель ЗдравоСохранения 5. Меньше увольнений- больше дохода! anaschu 24.03.2026 Теперь система здравосохранения уменьшает количество увольнений. 9TO2GP2bpX4 a42b81fb172ffc12ca589c7898261ccb/ https:/ / rutube. ru/ video/ a42b81fb172ffc12ca589c7898261ccb/ Слева синяя линия -. . .	Midnight Chicago Blues kumehtar 24.03.2026 Такой Midnight Chicago Blues, знаешь?. . Когда вечерние улицы становятся ночными, а ты не можешь уснуть. Ты идёшь в любимый старый бар, и бармен наливает тебе виски. Ты смотришь на пролетающие. . .
SDL3 для Desktop (MinGW): Вывод текста со шрифтом TTF с помощью библиотеки SDL3_ttf на Си и C++ 8Observer8 24.03.2026 Содержание блога Финальные проекты на Си и на C++: finish-text-sdl3-c. zip finish-text-sdl3-cpp. zip	Жизнь в неопределённости kumehtar 23.03.2026 Жизнь — это постоянное существование в неопределённости. Например, даже если у тебя есть список дел, невозможно дойти до точки, где всё окончательно завершено и больше ничего не осталось. В принципе,. . .	Модель здравоСохранения: работники работают быстрее после её введения. anaschu 23.03.2026 geJalZw1fLo Корпорация до введения программа здравоохранения имела много невыполненных работниками заданий, после введения программы количество заданий выросло. Но на выплатах по больничным это. . .	Контроль уникальности заводского номера Maks 23.03.2026 Алгоритм контроля уникальности заводского (или серийного) номера на примере нетипового документа выдачи шин для спецтехники с табличной частью, разработанного в конфигурации КА2. Номеклатура. . .

@maybecsharp 0 / 0 / 0 Регистрация: 02.03.2020 Сообщений: 4
	02.03.2020, 01:03 [ТС]
	Копируются указатели. Я предполагал, что ломается именно из-за этого. Но как это исправить? 0

@maybecsharp 0 / 0 / 0 Регистрация: 02.03.2020 Сообщений: 4
	02.03.2020, 01:13 [ТС]
	А для чего нужен конструктор CLab(CLab&& obj)? 0