样式化复合AND / OR if语句

如何为复杂的复合AND / OR if语句设置样式，以实现最大的可读性？您如何缩进以及将换行符放在何处？我的特殊情况如下。绝对比将所有内容都粉碎成一条线要好，但是看起来仍然很凌乱。

if (
    (
        x == y
        && a != b
        && p.isGood() 
        && (
            i + u == b
            || q >= a
        )
    )
    || k.isSomething()
    || m > n
) {
    doSomething();
}

为每个小步骤设置布尔变量：

bool step1 = i + u == b || q >= a;
bool step2 = a != b && p.isGood() && group1;
bool step3 = group2 || k.isSomething() || m > n;
if (step3) { doSomething(); }

当然，这与泪谱学的答案类似，只是每个步骤的名称不同。

如果你的名字step1，step2并step3以多种方式让好的概念的意义上，这应该是迄今为止最清晰可辨。p.isGood()并且k.isSomething()有时在它不会是你的原代码的情况下，如果你在一个非常紧密的循环中运行这段代码中调用，所以这不会是一个选项，如果这些功能是昂贵的或。

另一方面，您不必担心创建新变量可能会对性能造成的影响。一个好的编译器会优化它们。

带有矩形碰撞检测的示例（由于上述性能问题，您可能不会使用）：

if((a.x + a.width >= b.x || b.x + b.width >= a.x)
 && (a.y + a.height >= b.y || b.y + b.width >= a.y)
)
{ collision(); }

可能成为：

bool horizMatch = a.x + a.width >= b.x || b.x + b.width >= a.x;
bool vertMatch = a.y + a.height >= b.y || b.y + b.width >= a.y;
if(horizMatch && vertMatch) { collision(); }

另外，如果您想保留代码，我也认为完全可以。老实说，我认为您的代码清晰易读。显然我不知道到底a b x y i u p k m n是什么，但是就结构而言，对我来说看起来不错。

如果条件变得如此复杂，我通常会将代码重构为更加模块化的代码。

在这种复杂程度下，我会做更多类似的事情

bool doIt = x == y && a != b && p.isGood();
doIt &= ( i + u == b || q >= a);
doIt |= k.isSomething() || m > n;

if(doIt)
{
    doSomething();
}

它很丑陋，但是可读性强，我敢肯定编译器会知道如何重构它。

另一方面，如果我曾经遇到过编写这样的IF语句的情况，我会重新考虑解决方案，因为我确定有一种更简单的方法，或者至少可以抽象出某些条件（例如：x == y && a != b && p.isGood()真的只是刻薄this->isPolygon()，我可以采用这种方法；

随着时间的流逝，我对垂直对齐的痴迷程度降低了，但是我对多行表达式的一般形式是...

if (   (   (expr1 == expr2)
        || (expr3 == expr4)
        || (expr5 == expr6)
       )
    && (   (expr7 == expr8)
        || (expr9 == expra)
       )
   )
{
  blah;
}

关键点...

• 闭合的括号和括号一起与打开的括号垂直对齐。
• 一条线内的子表达式位于一条线内，并且在左侧垂直对齐。在提高可读性的地方，这些单行部分中的中缀运算符也垂直对齐。
• 右括号自然地产生了近乎空白的线条，有助于视觉上对事物进行分组。

有时，我也会格式化+和/ *或其他一些类似的运算符。相当多的复杂表达式采用乘积和或乘积形式（可以指布尔“和”和“乘积”），因此它很常见，因此值得使用一致的样式。

不过要小心。通常最好进行重构（将表达式的一部分移至函数中，或将中间部分计算并存储在变量中），而不是使用缩进来尝试使过于复杂的表达式更具可读性。

如果您喜欢在右侧堆叠您的近亲，我不会讨厌它，但是我想它还不错。太远了，您冒着这样的风险，即错误可能使缩进错误地表示括号的作用。

if (   (   (expr1 == expr2)
        || (expr3 == expr4)
        || (expr5 == expr6))

    && (   (expr7 == expr8)
        || (expr9 == expra)))
{
  blah;
}

http://www.codinghorror.com/blog/2006/01/flattening-arrow-code.html

我同意JohnFx以及Lacrymology的回答。我将构建一堆实现小目标的功能（最好是静态的），然后以一种聪明的方式在这些功能上进行构建。

那么，这样的事情怎么样？注意，这不是完美的解决方案，但它可以工作。有一些方法可以进一步清理此问题，但是需要更具体的信息。注意：由于编译器很聪明，因此此代码应以相同的速度运行。

// Currently based on members or global vars
// (which is often a bad idea too)
function doSomethingCondirionally()
{
  if (k.isSomething() || m > n)
  {
    doSomething();
    return;
  }

  // Else ... 
  if (x != y) return;
  if (a == b) return;
  if (!p.isGood()) return;

  // Final, positive check
  if (i + u == b || q >= a)
  {
    doSomething();
  }
}

对于它的价值，我很惊讶地看到您的示例看起来很像我编写的复杂谓词。我同意其他人的观点，即复杂的谓词对于可维护性或可读性而言并不是最大的选择，但有时会出现。

让我强调您已正确完成了此部分：&& a != b 切勿将逻辑连接器放在行尾，这很容易在视觉上遗漏。永远不要在行的末尾放置运算符的另一个地方是字符串连接，使用这种运算符的语言。

做这个：

String a = b
   + "something"
   + c
   ;

不要这样做：

String a = b +
   "something" +
   c;

如果条件是如此复杂，则通常表明应将其分解为多个部分。也许可以将一个子句分配给中间变量。也许一个子句可以变成辅助方法。我通常不希望一行中有那么多的“ and”或“ ors”。

您可以将代码分成多个语句，以使其易于理解。但是真正的忍者会做这样的事情。:-)

if
(
    (
        x == y
    &&
        a != b
    &&
        p.isGood()
    &&
        (
            i + u == b
        ||
            q >= a
        )
    )
||
    k.isSomething()
||
    m > n
)
{
    doSomething();
}