1

我有一个Block调用SDL_FreeSurface(surface)析构函数的类。在main()我创建 Block 的实例时，该对象正常运行，但是当我在另一个Control具有vector<Block> block_vector作为数据成员的类中使用它时，当我将实例添加Block到block_vector. 我缩小了Block调用时的析构函数的问题SDL_FreeSurface(surface)。向向量中添加对象与 is 有什么关系吗？问题是什么？

 class Block{

public:

    Block(int x, int y);
   ~Block();


    void Load_Image(MediaFunctions &M_Functions);

    void SetPosition(int x, int y);

    void BlitBlock(SDL_Event &event, MediaFunctions &M_Functions, SDL_Surface *destination);


    bool DetectionNames(SDL_Event &event, MediaFunctions &M_Functions, SDL_Surface *destination);

    bool DetectionHours(SDL_Event &event, MediaFunctions &M_Functions, SDL_Surface *destination);

    bool return_error();



private:


    SDL_Surface *block_surface_names;
    SDL_Surface *block_surface_hours;

    SDL_Surface *block_names_detected;
    SDL_Surface *block_hours_detected;

    SDL_Rect     block_rect_names;
    SDL_Rect     block_rect_hours;


    bool error;

    };


    //the problem
    Block::~Block(){


    SDL_FreeSurface(block_surface_hours);
    SDL_FreeSurface(block_surface_names);

    SDL_FreeSurface(block_hours_detected);
    SDL_FreeSurface(block_names_detected);

    }



    //when doing this 
    void Control::HandleEvents(SDL_Event &event, MediaFunctions &M_Functions){

if(event.type == SDL_KEYDOWN){

    if( event.key.keysym.sym == SDLK_a ){

            //append a block instance
            BlockVector.push_back(Block (Block(100,100)));
            BlockVector.at(block_index).Load_Image(M_Functions);

            block_index++;

    }

    }

    }
4

1 回答 1

3

当您将一个 Block 对象插入向量中时，会生成一个副本。由于您没有提供复制构造函数，因此使用了编译器生成的复制构造函数。编译器生成的复制构造函数将简单地复制您的指针。它不会复制实际的 SDL 表面。

这意味着您的原始 Block 对象和插入向量中的 Block 对象都具有指向相同 SDL 表面的指针。当其中一个 Block 对象的生命周期结束时，将调用析构函数，从而释放 SDL 表面。现在另一个 Block 对象有一个悬空指针，当它的生命周期结束并调用析构函数时，您在悬空指针上调用 SDL_FreeSurface。

这几乎肯定是导致您崩溃的原因。这就是为什么您需要遵循三法则，并正确实现复制构造函数和赋值运算符。实施取决于您。在大多数情况下，我建议对您的对象进行完整的深拷贝。但是对于像 SDL 表面这样的重量级对象，我建议您考虑实现某种引用计数机制，或者std::shared_ptr与调用SDL_FreeSurface而不是调用delete.

于 2012-10-09T05:19:50.950 回答