c++ - 我应该避免在这里使用 goto 吗？如果是这样，怎么做？

Question

我正在编写一个函数来检查配对:

int containsPairs(vector<Card> hand)
{
    int pairs{ 0 };

    loopstart:
    for (int i = 0; i < hand.size(); i++)
    {
        Card c1 = hand[i];
        for (int j = i + 1; j < hand.size(); j++)
        {
            Card c2 = hand[j];
            if (c1.getFace() == c2.getFace())
            {
                pairs++;
                hand.erase(hand.begin() + i);
                hand.erase(hand.begin() + (j - 1));
                goto loopstart;
            }
        }
    }
    return pairs;
}

当它在第 10 行找到对子时，我想删除它找到对子的手中的牌，然后用删除的牌重新开始整个循环以找到第二对(如果有的话)。对我来说，goto 是最直观的方法，但在这种情况下，这是真的吗？

Answer 1

试试这个:

int containsPairs(vector<int> hand)
{
    int pairs{ 0 };

    for (int i = 0; i < hand.size(); i++)
    {
        int c1 = hand[i];
        for (int j = i + 1; j < hand.size(); j++)
        {
            int c2 = hand[j];
            if (c1 == c2)
            {
                pairs++;
                hand.erase(hand.begin() + i);
                hand.erase(hand.begin() + (j - 1));
                i--;
                break;
            }
        }
    }
    return pairs;
}

这几乎是你的版本，唯一的区别是，不是 goto，而是 i--;打破;。这个版本比你的效率更高，因为它只执行一次双循环。

是不是更清楚了？嗯，这是个人喜好。我完全不反对goto，我认为应该修改它目前的“从不使用它”状态。在某些情况下，goto 是最好的解决方案。

---

这是另一个更简单的解决方案:

int containsPairs(vector<int> hand)
{
    int pairs{ 0 };

    for (int i = 0; i < hand.size(); i++)
    {
        int c1 = hand[i];
        for (int j = i + 1; j < hand.size(); j++)
        {
            int c2 = hand[j];
            if (c1 == c2)
            {
                pairs++;
                hand.erase(hand.begin() + j);
                break;
            }
        }
    }
    return pairs;
}

基本上，当它找到一对时，它会只删除更远的牌，然后打破循环。所以没有必要对 i 感到棘手。