DirectX COMオブジェクトでこのような状況でメモリリークが発生しましたか？

Question

私はこのようなクラスまたは構造体を持っている場合：

この関数何か中の任意の時点で失敗し、それが最後でクリーンアップを行いません、falseを返す場合

bool localScopeFunc() 
 
{ 
 
    bool result; 
 
    IDXGIFactory* pFactory; 
 
    IDXGIAdapter* pAdapter; 
 
    
 
    result = //Do something here with the objects 
 
\t if (!result) return false; 
 
    
 
\t result = //Do something here with the objects 
 
\t if (!result) return false; 
 
    
 
\t result = //Do something here with the objects 
 
\t if (!result) return false; 
 
    // And so on...... 
 
    
 
    
 
    //___Do cleanup here___// 
 
    pFactory->Release(); pFactory = nullptr; 
 
    pAdapter->Release(); pAdapter = nullptr; 
 
    
 
    return true; // If all passes 
 
}

、したがって、任意のオブジェクトに対して - > Release（）を呼び出すことはありません。これはメモリリークを意味しますか？

もしそうなら、実現可能な方法を見つけることができません。時々私は関数呼び出しのリストを持っていて、新しい段階を初期化する各段階で、逆にすべてをクリーンアップしなければならない

int main() 
 
{ 
 
if (!initTime()) {return -1;} 
 
if (!initD3D())  {shutDownTime(); return -2;} 
 
if (!initCamera()) {shutDownD3D(); shutDownTime(); return -3;} 
 
if (!initSound()) {shutDownCamera(); shutDownD3D(); shutDownTime(); return -3;} 
 
if (!initPhysics()) {shutDownSound(); shutDownD3D(); shutDownTime(); return -4;} 
 
// And so on. 
 
    
 
return 0; 
 
}

Answer 1

あなたはクリーンアップをスキップしているので、はい、それはリークが発生します：次のようになります。 COMオブジェクトは参照カウントを使用し、適切な時間にメモリを削除するためには、カウントが正確でなければなりません。

解決策は実際には非常に簡単です：Microsoft::WRL::ComPtrを使用してください。このスマートポインタは、スコープから離れる場合でも必要に応じてReleaseを呼び出す処理を行います。

その他の注意点は、COMオブジェクトはboolとしてエラーを返さないことです。彼らはHRESULTです。関数がHRESULTを返すと失敗する可能性があるので、無視しないでください。また、== S_OKなどを使うべきではありません。 FAILEDマクロ、SUCCEEDEDマクロ、またはDX::ThrowIfFailedなどのマクロを使用する必要があります。

#include <wrl/client.h> 

using Microsoft::WRL::ComPtr; 

bool localScopeFunc() 
{ 
    ComPtr<IDXGIFactory> pFactory; 
    ComPtr<IDXGIAdapter> pAdapter; 

    HRESULT result = //Do something here with the objects 
    if (FAILED(result)) return false; 

    result = //Do something here with the objects 
    if (FAILED(result)) return false; 

    result = //Do something here with the objects 
    if (FAILED(result)) return false; 

    // And so on...... 

    return true; // If all passes 
} 

ComPtrの詳細については、thisを参照してください。