Как я могу правильно освободить ID3D11ShaderResourceView **, чтобы избежать утечек памяти?

Question

У меня это как частный член класса.

ID3D11ShaderResourceView** texture_pool; 

Я поставил его в NULL в конструкторе класса, как это:

texture_pool = NULL; 

Тогда я инициализировать его, как это в инициализации класса:

texture_pool = new ID3D11ShaderResourceView*[texture_count]; 
for (int n = 0; n < texture_count; n++) texture_pool[n] = NULL; 

А в классе деструктор I отпустите его следующим образом:

for (int n = 0; n < texture_count; n++) SAFE_RELEASE(texture_pool[n]); 
SAFE_DELETE_ARRAY(texture_pool); 

Но иногда моя программа падает, когда я выйти из нее и точки debbuger на этой линии в качестве причины аварии:

for (int n = 0; n < texture_count; n++) SAFE_RELEASE(texture_pool[n]); 

Если удалить эту строку все работает гладко, но я беспокоюсь о возможных утечках памяти, если Я не выпускаю массив.

Итак, могу ли я просто удалить эту линию, и все будет очищено porperly?

Эти являются определенные линии для выпуска и удаления инструкции:

#define SAFE_RELEASE(p) { if ((p)) { (p)->Release(); (p) = 0; } } 
#define SAFE_DELETE(a) if((a) != NULL) delete (a); (a) = NULL; 
#define SAFE_DELETE_ARRAY(a) if((a) != NULL) delete[] (a); (a) = NULL; 

Answer 1

Вы показали это:

#define SAFE_RELEASE(p) { if ((p)) { (p)->Release(); (p) = 0; } } 
#define SAFE_DELETE(a) if((a) != NULL) delete (a); (a) = NULL; 
#define SAFE_DELETE_ARRAY(a) if((a) != NULL) delete[] (a); (a) = NULL; 

Попробуйте вместо этого:

#ifndef SAFE_RELEASE 
    #define SAFE_RELEASE(p) { if(p) { (p)->Release(); (p) = NULL; } } 
#endif 

#ifndef SAFE_DELETE 
    #define SAFE_DELETE(p) { if(p) { delete (p); (p) = nullptr; } } 
#endif 

#ifndef SAFE_DELETE_ARRAY 
    #define SAFE_DELETE_ARRAY(p) { if(p) { delete[](p); (a) = nullptr; } } 
#endif 

При работе с макросами вам нужно быть осторожным с использованием «()« & «{}".

Вам также не нужно (p)! = Проверка nullptr, поскольку, если у вас есть (p), он будет выполнять только следующую инструкцию, если эта проверка возвращает значение true, указывающее, что указатель действителен, а не null.

Вам также не хватает #ifndef и #endif вокруг определений макросов.

Если ваш компилятор не поддерживает nullptr, тогда вы можете использовать NULL, но nullptr намного чище и удобочитаемо.