language-agnostic - 单个 if 语句中谓词的正确平衡在哪里？

Question

我个人对以下代码没有问题

if (Object foo != null && !String.IsNullOrEmpty(foo["bar"]))
{
    // do something
}

因为我觉得下面太啰嗦了

if (Object foo != null)
{
    if (!String.IsNullOrEmpty(foo["bar"]))
    {
        // do something
    }
}

但是，如果说有 5 个谓词并且我必须将文本包装在编辑器中才能一次看到它们，那么我不会站在这个立场上走这么远，是否有一条逻辑“线”可以说明你有多少谓词包含在一个 if 语句中，类似于说方法不应该需要超过 7 个参数

Answer 1

我不认为将 if 语句重写为两个，特别是如果您考虑添加 else你的例子的子句给出了不同的结果。如果我想减少条件中原子命题的数量，我会将一些有意义的分解为它们自己的函数，例如:

if(won_jackpot(obj)) ...