如何在构造函数初始化列表中创建深层副本。 C ++

时间:2014-10-11 03:43:52

标签: c++ constructor linked-list initialization-list

这是列表类中节点的构造函数。我需要制作一个酒厂的深层副本,这是初始化列表中的另一个类。物品是酿酒厂的一个实例。

List::Node::Node(const Winery& winery) : 
    item(winery)
    // your initialization list here
{
    Winery w = winery;
    item = w;
}

酿酒厂的构造函数:

Winery::Winery(const char * const name, const char * const location,
        const int acres, const int rating) :
    name(name),
    location(location),
    acres(acres),
    rating(rating)
{
    char    *nm = new char[strlen(name) + 1];
    char    *loc = new char[strlen(location) + 1];
    strcpy(nm, this->name);
    strcpy(loc, this->location);
    this->name = nm;
    this->location = loc;
    this->acres = acres;
    this->rating = rating;
}

1 个答案:

答案 0 :(得分:2)

在Node-ctor中三次复制酒庄绝对没有共鸣 一次就够了:

List::Node::Node(const Winery& winery) : item(winery) {}

你可以添加一个move-ctor(C ++ 11及更高版本):

List::Node::Node(Winery&& winery) : item(std::move(winery)) {}

类似于Winery 如果这四个人都是成员,那么Winery - ctor已经做了深层复制。

我希望你能记住3的规则,并且还提供了一个自定义副本,复制分配操作员和dtor。
不过,更好的方法是使用std::unique_ptrstd::string

此外,顶级cv限定符是无用的,因此我删除了它们。

Winery::Winery(const char * name, const char * location,
        int acres, int rating) :
    name(strcpy(new char[strlen(name)+1], name),
    location(strcpy(new char[strlen(location)+1], location),
    acres(acres),
    rating(rating)
{}