C 内存泄漏与类成员

C++ memory leak with class members

本文关键字：成员 泄漏 内存      更新时间：2023-10-16

vs2010中的debug_vld揭示了来自类成员创建/初始化/删除的一些内存泄漏。

my_member是具有双重类型*的数据成员。在构造函数中，我有

my_member = NULL ;

然后，在某种方法中，我需要为my_member分配内存。我不能在构造函数中这样做由于我还不知道数组的尺寸，并且/或大小可能会在方法的不同调用中不同。我在这种方法中要做的是检查成员是否为null。如果是这样，我可以为此分配空间，如果不是这样，我可以在数组上操作（使用Accesor []更改其元素的值）。看起来像

void MyClass::my_method()
{
    if( my_member == NULL )
        my_member = new double[n_dim] ;
    for(int k = 0 ; k < n_dim ; k++ )
         my_member[k] = k ;
}

和内存泄漏发生在my_member = new double[n_dim] ;线上。

在破坏者中，我有

delete[] my_member ;

怎么了？如何正确分配？

谢谢！

使用 std::vector<double>是首选的方法，但是如果您想拥有RAW double*并写下复制构造函数，请手动移动constructor和 operator="，您应该做类似的事情：

#include <assert.h>  // for assert
#include <algorithm> // for std::swap
class MyClass
{
  //
  // Raw array
  //
  double * m_ptr; // raw pointer
  size_t m_dim;   // number of items in the array
public:
  // Default constructor - creates empty vector
  MyClass()
    : m_ptr(nullptr)
    , m_dim(0)
  {
  }

  // Copy constructor
  MyClass(const MyClass& src)
    : m_ptr(nullptr)
    , m_dim(0)
  {
    // Special case of empty source
    if (src.m_dim == 0)
    {
      assert(src.m_ptr == nullptr);
      return;
    }
    // Allocate and deep-copy from source
    m_ptr = new double[src.m_dim];
    m_dim = src.m_dim;
    for (size_t i = 0; i < m_dim; i++)
      m_ptr[i] = src.m_ptr[i];
  }
  // Move constructor: steal the "guts" from src
  MyClass(MyClass&& src)
  {
    m_ptr = src.m_ptr;
    src.m_ptr = nullptr;
    m_dim = src.m_dim;
    src.m_dim = 0;
  }
  // Destructor
  ~MyClass()
  {
    delete [] m_ptr;
  }
  // Unified operator=
  MyClass& operator=(MyClass src)
  {
    std::swap(m_ptr, src.m_ptr);
    std::swap(m_dim, src.m_dim);
    return *this;
  }
};

如果代码中还有其他位置，您将my_member设置为NULL而不调用delete[]，则是。如果您不遵守三个规则（正确实现的复制构造函数和分配运算符），则会遇到各种麻烦。

为了防止这种情况，请改用std::vector<double>，您可以在哪里进行：

void MyClass::my_method()
{
    my_member.resize(n_dim); // yay, learned something new here 
    for(int k = 0 ; k < n_dim ; k++ )
         my_member[k] = k ;
}

这样，您就不会管理内存，因此不需要驱动器（除非是virtual，在这种情况下，它可以为空）。

相关文章：