std :: list :: erase（iter）异常处理C ++

Question

我尝试从函数中获取一个迭代器，然后从列表中删除一个元素。

1. 有什么办法可以捕获该异常？
2. 有没有办法知道迭代器是否有效？

当我使用无效的迭代器运行程序时，它将引发以下异常：

 Exception thrown: read access violation.
 _Right.**_Myproxy** was 0xCCCCCCCC. occurred

我的功能是：

list<AccountStruct>::const_iterator SearchAccount(list<AccountStruct>& Acc,string p)
{

    for (list<AccountStruct>::const_iterator j = Acc.begin(); j !=Acc.end(); ++j)
    {
        if ((*j).phone == p) return j;
    }

}

void RemoveAccount(list<AccountStruct>& Acc)
{
    string Phone;
    cout << "Enter Phone Number ->" << endl;
    cin >> Phone;
    //cout << "Searching..." << endl;
    list<AccountStruct>::const_iterator iter = SearchAccount(Acc,Phone);
    try
    {
        Acc.erase(iter);
    }
    catch()// <- What to put inside??
    {
        cout << "No Phone Number;" << endl;
    }

}

Answer 1

我们可以通过一些更改来解决此问题。 首先，如果函数找不到对象，则SearchAccount不会返回任何内容。 这是未定义的行为，因为您必须在声明时返回某些内容。 如果find该项目，我们可以通过返回end()来解决此问题，就像find一样。 那给你

list<AccountStruct>::const_iterator SearchAccount(list<AccountStruct>& Acc,string p)
{
    for (list<AccountStruct>::const_iterator j = Acc.begin(); j !=Acc.end(); ++j)
    {
        if ((*j).phone == p) return j;
    }
    return Acc.end(); // item not found
}

现在SearchAccount可以正常运行了，我们可以检查是否返回了end()而不是使用异常机制。 看起来像

void RemoveAccount(list<AccountStruct>& Acc)
{
    string Phone;
    cout << "Enter Phone Number ->" << endl;
    cin >> Phone;
    //cout << "Searching..." << endl;
    list<AccountStruct>::const_iterator iter = SearchAccount(Acc,Phone);
    if (iter != Acc.end())
    {
        Acc.erase(iter);
    }
    else
    {
        cout << "No Phone Number;" << endl;
    }
}

现在，您不再需要异常机制的开销，并且发生了“正确的事情”。

Answer 2

您应该使用::std::remove_if算法

Acc.erase
(
    ::std::remove_if
    (
        Acc.begin()
    ,   Acc.end()
    ,   [&Phone](AccountStruct const & acc)
        {
            return(acc.phone == Phone);
        }
    )
,   Acc.end()
);

Answer 3

std::list::erase不会引发任何异常。

不需要try块。 但是，无效的迭代器将导致未定义的行为。

你可以返回Acc.end()在后的for中环SearchAccount和核对返回的值Acc.end而不是使用try 。