清除 C++ 中的 STL 指针列表

Question

我已经定义了一个指针列表。在清除列表之前我应该​​如何释放所有这些指针？删除所有列表成员的最佳方法是什么？在下面的程序中，是否需要释放为 struct 分配的内存？请参阅我的内联 cmets。

struct MyStruct {
    char *data;
    int len;
};

typedef std::list<struct MyStruct *> myStruct_list;
myStruct_list l_list;

/* Prepare a list */

for( int i = 0; i < 10; i++) {
    struct MyStruct *s = (MyStruct *)malloc(sizeof(struct MyStruct));
    s->data = (char*)malloc(MAX_LEN);
    get_random_data(s->data,size);
    s->len = strlen(s->data);
    l_list.push_back(s);
}


/* Delete all members from a list */

myStruct_list::iterator it;
for (it = l_list.begin(); it != l_list.end(); ++it) {
    if (*it) {
        free(*it);       // --->> do I need to free (*it)->data ?????
    }
}
l_list.clear();

Answer 1

我想了解一下下面的程序是否有内存泄漏？

是的，你就在这里：

p = (char*)malloc(MAX_LEN);
p = (char *)buf;

您分配内存并将其分配给p，下一行您将其释放。所以：

1. 您不应在 C++ 程序中使用 malloc()，除非您需要传递将由 C 代码管理的数据
2. 您应该使用std::string 等特殊数据结构来管理您的数据。
3. 如果您仍需要分配动态内存，请使用智能指针。

如果有内存泄漏，我应该如何调试？

你不会一开始就创建它们。例如，你怎么写get_random_str（假设你真的必须使用malloc分配它）：

using spchar = std::unique_ptr<char[], decltype(std::free) *>;

spchar get_random_str( int len )
{
    spchar s( static_cast<char *>( malloc( len + 1 ) ), std::free );
    static const char alphanum[] =  "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz";
     for (int i = 0; i < len; ++i) {
         s[i] = alphanum[rand() % (sizeof(alphanum) - 1)];
     }

     s[len] = '\0';
     return s;
 }

注意，这段代码我没有编译，只是给你看思路。

更新：看起来你认为这段代码：

p = (char *)buf;

会将字符串从buf 复制到p，但情况并非如此。相反，您使p 指向buf 的内存，从而丢失malloc() 之前返回的旧值（因此造成内存泄漏），然后您将buf 的地址分配给data，这会在您调用@987654336 时导致UB @ 就可以了，所以你需要什么：

strncpy( p, buf, MAX_LEN );

但即使这样也没有必要，因为您根本不需要 buf：

void myClass::fillData(void) 
{
    s = (MyStruct *)malloc(sizeof(struct MyStruct));
    s->data = (char*)malloc(MAX_LEN);
    get_random_str(s->data,950);
    s->len = strlen(s->data);
    l_list.push_back(s);
}

但这是更多的 C 代码，使用一些 C++ 语法。如果你真的想学习 C++，你应该得到更新的编译器和教科书。

Answer 2

使用for_each中的lambda函数删除元素：

std::for_each(l_list.begin(), l_list.end(), [](const MyStruct* elem){if(elem) free(elem)});

并明确指针：

l_list.clear();