0

私はそれらの継承クラスを持っています: 基本クラス: エンティティ

エンティティ クラスから派生: Actor、Obj、Enemy

基本クラスのエンティティには、「CollisionStuff」と呼ばれるユーザー定義型の obj が含まれています。プログラムを実行すると、CollisionStuff のデストラクタが、CollisionStuff コンストラクタが呼び出されるたびに呼び出され、ゲームループが実行されるたびに呼び出されます。

だから私の電話は：なぜこれが起こっているのですか？

以下に示すように、setRectangle メソッドでいくつかの配列を動的に割り当てます。プログラムはデストラクタを呼び出し、データを削除し、それらを使用しようとすると... "_ASSERTE(_BLOCK_TYPE_IS_VALID(pHead->nBlockUse)); を呼び出します。 "。

前にありがとう

ここに私のコード：Entity.h

enum e_Type {tActor = 0, tObj, tEnemy, tBackg};

class Entity
{
public:
    Entity(void);
    ~Entity(void);

    float getH();
    float getW();
    void setWH(float W, float H);
    bool CreateSprite(std::string path);
    sf::Sprite& getSprite();
    void setType(e_Type type);
    e_Type getType();
    CollisionStuff getColStuff();

    static std::list<Entity*> List;

protected:
    sf::Sprite m_sprite;
    sf::Texture m_texture;
    float m_h;
    float m_w;
    e_Type m_type;
    CollisionStuff m_colStuff;

    void addToList();
};

CollisionStuff.h

class CollisionStuff
{
public:
    CollisionStuff();
    ~CollisionStuff(void);

    void setRectangle(int  W, int H);
    void followTheSprite(Entity entity);

private:
    sf::Vector2f* m_a;
    sf::Vector2f* m_b;
    sf::Vector2f* m_c;
    sf::Vector2f* m_d;
    /* this member data are sides of rectangle used
    to manage collisions between object throughout the scenario 

            a
      -------------
      |           |
    c |           | d
      |           |
      -------------
            b
    */

};

CollisionStuff.cpp

CollisionStuff::CollisionStuff()
{
    //setRectangle(0, 0);
}

void CollisionStuff::setRectangle(int W, int H)
{
    m_a = new sf::Vector2f[W];
    m_b = new sf::Vector2f[W];
    m_c = new sf::Vector2f[H];
    m_d = new sf::Vector2f[H];
}

void CollisionStuff::followTheSprite(Entity entity)
{
    entity.getSprite().setOrigin(0, 0);
    sf::Vector2f UpLeftVertex = entity.getSprite().getPosition();


    for(int i = 0; i <  entity.getW(); i++)
    {
        m_a[i].x = UpLeftVertex.x + i;
        m_a[i].y = UpLeftVertex.y;

        m_b[i].x = UpLeftVertex.x + i;
        m_b[i].y = UpLeftVertex.y + entity.getH();
    }

    for(int i = 0; i <  entity.getH(); i++)
    {
        m_c[i].x = UpLeftVertex.x;
        m_c[i].y = UpLeftVertex.y + i;

        m_d[i].x = UpLeftVertex.x + entity.getW();
        m_d[i].y = UpLeftVertex.y + i;
    }

}



CollisionStuff::~CollisionStuff(void)
{
    delete [] m_a;
    delete [] m_b;
    delete [] m_c;
    delete [] m_d;
}

編集 回答ありがとうございます。CollisionStuff の使用例

Actor.cpp (エンティティの派生クラスです)

Actor::Actor(void)
{
    if(!CreateSprite("D://Sprites//MainChar.png"))
    {
        std::cout << "Impossibile creare sprite" << std::endl;
    }
    else
    {
        std::cout << "Creazione sprite riuscita" << std::endl;  
        m_sprite.setPosition(100.0f, 365.0f);
        m_sprite.setOrigin(20, 35);
        //m_sprite.setPosition(190.0f, 382.5f); // 200, 400
        setWH(40, 70);
        m_health = 100;
        m_status = Good;
        setType(tActor);
        m_jCounter = -1;
        m_action = Null;

        setColStuff();
    }
}

void Actor::setColStuff()
{
    m_colStuff.setRectangle(m_w, m_h);
}



void Actor::physic()
{
    //setColStuff();
    m_colStuff.followTheSprite(*this);
}

main.cpp

int main()
{
    sf::RenderWindow window(sf::VideoMode(800, 600), "Platform");

    std::list<Entity*>::iterator i;

    Background BG;
    Level1 FirstLev;
    Actor Doodle; 

    while(window.isOpen())
    {
        sf::Event event;
        if(window.pollEvent(event))
        {
            if (event.type == sf::Event::Closed)
                window.close();

            Doodle.inputEvts();
        }

        Doodle.act(Doodle.getAction());

        Doodle.physic();

        window.clear();

        window.draw(BG.getSprite());
        window.draw(Doodle.getSprite());

        FirstLev.drawLevel(window);

        window.display();
    }
    return 0;
}
4

1 に答える 1

2

あなたが投稿したコードの一部から判断するのは本当に難しいですが、推測する必要がある場合は、おそらくこれに関連していると思います:

CollisionStuff getColStuff();

つまり、これを呼び出しているCollisionStuff人によって新しいコピーが作成されます。元のオブジェクトが割り当てたのと同じポインターを持ち、CollisionStuffスコープ外になるとそれらを削除し、元のオブジェクトにぶら下がりポインターを残します。

参照またはポインターによって返してみることができますが、どちらの方法でも、コピー コンストラクターを作成し、CollisionStuff( Rule of Three ) の代入演算子をオーバーライドする必要があります。

別のアイデアは、配列を自分std::vector<sf::Vector2f>で割り当てる代わりに使用することです。sf::Vector2f

于 2013-08-24T15:01:19.710 に答える