language-agnostic - 您是否应该总是为“不可能发生”的其他情况编写代码？

Question

拿一些像

if (person.IsMale()) {
    doGuyStuff();
} else {
    doGirlStuff();
}

是否应该将其编写为显式检查 person.isFemale()，然后添加一个引发异常的新else？也许您正在检查枚举中的值，或类似的东西。您认为没有人会为枚举添加新元素，但是谁知道呢？ “永远不会发生”听起来像是著名的遗言。

Answer 1

我认为您已经回答了自己的问题。如果您知道您永远不会看到其他值：

bool isSet = ...
if (isSet)
{
    return foo;
}
return bar;

...然后就不用打扰了。但是，如果有可能存在两个以上的可能值，那就掩盖自己。您（或两年后的维护程序员）将不胜感激。