我正在尝试直接连接9个城市的图表。我遇到了“短路径”功能的这个问题。我一直收到错误“矢量迭代器不兼容”,我无法弄明白。请帮忙。感谢
这是功能:
void City::shortPath( vector<City*> & cities )
{
queue<City*> q;
q.push(this);
for( unsigned int i = 0; i < cities.size(); i++ )
{
cities[i]->cityDistance = -1;
cities[i]->visited1 = false;
}
q.front()->visited1 = true;
q.front()->cityDistance = 0;
while( !q.empty() )
{
City * v = q.front();
if( v->neighbor1.size() != 0 )
{
for( unsigned int i = 0; i < v->neighbor1.size(); i++ )
{
City * z = v->neighbor1[i];
q.push( z );
if( z->visited1 == false )
{
v->neighbor1[i]->cityDistance = v->cityDistance + 1;
v->neighbor1[i]->previous1.push_back( *v );
v->neighbor1[i]->visited1 = true;
}
}
}
q.pop();
}
}
这是主要的:
int main()
{
City * NewDelhi = &City( "New Delhi" );
City * HongKong = &City("Hong Kong");
City * Washington = &City("Washington");
City * Dublin = &City("Dublin");
City * Lisbon = &City("Lisbon");
City * Vienna = &City("Vienna");
City * Santiago = &City("Santiago");
City * RioDeJaneiro = &City("RioDeJaneiro");
City * Berlin = &City( "Berlin" );
City * NewYork = &City( "NewYork" );
vector<City*> vector1;
vector1.push_back(HongKong);
vector1.push_back(NewDelhi);
vector1.push_back(Washington);
vector1.push_back(Dublin);
vector1.push_back(Lisbon);
vector1.push_back(Vienna);
vector1.push_back(Santiago);
vector1.push_back(RioDeJaneiro);
vector1.push_back(Berlin);
vector1.push_back(NewYork);
(*HongKong).neighbor1.push_back( NewDelhi );
(*NewDelhi).neighbor1.push_back( Washington);
(*Washington).neighbor1.push_back( Dublin );
(*Dublin).neighbor1.push_back( Lisbon );
(*Lisbon).neighbor1.push_back( Vienna );
(*Vienna).neighbor1.push_back( Santiago );
(*Santiago).neighbor1.push_back( RioDeJaneiro );
(*RioDeJaneiro).neighbor1.push_back( Berlin );
(*Berlin).neighbor1.push_back( NewYork );
(*NewYork).neighbor1.push_back( HongKong );
(*NewYork).shortPath( vector1 );
答案 0 :(得分:1)
正如Zan Lynx在那里提到的那样......你正在使用糟糕的指针。
我写了一个快速测试,并且有g ++的输出
make a
g++ a.cpp -o a
a.cpp: In function ‘int main()’:
a.cpp:15:35: error: taking address of temporary [-fpermissive]
make: *** [a] Error 1
正如你所看到的,它告诉我,取一个临时的地址是不好的。不仅如此,它实际上并没有编译,因为这是一个错误。 cl肯定会在那个上做得更好!
我的测试代码,以防万一:
#include <iostream>
class City
{
public:
City(const std::string& name) : name_(name) { std::cout << "Constructed" << std::endl; }
~City() { std::cout << "Destructed" << std::endl; }
private:
const std::string name_;
};
int main()
{
std::cout << "Started" << std::endl;
City *SF = &City("San Francisco");
char c;
std::cin >> c;
}
答案 1 :(得分:0)
当您尝试使用属于同一类型的另一个向量的迭代器时,可能会发生这种情况。在某些时候,您可能正在制作矢量的副本,但是您使用原始矢量中的迭代器和新副本。
在这种情况下,罪魁祸首是通过将向量的副本与属于原始的迭代器一起传递给函数来返回向量或的副本而引起的。您需要检查代码并查看向量和iteratos如何传递。任何制作副本的地方都是问题的潜在根源。
此外,正如Roy指出的那样,如果在执行某些容器的修改后持有旧的迭代器,也会发生这种情况。在vector
的情况下,无论何时添加或删除元素,都可能需要调整内部存储大小,导致所有迭代器变为无效。任何使用这些迭代器的尝试都会导致未定义的行为,并且可能发生任何事情。
答案 2 :(得分:-1)
通常是因为你在保存迭代器之后有一些动作改变了向量,例如插入/擦除,先前保存的迭代器不再有效。
检查您的代码,看看是否属实。