C++：向量的第一个元素“损坏”

Question

我有一个包含向量的类 (foo)。

如果我尝试像这样迭代向量中的元素：

for(vector<random>::iterator it = foo.getVector().begin();
        it != foo.getVector().end(); ++it) {
  cout << (*it) << endl;

}

第一个元素总是损坏并返回垃圾数据。

但是，如果执行以下操作：

 vector<random> v = foo.getVector();
 for(vector<random>::iterator it = v.begin();
            it != v.end(); ++it) {
      cout << (*it) << endl;

 }

一切似乎都工作正常。 是否有我不知道的“陷阱”？

我也尝试过 cout << foo.getVector()[0] << 结束； 在循环之外，但似乎工作正常。

谢谢。

编辑：

这是我的头文件：

#ifndef HITS
#define HITS

#include <vector>
#include "wrappers.h"

class Hits {

    public:
        Hits();
        std::vector<word_idx_value> getVector() {return speech_hits;}
        const std::vector<word_idx_value> getVector() const {return speech_hits;}
        void add(const word_idx_value&);
        Hits &operator+=(const Hits&);
    private:
        std::vector<word_idx_value> speech_hits;
};

#endif

Answer 1

for(vector<random>::iterator it = foo.getVector().begin();

当您执行 foo.getVector() 时，会返回临时向量，并在 foo.getVector().begin(); 之后遇到 ; 时被销毁。 因此迭代器在循环内变得无效。

如果将 foo.getVector(); 的值存储在向量 v (v = foo.getVector();) 中，然后使用向量 v，则效果很好。 这是因为向量 v 在整个循环中都是有效的。

Answer 2

getVector() 按值返回向量。 getVector 的两次调用（begin() 和 end()）返回向量的不同副本，因此您对一个对象调用 begin()，对另一个对象调用 end()。 您得到的是两个迭代器到两个不同的容器中。 将这两个迭代器与 != 进行比较会产生一个未定义的值。

Answer 3

getVector() 按值返回向量，在第一种情况下，您会得到一个临时变量，一旦进入循环，该变量就会被销毁。 在第二种情况下，您将结果复制到在循环内仍然有效的局部变量中。 可能的解决方案是通过 const 引用返回向量。

Answer 4

您的错误出现在 getVector() 方法中。
通过引用返回。

class Hits
{
    public:
    std::vector<word_idx_value>&   getVector() {return speech_hits;}
    //                         ^
    //                      Add the & to return by reference.

    // You may also want a const version at some point.
    std::vector<word_idx_value> const&   getVector() const {return speech_hits;}

如果您不通过引用返回，您将创建一个临时副本。 该副本在使用后将被销毁。 在这种情况下，在 begin() 执行后，临时对象被销毁，因此 begin() 返回的迭代器无效。

Answer 5

修改 getVector 函数以返回对象引用，如下所示：
std::vector& getVector() {返回speech_hits;}