java - 返回令人困惑的 boolean 语句 - 重构

Question

我写了一个方法:

private boolean checkIfCanBeSent(Product product, Type type, TypeToSend typeToSend) {
    boolean isDisabled = isStatusDisabled(product, type);
    boolean isManual = isManual(typeToSend);

    return !(isDisabled && !isManual);
}

它工作正常，但是在同行评审期间得到评论说这个返回声明令人困惑。如何改进它并使其更具可读性？

Answer 1

三点建议。

• 应用德摩根定律给出等效的陈述。
• “启用”比“未禁用”更容易理解。
• 不要为变量指定与方法相同的名称。

所以

boolean isEnabled = ! isStatusDisabled(product, type);
boolean isManualType = isManual(typeToSend);

return isEnabled || isManualType;