为什么空的els-if语句是不好的风格，我应该如何重写它？

Question

if (! seesWater(LEFT))
{
    turn(LEFT);
}
else if (! seesWater(AHEAD));
else if (! seesWater(RIGHT))
{
    turn(RIGHT);
}
else
{
    turn180();
}

else-if有但什么也不做的原因是因为我希望代码执行的优先级：

如果（！seesWater（header）），那么我根本不希望其他条件运行，因为它们无关紧要。

Answer 1

谁说是“歪风邪气”？

要问的相关问题是，这比替代方案更清楚吗？在你的具体情况下，我会说是的。代码清楚地表达了4个选项中的一个选择，其中一个是“什么都不做”。

我所做的唯一改变就是用一对空的大括号替换那个无关紧要的分号，可能还有一个注释来说明这不是一个错误。

if (! seesWater(LEFT)) {
    turn(LEFT);
}
else if (! seesWater(AHEAD)) {
    // nothing required
}
else if (! seesWater(RIGHT)) {
    turn(RIGHT);
}
else {
    turn180();
}