c++ - 从 C++ 列表中删除元素的问题

Question

我有以下代码:

list<Item> playerItems = player.GetPlayerItems();

        list<Item>::iterator iterator;
        for (iterator = playerItems.begin(); iterator != playerItems.end(); ++iterator)
        {
            if (itemToSell == (*iterator).GetName())
            {
                if ((*iterator).GetCost() < GetShopGold())
                {
                    SetShopItem(*iterator);
                    SetShopGold(GetShopGold() - (*iterator).GetCost());

                    player.SetPlayerGold(player.GetPlayerGold() + (*iterator).GetCost());

                    //The issue is here
                    iterator = player.GetPlayerItems().erase(iterator);
                    return true;
                }
                else
                {
                    cout << "I can't afford that..." << endl;
                }
            }
        }

这是 RemovePlayerItem 代码:​​

void Player::RemovePlayerItem(list<Item>::iterator& iterator)
{
    iterator = _playerItems.erase(iterator);
}

当我运行代码时，出现以下错误:

_DEBUG_ERROR("list erase iterator outside range");

我google了一下，好像和我的迭代器有关，但是不确定错误和迭代器的关系。

相比之下，如果我将 _playerItems 变量设置为全局变量，然后使用该全局变量启动/结束迭代器，则如果我使用以下代码，代码运行良好 iterator = player._playerItems.erase(iterator) ;

编辑

这是 GetPlayerItems() 函数

list<Item> Player::GetPlayerItems()
{
    return _playerItems;
} 

谁能用这个给我指出正确的方向？

Answer 1

iterator 是 playerItems 的迭代器。不清楚 player.GetPlayerItems() 返回什么(因为您忽略了显示它的定义甚至声明)，但不管它是什么，playerItems 都是一个单独的、不同的列表- GetPlayerItems() 返回的拷贝。

然后您尝试使用指向另一个列表的迭代器从一个列表中删除。这表现出未定义的行为 - someList.erase(iter) 期望 iter 成为 someList 的有效迭代器。