我正在尝试创建一个迷宫类,该类可以读取具有迷宫描述的输入流并返回迷宫。然而,当我用这个给定的输入流运行测试时:
20 10
####################
#................<.#
#..................#
#...###............#
#.....#............#
#.....#............#
#...###............#
#..................#
#..................#
####################
它给出了一个分段错误,我在valgrind上运行对象文件来检查发生了什么:
Invalid write of size 8
==2482545== at 0x4032CD: Maze::setTile(Position const&, Tile*) (maze.cpp:47)
==2482545== by 0x40347B: Maze::read(std::istream&) (maze.cpp:67)
.....
==2482545== Address 0x0 is not stack'd, malloc'd or (recently) free'd
我真的不明白为什么在我的代码中存在分段错误或无效写入,我应该在setTile函数中为每个tile分配空间,所以应该有空间供我写入。我还将tile_collection与构造函数堆叠在一起,因此当我调用Maze(20,10(时,tile_collective应该被初始化,并且setTile内部的大小调整应该可以工作。你能指出我遗漏了什么吗?提前谢谢。我知道我在三个小时前问了一个非常类似的问题,但我仍然坚持写这篇无效的文章。(Position类和tilefactory类是2个给定的文件,因此它们应该可以正常工作(这是头文件中声明的类:
class Maze {
private:
// TODO: add fields
int Width;
int Height;
std::vector<Tile*> tile_collection;
这是我的cpp文件:
Maze::Maze(int width,int height):
Width(width),Height(height){
tile_collection[(width-1)*(height)];
}
}
void Maze::setTile(const Position &pos,Tile *tile){
tile_collection[pos.getX()+pos.getY()*(Width)]=tile;
}
Maze *Maze::read(std::istream &in){
int x;int y;char c;
if ((in>>x)&&(in>>y)){
Maze *new_maze=new Maze(x,y);
//loop over the specified maze dimension
for (int i=0;i<y;i++){
for (int j=0;j<x;j++){
if (in>>c){
//using tilefactory to change character into a tile
TileFactory *fac=TileFactory::getInstance();
Tile* temp=fac->createFromChar(c);
//if createFromChar fails, return nullptr, otherwise set tile at position j,i
if (temp==nullptr){
return nullptr;
}
else{
new_maze->setTile(Position(j,i),temp);
}
}
}
}
return new_maze;
}
else{
return nullptr;
}
}
我怀疑您看到的故障是由以下行引起的:
TileFactory *fac=fac->getInstance();
您正在声明一个名为fac
的本地指针变量,但您试图在将该指针变量设置为任何值之前通过该指针变量调用方法(getInstance()
(。对我来说,这似乎是一场灾难!