这是列表类中节点的构造函数。我需要制作一个酒厂的深层副本,这是初始化列表中的另一个类。物品是酿酒厂的一个实例。
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;
}
答案 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_ptr
或std::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)
{}