c++ 类向量中的段错误

Question

我一直在玩用 C++ 制作游戏，只是为了练习编程。我创建了一个类，它本质上是一个 2d 单元格向量，用作地牢的不同图块。当我尝试调用我的函数将边缘单元格变成墙时，我得到一个段错误 11。根据我的研究，错误是试图访问超出向量范围的元素，或者我已经用完了堆内存但我不确定这里是否属于这种情况。

int main(){
    floor myfloor(18,9);
    myfloor.setwalls();
}

#ifndef FLOOR_H
#define FLOOR_H
#include "cell.h"
#include <vector>

using namespace std;
class floor{
public:
    floor(int xmax, int ymax);
     void setwalls();
private:
     int sizeX;
     int sizeY;
     vector< vector<cell*> > layout;
};
#endif

floor::floor(int xmax, int ymax){
    sizeX = xmax;
    sizeY = ymax;
    layout.resize(xmax, vector<cell*>(ymax));

    for(int i = 0; i < xmax; i++){
        for(int j = 0; j < ymax; j++){
            layout[i][j] = new cell();
        }
     }
}
void floor::setwalls(){
    for(int i = 0; i < sizeY; i++){
        layout[0][i]->setwall();
        layout[sizeX][i]->setwall();
    }
    for(int i = 0; i < sizeX; i++){
        layout[i][0]->setwall();
        layout[i][sizeY]->setwall();
 }
}

是什么导致了这里的 segfault 11？我所做的测试表明我的程序可以到达 setwalls() 但它似乎会在尝试访问布局向量的第一个元素时立即出现段错误。

Answer 1

你做错了一些事情，所以我给你写了一个关于如何使用多维向量的例子。

#include<vector>
   #include<iostream>

 using namespace std;

 int main()
 {
     vector < vector < int >>myarr;

     myarr.resize(100);         // resizes 1st dimension 
     for (auto i = 0; i < 100; i++)
    myarr[i].resize(100);   // resizes 2nd dimension

// initilization
for (int i = 0; i < 100; i++)
    {
    for (int j = 0; j < 100; j++)
        myarr[i][j] = j;    // 0- 99 100 times.
     }

     for (int i = 0; i < 100; i++)
     {
    for (int j = 0; j < 100; j++)
              cout << myarr[i][j] << " ";

        cout << endl;
     }
 }

您没有正确调整矢量的大小，并且没有正确初始化它以适应您的使用方式。此外，您的类型应该是 std::unique_ptr 并且您使用 std::make_unique 对其进行初始化，并且两者都存在于内存标头中。

你的代码应该是

 int main(){
     floor myfloor(18,9);
     myfloor.setwalls();
 }

 #ifndef FLOOR_H
#define FLOOR_H
#include "cell.h"
 #include <vector>
 #include<memory>

 using namespace std;
 class floor{
 public:
     floor(int xmax, int ymax);
      void setwalls();
 private:
      int sizeX;
      int sizeY;
      vector<vector<unique_ptr<cell*>>> layout;
 };
 #endif

 floor::floor(int xmax, int ymax){
     sizeX = xmax;
     sizeY = ymax;
     layout.resize(xmax);

     for(int i = 0; i < xmax; i++)
          layout[i].resize(ymax);

     for(int i = 0; i < xmax; i++){
         for(int j = 0; j < ymax; j++){
             layout[i][j] = make_unique(cell());
         }
      }
 }
 void floor::setwalls(){
     for(int i = 0; i < sizeY; i++){
         layout[0][i]->setwall();
         layout[sizeX - 1][i]->setwall();
     }
     for(int i = 0; i < sizeX; i++){
         layout[i][0]->setwall();
         layout[i][sizeY - 1]->setwall();
  }
 }

现在你的板子将是你想要的大小。

Answer 2

索引从 0 开始。因此，bounds 的有效值为 0

您需要使用 sizeX-1 而不是 sizeX。 sizeY 也一样。