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;
    }
 }

我认为原因可能是，有些东西在c ++中是禁止的，我只是不知道，但是在输出调试方法中我发现崩溃是在中断之后立即发生的。也许问题不是我和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)
    }

但声明并未触发。

INFO

我不知道我是否已经说过但是崩溃是在方法完成之后和下一次之前发生的。这就是共鸣，为什么我不在这里发布主要课程，但我想我应该...

 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();
 }
}

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

t h a n k y o 。

Answer 1

不要将ev声明为指针。并在SDL_Event e方法中声明gameLoop，然后通过值或引用将其传递给其他函数，它总是在教程和书籍中完成：

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所说的那样检查它们，并尝试一直调试到崩溃的地方，它可能会给你一些答案。我强烈建议您使用C ++ 11标准提供的智能指针，如shared_ptr，而不是原始指针。

Answer 2

小心使用指针。你的函数的参数是SDL_Event * e, Graphics * g，所以在访问指针之前检查它是否不是NULL。因为，每次尝试取消引用NULL指针都会导致崩溃。

C ++ SDL2程序在交换机环路上崩溃

2 个答案: