删除循环中的内存泄漏

Question

“由于删除循环中的逻辑错误，您最终出现了一些内存泄漏。”

我朋友说了这个，我没看到。

for(int i=0; i<nrOfAvailableSeats; i++)
{
    delete passengers[i];
}
delete [] passengers;

Answer 1

您从 nrOfAvailableSeats 设置为初始值 (100) 开始，然后在程序运行时它可能会递减（因为您有

nrOfAvailableSeats--;

在您的代码中），因此当您在循环中使用它时，您不会delete 某些对象。

至少你必须定义一个全局常量：

const int PlaneCapacity = 100;

并使用它来初始化nrOfAvailableSeats 以及在数组创建和删除循环中。

Answer 2

这是因为您在删除仅部分元素后删除了一系列乘客。 因此，当您执行 delete [] passengers; 时，某些 ( [i], [i+1], ... ) 元素可能不会被删除。

Answer 3

在链接文件中有代码：nrOfAvailableSeats--;

当您释放数据时，您可能会遇到 memleak，因为 nrOfAvailableSeats 可以是

Answer 4

您将所有可用席位初始化为 0/null，因此，在清理阶段，只需遍历所有 100 个元素并删除每个项目。 （编辑：在删除之前检查空指针是多余的，没关系。）

如果您被允许使用 STL，也可以使用 std::vector。 然后只需遍历向量并将它们全部删除，因为向量中只有实际使用它们的项目。 （如果您只使用 push 和 pop）。

要从正在使用的向量中删除某些内容（不会完全清空），是将要删除的项目替换为vector.back()，然后删除vector.back()，然后再删除pop_back()。 注意：这会使任何迭代器无效，您必须正确重新分配它们，或者如果您只想删除一项，只需通过break 或return 跳出迭代循环。

#include "Passenger.h"
#include "EconomyClass.h"
#include "FirstClass.h"
#include <vector>



int main()
{
std::vector<Passenger*> passengers

//add passengers the vector grows when needed, it does so by copying the items, which, in the 
//case of raw pointers like this, fast.
passengers.push_back(new FirstClass(name, purpose, chair));
passengers.push_back(new EconomyClass(name, purpose, food));

//delete all, starting from the back of the stack
//check the std::vector reference online for details
std::vector<Passenger*>::reverse_iterator start,end;
start = passengers.rbegin();
end = passengers.rend();

//iterate through all items, in this case, 2
for(;start!=end;++start)
{
delete *start;
}
//optional since it would get called on cleanup anyway)
passengers.clear();

}