使用动态分配将 char* 复制到另一个 char**

Question

我还是这个主题的新手，但谁能解释我如何将一个 char* 复制到另一个 char** 作为参数而不发生内存泄漏？

void Hardware::copyString(char** dest, const char* source)
{
  size_t length = strlen(source);
  auto string = new char[length+1];
  strncpy(string, source, length);
  string[length] = '\0';
  *dest = string;
  //need to be freed
  free(string); //if i free here the data would getting lost
}

和一个演员：

Hardware::Hardware(const char* name, int cost)
{
 copyString(&name_, name);
 cost_ = cost;
}

main.cpp

Hardware hard("CPU", 250)

我应该在哪里释放它而不发生内存泄漏并获得正确的输出？

Answer 1

无论您的实现中存在什么问题，您的问题的简单答案是在析构函数处释放它。

如果您不想暴露name_ 变量，您可以在Hardware 类的析构函数中free 它。

Hardware::~Hardware()
{
    delete[] name_;
}

如果您 free 在函数本身内部，则您正在删除堆分配，即使您传递不同的变量，因为它们指向同一个内存位置。

但是，正如其他人所建议的，使用字符串的最佳方法是使用 std::string。