java - 使用 boolean setter 来查看它们是否执行成功是一种不好的做法吗？

Question

将我的 setter 写为 boolean 值来检查它们是否设置正确是不好的做法吗？

例如，以下代码将设置我的框架是否始终位于顶部，如果成功设置为我决定的内容，则返回 true。这样做是不好的做法，还是应该不带返回类型？

    public boolean setAlwaysOnTop(boolean alwaysOnTop) {
        frame.setAlwaysOnTop(alwaysOnTop);
        return frame.isAlwaysOnTop() == alwaysOnTop;
    }

请记住，在这个示例中，当我想在不尝试设置值时检查该值时，我还有一个 setter/getter :

    public boolean isAlwaysOnTop() {
    return frame.isAlwaysOnTop();
}

提前致谢。如果您想了解更多信息，请随时询问，我会提供。

编辑:

只是我想知道它是否有用，因为我可以做一个

if(setAlwaysOnTop(true)) 
do this 

而不是仅仅使用这样的 void:

  setAlwaysOnTop(true);
  do this 

Answer 1

添加额外条件来检查 API 工作通常不是一个好主意。 frame.setAlwaysOnTop 必须完成其工作。如果您不信任该 API，请使用单元测试来覆盖它。仅当测试失败时，您才应该考虑解决方案 - 找到软件的工作版本，报告错误或修复问题(如果您有权访问代码库)。

只有当您了解某些现有问题并且现在无法修复该问题时，此类额外检查才有意义。在这种情况下，我将从 setAlwaysOnTop 方法引发自定义异常(因为您知道这是一个异常(exception)情况)，记录错误并执行您的理智操作。

public void setAlwaysOnTop(boolean alwaysOnTop) throws UIModificationExcepion {
    frame.setAlwaysOnTop(alwaysOnTop);
    // due to existing bug ... is not updated for all cases
    if (frame.isAlwaysOnTop() != alwaysOnTop) {
        throw new UIModificationException("Unable to change 'always on top' property");
    }
}

客户端代码

try {
    setAlwaysOnTop(true);
} catch (Exception e) {
    log.warn("Could not update always on top", e);
    // do some stuff
}