Как вы называете конструктор копирования внутри функции-члена класса?

Question

Вот что я получил:

void set::operator =(const set& source) 
{ 
    if (&source == this) 
     return; 

    clear(); 

    set(source); 
} 

А вот ошибка я получаю:

vset.cxx:33: error: declaration of 'source' shadows a parameter

Как правильно это сделать?

Answer 1

Я считаю, что с помощью set(source); вы пытаетесь вызвать копию ctor. Вы не можете сделать это на C++, то есть вы не можете явно вызвать ctor. Что вы можете сделать, это написать приватный метод clone и вызвать его как в операторе copy ctor, так и в присваивании.

Answer 2

Эта ошибка обычно является результатом наличия локальной переменной с именем, аналогичной аргументу функции. Можете ли вы разместить больше своего кода?

Answer 3

Сообщение об ошибке, которое вы видите, не соответствует вопросу, но я не знаю, что это имеет значение. Ответ на ваш вопрос заключается в том, что вы не можете вызвать конструктор копирования изнутри вашего кода, потому что объект уже сконструирован.

Если вы хотите избежать дублирования кода между конструктором копирования и оператором =, я предлагаю иметь частную функцию, которая выполняет общую работу, и называть это как из конструктора копирования, так и для оператора =.

Для справки, вы можете позвонить оператору = от конструктора копирования, выполнив:

*this = source; 

Однако, я не думаю, что это хорошая идея, особенно если у вас есть виртуальные функции, или если оператор =() функция принимает полностью построенный объект (что, вероятно, делает).

Answer 4

Как вы уже отметили, set(source); является источником (не предназначен для каламбур) проблемы. Это не совсем то, что вы думаете, это не попытка вызвать копию ctor. Вместо этого он в основном эквивалентен: set source; - т. Е. Он пытается определить объект set с именем source - круглые скобки являются избыточными, но разрешены.

Вы можете вызвать копию ctor в ctor (или примерно в любом месте, где захотите), но это не будет так, как вы хотите, - копия ctor создает копию, поэтому, даже если вы ее вызывали, просто создаст временный объект, который испарится в конце этого утверждения.

Как уже упоминалось, то, что вы, вероятно, хотите, является частной функцией для копирования данных с одного объекта на другой, а затем используйте это как из вашей копии ctor, так и для оператора копирования. Еще лучше определите его, используя объекты, которые можно корректно обрабатывать по умолчанию ctor и операторы присваивания копий.

Answer 5

Вы ищете копию своп идиома:

set& set::operator=(set const& source) 
{ 
    /* You actually don't need this. But if creating a copy is expensive then feel free */ 
    if (&source == this) 
     return *this; 

    /* 
    * This line is invoking the copy constructor. 
    * You are copying 'source' into a temporary object not the current one. 
    * But the use of the swap() immediately after the copy makes it logically 
    * equivalent. 
    */ 
    set tmp(source); 
    this->swap(tmp); 

    return *this; 
} 

void swap(set& dst) throw() 
{ 
    // swap member of this with members of dst 
} 

Answer 6

Я не знаю подробностей того, что ваш файл определяет заголовок, но я хотел бы попробовать что-то вроде этого (возможно, потребуется изменить его для вашего конкретное применение):

---

недействительными устанавливается :: оператор = (Const установить & источник)
{

if (this == &source) 
    { 
    return; 
    } 

size_t i; 
this->clear(); 
data=source.data; 
for (i=0; i<source.child.size(); i++) 
    { 
    child.push_back(new set(*(source.child[i]))); 
    } 

}

---

-Joel