C++のベクタの問題

Question

私のアプリケーションは何らかの理由でクラッシュし続けます。私はベクタにとって新しいので、私のところには何か愚かなものがあります。

#include <iostream> 
#include <vector>  

using namespace std; 

class Projectile 
{ 
private: 
    SDL_Surface *projectile; 
    void load(); 

public: 
    int count; 

    vector<int> c; 
    vector< vector<int> > p; 

    int positionX; 
    int positionY; 

    Projectile(); 
    void newProjectile(int, int); 
    void drawCurrentState(SDL_Surface*); 
}; 

... 
... 

void Projectile::newProjectile(int x, int y) 
{ 
    positionX = x; 
    positionY = y; 

    c.push_back(10); 
    c.push_back(10); 

    //p.push_back(c); //trying to start off simple before i do multidimensional. 
} 

void Projectile::drawCurrentState(SDL_Surface* destination) 
{ 
    SDL_Rect offset; 
    offset.x = c[0]; //will eventually me the multidimensional p vector 
    offset.y = c[1]; // 

    SDL_BlitSurface(projectile, NULL, destination, &offset); 
} 

私はここで間違っていますか？私は2つの値を戻しますが（テストを終えた後xとyのintになります）、スクリプトのoffset.x = c[0];部分に到達するとクラッシュするようです。

確かに、c [0]とc [1]の両方が10になるはずです。

Answer 1

そうだね、両方とも[0]と[1]最初に）ちょうどあなたが本当にnewProjectileを（呼び出して確認してください10.

こと、および無効な宛先ポインタなど他の問題がないことを確認しなければならないとそうです。

Answer 2

多分、ベクトルcは、c [0]が読み取られる行で空です。 c.size（）の結果を確認してください。

Answer 3

コードは大丈夫ですが、呼び出しの順序は重要です。 drawCurrentState（）を呼び出す前にnewProjectile（）を呼び出すようにしてください。たとえば、次のようになります。

Projectile * projectile = new Projectile(); 
projectile->newProjectile(1,1); 
projectile->drawCurrentState(...);