0

我想知道是否有一种方法可以实现复制构造函数和赋值运算符，以便在为类重新定义它们时只需要进行少量修改。

例如，考虑这样一个类：

class Foo {
private:
    int* mInt_ptr;
    /* many other member variables
       of different types that aren't
       pointers */
public:
    Foo();
    Foo(const Foo&);
    Foo& operator=(const Foo&);
    ~Foo();
};

现在，为了处理指针，mInt_ptr我需要在复制构造函数和赋值运算符中适当地处理它。但是，其余的成员变量可以安全地进行浅拷贝。有没有办法自动做到这一点？

一旦一个类变大，显式写出复制非指针成员变量的操作可能会变得乏味和笨拙，所以我想知道是否有一种方法可以编写一个复制构造函数，例如：

Foo::Foo(const Foo& tocopy)
{
    mInt_ptr = new int(*tocopy.mInt_ptr);
    /* Do shallow copy here somehow? */
}

而不是以下的显式形式：

Foo::Foo(const Foo& tocopy)
{
    mInt_ptr = new int(*tocopy.mInt_ptr);
    mVar1 = tocopy.mVar1;
    mVar2 = tocopy.mVar2;
    ...
    ...
    mVarN = tocopy.mVarN;
}
4

3 回答 3

4

通常，不要使用原始指针，这正是您现在正在与之抗争的原因。相反，使用合适的智能指针，并使用复制交换分配：

class Foo
{
     int a;
     Zip z;
     std::string name;
     value_ptr<Bar> p;

public:
     Foo(Foo const &) = default;

     Foo & operator=(Foo rhs)
     {
         rhs.swap(*this);
         return *this;
     }

     void swap(Foo & rhs)
     {
         using std::swap;
         swap(a, rhs.a);
         swap(z, rhs.z);
         swap(name, rhs.name);
         swap(p, rhs.p);
     }
};

namespace std { template <> void swap<Foo>(Foo & a, Foo & b) { a.swap(b); } }

这value_ptr可能是一个成熟的实现，或者像这样简单的东西：

template <typename T>    // suitable for small children,
class value_ptr          // but not polymorphic base classes.
{
    T * ptr;

public:
    constexpr value_ptr() : ptr(nullptr) { }
    value_ptr(T * p) noexcept : ptr(p) { }
    value_ptr(value_ptr const & rhs) : ptr(::new T(*rhs.ptr)) { }
    ~value_ptr() { delete ptr; }
    value_ptr & operator=(value_ptr rhs) { rhs.swap(*this); return *this; }
    void swap(value_ptr & rhs) { std::swap(ptr, rhs.ptr); }

    T & operator*() { return *ptr; }
    T * operator->() { return ptr; }
};
于 2012-09-05T17:06:55.643 回答
2

您如何将所有浅拷贝位包装在一个小的帮助器结构中并在那里使用默认的复制行为。

class Foo {
private:
    int* mInt_ptr;
    struct helper_t
    /* many other member variables
       of different types that aren't
       pointers */
    } mHelper;
public:
    Foo();
    Foo(const Foo&);
    Foo& operator=(const Foo&);
    ~Foo();
};

Foo::Foo(const Foo& tocopy)
{
    mInt_ptr = new int(*tocopy.mInt_ptr);
    mHelper = tocopy.mHelper;
}

正如 Kerrek 建议的那样，使用更好的原语似乎是更好的设计。这只是另一种可能性。

于 2012-09-05T17:11:36.370 回答
1

无论您使用原始指针还是智能指针，Kerrek 的解决方案都是正确的，因为您应该使用这些创建复制构造函数、析构函数和交换并实现赋值：

class Foo 
{
private:
    int* mInt_ptr;
    // many other member variables
    // of different types
public:
    Foo()
        : mInt_ptr(NULL)
        // initialize all other members
    {}
    Foo(const Foo& that)
        : mInt_ptr(new int(*that.mInt_ptr) )
        // copy-construct all other members
    {}
    Foo& operator=(const Foo& that)
    {
        // you may check if(this == &that) here
        Foo(that).swap(*this);
        return *this;
    }
    ~Foo()
    {
        delete mInt_ptr;
        // and release other resources
    }
    void swap(Foo& that)
    {
        std::swap(mInt_ptr, that.mInt_ptr);
        // swap all members
    }
};

成员在这里是内联的，只是为了保持紧凑，通常不建议使用内联成员定义来负担类定义。

于 2012-09-05T17:23:55.883 回答