我需要释放使用'new'时分配的wchar内存还是delete []也释放它？

Question

试图找出释放内存的正确方法，但我找不到明确的答案。我确定它就在那里，但我似乎找不到它，我有点困惑。

这就是我正在做的事情。我从 sUsers 创建一个新的 sStructUsers，然后为用户名分配内存

    struct sStructUsers 
        {
        TCHAR *sName;
        };

    sStructUsers *sUsers = new sStructUsers[TOTALUSERS]();

    // Allocate memory for the TCHARs in sStruct
    for (INT k = 0; k < TOTALUSERS; k++)
        {
        sUsers [k].sName =  (TCHAR*)calloc(128,sizeof(TCHAR))
        }

接下来，我对 sName 成员做任何我需要做的工作

    // do work here

现在，要清理，我要这样做吗：

    delete[] sUsers;

还是我这样做？

    for (INT k = 0; k < TOTALUSERS; k++)
        {
        free(sUsers[k].sName);
        }
    delete[] sUsers?

我不确定是否需要释放内存或者 delete[] 是否会处理它？

Answer 1

如果你能提供帮助，请不要使用new。在大多数情况下，现代 C++ 中不需要 new。 您可以使用std::vector、std::array、std::string（可能是std::basic_string<TCHAR>）、std::unique_ptr 或std::shared_ptr。

如果你觉得你真的需要使用new，请创建一个特殊的类来处理这个问题。 IE。例如在构造函数中创建new，在析构函数中创建delete。

当然，正如 cmets 中所说，永远不要将 new/delete 与 malloc/free 混为一谈。完全不需要在 C++ 中使用这些函数，因为这是 C 的残余。

编辑：添加示例，如何由类本身处理。

struct sStructUsers 
{
    const int MAX_NAME_SIZE = 128;
    sStructUsers()
    {
        sName = new TCHAR[MAX_NAME_SIZE];
    }
    ~sStructUsers()
    {
        delete sName;
    }
    TCHAR *sName;
};

那么你在使用这个类时不需要任何初始化/清理。

请注意，您通常甚至不需要这样做，您可以将您的类定义为：

struct sStructUsers 
{
    std::basic_string<TCHAR> sName;
};

并且免费拥有完整的string 功能。或者像这样，如果你真的需要一个指针：

struct sStructUsers 
{
    const int MAX_NAME_SIZE = 128;
    sStructUsers() : sName(new TCHAR[MAX_NAME_SIZE]) {}
    std::unique_ptr<TCHAR[]> sName;
};

或者，正如我之前提到的，使用其他标准容器。

Answer 2

建议更可取的容器、更惯用的 C++ 和更现代的实践的 cmets 和答案都很好。

但是您的第二个选项回答了您的直接问题

for (INT k = 0; k < TOTALUSERS; k++)
    {
    free(sUsers[k].sName);
    }
delete[] sUsers;

这不会泄漏内存。但是，您的第一个选项会泄漏，因为仅数组的 delete[] 不会释放其元素的堆内存。

简而言之，对每个new[] 使用delete[]，对每个calloc 使用free。

（只有这样才可能重构为更接近 C++，而不是 C 和 C++ 的混合）

Answer 3

每个分配都应该被释放。通过一些代码。正如sklott 所写，有一些工具（容器和其他类）可以为您完成工作。如果其他一些代码不这样做，那么您的代码应该执行释放。

简而言之，您应该同时调用free(sUsers[k].sName) 和delete[] sUsers。就目前而言，删除[] sStructUserss 不会释放其中的指针。

（您也可以修改sStructUsers，使其析构函数直接或通过智能指针进行释放。）

（对于发布“永远不要将 new/delete 与 malloc/free 混淆”的回答者和评论者，请用 cmets 解释原因。了解建议和最佳实践背后的基本原理始终很重要。更新：显然你不能混合使用 new+free 或 malloc+delete 等；这些函数必须正确配对，以免出现堆损坏和泄漏。更好的问题是为什么不使用 new+delete 进行某些分配和malloc/calloc+free for others [更精细：CoTaskMemAlloc()、HeapAlloc()、VirtualAlloc() 或其他平台函数...]。)