C++ SDL2 程序在切换循环时崩溃

Question

我在“学习”SDL2 时遇到了一些麻烦。 每当我按下开关循环中的一个键时，程序就会因“errormodulename”StackHash_0a9e(无论是什么)而崩溃。 这是方法:

 void InputMan::acceptInput(SDL_Event * e,Graphics * g){
std::cout<<"handling input"<<std::endl;
switch(e->key.keysym.sym){
    case SDLK_UP:
    {
    Sprite * s=g->spriteByName("Filip");
    if(s->getRow()==2){
        s->action();
    }
    else{
        s->setRow(2);
        s->rollBack();
    }
    std::cout<<"Key up"<<std::endl;
    break;
    }
    case SDLK_DOWN:
    {
        Sprite * s=g->spriteByName("Filip");
        if(s->getRow()==0){
            s->action();
        }
        else{
            s->setRow(0);
            s->rollBack();
        }
        std::cout<<"Key down"<<std::endl;
        break;
    }
    case SDLK_LEFT:
    {
        Sprite * s=g->spriteByName("Filip");
        if(s->getRow()==1){
            s->action();
        }
        else{
            s->setRow(1);
            s->rollBack();
        }
        std::cout<<"Key left"<<std::endl;
        break;
    }
    case SDLK_RIGHT:
    {
        Sprite * s=g->spriteByName("Filip");
        if(s->getRow()==4){
            s->action();
        }
        else{
            s->setRow(4);
            s->rollBack();
        }
        std::cout<<"Key right"<<std::endl;
        break;
    }
    default:
     break;
    }
 }

我认为原因可能是 case 中有一些东西，这在 c++ 中是被禁止的，我只是不知道，但是通过输出调试方法我发现崩溃是在 break 之后立即发生的. 也许问题不在于我和 SDL，而在于我和 C++。

编辑

调试器说:

 Program received signal SIGSEGV, Segmentation fault.
 0x00000190 in ?? ()

(如果我按向右箭头按钮。在左侧，它是 0x00000064，向上 0x000000c8，向下 0x00000000) 显然我有一个指向无效区域的引用。

编辑 2:

@Jarod 给了我提示，也许 Sprite (Sprite * s=g->spriteByName("Filip");) 是一个 nullptr 但它似乎不是. 我添加了一个 if 语句:

 Sprite * s=g->spriteByName("Filip");
    if(s==NULL){
        starter.die("Sprite \"Filip\" not found");//<-closes the program with an error message)
    } 

但是没有触发语句。

信息

我不知道我是否已经说过，但崩溃发生在该方法完成之后和下一个方法之前。这就是原因，为什么我不在这里发布主要类(class)，但我认为我应该...

 void Starter::gameLoop(){
 std::cout<<"Entering Gameloop"<<std::endl;
 while(!quit){
    SDL_PollEvent(ev);
    if(ev->type==SDL_QUIT){
        quit=true;
        std::cout<<"successfull end"<<std::endl;
    }
    else if(ev->type==SDL_KEYDOWN){
        input->acceptInput(ev,graphics);//<-- the method I posted already
        std::cout<<"Paint"<<std::endl;//<-- the program crashes before that happens
    }
    else if(ev->type==SDL_KEYUP){
        graphics->spriteByName("Filip")->rollBack();
    }
    graphics->paint();
 }
}

我希望你能帮我解决这个问题。

谢谢你。

Answer 1

不要将 ev 声明为指针。并在gameLoop 方法中声明SDL_Event e，然后通过值或引用将其传递给另一个函数，在教程和书籍中总是这样做:

void InputMan::acceptInput(SDL_Event e,Graphics * g){...}

或

void InputMan::acceptInput(SDL_Event &e,Graphics * g){...}

并像这样在你的游戏循环中使用它:

void Starter::gameLoop()
{
    std::cout<<"Entering Gameloop"<<std::endl;

    while(!quit)
    {

    SDL_event e;
    SDL_PollEvent(&e);

    switch(e.type)
    {
     case SDL_QUIT:
       quit=true;
       std::cout<<"successfull end"<<std::endl;
       break;

     case SDL_KEYDOWN:
       input->acceptInput(e,graphics);
       std::cout<<"Paint"<<std::endl;
       break;

     /*another cases*/
    }
    }
}

但问题可能出在您的图形类实例中。它可能没有正确初始化。总是像 troyane 说的那样在 NULL 上检查它们，并尝试一直调试到崩溃的地方，它可能会给你一些答案。我强烈建议您使用 C++ 11 标准提供的智能指针，例如 shared_ptr，而不是原始指针。