列表插入迭代器超出范围错误

Question

我有这个功能

void Deck::draw(Hand &hand)
{
int random = (rand() % deckCards.size()) + 1;
list<Card*>::iterator cardID = deckCards.begin();

for (int i = 1; i < random; i++) {
    cardID++;
}

Card* selected = *cardID;

deckCards.erase(cardID);

hand.getHandCards().insert(hand.getHandCards().end(), selected);
}

使用以下吸气剂

list<Card*> Hand::getHandCards() const
{
return handCards;
}

我在运行时遇到此错误

Answer 1

迭代器仅对一个容器有效。您的迭代器用于其他容器，因此不能用于将元素插入另一个容器。

Answer 2

此函数返回handCards

的副本

list<Card*> Hand::getHandCards() const
{
    return handCards;
}

所以，在下面

• 创建一个临时副本，然后选择其end()迭代器。
• 已创建另一个临时副本，您可以使用另一个副本中的end()迭代器进行插入。

hand.getHandCards().insert(hand.getHandCards().end(), selected);

这使程序具有未定义的行为，甚至如果它是合法的，两个副本都是临时的，并且在语句末尾将被销毁。

在没有看到其余代码的情况下，很难看出这是否是您执行此操作的唯一位置，但是要修复此特定功能，您应该返回对handCards变量的引用：

list<Card*>& Hand::getHandCards()
{
    return handCards;
}

对此进行进一步的思考：您有一个返回的成员函数，我想是，一个private成员变量，让每个人都可以用它来做自己喜欢的事情。为什么不将其设为public？如果您不喜欢这种想法，请向Hand添加成员函数，以使类的用户仅执行您允许的那些事情。

示例：

void Hand::insert(Card* card) {
    handCards.emplace_back(card);
}