C ++函数从列表中查找最小值。 此函数是否正确，因为返回的迭代器无效

Question

list<LNode>::iterator minI(list<LNode>::iterator start,list<LNode>::iterator end)
{
     list<LNode>::iterator min_index=start;

     for(auto it=start;it!=end;it++)
     {
        if(it->len<min_index->len)
            min_index=it;
     }
     return min_index;
}

在main（）中

..

    cout<<(&(*vnode.begin()))<<endl;
    auto min_it=minI(vnode.begin(),vnode.end());
    cout<<(&(*vnode.begin()))<<endl;
    cout<<(&(*min_it))<<endl;
    char ch;
    int i=min_it->i,j=min_it->j;
    printf("\n%s\t%s\t%s\t%s\t%s\t",t1[i].from,t1[i].to,t1[i].flight,t2[j].city,t2[j].discount);
    scanf("%s",&ch);

    cout<<(&(*min_it))<<endl;

..

前三个cout打印0x600d88，最后一个cout打印0x600d08，谁能解释我的原因？

,,

Answer 1

迭代器不会失效。

通常，只有在修改容器后，才能使标准容器上的迭代器无效。 对于list ，仅当删除了它们所引用的元素时，它们才会无效。

在第二段代码中， scanf被赋予指向单个字符的指针，但是它将覆盖至少两个：用户输入的任何字符，后跟终止符。 这给出了未定义的行为，这很可能会修改局部变量的值（例如min_it ）。 尝试用更安全的C ++替代它

std::string ch;
std::cin >> ch;             // to read one word, or 
std::getline(std::cin, ch); // to read a whole line