如何正确返回vector的iterator？下述代码中cout<<*p<<endl输出异常

你的代码里出现输出错误的核心原因，是函数参数传递方式导致了迭代器悬空——咱们一步步拆解：

问题根源 #

你定义的int_begin函数接收的是vector<int>的值拷贝：

vector<int>::iterator int_begin(vector<int> V)

当你在main里调用int_begin(V)时，程序会创建一个原始vector的临时副本。函数里返回的V.begin()是这个临时副本的迭代器，但函数执行完毕后，这个临时vector会被立即销毁，此时返回的迭代器就变成了「悬空迭代器」——指向已经被释放的内存空间，解引用它属于C++里的未定义行为，自然得不到正确结果。

最佳解决方案：传递引用而非值拷贝 #

要解决这个问题，只需要把函数的参数改成引用类型，让函数直接操作原始的vector对象，这样返回的迭代器就始终指向有效的内存区域。

修正后的完整代码

#include<iostream>
#include<vector>
using namespace std;

// 将参数改为非const引用，避免拷贝并操作原始对象
vector<int>::iterator int_begin(vector<int>& V);

int main() {
    vector<int> V;
    V.push_back(3);
    V.push_back(1);
    vector<int>::iterator p = int_begin(V);
    cout << *p << endl; // 现在会正确输出3
    return 0;
}

vector<int>::iterator int_begin(vector<int>& V) {
    cout << *V.begin() << endl; // 这里正常输出3
    return V.begin();
}

进阶：遵循const正确性

如果你的函数不需要修改传入的vector，推荐使用const引用作为参数，同时返回const_iterator，这样能避免意外修改原始容器，让代码更健壮：

// 只读场景下的版本
vector<int>::const_iterator int_begin(const vector<int>& V) {
    cout << *V.begin() << endl;
    return V.begin();
}

关键注意事项 #

• 永远不要返回指向临时对象的迭代器、指针或引用，临时对象在函数结束后会被销毁，这些值都会变成悬空的，访问它们会导致不可预测的错误。
• 传递容器（比如vector、map等）时，优先使用引用而非值拷贝，既能避免不必要的性能开销，又能保证迭代器/指针的有效性。

内容的提问来源于stack exchange，提问作者BODAPATI NIRUPAMA SAI