哪个指针分配更好（C++）

Question

foo.h

#include "class1.h"

class foo{

    private:
        class1* class1ObjectPointer;
    
    public:
        foo();
        virtual ~foo();
    
        void foomethod();

}

foo.cpp（版本 1）

#include "foo.h"

foo::foo()
{
    this->class1ObjectPointer = new class1();
}


foo::~foo()
{
    if( this->class1ObjectPointer != NULL ){

        delete class1ObjectPointer;
        this->class1ObjectPointer = NULL;
    }
}


foo::foomethod(){

    *(this->class1ObjectPointer) = some_method(); 
    //suppose some method returns an object of class1

}

foo.cpp（版本 2）

#include "foo.h"

foo::foo()
{
    this->class1ObjectPointer = NULL;
}


foo::~foo()
{
    if( this->class1ObjectPointer != NULL ){

        delete class1ObjectPointer;
        this->class1ObjectPointer = NULL;
    }
}


foo::foomethod(){

    class1 object;
    object = some_method(); 
    //suppose some method returns an object of class1
    this->class1ObjectPointer = new class1(object); // copy constructor
}

以下情况下哪种指针赋值更好：

1. class1 的对象始终具有固定大小（例如，其中包含固定变量的类）
2. class1 的对象可能具有可变大小的成员（例如，其中包含可能具有不同大小的矩阵的类）

---

你会建议任何其他更好的方法来做我在这些 sn-ps 中所做的事情吗？

Answer 1

根据第一个示例重新分配对象可能比分配新对象更好（更有效，更容易避免内存泄漏） - 正如第二个示例中的内存泄漏所证明的那样。

看起来你根本不需要指针；只需在你的类中嵌入一个对象：

class foo {
private:
    class1 class1Object;

public:
    // implicit constructors, assignment and destructor are fine
    void foomethod() {class1Object = some_method();}
};

您的第二种情况没有意义 - 特定类的对象都相同大小。

如果你确实需要存储一个指针（也许是因为你需要多态性），那么处理它的最简单方法是使用智能指针：

class foo {
private:
    std::unique_ptr<class1> class1ObjectPointer;

public:
    foo() : class1ObjectPointer(new class1) {}
    // implicit copying and destructor are fine

    foomethod() {*class1ObjectPointer = some_method();}
    // or, if not assignable, class1ObjectPointer.reset(new class1(*some_method()));
};

如果您真的想自己管理它，那么您需要根据Rule of Three 覆盖默认的复制构造函数和复制赋值运算符，确保在重新分配指针时删除旧对象，并小心异常安全。并且不需要在删除之前检查NULL，也不需要在析构函数中分配NULL。

Answer 2

无论任何分配如何，类的实例都具有恒定的大小。 如果将成员变量分配给某个指针，则实例的大小不会改变。只是内存中的一些垃圾（或 NULL 值，如果你先初始化它）将被其他分配对象的地址替换。

Answer 3

1. if ptr != NULL 检查在 delete 前面是多余的，因为它在内部进行检查。

2. 这些变体中哪个更好取决于具体情况。两者各有千秋。

3. 这里根本不要使用指针。如果必须，请使用 std::unique_ptr（需要 C++11）或 boost::shared_ptr。

4. 在任何情况下，构造函数中的成员都应该初始化，而不是赋值。也就是这样写：

   foo::foo() : class1ObjectPointer(new class1) { }
   

   这使用initialisation list。