关于如何格式化长 if 语句的愚蠢问题

Question

对于占用多行的长 if 语句，您是否将 AND 或 OR 等条件放在新行上，如下所示：

               if (something
                   && something else)

或者像这样：

               if (something &&
                   something else)

Answer 1

对于复杂的条件，请考虑将其提取到函数或变量中：

if (complexCondition(foo)) { ..

作为奖励，函数或变量的名称可用于传达条件的含义。这使您的代码更易于阅读。

Answer 2

我通常采用第二种方式，因为我可以排列语句。然而，在编写代码时，只要保持一致，任何一种方法都可以。

Answer 3

我更喜欢第一首的演绎。我的理由是，出于任何测试目的通过剪切/粘贴/注释删除条件更容易。注释掉一行比从上面的行中删除 and 并注释掉一行要容易得多。当我在 SQL 中执行 where 子句时，这种情况比在任何其他给定语言中的 if 语句中发生的情况更多，但很相似。

Answer 4

鉴于我的建议，我首先会避免进行长时间的 if 测试。我宁愿做这样的事情：

bool fTest1 = A == B ;
bool fTest2 = C ;
bool fTest3 = f(1,2,3) ;
bool fSuccess = ( fTest1 | ftest2 ) & fTest3 ;
if ( fSuccess )
...

否则就是这样的事情：

if (  A == B
&& (  C == D
   || E == F
   )
&&    Z >  Y
) {
    ...
  }
else
  {
    ...
  }

YMMV，当然。

前者更容易调试、测试、记录等。

Answer 5

我通常使用 IDE 格式化程序进行格式化，然后重新排列一下以使其看起来更漂亮。

Answer 6

我在 VSC 工作，最近不仅成功地在嵌套 if 语句中编写了非常长的条件，而且使其可读。只需使用括号和新行即可。它应该自动缩进：

foreach ($panstwa as $key => $value) {
                            if (is_object($value)) {

                                if (
                                    (
                                        ($value->checkbox1 === true) && (is_string($value->panstwoZListy)) && ($value->panstwoZListy !== 'none') && ($value->panstwo === '') && ($value->panstwoZListy !== '')
                                    ) ||
                                    (
                                        (
                                            ($value->checkbox2 === true &&
                                                ($value->checkbox2_1 === true || $value->checkbox2_2 === true || $value->checkbox2_3 === true || $value->checkbox2_4 === true || $value->checkbox2_5 === true || $value->checkbox2_6 === true)
                                            ) ||
                                            ($value->checkbox3 === true &&
                                                ($value->checkbox3_1 === true || $value->checkbox3_2 === true)
                                            ) ||
                                            ($value->checkbox4 === true &&
                                                (
                                                    (
                                                        ($value->checkbox4_1 === true || $value->checkbox4_2 === true || $value->checkbox4_3 === true || $value->checkbox4_4 === true || $value->checkbox4_5 === true || $value->checkbox4_6 === true || $value->checkbox4_7 === true) && ($value->checkbox4_8 === false)
                                                    ) ||
                                                    (
                                                        ($value->checkbox4_1 === false && $value->checkbox4_2 === false && $value->checkbox4_3 === false && $value->checkbox4_4 === false && $value->checkbox4_5 === false && $value->checkbox4_6 === false && $value->checkbox4_7 === false) && ($value->checkbox4_8 === true) && (sprawdzRegexTextInput($value->prawnieUzasadnionyInteres)) && (is_object($value->dokumentacjaOceny) || is_string($value->dokumentacjaOceny))
                                                    )
                                                )
                                            )
                                        ) &&
                                        (is_string($value->panstwo)) && ($value->panstwoZListy === 'none') && ($value->panstwo !== '') && (sprawdzRegexTextInput($value->panstwo)
                                        )
                                    ) &&
                                    ((is_int($value->panstwoid) && is_numeric($value->panstwoid)) || (is_bool($value->panstwoid) && $value->panstwoid === false)) &&
                                    (is_bool($value->zmiana))
                                ) {
                                    echo "ok";
                                    //nie robię nic
                                } else {
                                    $flagaPanstwa = false;
                                }
                            } else {
                                $flagaPanstwa = false;
                            }
                        }