迭代器解引用中的分段错误

Question

下面列出的代码在基于迭代器的循环中触发分段错误：

#include <iostream>
#include <vector>

class A {
public:
    A(unsigned id = 0) {id_ = id;}
    unsigned get_id() {return id_;}
private:
    unsigned id_;
};

class B {
public:
    B() {}
    B(std::vector<A*> entries) : entries_(entries) {}
    const std::vector<A*> get_entries() const {
        return entries_;
    }
private:
    std::vector<A*> entries_;
};

int main() {
    std::vector<A*> entries;
    for (unsigned i = 0; i < 5; i++) {
        entries.push_back(new A(i));
    }
    B b(entries);

    // index based access (ok)
    for (unsigned i = 0; i < b.get_entries().size(); i++) {
        std::cout << b.get_entries()[i]->get_id() << std::endl;
    }

    // iterator based access (segmentation fault)
    for (std::vector<A*>::const_iterator i = b.get_entries().begin();
        i != b.get_entries().end();
        ++i) {
        std::cout << (*i)->get_id() << std::endl;
    }   
}

另一方面，基于索引的循环工作正常。

当返回std::vector的副本时触发此行为（请参阅： const std::vector<A*> get_entries() const ），而不是对其的const引用，例如const std::vector<A*>& get_entries() const 。 后一种情况很好。

如何解释这种行为？

Answer 1

由于get_entries()按值返回向量，因此get_entries()都使用不同的std::vector<A*> get_entries()对象。 比较来自不同向量的迭代器是“未定义行为”，因此即使仅使用get_entries().begin() != get_entries().end()您也已经遇到麻烦了。

Answer 2

问题在于get_entries返回向量的临时副本，而不是原始副本的引用。 因此，每次调用它并将迭代器获取一个临时返回值时，该迭代器在您使用它时将无效。 这导致不确定的行为，其中崩溃很常见。

您在这里有两个选择。

选项1：将返回的向量存储在局部变量中并对其进行迭代。

选项2：修改函数以返回引用const std::vector<A*> & get_entries() const 。