干燥 if 语句

我有一个 C++ 程序，在许多不同的 .cpp 文件中，我执行如下操作：

if (!thing1.empty() && !thing2.empty())
{
    if (thing1.property < thing2.property)
        return func1();
    else if (thing2.property < thing1.property)
        return func2();
    else
        return func3();
}
else if (!thing1.empty())
{
    return func1();
}
else if (!thing2.empty())
{
    return func2();
}
else
{
   return func4();
}

如果 thing1 大于 thing2，我尝试以一种方式执行 func，如果情况相反，则向后执行 func，但如果不存在，那么我只对那一半执行 func。如果两者都不存在，我会做一些完全不同的事情。每次我使用此模式时，属性、函数和返回类型都不同。对于我想要做的事情，有比这种丑陋的混乱的嵌套 if 语句更好的设计吗？

EDIT:意识到我的示例代码过于简单化。这是我的一些真实代码，希望能更好地解释问题（尽管它更混乱）：

if (!diamondsOnly.empty() && !clubsOnly.empty())
{
    if (diamondsOnly.size() < clubsOnly.size())
    {
        if (passHighCards(player.hand, getHighCards(Card::DIAMONDS), result))
            return result;
        if (passHighCards(player.hand, getHighCards(Card::CLUBS), result))
            return result;
    }
    else if (clubsOnly.size() < diamondsOnly.size())
    {
        if (passHighCards(player.hand, getHighCards(Card::CLUBS), result))
            return result;
        if (passHighCards(player.hand, getHighCards(Card::DIAMONDS), result))
            return result;
    }
    else
    {
        if (diamondsOnly.back().value > clubsOnly.back().value)
        {
            if (passHighCards(player.hand, getHighCards(Card::DIAMONDS), result))
                return result;
            if (passHighCards(player.hand, getHighCards(Card::CLUBS), result))
                return result;
        }
        else
        {
            if (passHighCards(player.hand, getHighCards(Card::CLUBS), result))
                return result;
            if (passHighCards(player.hand, getHighCards(Card::DIAMONDS), result))
                return result;
        }
    }
}
else if (!diamondsOnly.empty())
{
    if (passHighCards(player.hand, getHighCards(Card::DIAMONDS), result))
        return result;
}
else if (!clubsOnly.empty())
{
    if (passHighCards(player.hand, getHighCards(Card::CLUBS), result))
        return result;
}

先决定再做

看看真实的代码，我注意到的第一件事是有很多几乎相同的调用，仅变化一个常量。我将使用在复杂逻辑中设置的参数在一处进行调用。

// Decide what to do.
std::vector<Card::Suit> passOrder;
if (!diamondsOnly.empty() && !clubsOnly.empty()) {
    // .. complicated logic that adds suits to passOrder ..
}

// Do it.
for (auto suit : passOrder) {  // This is C++11 style -- alter as needed
    if (passHighCards(player.hand, getHighCards(suit), result))
        return result;
}

（如果向量总是只有一两个，那么使用向量可能有点矫枉过正，但我​​假设真正的代码可能会处理所有的花色。）

这使得它更容易阅读。程序员可以看到，您首先决定传递卡片的顺序，然后才真正传递它们。两个单独的步骤会更清晰。与将其副本分散在整个决策逻辑中相比，只有一个调用 passCards 的地方更不容易出现愚蠢的拼写错误。它还将使调试变得更容易，因为您可以在非常特定的情况下设置断点，或者您可以只在循环开头设置断点并检查 passOrder。

简化逻辑

接下来我们要简化决策逻辑。

Options:

• Sentinels：复杂性的部分原因在于，在某些情况下，您需要取消引用其中一个容器中的最后一张卡，如果容器为空，则无法执行此操作。有时值得考虑向容器添加哨兵，这样您就不需要测试空的情况——可以保证它永远不会空。这可能可行，也可能不可行。您需要使处理容器的所有其他代码都能理解哨兵。

• 只是例外：您可以通过选择默认顺序来消除某些条款，例如，钻石然后俱乐部，然后仅测试您需要俱乐部然后钻石的情况。

• 使用临时对象表示：创建命名良好的临时对象，以简化您必须进行的比较，并根据这些临时对象来表达比较。请注意，将空/非空情况分解到临时变量中后，您可以通过选择适当的 SENTINEL_VALUE（例如 0 或 -1）来消除某些情况。

把它们放在一起：

// For readability.
const bool fewerClubs = clubsOnly.size() < diamondsOnly.size();
const bool sameNumber = clubsOnly.size() == diamondsOnly.size();
const int lastDiamondValue =  diamondsOnly.empty() ? -1 : diamondsOnly.back().value;
const int lastClubValue    =  clubsOnly   .empty() ? -1 : clubsOnly   .back().value;

// Decide what order to select cards for passing.
std::vector<Card::Suit> passOrder;
passOrder.push_back(Cards::DIAMONDS);  // default order
passOrder.push_back(Cards::CLUBS);

// Do we need to change the order?
if (fewerClubs || (sameNumber && lastClubValue > lastDiamondValue)) {
    // Yep, so start with the clubs instead.
    passOrder[0] = Cards::CLUBS;
    passOrder[1] = Cards::DIAMONDS;
}

// Do it.
for (auto suit : passOrder) {  // This is C++11 style -- alter as needed
    if (passHighCards(player.hand, getHighCards(suit), result))
        return result;
}

这假设 getHighCards 处理可能为空的容器作为输入。