在 if-then-else 语句中明确声明范围是否有害？ [关闭]

Question

（我搜索了“else if range”，但没有找到任何回答我问题的帖子）。

当使用 if .. elseif .. else 来选择基于某个（互斥）范围内的变量要做什么时，我喜欢明确指定范围：

int num = 55;
String message;

if (num >= 20) {
    // update message variable
    message = "Bigger or equal to than 20";
}
else if (10 <= num && num < 20) {
    // update message variable
}
else if (0 <= num && num < 10) {
   // update message variable
}
else if (num < 0) {
   // update message variable
}

System.out.println(message);

但是我看到的所有教科书和讲义都写了这样一个例子：

int num = 55;
String message;

if (num >= 20) {
    // update message variable
    message = "Bigger or equal to than 20";
}
else if (num >= 10) {
    // update message variable
}
else if (num >= 0) {
   // update message variable
}
else {
   // update message variable
}

System.out.println(message);

我理解他们为什么要确保在最后使用 else （即防止 Java 编译器认为像消息这样的变量可能不会被初始化，如果该变量是一个原始变量），但考虑到所有教科书和讲义显示另一种风格，是否像我喜欢做的那样，明确写出所有其他条件的范围会导致任何问题？

Answer 1

不要重复自己。

使用您的方法，您在代码中重复相同的值两次，这是不必要的。这有几个缺点，包括：

• 通过更改其中一个值而不是另一个值，可以更轻松地在维护期间引入错误。
• 在执行期间，它会执行冗余检查。如果您在else if 中，您已经知道之前的条件是错误的。
• 代码更长，没有添加任何附加信息。阅读时间更长，检查时间更长。

后期维护可能会更改其中一个值，但不会更改另一个值，从而导致一些令人惊讶的行为。

if ( num >= 30 ) {
    // update message variable
}
else if (10 <= num && num < 20 ) { // Whoops! 
    // update message variable
}
...

您可以通过为每个值定义一个常量来避免这种情况。但是您的代码仍然指定了冗余检查。

final int HIGH = 20;
final int MEDIUM = 10;
final int LOW = 0;

if (num >= HIGH ) {
   // update message variable
}
else if (MEDIUM <= num && num < HIGH ) { // We already know num<HIGH
    // update message variable
}
...

Answer 2

如果我正在构建一些代码，这些代码将在某种交通工具上运行以成功降落在另一个星球上，那么我总是会为清晰而丰满。请记住，一行代码将被编写一次，但可能会被读取数百次。在凌晨 4 点调试会话期间，您的第一个 sn-p 将更容易理解。

您的第一种方法确实更清晰，如果错误的重构器重新排序条件检查，也不会引入错误。

不过，我总是会以 } else { 结束，如果程序控制不应该达到这一点，理想情况下会带有某种断言。