代码之家 › 专栏 › 技术社区 › Pavel

复制构造函数首先复制指针,然后取消引用以获得“深度”复制

copy-constructor c++

Pavel · 技术社区 · 6 年前

我正在查看以下(坏的)示例代码 http://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#c22-make-default-operations-consistent :

class Silly {   // BAD: Inconsistent copy operations
    class Impl {
        // ...
    };
    shared_ptr<Impl> p;
public:
    Silly(const Silly& a) : p{a.p} { *p = *a.p; }   // deep copy
    Silly& operator=(const Silly& a) { p = a.p; }   // shallow copy
    // ...
};

作者试图说明复制任务和复制分配之间的不匹配,但我想知道代码是否有其他方面的缺陷。

“深度复制”构造函数是否有意义?看起来像是进入身体, p 将与 a.p 而自我分配并没有做任何事情(充其量)。

我错过什么了吗?

2 回复 | 直到 6 年前

Lightness Races in Orbit 6 年前

事实上,那不过是胡说八道。

大概这就是例子的重点,尽管我不相信它是一个很好的例子。

否则,作者的意思可能是:

Silly(const Silly& a) : p{std::make_shared<Impl>()} { *p = *a.p; }

或:

Silly(const Silly& a) : p{std::make_shared<Impl>(*a.p)} {}

或者一些。

Vivick 6 年前

Silly(const Silly& a) : p{a.p} { *p = *a.p; }

将,除了复制基础的共享\指针之外,还将复制它所指向的值。我担心的是 shared_ptr<T>::operator* 实际回报 T& 因此,我们的分配将更改控制块的值,实际上什么也没有完成。

但是,同样,目的不是演示实际的深度复制,而是提供不一致的复制操作。

A的深度拷贝 shared_ptr 像这样吗

Silly(const Silly& a) : p{make_shared<Impl>(*a.p)} {}

推荐文章

patrick · 从boost::绑定1和2占位符运行make失败。。。?

1 年前

rookie · 检查函数模板的所有参数包参数是否属于int

1 年前

ivaigult · -W转换和隐式字符串到布尔类型转换

1 年前

David Bien · declval<_Xp(&)()>()()-这在下面的上下文中意味着什么?

1 年前

khallad sayed · 在%上,C++表达式必须具有整型或无范围枚举类型

1 年前

rainer · 后台插入程序的初始化

1 年前

Community wiki · 以理智、安全和高效的方式复制文件

1 年前

Shefali Kanaujia · 对C中向量的向量进行排序++

1 年前

Prakritish Ghosh · 现在显示正确输出的函数中的Infix到Postfix错误[重复]

1 年前

Ma Joonyoung · 粗粒度和细粒度链表的时间比较

1 年前