c# - 在方法返回的 If-Else 语句中，如果可以隐式遵循 Else，是否应该显式声明 Else？

Question

我有一个方法可以检查某些事情并根据这些检查返回一个 bool 值。它涉及一个单独的分支 If 部分，该部分按顺序检查大约 5 个条件。如果这些条件中的任何一个返回真，则该方法将返回真；。如果没有任何条件返回真，则该方法将返回假；。由于 If 部分之后的代码仅在所有条件都不为真时才会运行，因此该代码在逻辑上与包含实际的 Else 语句相同。

那么对于这种情况，实际写在 Else 语句中是更好的主意吗？

编辑

事实证明，我需要有关哪些条件实际上触发了其中一些情况的“真”的信息，因此我更改了返回 int 的方法，其中 -1 表示“假”情况。逻辑仍然存在，如果所有条件都不为真，它将返回 -1。所以，我不再有 return (cond1 || cond2 || cond3 || cond4 || cond5); 的可压缩选项，但我也感谢大家的建议，因为我确实没有想到关于它(主要是因为 cond3 是一个非常复杂的条件，涉及检查两对 DateTime 对象的中点的交集，所以它看起来很难看)。虽然方法的性质变了，但这个问题的性质没有变，所有答案仍然基本适用......

目前的代码，解释它并删除所有定义 cond1 到 cond5 的无关代码......

if (cond1) { return 1; }
else if (cond2) { return 2; }
else if (cond3) { return 3; }
else if (cond4) { return 4; }
else if (cond5) { return 5; }

Answer 1

这真的是一个风格问题，你(和你的同事)发现什么更清晰。一般来说，我个人认为结构:

if( a )
   someResult = doSomething();
else if( b )
   someResult = doSomethingElse();
else
   someResult = doSomethingAnyways();

return someResult;

比:

更清晰

if( a )
    return doSomething();
if( b )
    return doSomethingElse();
return doSomethingAnyways();