分段故障

Segmentation fault

本文关键字:故障 分段      更新时间:2023-10-16
#include <iostream>
using namespace std;
void initializeMap(int mapSizeX, int mapSizeY, int map[][10])
{
    // Map details:
    // 0 = # (wall)
    // 1 = space (free space)
    // 2 = x (player)
    for(int x = 0; x < mapSizeX; x++)
    {
        map[x][0] = 0;
    }
    for(int y = 0; y < (mapSizeY - 2); y++)
    {
        map[0][y] = 0;
        for(int x = 0; x < (mapSizeX - 2); x++)
        {
            map[x][y] = 1;
        }
        map[mapSizeX][y] = 0;
    }
    for(int x = 0; x < mapSizeX; x++)
    {
        map[x][mapSizeY - 1] = 0;
    }
}
void paintMap(int mapSizeX, int mapSizeY, int map[][10])
{
    for(int y = 0; y < mapSizeY; y++)
    {
        for(int x = 0; x < mapSizeX; x++)
        {   
            switch(map[x][y])
            {
                case 0:
                    cout << "#";
                    break;
                case 1:
                    cout << " ";
                    break;
                case 2:
                    cout << "x";
                    break;
            }
            cout << map[x][y];
        }
        cout << endl;
    }
}
int main()
{
    int mapSizeX = 10;
    int mapSizeY = 10;
    int map[10][10];
    initializeMap(mapSizeX, mapSizeY, map);
    paintMap(mapSizeX, mapSizeY, map);
    cout << endl << endl;
    return 0;
}

我的代码编译得很好,没有错误,但当我尝试运行它时,它只会说"分段错误"。我做了一些研究,我不明白为什么我会得到它,因为我根本不使用指针。我该如何解决这个问题?我使用g++编译它,只需键入即可运行它/主要在终端。

map[mapSizeX][y] = 0;

这是违法的。索引的有效值从0运行到mapSizeX - 1

线路应为:

map[mapSizeX][y] = 0;

有人假设这是想要的输出?

#0#0#0#0#0#0#0#0#0#0
#0 1 1 1 1 1 1 1 1#0
#0 1 1 1 1 1 1 1 1#0
#0 1 1 1 1 1 1 1 1#0
#0 1 1 1 1 1 1 1 1#0
#0 1 1 1 1 1 1 1 1#0
#0 1 1 1 1 1 1 1 1#0
#0 1 1 1 1 1 1 1 1#0
#0 1 1 1 1 1 1 1 1#0
#0#0#0#0#0#0#0#0#0#0

如果是这样,则在initializeMap函数中存在许多其他逐个关闭的错误。代替:

for(int y = 0; y < (mapSizeY - 2); y++)

for(int x = 0; x < (mapSizeX - 2); x++)

你应该使用

for(int y = 1; y < (mapSizeY - 1); y++)

for(int x = 1; x < (mapSizeX - 1); x++)

分别。


顺便说一句,这里有一种更干净的写initializeMap:的方法

template<int mapSizeX, int mapSizeY>
void initializeMap(int (&map)[mapSizeX][mapSizeY])
{
  for( int y = 0; y < mapSizeY; y++ ) {
    for( int x = 0; x < mapSizeX; x++ ) {
      if (x == 0 || x + 1 == mapSizeX || y == 0 || y == mapSizeY)
         map[x][y] = 0;
      else
         map[x][y] = 1;
    }
  }
}

你可以用来称呼它

initializeMap(map);

不需要传递大小,编译器会自动计算出来。

我没有深入了解这是否是您代码的唯一问题,但

map[mapSizeX][y] = 0;

将在数组的边界之上写入。

initializeMap-的循环的第二个

map[mapSizeX][y] = 0;

CCD_ 6是10,并且在矩阵中不存在CCD_。