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;

请点击查看调试图片，

这很奇怪，因为

在监视表中，

定义了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;
}