#include<iostream>
#include<vector>
using namespace std;
vector<int>::iterator search(vector<int> vec,int n)
{
    for(vector<int>::iterator iter=vec.begin();iter!=vec.end();iter++)
    {
        if(*iter==n)
        {
        
        return iter;
        }
    }

    return NULL;
}
void main()
{
    vector<int> test;
    int num;
    cout<<"please input the members"<<endl;
    for(int i=0;i<5;i++)
    {
        cin>>num;
        test.push_back(num);
    }
    cout<<"please input the num you want to search"<<endl;
    cin>>num;
    vector<int>::iterator temp=search(test,num);
    if(!temp==NULL)
        cout<<*temp;
    else
        cout<<"the num  "<<num<<"  doesn't exist"<<endl;
}
问为什么运行之后，能找到，但输出*temp却是一个未定义的值。

这个函数vector<int>::iterator search(vector<int> vec,int n)
{
    for(vector<int>::iterator iter=vec.begin();iter!=vec.end();iter++)
    {
        if(*iter==n)
        {
        
        return iter;
        }
    }

    return NULL;
}

退出函数时,返回的迭代器已经无效,建议直接返回值

楼上说的对，search(vector<int> vec,int n)句中，你只传了个拷贝，最好还是传引用，
search(vector<int> &vec,int n)
因为你传的是拷贝，所以iter是一个局部指针，当函数结束时分配给vec的内存被回收。。
#include<iostream>
#include<vector>
using namespace std;
vector<int>::iterator search(vector<int> &vec,int n)
{
    for(vector<int>::iterator iter=vec.begin();iter!=vec.end();iter++)
    {
        if(*iter==n)
        {
        
        return iter;
        }
    }

    return NULL;
}
void main()
{
    vector<int> test;
    int num;
    cout<<"please input the members"<<endl;
    for(int i=0;i<5;i++)
    {
        cin>>num;
        test.push_back(num);
    }
    cout<<"please input the num you want to search"<<endl;
    cin>>num;
    vector<int>::iterator temp=search(test,num);
    if(!temp==NULL)
        cout<<*temp;
    else
        cout<<"the num  "<<num<<"  doesn't exist"<<endl;
}

收到的鲜花

• mxs810 于 2010-10-09 13:25 送鲜花  3朵   附言：我很赞同

是这样，题目要求返回一个迭代器，我也尝试过定义一个全局变量。但还是有问题~~~~

你把vector<int>::iterator search(vector<int> vec,int n)定义成
vector<int>::iterator search(vector<int> *vec,int n)就应该符合你的要求了.