为什么在构造函数的初始化列表中使用 new 运算符进行 memleak？

发布于 2024-12-17 02:12:05 字数 279 浏览 0 评论 0原文

给定一个带有私有成员变量 name 和基本构造函数的 C++ 简单类：

#include <QString>

class Testclass
{
  private:
    QString *name;

  public:
    Testclass(): name(new QString()) {}
};

为什么 valgrind 的 memcheck 会抱怨 1 个块中的 8 个字节，而在使用此构造函数时这些字节肯定会丢失？

原文

Given a simple class in C++ with a private member variable name and a basic constructor:

#include <QString>

class Testclass
{
  private:
    QString *name;

  public:
    Testclass(): name(new QString()) {}
};

Why does valgrind's memcheck complains about 8 bytes in 1 block, which are definitely lost when using this constructor?

分享到QQ

分享到微博

如果你对这篇内容有疑问，欢迎到本站社区发帖提问参与讨论，获取更多帮助，或者扫码二维码加入 Web 技术交流群。

发布评论

需要登录才能够评论，你可以免费注册一个本站的账号。

辞慾 2024-12-24 02:12:05

~Testclass(){delete name;}

会堵住你的漏洞。 C++ 不会（也不应该）为您做这件事。

ETA：ildjarn 正确地指出您还应该有一个复制构造函数和赋值运算符。

TestClass(const TestClass &cp): name(new QString(*(cp.name)) ) {}
const TestClass& operator=(const Testclass&rhs)
{
  (*name)=(*hrs.name);
  return *this;
}

否则，默认的复制构造函数或赋值运算符将导致同一内存被删除两次。大多数需要析构函数的类应该替换或禁用默认的复制构造函数和赋值运算符。这就是所谓的“三法则”。

您可能需要考虑简单地按值保存 QString，因为它本身可能是一个轻量级容器类，如 std::string 或 std::vector。但如果您是 C++ 初学者，这样做一次就是一个宝贵的教训。

~Testclass(){delete name;}

Will plug your leak. C++ doesn't (and shouldn't) do this for you.

ETA: ildjarn correctly points out that you should also have a copy constructor and assignment operator.

TestClass(const TestClass &cp): name(new QString(*(cp.name)) ) {}
const TestClass& operator=(const Testclass&rhs)
{
  (*name)=(*hrs.name);
  return *this;
}

Otherwise, the default copy constructor or assignment operator will cause the same memory to be deleted twice. Most classes that require a destructor should replace or disable the default copy constructor and assignment operator. This is called the "Rule of Three."

You might want to consider simply holding the QString by value, because it's likely itself a lightweight container class, like std::string or std::vector. But if you're a C++ beginner, doing it this way once is a valuable lesson.

回复收藏 0 原文

~没有更多了~