我有一个实用程序方法,它读取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的责任留给调用者是不是一个坏主意?
答案 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);
}