返回 C++ 迭代器引用

Question

按预期工作的示例

#include <iostream>
#include <vector>

struct MyClass{


  const std::vector<float>::iterator& begin(){
    return myvec.begin();
  }

  const std::vector<float>::iterator& end(){
    return myvec.end();
  }
  std::vector<float> myvec;
};

int main(){

  std::vector<float> mainvec(8,0);

  MyClass myClass;

  myClass.myvec = mainvec;

  for (std::vector<float>::iterator it = myClass.begin();
      it != myClass.end();++it){
    std::cout << *it << " " ;
  }

  std::cout << std::endl;
}

在这段代码中，我得到以下输出：

0 0 0 0 0 0 0 0 

一个没有按预期工作的例子：

#include <iostream>
#include <vector>

struct MyClass{


  const std::vector<float>::iterator& begin(){
    return myvec.begin();
  }

  const std::vector<float>::iterator& end(){
    return myvec.end();
  }
  std::vector<float> myvec;
};

int main(){

  std::vector<float> mainvec(8,0);

  MyClass myClass;

  myClass.myvec = mainvec;

  const std::vector<float>::iterator& end_reference = myClass.end();

  for (std::vector<float>::iterator it = myClass.begin();
      it != end_reference;++it){
    std::cout << *it << " " ;
  }

  std::cout << std::endl;
}

在这段代码中，我得到以下输出：

"empty output"

第一个代码示例

问题在于我调用（错误地）向量 begin() 和 end() 而不是 MyClass 方法。

我有以下最少的代码来表示我的疑问：

#include <iostream>
#include <vector>

struct MyClass{


  const std::vector<float>::iterator& begin(){
    return myvec.begin();
  }

  const std::vector<float>::iterator& end(){
    return myvec.end();
  }
  std::vector<float> myvec;
};

int main(){

  std::vector<float> mainvec(8,0);

  MyClass myClass;

  myClass.myvec = mainvec;

  for (std::vector<float>::iterator it = myClass.myvec.begin();
      it != myClass.myvec.end();++it){
    std::cout << *it << " " ;
  }

  std::cout << std::endl;
}

我在第 8 行和第 12 行收到以下警告：

returning reference to local temporary object [-Wreturn-stack-address] [cpp/gcc] 

但是当我编译并运行程序时，我得到：

0 0 0 0 0 0 0 0 

所以，似乎本地引用在返回myvec.begin() 时没有被破坏。当我第一次编写代码时，我并不认为这会是一个问题，因为在我的脑海中，来自向量的 begin() 方法将返回一个指向第一个向量位置的迭代器引用，这个迭代器对我来说不是当我执行 myvec.begin() 时分配，但它是对这个迭代器的引用。所以，这个警告不应该出现，因为我没有分配内存。但是由于我不知道这种机制是如何工作的，所以我想学习它以编写一致的代码。看来我可以忽略这个警告，不是吗？

Answer 1

std::vector::begin 确实返回一个临时对象 （见http://www.cplusplus.com/reference/vector/vector/begin/）

您的代码是有效的，因为您将临时对象绑定到 const 引用（使用非 const 引用是不合法的），并立即将其复制到局部变量 it。
编辑：我的错误。这确实也是 UB 从一个临时返回一个 const 引用。

但是，将此引用分配给另一个具有更长生命周期的 const 引用会导致未定义的行为，因为临时对象的生命周期只会延长到直接分配给它的 const 引用的生命周期。

我建议考虑警告并返回一个迭代器。

我也推荐阅读http://herbsutter.com/2008/01/01/gotw-88-a-candidate-for-the-most-important-const/