此代码是否会导致内存泄漏问题

Will this code cause memory leak issue?

本文关键字：内存 泄漏 问题 代码 是否      更新时间：2023-10-16

考虑到下面的代码，如果我像这样使用Die类的实例会发生什么：

Die d;
d.Roll(20);
d.Roll(15);
d.Roll(30);

在再次为其分配内存之前，我应该或不应该释放值占用的内存吗？ delete[ ]之前new？

哒��

#ifndef DIE_H
#define DIE_H
#include<iostream>
#include<time.h>
using namespace std;

class Die
{
private:
    int number;
    int* values;
    int count;
    void roll();
public:
    Die(){srand(static_cast<int>(time(NULL)));number=0;values=NULL;count=0;}
    void Roll(int n);
    int getNumber()const{return number;}
    void printLastValue();
    void printValues();
    ~Die(){delete [] values;}
};
#endif

死.cpp

#include"die.h"
#include<iostream>
#include<time.h>
using namespace std;
void Die::roll()
{
    number=1+rand()%6;
}
void Die::printLastValue()
{
    cout<<number<<endl;
}
void Die::Roll(int n)
{
    count=n;
    values=new int[count];
    for(int i=0;i<count;i++)
    {
        roll();
        values[i]=number;
    }
}
void Die::printValues()
{
    for(int i=0;i<count;i++)
    {
        cout<<values[i]<<endl;
    }
}

主.cpp

#include"die.h"
#include<iostream>
using namespace std;
int main()
{
    Die d;
    d.Roll(25);
    d.printValues();
    d.Roll(40);
    d.printValues();
    d.Roll(100);
    d.printValues();
    d.printLastValue();
}

是的，如果您多次调用Roll，这将导致内存泄漏。您应该检查值是否NULL，如果不是，则调用delete []。

编辑：
如下所述，您不必检查 null，您可以安全地在 null 指针上调用 delete。这只是我工作时公司标准中根深蒂固的习惯。

您应该考虑使用std::vector而不是数组。 通过这样做，您将消除内存泄漏的危险，并且不再需要显式定义析构函数。你可以用这个代替你是values：

std::vector<int> values;

然后在您的 Roll 代码中，您可以执行以下操作：

void Die::Roll(int n) {
    count=n;
    values.clear();
    for(int i=0;i<count;i++)
    {
        roll();
        values.push_back(number);
    }
}

您肯定需要删除它们，因为您正在重新分配 Die：：values，从而导致内存泄漏。

编辑：在这种情况下，最好使用std：：vector而不是原始数组。然后你不需要删除任何内容，只需在 Die：：Roll 的开头调用 std：：vector：：clear。

是的，它会泄漏内存。 当你这样做时

值 = 新整数 [len];

它使用数组分配新内存，并将值指向新的内存位置。旧内存位置仍包含旧数据，在分配新数据之前需要删除这些数据。

相关文章：