捕获DocumentException是个好主意吗?

时间:2014-07-11 06:39:25

标签: java

我有一个实用程序方法,它读取xml文件并转换为字符串,如下所示:

public static String readFile(String xmlFileName) throws IOException, DocumentException{
        String xmlMsg = null;
        Resource resource = null;
        InputStream inputStream = null;
        try{
            resource = new ClassPathResource(xmlFileName);
            inputStream = resource.getInputStream();
            SAXReader reader = new SAXReader();
            Document doc = reader.read( inputStream );
            xmlMsg = doc.asXML();
        }finally{
            if(inputStream != null){
                inputStream.close();
            }
        }
        return xmlMsg;
    }

如果我在上面的代码中捕获DocumentException并重新抛出它,这是一个坏主意:

public static String readFile(String xmlFileName) throws IOException, DocumentException{
        String xmlMsg = null;
        Resource resource = null;
        InputStream inputStream = null;
        try{
            resource = new ClassPathResource(xmlFileName);
            inputStream = resource.getInputStream();
            SAXReader reader = new SAXReader();
            Document doc = reader.read( inputStream );
            xmlMsg = doc.asXML();
        }catch (DocumentException e){
           throw new DocumentException("some message");
        }finally{
            if(inputStream != null){
                inputStream.close();
            }
        }
        return xmlMsg;
    }

那么,将处理DocumentException的责任留给调用者是不是一个坏主意?

2 个答案:

答案 0 :(得分:5)

不,让来电者处理Exception很好 - throw early catch late

我遇到的问题是:

}catch (DocumentException e){
    throw new DocumentException("some message");

为什么你会catch (DocumentException e)然后抛出删除所有有用信息的新实例?你可以在第一时间不抓住它,让它渗透到可以处理它的人身上。

此外,使用Java 7 try-with-resources而不是finally。所以,你的代码应该是:

public static String readFile(String xmlFileName) throws IOException, DocumentException {
    try (final InputStream is = new ClassPathResource(xmlFileName).getInputStream()) {
        final SAXReader reader = new SAXReader();
        final Document doc = reader.read(inputStream);
        return doc.asXML();
    }
}

我已经删除了声明为null然后重新分配的变量,我讨厌这种做法,许多其他Java开发人员也是如此 - 摆脱这种习惯。在需要时声明 立即分配。在垃圾收集语言中,最小范围的原则非常重要。

我还将其直接更改为return,而不是出于某种原因存储该值。

答案 1 :(得分:0)

    catch (DocumentException e){ // here you are catching exception
       throw new DocumentException("some message");// now you are changing
       // exception message 
    }

这是一个不好的做法,你必须将最初抛出的异常抛给上层,然后你可以在那里处理Exception

这种方式更好

   catch (DocumentException e){
       throw new DocumentException("some message",e);
   }