C中的简单游戏使用getch - segfault问题?

时间:2010-11-05 06:14:26

标签: c arrays segmentation-fault

以下代码存在段错误问题(也发布了at pastebin):

#include <stdio.h>
#include <string.h>
#include <stdbool.h>
#include "mygetch.h"

#define MAX_SIZE 255
#define SCR_CLEAR "\033[2J\033[;H"

void getgrid(int, int);
void resetgrid(void);
void getkey(void);


static bool grid[5][5] = {{0,0,0,0,0},
                           {0,0,0,0,0},
                           {0,0,0,0,0},
                           {0,0,0,0,0},
                           {0,0,0,0,0}};


int main() {

    while(1) {
        getkey();
    }


    return 0;
}

void getgrid(int xpos, int ypos) {
    int x = 0;
    int y = 0;

    grid[xpos][ypos] = 1;

    for(x = 0; x <= 4; x++) {
        for(y = 0; y <= 4; y++) {
            printf("%i ", grid[x][y]);
        }
        printf("\n");
    }
}


void resetgrid() {
    int x = 0;
    int y = 0;

    for(x = 0; x <= 4; x++) {
        for(y = 0; y <= 4; y++) {
            grid[x][y] = 0;
        }
    }
}

void getkey() {
    static int xpos = 0;
    static int ypos = 0;
    int c = mygetch();

    //0x41 = up.. apparently on my linux console?
    //0x42 = down
    //0x44 = left
    //0x43 = right

    if(c == 0x41 && ypos != 0) {
        ypos--;
    } else if(c == 0x42 && ypos != 4) {
        ypos++;
    } else if(c == 0x44 && xpos != 4) {
        xpos--;
    } else if(c == 0x43 && xpos != 0) {
        xpos++;
    }

    resetgrid();
    printf(SCR_CLEAR);
    getgrid(xpos, ypos);
}

你可以假设mygetch()返回一个ASCII整数代码点,在我的linux控制台上,左下方是A / B / C / D,所以我将它们映射为。

我的问题出于某种原因,即使我正确地定义了多维数组,当我向上/向下/向左/向右/向右按下它们不起作用时,它们会离开屏幕并导致段错误,现在我知道了键是正确映射的,所以我不知道为什么y-- y++等不会正确起作用,除非我在定义数组或其他地方时出错。

我肯定会从解决这个问题和做更多事情中学到很多东西,但这只是我希望做的事情。

1 个答案:

答案 0 :(得分:3)

left right 的边界条件被反转。你应该:

if (c == 0x41 && ypos != 0) {
    ypos--;
} else if(c == 0x42 && ypos != 4) {
    ypos++;
} else if(c == 0x44 && xpos != 0) {
    xpos--;
} else if(c == 0x43 && xpos != 4) {
    xpos++;
}

而不是:

if (c == 0x41 && ypos != 0) {
    ypos--;
} else if(c == 0x42 && ypos != 4) {
    ypos++;
} else if(c == 0x44 && xpos != 4) {
    xpos--;
} else if(c == 0x43 && xpos != 0) {
    xpos++;
}