Question

我最近发现，当我在一个类中有指针时，我需要指定一个Copy构造函数。

要了解这一点，我制作了以下简单代码。它编译，但在执行复制构造函数时给出了运行时错误。

我试图只复制复制对象指针的值，但是避免分配相同的地址。

那么，这里有什么不对？

    class TRY{
        public:
        TRY();
    ~TRY();
        TRY(TRY const &);

        int *pointer;

        void setPointer(int);
    };


    void TRY::setPointer(int a){
        *pointer = a;

        return;
    }


    TRY::TRY(){}


    TRY::~TRY(){}


    TRY::TRY(TRY const & copyTRY){
        int a = *copyTRY.pointer;
        *pointer = a;
    }



    int main(){

        TRY a;
        a.setPointer(5);

        TRY b = a;

        b.setPointer(8);

        cout << "Address of object a = " << &a << endl;
        cout << "Address of object b = " << &b << endl;

        cout << "Address of a.pointer = " << a.pointer << endl;
        cout << "Address of b.pointer = " << b.pointer << endl;

        cout << "Value in a.pointer = " << *a.pointer << endl;
        cout << "Value in b.pointer = " << *b.pointer << endl;

        return 0;
    }

我将把这个概念用于其他有很多指针的类，我需要将所有值从on对象复制到另一个。最初需要复制此代码，因此我希望保持复制的可能性（我不会将复制构造函数隐藏为私有）。

此外，我需要实现的真正类有10个指针，它可能会随着时间而变化。在C ++中使用深层复制构造函数是否有一种更聪明的方法？...

Answer 1

使用语句int* pointer，您刚刚定义了一个指针，但没有分配任何内存。首先，您应该通过分配一些内存来指向正确的内存位置：int* pointer = new int。然后再次在复制构造函数中，您必须为复制的对象分配内存。另外，不要忘记在析构函数中使用delete释放内存。

我希望这个例子有所帮助：

class B
{

public:
    B();
    B(const B& b);
    ~B();
    void setVal(int val);

private:
    int* m_p;
};

B::B() 
{
    //Allocate the memory to hold an int
    m_p = new int;

    *m_p = 0;
}

B::B(const B& b)
{
    //Allocate the memory first
    m_p = new int;

    //Then copy the value from the passed object
    *m_p = *b.m_p;
}

B::~B()
{

    //Release the memory allocated
    delete m_p;
    m_p = NULL;
}

void B::setVal(int val)
{
    *m_p = val;
}

Answer 2

我最近发现，当我我需要在课堂上有指针指定一个Copy构造函数。

这不完全正确。当你的类中有指针并使用new分配内存时，你必须担心复制构造函数。另外，不要忘记赋值运算符和析构函数。您必须删除使用delete分配的内存。

它被称为Law Of The Big Three。

示例：

  ~Matrix();  //Destructor
  Matrix(const Matrix& m); //Copy constructor
  Matrix& operator= (const Matrix& m); //Assignment operator

Answer 3

如果要进行深层复制，当然还必须分配新内存来保存值。如果原始文件有一个指向int的指针，并且您不希望该副本使用相同的指针值，则必须分配新内存来保存int，然后将值复制到那里。

您的示例不是很清楚，它不会显示您的复制构造函数的实现，也不会显示pointer成员的初始化方式。

Answer 4

我最近发现，当我我需要在课堂上有指针指定复制构造函数

通常，通过声明（和分配运算符）私有而不实现它来简单地禁用它是个好主意。

Answer 5

如果它有一个指向常规类型的指针，那么

A::A(const A& a):
  pointer_( new int( *a.pointer_ ) )
{
}

如果它有一个指向某个基类的指针，那么

A::A(const &a ):
  pointer_( a.pointer_->clone() )
{
}

克隆是prototype pattern

的实现

不要忘记删除析构函数中的指针

A::~A()
{
    delete pointer_;
}

修复你的例子

TRY::TRY(TRY const & copyTRY){
    int a = *copyTRY.pointer;
    pointer = new int(a);
}

Answer 6

你的问题就在这一行：

    *pointer = a;

通常在默认构造函数中发生的所有事情尚未发生，包括*pointer的内存分配。

修复是为整数分配内存。您可以使用malloc和朋友或new，但请确保它与您在默认构造函数中使用的方法相同，因为您只获得一个析构函数，并且调用必须匹配。

Answer 7

如果成员（浅）副本没问题，那么您不必做任何事情。如果您需要深层复制，则必须为所有成员的副本分配新存储空间。

Answer 8

编写复制构造函数时，应为所有成员分配内存。在你的情况下：

TRY::TRY(TRY const & copyTRY){
    pointer = new int(*(copyTry.pointer));
}

运算符=在某种程度上相似，但没有内存分配。

TRY& operator=(TRY const& otherTRY){
      this->a  = *(otherTry.pointer)
      return *this
}

Answer 9

通常情况下，如果你需要编写一个复制构造函数或赋值运算符，那么你做错了。将复制构造函数和赋值运算符保留给标准库的实现者。编写已经可复制和可分配元素的类，您不必编写自己的类。

例如，也许int * member应该是std :: vector。

如果你不能使类默认可复制/可赋值，也许你可以通过声明但不实现私有拷贝构造函数和赋值运算符使其不可复制/可赋值。

只有当上述任何一个都不可行时，才应该实现自己的复制构造函数或赋值运算符。

Answer 10

这里是复制构造函数的例子

class Test
{
private:
    int *s;
    int size;
public:
    Test(int a, int b);
    Test(const Test&t);
    ~Test();
    void setValue(int l);
    void getValues();
};


Test::Test(int a, int b)
{
    s = new int;
    *s = a;
    this->size = b;
}
Test::Test(const Test&t) {
    s = new int;
    *s = *(t.s);
    this->size = t.size;
}
void Test::setValue(int l) {
    *s = l;
}
void Test::getValues() {
    cout << "value of s: " << *s << endl;
    cout << "value of size: " << this->size << endl;
}


Test::~Test() {
    cout << "memory de allocated!!!" << endl;
    delete s;
}

使用指针复制构造函数

10 个答案: