java - 捕获DocumentException是个好主意吗？

我有一个实用程序方法，该方法读取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的责任留给调用者是一个坏主意吗？

最佳答案

不，让呼叫者处理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，而不是出于某种原因而不是存储值。