数组元素消失

Question

我正在尝试用 x 标记他们的位置来跟踪玩家的位置。当玩家输入一个字符串时，我相应地增加坐标。但是，当玩家位于距离边界一格的位置，然后尝试移动到地图边缘时，玩家就会消失。

例子：

.....  
...x.  
.....  
.....  
.....

玩家位于'x'

如果玩家输入字符串“right”并且我移动player_loc，则数组简单地返回：

.....  
.....  
.....  
.....  
.....

我试图通过增加数组的大小来添加一种缓冲区。没有运气。我已经坚持了将近一个星期了。任何帮助，将不胜感激。我为混乱的代码道歉。我在这方面完全是新手，我真的只是在黑暗中摸索所有这些东西。我在这里的论坛上对此进行了研究，但没有找到解决方案。如果您知道我可能（可能）错过的某些事情，请随时指出我的方向。

#include <stdio.h>
#include <string.h>
char map[6][6];
char player_loc = 'x';
int row;
int col;

void init_map()
{
    for (int i = 0; i < 5; i++) {
        for (int j = 0; j < 5; j++) {

        map[i][j] = '.';

        }
    }
}

void print_map()
{
        for (int i = 0; i < 5; i++) {
            for (int j = 0; j < 5; j++) {

                printf("%c", map[i][j]);

            }
        printf("\n");
        }
}

int get_player_loc()
{
    for (int j = 0; j < 5; j++) {
        for (int k = 0; k < 5; k++) {
            if(map[j][k] == player_loc)
            {
                row = k;
                col = j;
            }
        }
    }
    return row;
    return col;
}

void init_player_loc()
{
    int check = 1;
    for (int g = 0; g < 5; g++) {
        for (int h = 0; h < 5; h++) {
            if (map[g][h] == 'x') {
                check = 0;
            }

        }   
    }
    if(check == 1) {
        map[0][0] = player_loc;
    } else {
        get_player_loc();
    }

}

void move_left()
{
    for (int i = 0; i < 5; i++) {
        for (int j = 0; j < 5; j++) {
            if (map[i][j] == player_loc) {
                map[i][j-1] = player_loc;
                map[i][j] = '.';
            }
        }
    }

}

void move_right()
{
    for (int i = 0; i < 5; i++) {
        for (int j = 0; j < 5; j++) {
            if (map[i][j] == player_loc) {
                map[i][j+1] = player_loc;
                map[i][j] = '.';
            }
        }
    }
}




int main(int argc, char* argv[])
{
    char input[15];
    printf("You enter a room...you can go left, right, or straight. Which way do you go?\n");

    int done = 0;

    init_map();
    map[3][3] = player_loc;
    //init_player_loc();
    print_map();

    while (!done)  {
        scanf("%s", input);

            if (strcmp("left", input) == 0) {
                move_left();
                printf("You go left...\n");
                print_map();
                get_player_loc();
                printf("%d %d\n", row, col);
                done = 1;
            } 
            else if (strcmp("right", input) == 0) {
                move_right();
                printf("You go right...\n");
                print_map();
                get_player_loc();
                printf("%d %d\n", row, col);
                done = 1;
            }
            else if (strcmp("straight", input) == 0) {
                printf("You go straight...");
                done = 1;
            }
            else {
                printf("Sorry, can't do that.\n");
            }
        }
}

Answer 1

如果找到播放器位置，则必须中断循环，例如

void move_right()
{
    for (int i = 0; i < 5; i++) {
        for (int j = 0; j < 5; j++) {
            if (map[i][j] == player_loc) {
                map[i][j+1] = player_loc;
                map[i][j] = '.';
                return;
            }
        }
    }
}

在您的代码中，您将玩家向右移动，下一个循环将在新位置找到玩家并再次向右移动，直到永远。

此外，在您的代码中，您没有处理二维矩阵的边界：j+1 仅在 j<5 时有效。 那么更好的代码应该是

void move_right()
{
    for (int i = 0; i < 4; i++) {
        for (int j = 0; j < 4; j++) {
            if (map[i][j] == player_loc) {
                map[i][j+1] = player_loc;
                map[i][j] = '.';
                return;
            }
        }
    }
}

Answer 2

问题在于您的move_right 函数会拾取玩家并将他们完全移出地图。假设您的播放器位于[0, 2] 并单步执行代码。

for (int j = 0; j < 5; j++) {
    if (map[i][j] == player_loc) {
        map[i][j+1] = player_loc;
        map[i][j] = '.';
    }
}

• [0, 0]这里没有玩家，继续前进
• [0, 1]这里没有玩家，继续前进
• [0, 2]找到玩家了！将它们右移至[0, 3]
• [0, 3]找到玩家了！将它们右移至[0, 4]
• [0, 4]找到玩家了！将它们右移至[0, 5]

在 5 点，循环结束。由于您添加了缓冲区，您的数组是 6x6，因此玩家被隐藏在机翼中而不会导致程序崩溃。您应该做几件事：

• 找到并移动播放器后，break 或 return 这样它们只会移动一次。
• 将您的阵列设为 5x5（或全部打印 6x6），以便您可以查看所有内容。
• 做一些边界检查，这样玩家就不能从j = 5向右移动。
• 注意move_up 中的相同错误，当您增加i 时会发生这种错误。

Answer 3

您的循环允许检查位置两次，一次是在 i,j，另一次是在 i,(j+1)（或其他一些变体）。这可能不是你想要的。找到播放器后，您应该进行更新，然后跳出循环。

此外，理论上，代码原样允许通过数组边界进行索引。也不是想要的。您可以考虑边界检查。我不知道当玩家向右移动并且右侧有一堵墙时会发生什么。他不动吗？环绕？ LR 拐角可能会导致 seg 错误，就像现在一样。

Answer 4

您似乎在get_player_loc 函数中转置了行和列索引，并且有两个返回语句（编译器应该警告您有关无法访问的代码），调用代码都不需要或使用这两个语句。

一开始，初始化row 和col 变量。 （取自您的 main 的值。）

int row = 3;
int col = 3;

更改get_player_loc 函数，使其仅更新全局变量row 和col。如果没有找到播放器，它会将row 和col 设置为0，按照原始设置。

void get_player_loc(void)
{
    for (int j = 0; j < 5; j++) {
        for (int k = 0; k < 5; k++) {
            if(map[j][k] == player_loc)
            {
                // The meaning of row and col is set by how they are used
                // to index the array in the move and print functions.  Keep
                // the same order and meaning here.
                row = j;
                col = k;
                return;
            }
        }
    }
    // Set row and col to 0 if the location is not found.
    row = 0;
    col = 0;
    map[0][0] = player_loc;
}

当它们到达边缘时你仍然会遇到问题，因为数组的索引超出了移动函数的范围，但这是一个不同的问题。