この状況でメモリリークが発生しましたか？

Question

新しいキーワードを繰り返し使用すると、私はメモリリークに非常に注意しています。次の例では、メモリリークがありますか？私の腸の本能ははいと言います。

class Handler  // Class definition 
{public: 
    ~Handler(); 
    int* ptrToInts;  
}; 

Handler::~Handler() // Class destructor 
{ 
    delete[] ptrToInts; 
} 

Handler handler;  // Global object 


void aFunction() 
{ 
    handler.ptrToInts = new int[20]; 
} 


int main() 
{ 
    bool quit = false; 

    while(!quit) 
    { 
     aFunction(); 
    } 

    return 0; 
} 

ptrToIntsは、heapeach時間に別々のメモリに20個の別々の新しいintを作成するでしょうか？

また、デストラクタ用でない場合、動的に割り当てられたメモリを解放するかどうかが問題です。クラスの生存期間はプログラムの継続時間であるかのように見て、すべての「新しい」メモリーのクリーンアップを行いますか？

編集：ありがとうございました。私がこれを求めている理由は、WndProcが基本的にRaw Inputのために呼び出されるたびに、新しい呼び出しと削除を呼び出そうとしているからです。これはMSDNがあなたに指示する方法です。非常に非効率だと思われる。

Answer 1

、あなたはメモリを作成リーク。これは、ポインタが呼び出されるたびにポインタを再割り当てするため、aFunction()をループすると発生します。

2番目の質問では、デストラクタはポインタに割り当てられた最後の配列delete[]だけです。

Answer 2

delete[]は、newによって割り当てられたメモリを解放します。 newを使用するたびに、には、deleteが必要です。 Documentationに基づいて他の質問については、

：とすぐに、ヒープ上の割り当て解除その割り当てられたメモリにdelete[]を使用せずにポインタを再割り当てとして

MyClass * p1 = new MyClass[5]; // allocates and constructs five objects

Answer 3

はい、呼び出すたびにhandler.ptrToIntsを明示的に割り当て解除せずに、関数を複数回呼び出すとメモリリークが発生します。

void aFunction() 
{ 
    handler.ptrToInts = new int[20]; 
} 

//-----somewhere we see the caller 

while(!quit) 
    { 
     aFunction(); 
    } 

しかし、これはリークを検出する些細なケースです...あなたはLeak detectorsとstatic analyzersを使用することを学ぶ必要があります。

は、メモリリークがあるもちろんHow do you detect/avoid Memory leaks in your (Unmanaged) code?

Answer 4

を参照してください。あなたはどうなる

void aFunction() 
{ 
    delete [] handler.ptrToInts; 
    handler.ptrToInts = new int[20]; 
} 

のように、最初の古いint型の割り当てを解除せずに

void aFunction() 
{ 
    handler.ptrToInts = new int[20]; 
} 

にint型を割り当てます。

aFunction()を呼び出すと、「無限の」メモリ割り当てが行われます。 最後に割り当てられたintだけを解放するデストラクタも呼び出されません。

あなたのハンドラがそれ自身のメモリを管理しないのはなぜですか？

オブジェクトの外部にメモリを割り当てて、その内部でメモリを解放することは非常に悪いことです。あなたは `回以上new`と` [] `一度だけ[はい、あなたがメモリリークを持って削除しない場合は

class Handler 
{ 
public: 
    Handler(); 
    ~Handler(); 
    void aMethod(); 
private: 
    int* ptrToInts;  
}; 

Handler::Handler() { 
    handler.ptrToInts = new int[20]; 
} 

Handler::~Handler() { 
    delete[] ptrToInts; 
} 

void Handler::aMethod() { 
    delete[] ptrToInts; 
    handler.ptrToInts = new int[20]; 
} 

int main() { 
    bool quit = false; 
    Handler handler; 

    while(!quit) { 
    handler.aMethod(); 
    } 
}