C++ STL 向量迭代器...但出现运行时错误

Question

我正在学习STL并制作了win32项目..

但我陷入了运行时错误..

我试图调试它但是..

（部分代码）

vector<Vertex> currPoly=polygons.back();
vector<Vertex>::iterator it;


for(it=currPoly.begin();it!=currPoly.end();++it){
    vector<Vertex>::iterator p1;
    vector<Vertex>::iterator n1;
    vector<Vertex>::iterator n2;

    if(         it==currPoly.begin()){
        p1=currPoly.end();
        n1=it+1;
        n2=it+2;
    }else if(   it==currPoly.end()-1){
        p1=it-1;
        n1=it+1;
        n2=currPoly.begin();
    }else if(   it==currPoly.end()){
        p1=it-1;
        n1=currPoly.begin();
        n2=currPoly.begin()+1;
    }else{
        p1=it-1;
        n1=it+1;
        n2=it+2;
    }
    int tmp;
    tmp=it->x;
    tmp=p1->x;

请click看调试图

这很奇怪，因为

在表中，

n1,p1,它已定义但 n2 未定义 tmp 也不是..

我找不到问题所在...

请帮忙..

Answer 1

你应该更清楚你的问题到底是什么......

如果您想知道为什么 n1 和 tmp 的值无法在调试器中显示，我猜这是因为您正在调试发布版本（或某种使用优化），并且编译器可能已经在执行流程中的那个点“优化掉”了这些变量（它决定不再使用它们或者它们的值可以在其他地方获得）。

尝试调试未优化的构建。

顺便说一句，错误 CXX0017（调试器为这些变量显示的内容）表示“表达式评估器错误”。

Answer 2

你保证多边形总是至少有 3 个点吗？另外，请仔细查看您的第一个 else if。你还记得 end() 是一个过去的结尾吗？ （您似乎大多如此，但可能/有些地方出了问题……）

Answer 3

作为说明：

}else if(   it==currPoly.end()){
    p1=it-1;
    n1=currPoly.begin();
    n2=currPoly.begin()+1;
}

这绝不应该发生。

int tmp;
tmp=it->x;
tmp=p1->x;

这看起来有点没有意义。

Answer 4

您仍然对 currPoly.end() 不指向有效元素的想法感到困惑。 end() 指向最后一个有效元素之后的一个。

因为您将向量用作环，所以在向量中使用整数索引实际上是编写代码的更好方法。您可以根据向量大小修改索引，而不是编写特殊情况代码。但既然你这样做是为了学习 STL，我们将坚持使用迭代器。这是我认为正确的代码应该是什么样子的（我没有编译它来查看它是否有效）：

vector<Vertex> currPoly=polygons.back();
vector<Vertex>::iterator it;


for(it=currPoly.begin();it!=currPoly.end();++it){
    vector<Vertex>::iterator p1;
    vector<Vertex>::iterator n1;
    vector<Vertex>::iterator n2;

    if(it==currPoly.begin()){
        p1=currPoly.end()-1;
        n1=it+1;
        n2=it+2;
    }else if(it==currPoly.end()-1){
        p1=it-1;
        n1=currPoly.begin();
        n2=n1+1;
    }else if(it==currPoly.end()-2){
        p1=it-1;
        n1=it+1;
        n2=currPoly.begin();
    }else{
        p1=it-1;
        n1=it+1;
        n2=it+2;
    }
    int tmp;
    tmp=it->x;
    tmp=p1->x;
}

这就是我认为基于索引的版本的样子（同样，没有用编译器检查）：

vector<Vertex> currPoly=polygons.back();


for(int i=0; i < currPoly.size(); ++i){
    int p1 = (i+currPoly.size()-1)%currPoly.size();
    int n1 = (i+1)%currPoly.size();
    int n2 = (i+2)%currPoly.size();

    int tmp;
    tmp=currPoly[i].x;
    tmp=currPoly[p1].x;
}