Правильно используя конструкторы в C++

Question

У меня есть простая функция присваивания следующим образом:

LinkedList& LinkedList::operator=(const LinkedList &l) { 
    // handle self assignment 
    if (this == &l) { 
     return *this; 
    } 

    // free old elements of the list before the new elements from l are assigned 
    ~*this(); 

    // build the list as a deep copy of l (copy constructor handles empty case) 
    this(l); 

    return *this; 
} 

и всякий раз, когда я запускаю свою программу, я получаю error: ‘this’ cannot be used as a function ответ. Как я должен использовать конструкторы в их реальном контексте? Любая помощь очень ценится!

Answer 1

Правильный синтаксис для того, что вы пытаетесь это:

this->~LinkedList(); 
new(this) LinkedList(l); 

Вы ясно поняли, что это хорошо, чтобы избежать дублирования кода, однако предпочтительный способ пойти об этом, чтобы использовать copy and swap idiom писать оператор присваивания.

Answer 2

Конструкторы или деструкторы ручного вызова - почти всегда очень плохая идея. Они не предназначены для этого.

Вы должны создать отдельные функции для очистки и копирования списка. Эти конструкторы могут использовать конструктор и деструктор.