操作成功/失败的返回布尔值是Java中的一个好习惯吗？

Question

我有一些功能适用于数据库。 我在这里设置了一个 try/catch 用于错误处理，并显示一条消息，它工作正常。

现在调用这个删除函数的类需要知道是否有错误。在我的情况下：如果成功则刷新 GUI，如果失败则什么都不做（因为已经显示了一个消息消息对话框）。

我想出了一个在这个函数中返回布尔值的想法。

public static Boolean delete(int id){

    String id2 = Integer.toString(id);
    try {
        String sql = 
                "DELETE FROM toDoItem " +
                "WHERE id = ?;";
        String[] values = {id2};
        SQLiteConnection.start();
        SQLiteConnection.updateWithPara(sql, values);
    } catch (SQLException e) {
        Main.getGui().alert("Fail when doing delete in DataBase.");
        System.out.println("Exception : "+ e.getMessage());
        return false;
    }
    return true;
}

不知道是好是坏，请告诉。

---

编辑：

这里是我如何使用的更多细节：

假设上面的代码在 A 类中， B类：

public boolean deleteItem(int id){
    int i = index.get(id);
    if(theList[i].delete()){  //<---- here is the function from Class A
        theList[i] = null;
        index.remove(id);
        retutn true;
    }
    retutn false; 
}

我需要在不止一个类中传递布尔值，我不知道这是否可以更好地通过......

C 类：

public void toDoList_deleteItem(){
    MyButton btn = (MyButton)source;
    int id = btn.getRefId();
    List toDoList = Main.getToDoList();
    if(toDoList.deleteItem(id)){  //<-------function in Class B
        Main.getGui().refresh();
    }
}

---

编辑 2：

我注意到这个问题更有可能是在问“我应该如何处理影响 GUI 层的数据库层的异常？”......类似的事情。如果应该编辑问题标题，请纠正我。

Answer 1

当您预计不会失败时，应使用异常。

在您的情况下，如果抛出 SQLException 并且不影响您的程序对您来说没问题，则可以返回布尔值。

如果导致删除失败的 SQLExcetion 可能导致应用程序的另一部分出现问题，则最好抛出异常。

编辑：

根据您的编辑，当发生错误时，您似乎正在进行一些维护和清理工作。在这种情况下，我建议使用 Exceptions 比使用布尔值来控制执行更好。

Answer 2

您似乎正在返回 boolean 状态以指示发生了异常情况。一般来说，这不是一个好的做法，原因有两个：

• 它鼓励一种容易出错的异常处理方式 - 很容易错过状态检查，从而导致错误被忽略
• 它限制了 API 报告错误的能力 - 单个通过/失败位并不总是足够的，可能需要传递有关错误的更多信息。

更好的方法是定义一个特定于应用程序的异常，并在您的 API 中使用它。这迫使您的 API 用户注意可能发生的异常情况，同时让您传递尽可能多（或尽可能少）您认为必要的附加信息。同时，您的代码不会在每次 API 调用时被 if (!delete(id)) { /* handle error */ } 代码污染，从而缩小代码库并提高其可读性。

你能告诉我更多关于“定义一个特定于应用程序的异常”的信息吗，或者请展示一些代码示例？

我会这样做：

public class DataAccessException extends Exception {
    ... // Define getters/setters for passing more info about the problem
}
...
public static void delete(int id) throws DataAccessException {
    try {
        ... // Do something that may lead to SQLException
    } catch (SQLException se) {
        // Do additional logging etc., then
        throw new DataAccessException("Error deleting "+id, se);
    }
}

注意：通常为自定义异常提供四个镜像Exception 类的构造函数以允许异常链接。构造函数描述为here。

Answer 3

只要你不希望调用者知道发生了什么，只是它失败了（失败是它预期行为的一部分）你应该没问题。

话虽如此，我注意到了这一点：Main.getGui().alert("Fail when doing delete in DataBase.");。

您似乎正在从其他地方访问 GUI 层。如果您决定对应用程序进行多线程处理，这可能会导致问题。此外，您的图层不相交通常被认为是一种很好的做法。

Answer 4

不要返回Boolean，而是返回boolean。由于这不是 异常/错误 条件，所以没问题。

Answer 5

这个问题主要是基于意见的。就我个人而言，我不希望此时捕获异常。

根据delete() 的caller 应该做什么，您可能需要其他结果。所以你最好添加一个 throw 语句，让调用方法来决定错误是否严重 - 或者它是否可以继续。

仅仅true 和false 不足以让调用者做出正确决定。他不知道删除是否由于数据库错误、外键约束或其他原因而失败。

让异常在调用堆栈中冒泡将为调用者提供正在发生的确切错误，增加以正确方式处理错误的机会，或者仅显示自定义错误消息以帮助用户采取正确的措施。