2

Die以下のコードを考えると、次のようなクラスのインスタンスを使用するとどうなりますか:

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

メモリを再度割り当てる前に、値によって占有されているメモリを解放する必要がありますか? delete[ ]前にnew？

ダイ.h

#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;
    }
}

main.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();
}
4

3 に答える 3

5

はい、Roll複数回呼び出すとメモリ リークが発生します。値がそうであるかどうかを確認し、そうでない場合はNULL呼び出すdelete []必要があります。

編集:
以下にコメントされているように、null をチェックする必要はありません。null ポインターで delete を安全に呼び出すことができます。それは、私がかつて働いていた会社の基準からの長い間染み込んだ習慣です.

std::vector配列の代わりにa を使用することを検討する必要があります。これにより、メモリ リークの危険がなくなり、デストラクタを明示的に定義する必要がなくなります。を次のように置き換えることができます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);
    }
}
于 2012-11-21T20:21:22.467 に答える
3

Die::values を再割り当てしているため、メモリリークが発生するため、それらを削除する必要があります。

編集: この場合、生の配列よりも std::vector を使用する方が適切です。その後、何も削除する必要はなく、Die::Roll の先頭で std::vector::clear を呼び出すだけです。

于 2012-11-21T20:21:32.120 に答える
1

はい、メモリリークします。あなたがするとき

値 = 新しい int [len];

配列を使用して新しいメモリを割り当て、値を新しいメモリ位置にポイントします。古いメモリ位置には、新しいデータを割り当てる前に削除する必要がある古いデータがまだ含まれています。

于 2012-11-21T20:23:52.777 に答える