避免 C++ 指针中的内存泄漏 [关闭]

Question

void append(char*& input,const char* str){
  size_t input_len = strlen(input);
  size_t str_len = strlen(str);

  size_t len = input_len + str_len;
  char* temp = new char[len + 1];

  memcpy(temp, input, input_len);
  memcpy(temp + input_len, str, str_len);
  // **#3** delete[] input;
  temp[len] = '\0';
  input = temp;
}
int main(){
  char* s = "string";

  cout << s << endl;
  append(s, " hello"); // **#1** not initialized by an array
  cout << s << endl;
  append(s, " welcome"); // **#2** now having an array
  cout << s << endl;
}

我想在分配新数组后删除以前分配的数组(#3)。但第一次 (#1) 将其称为具有动态分配的指针。

如何避免此类内存泄漏？
有没有办法用“新”识别内存分配？还是其他？

    if(p == new char[])...

看看这个，http://www.cplusplus.com/reference/cstring/strcat/它改变了原来的字符串

Answer 1

我遵循一些简单的规则：

• 不要手动分配内存。如果您发现自己正在编写手册 delete，请停下来三思而后行。
• 使用 std::string 代替 C 风格的字符串
• 使用std::vector<T> 代替C 数组（或std::array<T> 用于固定大小的数组）
• 默认使用std::unique_ptr和std::make_unique
• 如有必要，请使用std::shared_ptr
• 对其他资源使用 RAII 包装器，例如文件

你的代码可以写得这么简单

// With C++17, we could use std::string_view for the input parameters
// I strongly prefer pure functions that don't mutate their arguments
std::string append(std::string const& s1, std::string const& s2) {
    return s1 + s2;
}

// the same function mutating the input argument. 
void append_mutate(std::string& s1, std::string const& s2) {
    s1 += s2;
}

int main(){
  std::string s = "string";

  cout << s << endl;
  s = append(s, " hello");
  cout << s << endl;
  append2(s, " welcome");
  cout << s << endl;
}

我也强烈建议不要使用 C 风格的字符串，尤其是 C 字符串。如果您有 C 背景或没有太多 C++ 经验，我会推荐 A Tour of C++。

这些类的想法是RAII 的一个例子。基本原理是封装资源，例如内存或文件，放入拥有资源并在构造函数/析构函数中处理获取和释放的包装类。这提供了异常安全、确定性的资源处理。