在禁用的 GUI 选项中添加额外的检查是一个好习惯吗？

Question

我指的是以下内容：

void setup_gui()
{
    if (some_condition)
        some_button.disable();

    ...
}

void some_button_click()
{
    // Is this a good practice?
    if (some_condition)
        return;

   ...
}

添加该检查可确保程序不会运行该操作，但它也可以被视为隐藏错误（some_button_click() 必须根本没有运行）。

那么，您对此有何看法？这是安全的编码实践还是隐藏错误？

Answer 1

防御性编程与防御性驾驶一样合理。

从单独的关注点角度来思考这一点可能会有所帮助。一个问题是演示。另一个可能是一组业务规则。在两个地方进行相同的检查是合理的。

您希望在表示层中进行检查以与用户进行通信。

您可能还想在表示层下方进行检查：

• 以防止表示层中当前和未来的错误。
• 以防表示层下面的代码在其他地方重复使用。
• [来自 mvds 的评论] 如果由于启用或禁用控件而导致条件可能发生变化。

编辑： David Heffernan 的下面的 DRY 问题可以通过仅定义一次条件并在其他地方访问它来轻松解决。

void setup_gui()
{
    some_button.setEnabled( context.isThisActionAvailable() );

    ...
}

void some_button_click()
{
    if ( context.isThisActionAvailable() )
        return;

   ...
}

Answer 2

我不采用这种腰带和背带的方法。问题是你双重使用 some_condition 违反了 DRY 原则。在一个地方更改此设置非常容易，而在另一个地方则不然。

当然， some_condition 在这个虚构的示例中非常简单，但实际上它通常要复杂得多。

如果您不能相信 GUI 框架会在您请求阻止操作时阻止这些操作，那么您需要修复该框架。

Answer 3

它可以被认为是安全编码，也可以被认为是隐藏错误。现在也许您应该重新评估您的程序逻辑。如果可能的话，代码冗余总是最好避免的。如果您的程序可以通过确保程序的实际逻辑正确且按预期工作而无需检查两次的方式构建，那就更好了。

当然，如果你很着急，这是一个快速修复并且它有效，但它在程序的其他地方散发着糟糕的设计和/或逻辑的味道。

Answer 4

some_button_click() 中的静默返回可能隐藏了一个错误。我要么根本不做检查，要么大声而猛烈地撞车。