具有相同异常类型的多个catch语句

时间:2013-07-01 21:38:01

标签: java exception types try-catch

我一直在寻找答案,但尚未找到答案。

基本上我试图通过GUI连接到数据库服务器。我的老板希望能够输入所有字段,然后检查它们是否是有效条目,然后如果有任何无效条目,他希望我将文本变为红色,表示该字段无效。我有try语句catch ClassNotFoundException和SQLException。因为需要检查多个字段,所以我尝试使用一组if语句来检查连接信息。这是下面的代码,我希望这是有道理的......

    //The cancel boolean values in this code are used elsewhere to regulate the Threads
    try 
    {
                   //attempt connection here
    } 
    catch(ClassNotFoundException | SQLException e)
    {
        String[] errors = new String[4]; //This will create a String array of the errors it catches
                                         //and will later get called into a method that displays
                                         //the messages in a JOptionPane.showMessageDialog()
        if (e.getMessage().startsWith("The TCP/IP connection to the host"))
        {
            errors[0] = "SQL CONNECTION FAILED: Please check the server URL you entered to make sure it is correct.";
            cancel = true;
            mGUI.serverNameTextField.setForeground(Color.RED);
        }

        if (e.getMessage().startsWith("Login failed for user"))
        {
            errors[1] = "LOGIN FAILED: You do not have sufficient access to the server.";
            cancel = true;

        }
        if (e.getMessage().startsWith("Cannot open database"))
        {
            errors[2] = "SQL CONNECTION FAILED: Please check the database name you entered to make sure it is correct.";
            cancel = true;
            mGUI.dbNameTextField.setForeground(Color.RED);
        }

        mGUI.reportErrors(errors); //Method where it reports the String[] array of errors
                                   //However, the 'errors' parameter only returns one error
                                   //message at a time, which is the problem.

感谢您的帮助!

**** 修改 * ** < EM> * ** 我找到了一个解决方案,所以希望这会对某人有所帮助。我更改了我的if语句,以添加AND参数检查特定的错误代码。您可以通过设置断点并查看调试透视图来查找错误代码,或者您可以执行我所做的操作并设置print语句以查看错误代码。这是打印声明:

    System.out.println(((SQLException) e).getErrorCode());

以下是我的新陈述:

    try 
    {
        //attempt connection here
    } 
    catch(SQLException | ClassNotFoundException e)
    {
        if (e instanceof SQLServerException && ((SQLServerException) e).getErrorCode() == 0)
        {
            //code here
        }
        else{
            //code here
        }
        System.out.println(((SQLException) e).getErrorCode()); //Here is the print statement to see the error code.
        if (e instanceof SQLServerException && ((SQLServerException) e).getErrorCode() == 4060)
        {
            //code here
        }else{
            //code here
        }
        if(cancel != true)
        {
            //code here
        }
    }

3 个答案:

答案 0 :(得分:0)

您可以通过多种方式实现

1具有多个具有共同功能的捕获

}catch (ClassNotFoundException e){
    handleError(e);

}catch (SQLException e){
    handleError(e);
}

其中handleError将异常作为参数。

您似乎没有做任何其他事情,因此您可以将它们组合成一个例外

}catch(Exception e){

}

它会捕获所有内容,但您对错误处理的控制力会大大降低。

答案 1 :(得分:0)

例外的一般原则是它们在最有可能被处理的那一点处理。

你似乎有非常不同的异常,并且可能是在代码中某处抛出的TCP异常与连接到数据库时抛出的SQLException不同(我可能在这里错了,因为我不知道其余的是什么代码看起来像)。因此,不会有一组异常处理程序,每种类型的一个更有意义。同样来自Bryan Roach,文本消歧也不是一个好主意。

try {
...
} catch (java.net.SocketException e) {
 e[0] = "tcp error";
} catch (java.sql.SQLException e) {
 e[1] = "sql exception happened";
}

你的字符串数组似乎有点风险,可能是

ArrayList errors = new ArrayList();
errors.add("Some tcp error");
errors.add("Some db error");

然后为您报错

mGUI.reportErrors(errors.toArray())

将保留您的界面,而不是浪费您必须为数组分配额外的元素并具有空条目。我不确切地知道你的问题是什么,但你提到GUI没有显示多个错误。可能有一个检查停止在数组中的第一个空元素。假设e [2]和e [4]已填充,当e [3]为空时,它可能会在迭代错误时停止。我再次假设,因为我不知道代码是什么样的

答案 2 :(得分:0)

从上面的注释中可以看出,您想要做的就是在单个catch块中捕获的各种Exception类型具有不同的逻辑。如果是这种情况,你可以去:

...
catch(ClassNotFoundException | SQLException e) {
    String[] errors = new String[4];
    if (e instanceof ClassNotFoundException) {
       //do something here
    }
    if (e instanceof SQLException) {
       //do something else here
    }
    ...etc
}

这应该可行,但是其他人建议使用多个catch块可能同样容易:

}catch (ClassNotFoundException e){
    handleError(e);
}catch (SQLException e){
    handleError(e);
}

我并不是指任何冒犯,但代码处理异常的方式可能会引起一些麻烦。例如:

if (e.getMessage().startsWith("Cannot open database")) {

这里的代码依赖于抛出异常以使用相同文本描述的支持库,但是如果切换到另一个JVM版本,使用不同的数据库驱动程序等,此描述可能会更改。可能更安全。异常类型,而不是异常描述。