遍历向量（分段错误）

Question

我想迭代器有问题，但我不明白为什么！你能给我更多信息吗？

我有电影class，内容如下：

public:
    vector<string> casting() const;

private:
    string _titol;
    short _year;
    vector<string> _alies;
    string _director;
    vector<string> _casting;

casting() 方法只是 returns _casting 向量

我有我的 main.cpp 我打电话的地方：

void Movies::actorMovies(string actor){
bool existeix = false;
std::map<titleyear,Movie>::iterator it = _pelis.begin();
std::vector<string>::iterator it2;

for(it; it!=_pelis.end(); it++){
    for(it2=it->second.casting().begin(); it2!=it->second.casting().end(); it2++){
        /*if((*it2).compare(actor)==0){
            cout<<"Titol: "<<it->first.t<<endl<<"Any: "<<it->first.y<<endl;
            existeix = true;
        }*/
    }


}
if(!existeix) 
    cout<<"NOT FOUND"<<endl;
}

当我想取消注释 if 语句时出现分段错误。有人能看到这里发生了什么吗？

Answer 1

你的前提是错误的。 casting() 方法 returns 每次调用 _casting 向量的新副本。因此 it2 永远不会等于 it->second.casting().end()，因为它是一个完全不同的容器的迭代器！

事实上，it2 在完整表达式的末尾立即失效，因为它是一个指向临时容器的迭代器，该容器会立即消失。

如果 casting() 旨在提供实际 _casting 向量的视图，它应该 return 一个左值：

const std::vector<std::string> & casting() const { return _casting; }
//                            ^^^

Answer 2

casting() 函数 return 是一个临时向量按值。对该函数的不同调用每个 return 向量的一个新副本。

所以当你写的时候：

for(it2=it->second.casting().begin();

it2 立即悬空，因为临时向量现在被破坏了。

测试 it2!=it->second.casting().end(); 将悬空迭代器与 _casting 的新副本的末尾进行比较，导致未定义的行为。

有两个选项可以解决此问题：

存储结果：vector<string> casting = it->second.casting(); for (it2 = casting.begin(); ..........
通过引用制作 casting() return，这样您就可以在 it->second.

遍历向量（分段错误）

Going over a vector (Segmentation fault)

c++

vector

std

fault