9

我还没有看到同时使用 unique_ptr 和 move-semantics 的 pimpl 示例。

我想向 STL 派生容器添加一个 CHelper 类，并使用 pimpl 隐藏 CHelper 的功能。

这看起来对吗？

派生的.h

class CDerived : public set<CSomeSharedPtr>, public CHelper  
{
//...
};

`

助手.h

// derived containers need to support both copy and move, so CHelper does too  

class CHelper  
{  
private:  
    class impl;  
    unique_ptr<impl> pimpl;  

public:  
//--- default: need both cotr & cotr (complete class) in order to use unique_ptr<impl>  
    CHelper();  
    ~CHelper();  

//--- copy  
    CHelper(const CHelper &src);         //copy constructor  
    CHelper& operator=(const CHelper &src);//assignment operator  

//--- move  
    CHelper(CHelper &&src);         //move constructor  
    CHelper& operator=(CHelper &&src);//move operator  

//--- expose public methods here  
    void SetModified(BOOL bSet=TRUE);  
};

助手.cpp

//===========================  
class CHelper::impl  
{  
public:  
BOOL m_bModified; //has the container been modified (needs to be saved)  
// ... other data  

impl() {m_bModified = FALSE;}  

//--- copy cotr/assign  
impl(const impl &src)  
{  
  *this = src;  
}  

void operator=(const impl &src)   
{  
  m_bModified = src.m_bModified;  
  // ...other data  
}  

//--- move cotr/assign ?? do I need to write move cotr/assign ??   

};  

//============================  
CHelper::CHelper() : pimpl(unique_ptr<impl>(new impl())) {}

CHelper::~CHelper() {}  

//--- copy  
CHelper::CHelper(const CHelper &src)  
: pimpl(unique_ptr<impl>(new impl(*src.pimpl))) {}

CHelper& CHelper::operator=(const CHelper &src)  
{  
  if (this != &src)  
    *pimpl = *src.pimpl;  

  return *this;  
}  

//--- move  
CHelper::CHelper(CHelper &&src)  
{  
  if (this != &src)  
  {  
    pimpl = move(src.pimpl); //use move for unique_ptr  
    src.pimpl.reset(nullptr);//leave pimpl in defined / destructable state  
  }  
}  

CHelper& CHelper::operator=(CHelper &&src)  
{  
    if (this != &src)  
    {  
      pimpl = move(src.pimpl); //use 'move' for unique_ptr  
      src.pimpl.reset(nullptr);//leave pimpl in defined / destructable state  
    }  
    return *this;  
}
4

1 回答 1

4

考虑到唯一的成员CHelper是 aunique_ptr并且 copy 的默认实现是调用基数和成员的副本，move的默认实现是调用基数和成员的移动，不需要覆盖CHelperctors并分配。让默认值完成他们的工作。他们只会调用适当的 unique_ptr 移动构造函数和运算符。

关于组合CHelper并set<...>形成 CDerived... 这不是“规范设计”（set不是 OOP 类...，而 CHelper 也不是）但如果使用得当可以工作（不要尝试分配CDerived到CHelper*广告调用删除就可以了，否则你会流泪）。只是没有足够的信息来理解他们的目的。

如果问题是“我希望 CHelper 也能够复制”，那么您最好遵循这样的成语（#include并且using namespace分开......）

class CHelper
{
    struct impl
    {
       .....
    };
public:
    // create and initialize
    CHelper() :pimpl(new impl) {}

    // move: just keep the default
    CHelper(CHelper&& a) = default;

    // copy: initialize with a copy of impl
    CHelper(const CHelper& a) :pimpl(new impl(*a.pimpl)) {}

    CHelper& operator=(CHelper a) //note: pass by value and let compiler do the magics
    { 
        pimpl = move(a.pimpl); //a now nullifyed, but that's ok, it's just a value
        return *this; 
    }

    ~CHelper() = default; //not really necessary
private:
    unique_ptr<impl> pimpl;
};

当然，您可以根据需要随意分离声明和实现。

根据 John Balcom 的评论进行编辑。

是的，当然代码会发生变化，但实质上不会发生变化。您只需将声明转发struct impl;到CHelper（以便 unique_ptr 具有含义），然后CHelper::impl在其他地方声明结构（可能在所有 CHelper 实施将完成的 CPP 文件中）。

这里唯一需要注意的是，在 CPP 文件中，CHelper::impl 必须同时定义构造函数和析构函数，以便在 CPP 文件中具有一致的 unique_ptr 实例化（必须调用 impl 析构函数）。否则，某些编译器可能会在所有包含 CHelper 声明的文件中出现“类型使用不完整”错误。

关于第二点（源自std::set），这是 C++ 编程的一个有争议的方面。由于与 C++ 本身无关的原因，但在面向对象编程学派中，“inhertance”表示“is a”，“is a”表示“object subtitution possible”。正因为如此，如果基 dtor 不是虚拟的，由于通过基指针删除对象是 UB，从而使对象替代 UB，OOP 学校拒绝将任何没有虚拟 dtor 的类的继承作为教条，并且因为方式他们在开始编程时就受过教育，如果您这样做，他们就会开始向您吐火。

对我来说，这在你的设计中不是问题，但是他们理解 C++ 继承并不意味着“是”而是“就像”并且并不意味着对象替换是他们的错误（对我来说，这是他们认为每个 C++ 类都是错误的）一个 OOP 对象，而不是您使用工具来做对您有用的事情，如果您想更清楚地说明我的立场，请看这里或这里：C++ 中的对象替换不是针对“对象”而是逐个方法的，因为每个方法都可以虚拟或不相互独立）。就是说，也许你必须和那些人一起工作，所以......评估在他们自己喜欢的宗教实践中不跟随他们的利弊。

于 2012-06-26T17:03:56.487 回答