NULLをチェックしようとしたときにEXC_BAD_ACCESS

Question

私はLazyFooの素晴らしいチュートリアル（http://lazyfoo.net/tutorials/SDL/10_color_keying/index.php）に基づいて単純なカラーキーイングエンジンの例を実装しようとしていますが、実行しようとするとスレッドはEXC_BAD_ACCESS 1、address = 0x0）がポインタにヌルかどうかをテストしようとしているにもかかわらず、

//Texture wrapper class 
class LTexture 
{ 
public: 
    //Initializes variables 
    LTexture(); 
    //Deallocates memory 
    ~LTexture(); 
    //Loads image at specified path 
    bool loadFromFile(std::string path); 
    //Deallocates texture 
    void free(); 
    //Renders a texture at a given point 
    void render(int x, int y); 
    //Gets an image's dimensions 
    int getWidth(); 
    int getHeight(); 
private: 
    //The actual hardware texture 
    SDL_Texture* mTexture = NULL; 
    //Image dimensions 
    int mWidth; 
    int mHeight; 
}; 

、ここでintializeだとメソッドを破壊する：ここでは、クラスは次のようになります

LTexture::LTexture() 
{ 
    //Initialize 
    mTexture = NULL; 
    mWidth = 0; 
    mHeight = 0; 
    printf("I initialized"); 
} 

LTexture::~LTexture() 
{ 
    free(); 
} 

そして、私のエラーがLTexture::free方法にあります。

void LTexture::free() 
{ 
    //Free texture if it exists 
    if (mTexture != NULL) //HERE IS THE ISSUE. WHAT IS WRONG WITH THIS? 
    { 
     SDL_DestroyTexture(mTexture); 
     mTexture = NULL; 
     mWidth = 0; 
     mHeight = 0; 
    } 
} 

あなたはインラインで見ることができるように、mTextureがNULLであれば、私がテストしたときの問題は、私が有効であるべきだと考えており、表示されますが、何らかの理由でそうではありません。私は間違って何をしていますか？コードの詳細を投稿するのに役立つでしょうか？

Answer 1

おそらく、正しくコピーして移動することができないという問題があります。

LTextureをコピーするたびに、ポインタだけがコピーされます。コピーが範囲外になると、デストラクタが呼び出されます。次に、ある点で元のスコープが範囲外になり、デストラクターが同じポインター上で再度呼び出され、二重解放が行われます。

私は、スマートポインタを使用することをお勧めします：

#include <memory> 
class TextureDeleter { void operator()(SDL_Texture* t) { SDL_DestroyTexture(t); }; 

// in the class 
std::unique_ptr<SDL_Texture, TextureDeleter> mTexture; 

あなたは、あなたのデストラクタを削除することができます。

編集：あなた本当にが<memory>を使用しない場合、その後、あなたは自分のクラスに

LTexture(const LTexture &) = delete; 
LTexture& operator=(const LTexture &) = delete; 
LTexture(LTexture &&) = delete; 
LTexture& operator=(LTexture &&) = delete; 

を追加することにより、安全性の同じ種類を達成することができます。

しかし、コメントに指摘されているように、これは実際にクラスを移動またはコピーする必要がない場合にのみ機能します。あなたがそうするなら、あなた自身を作ることは自明ではないshared_ptrを使用しなければなりません。