C++中的函数导致的内存泄漏

Question

有一个函数可以创建新内存并在构造函数中使用。 但是，我不知道如何在析构函数中访问它来删除它。

在构造函数中使用（相同文件）player.cpp

#include <string.h>

#include "Player.h"

// TODO: Fix the bugs in this file

Player::Player(const char* name) : name_(0)
{
  copyString(&name_, name);
}

Player::Player(const Player& copy) : name_(0)
{
  name_ = copy.name_;
}

Player::~Player()
{ 
  delete [] name_; // Not sure if it works. errors promts- double free
}

Player& Player::operator=(const Player& copy)
{
  return *this;
}

void Player::copyString(char** dest, const char* source)
{
  unsigned int str_len = strlen(source);
  char* str = new char[str_len+1]; //This line
  strncpy(str, source, str_len);
  str[str_len] = '\0';

  *dest = str;
}

std::ostream& operator<<(std::ostream& out, const Player& player)
{
  out << player.name_ << std::endl;
  return out;
}

我只能更改 .cpp 文件。 我添加了删除行 i 析构函数，但出现错误。

Answer 1

拷贝构造函数需要做深拷贝，否则有2个东西指向同一个地方……

Player::Player(const Player& copy) : name_(0)
{
  copyString(&name_, copy.name_);
}

operator= 不起作用 - 只是忽略了旧值，所以也应该复制一份....

Player& Player::operator=(const Player& copy)
{
  if( this != & copy ){
      delete [] name_;
      copyString(&name_, copy.name_);
  }
  return *this;
}

你已经在析构函数中有一个delete []。

3（或0）规则是关于如何管理资源的规则。如果您实现了析构函数、复制构造函数或 operator=，那么您可能需要同时实现这三个。

如果你不这样做，你就可以不使用它们（0 规则）。其思路是，如果资源在复制、构造或破坏时需要注意，则将其制成一个单独的类，该类对所有三种行为都进行了充分考虑。

如果您创建基本类（例如这个“字符串”类 - Player），那么包含 Player 的东西可以在不了解副本语义的情况下存在，并且知道为它们管理资源是安全的。