Я еще не видел пример pimpl, который использует и unique_ptr, и семантику перемещения.
Я хочу добавить класс CHelper в контейнеры, производные от STL, и использовать 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);
};
Helper.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;
}
class CDerived : public set<CSomeSharedPtr>Это действительно плохая отправная точка. Стандартный контейнер не предназначен для расширения, не наследуйте от них, вместо этого используйте композицию. - person David Rodríguez - dribeas schedule 26.06.2012