Конструктор копий для шаблонного класса(связный список)

@SadiQ228 · Регистрация: 16.12.2016

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

Не совсем понятно как копировать объекты из динамической памяти, в частности правильная реализация конструктора копий.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
template<typename T>
inline List<T>::List(const List &obj)
{
    this->head = obj.head;
    this->count = obj.count;
    if (this->head->next == nullptr)
    {
        this->head->data = obj.head->data;
    }
    else
    {
        Node<T>*tmp = this->head;
        while (tmp->next != nullptr)
        {
            obj.push_back(tmp->data);
            tmp = tmp->next;
        }
    }
}

это то что я придумал, но не работает
вот main

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
int main()
{
    List <int> lst;
    lst.push_back(5);
    lst.push_back(10);
    lst.push_back(15);
    for (int i = 0; i < lst.size(); i++)
    {
        cout << lst[i] << endl;
    }
    cout << "!!!!!!!!!!!!!!!" << endl;
    List<int>qwe(lst);
    for (int i = 0; i < qwe.size(); i++)
    {
        cout << qwe[i] << endl;
    }
    cout << "!!!!!!!!!!!!!!!" << endl;
    return 0;
}

Конструктор копий для шаблонного класса(связный список)

@SadiQ228 · 02.10.2018, 00:01 **[ТС]**

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

@JaponDemon · 02.10.2018, 00:44

конструктор копии

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
template<typename T>
 List<T>::List(const List<T> &other)
{
    if (other.head != nullptr)
    {
            Node<T>*tmp = other.head;
 
            while (tmp != nullptr)
            {
                this->push_back(tmp->data);
                tmp = tmp->next;
            }
        this->count = other.count;
    }
}

оператор присваивания

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
template<class T>
List<T>& List<T>::operator=( const List<T>& other )
{
    if ( this != &other )
    {
        if (other.head != nullptr)
        {
            Node<T>*tmp = other.head;
 
            while (tmp != nullptr)
            {
                this->push_back(tmp->data);
                tmp = tmp->next;
            }
            this->count = other.count;
        }
    }
    return *this; 
}

деструктор нужен для полноты картины

Не по теме:

move конструктор и оператор = move пока не умею.

@SadiQ228 · 02.10.2018, 00:45 **[ТС]**

Спасибо ща проверю
А деструктор у меня делает popFront пока size не ноль

@JaponDemon · 02.10.2018, 00:55

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

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
template<class T>
List<T>& List<T>::operator=( const List<T>& other )
{
    if ( this != &other )
    {
        if (other.head != nullptr)
        {
           this->Clear();      // Очистка списка
            .......
        }
    }
    return *this; 
}

@SadiQ228 · 02.10.2018, 01:01 **[ТС]**

бро твой код отработал, не совсем понял зачем чистить список?
P.s. понял, ты про присваивание) да спасибо учту!

@JaponDemon · 02.10.2018, 01:23

Посмотрел со стороны
и что-то не нравится copy-конструктор.
Голову списка обнулить и задать размер лучше в списке инициализации.
А то может случиться, что остануться неинициализированы.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
template<typename T>
 List<T>::List(const List<T> &other) 
      : head(nullptr), count(other.count)   //  список инициализации
{
    if (other.head != nullptr)
    {
            Node<T>*tmp = other.head;
 
            while (tmp != nullptr)
            {
                this->push_back(tmp->data);
                tmp = tmp->next;
            }
    }
}

@DrOffset · 02.10.2018, 08:31

JaponDemon, а оператор присваивания можно красиво реализовать через copy-and-swap

@JaponDemon · 02.10.2018, 09:47

copy-and-swap idiom
оператора присваивания параметр передаётся по значению

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
template<class T>
List<T>& List<T>::operator=(List<T> other) 
{
    swap(*this, other); 
 
    return *this;
}
 
// прототип template<class T> friend void  swap( List<T> &lhs, List<T> &rhs ) noexcept;
template<class T>
void  List<T>::swap( List<T> &lhs, List<T> &rhs ) noexcept
{
   std::swap(lhs.head, rhs.head);
   std::swap( lhs.count, rhs.count);
}

@DrOffset · 02.10.2018, 11:59

JaponDemon,
теперь и до move-вариантов недалеко

@JaponDemon · 02.10.2018, 12:04

Не по теме:

рановато , почитаю ещё )

@SadiQ228 · 02.10.2018, 14:04 **[ТС]**

Не по теме:

это мы не проходили это нам не задавали)))

Новые блоги и статьи Все статьи Все блоги /
Расчёт переходных процессов в цепи постоянного тока igorrr37 16.01.2026 / * Дана цепь постоянного тока с R, L, C, k(ключ), U, E, J. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа, решает её и находит токи на L и напряжения на C в установ. режимах до и. . .	Восстановить юзерскрипты Greasemonkey из бэкапа браузера damix 15.01.2026 Если восстановить из бэкапа профиль Firefox после переустановки винды, то список юзерскриптов в Greasemonkey будет пустым. Но восстановить их можно так. Для этого понадобится консольная утилита. . .	Изучаю kubernetes lagorue 13.01.2026 А пригодятся-ли мне знания kubernetes в России?	Сукцессия микоризы: основная теория в виде двух уравнений. anaschu 11.01.2026 https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/
WordPad для Windows 11 Jel 10.01.2026 WordPad для Windows 11 — это приложение, которое восстанавливает классический текстовый редактор WordPad в операционной системе Windows 11. После того как Microsoft исключила WordPad из. . .	Classic Notepad for Windows 11 Jel 10.01.2026 Old Classic Notepad for Windows 11 Приложение для Windows 11, позволяющее пользователям вернуть классическую версию текстового редактора «Блокнот» из Windows 10. Программа предоставляет более. . .	Почему дизайн решает? Neotwalker 09.01.2026 В современном мире, где конкуренция за внимание потребителя достигла пика, дизайн становится мощным инструментом для успеха бренда. Это не просто красивый внешний вид продукта или сайта — это. . .	Модель микоризы: классовый агентный подход 3 anaschu 06.01.2026 aa0a7f55b50dd51c5ec569d2d10c54f6/ O1rJuneU_ls https:/ / vkvideo. ru/ video-115721503_456239114

@SadiQ228 -4 / 24 / 7 Регистрация: 16.12.2016 Сообщений: 716
	02.10.2018, 00:01 [ТС]
	стер из аргументов const, стало компилировать, но не работает 0

@SadiQ228 -4 / 24 / 7 Регистрация: 16.12.2016 Сообщений: 716
	02.10.2018, 00:45 [ТС]
	Спасибо ща проверю А деструктор у меня делает popFront пока size не ноль 0

@SadiQ228 -4 / 24 / 7 Регистрация: 16.12.2016 Сообщений: 716
	02.10.2018, 01:01 [ТС]
	бро твой код отработал, не совсем понял зачем чистить список? P.s. понял, ты про присваивание) да спасибо учту! 0

@DrOffset 19491 / 10097 / 2460 Регистрация: 30.01.2014 Сообщений: 17,805
	02.10.2018, 08:31
	JaponDemon, а оператор присваивания можно красиво реализовать через copy-and-swap 1

@DrOffset 19491 / 10097 / 2460 Регистрация: 30.01.2014 Сообщений: 17,805
	02.10.2018, 11:59
	JaponDemon, теперь и до move-вариантов недалеко 0

@JaponDemon
	02.10.2018, 12:04
	Не по теме: рановато , почитаю ещё ) 0

@SadiQ228
	02.10.2018, 14:04 [ТС]
	Не по теме: это мы не проходили это нам не задавали))) 0