【发布时间】：2018-02-19 04:00:12
【问题描述】：
我正在尝试编写一个广度优先搜索程序来解决 8 难题。当我运行以下代码时，出现以下错误：
* `/home/a.out' 中的错误：free()：无效指针：0x0000000001f81430 *
中止
我很确定问题在于 this 指针的使用以及我如何存储父节点。有什么帮助吗？
#include <iostream>
#include <string>
#include <iostream>
#include <algorithm>
#include <vector>
#include <queue>
using namespace std;
class Node {
public:
vector<Node> children;
vector<int> puzzle;
vector<int> goal = {1, 2, 3, 4, 5, 6, 7, 8, 0};
Node *parent;
Node(vector<int> _puzzle, Node *_parent){ // constructor for node
puzzle=_puzzle;
parent=_parent;
}
void moveUp(){ //function to move up
int zPos = findZero();
vector<int> temp = puzzle;
if ( zPos != 0 || zPos != 1 || zPos != 2 )
std::swap(temp[zPos], temp[zPos-3]);
Node child = Node(temp, this);
children.push_back(child);
}
void moveDown(){ //function to move down
int zPos = findZero();
vector<int> temp = puzzle;
if ( zPos != 6 || zPos != 7 || zPos != 8 )
std::swap(temp[zPos], temp[zPos+3]);
Node child = Node(temp, this);
children.push_back(child);
}
void moveRight(){ //function to move right
int zPos = findZero();
vector<int> temp = puzzle;
if ( zPos != 2 || zPos != 5 || zPos != 8 )
std::swap(temp[zPos], temp[zPos+1]);
Node child = Node(temp, this);
children.push_back(child);
}
void moveLeft(){ //function to move left
int zPos = findZero();
vector<int> temp = puzzle;
if ( zPos != 0 || zPos != 3 || zPos != 6 )
std::swap(temp[zPos], temp[zPos-1]);
Node child = Node(temp, this);
children.push_back(child);
}
void printPuzzle() { //function to print the puzzle
int count = 0;
for (auto i: puzzle) {
if ( count % 3 == 0)
std::cout << std::endl;
std::cout << i << ' ';
count++;
}
}
int findZero(){ // function to find the location of zero
std::vector<int>::iterator it;
it = find (puzzle.begin(), puzzle.end(), 0);
auto z = std::distance(puzzle.begin(), it);
return (int)z;
}
bool isGoal(){ //function to check if goal state is reached
bool goalFound = false;
if(puzzle == goal)
goalFound = true;
return goalFound;
}
};
bool contains(std::queue<Node> q, Node n){ // checks repeated nodes
std::queue<Node> tempQ = q;
bool exist = false;
while (!tempQ.empty()){
if (tempQ.front().puzzle == n.puzzle)
exist = true;
tempQ.pop();
}
return exist;
}
int main()
{
std::vector<int> initial = {3, 5, 8, 1, 0, 4, 2, 7, 6};
Node init = Node(initial, NULL);
std::queue<Node> openList;
std::queue<Node> closedList;
openList.push(init);
bool goalFound = false;
while(!openList.empty() && !goalFound){
Node currentNode = openList.front();
closedList.push(currentNode);
openList.pop();
currentNode.moveUp();
currentNode.moveDown();
currentNode.moveRight();
currentNode.moveLeft();
for (auto i: currentNode.children){
Node currentChild = i;
if (currentChild.isGoal()){
std::cout << "Goal Found." << endl;
goalFound = true;
}
if (!contains(openList, currentChild) && !contains(closedList, currentChild))
openList.push(currentChild);
}
}
}
现在我只专注于寻找目标。我还没有实现目标路径并打印最终解决方案。
【问题讨论】：
-
你到处复制节点，但是你没有一个行为良好的复制构造函数或赋值运算符。您的大多数
parent指针无效（它们指向已被销毁的旧currentNode）。 -
我刚刚注释掉了所有关于父指针的行。我仍然得到同样的错误。也许问题出在其他地方。
-
parent稍后会联系您。vectors 是一个非常棒的工具，但是在将指向项目的指针存储在一个中时，你必须保持警惕。更多阅读：en.cppreference.com/w/cpp/container/… 和 stackoverflow.com/questions/6438086/iterator-invalidation-rules -
如果不是太不方便的话，您能否更详细地解释一下我哪里出错了。
-
阅读链接，但一般问题是向量包含您输入的内容的副本。副本的地址与原始地址不同。当您在
vector中移动东西时，项目的地址会根据vector中的时间而改变。一次交换和parent指针指向错误的位置。另外，当您向vector添加内容时，迟早它必须分配更多存储空间并将其当前包含的内容复制到新存储空间中。parent指针指向旧存储空间，vector会在它已经完成了。试图访问已删除的内存是不好的。
标签： c++ pointers this breadth-first-search sliding-tile-puzzle