在Java7之后有一个try(参数){}的方法,然后把流的实例方法写到参数,执行完之后会自动关闭流。
先把正确写法放出来例如:
try(FileReader fr = new FileReader(fileName);
BufferedReader br = new BufferedReader(fr)
){
······
}catch(······){}
或者:
try(BufferedReader br = new BufferedReader(new FileReader(fileName))
){
······
}catch(······){}
然后说说我的修正过程
1.我本来是在try里面实例流,然后关闭:
try{
FileReader fr = new FileReader(fileName);
BufferedReader br = new BufferedReader(fr);
......
br.close();
fr.close();
}catch(){}
Sonar报这种错:Close this "FileReader".
提示详情里我要把close方法放在finally里面(其实这里的提示还有提示Java7之后的写法,就是开头那个正确的写法,但我就是实力眼瞎-_-||)
2.然后我改成这样写的:
FileReader fr = null;
BufferedReader br = null;
try{
fr = new FileReader(fileName);
br = new BufferedReader(fr);
......
}catch(...){}
finally{
br.close();
fr.close();//这里的异常直接抛出去了,因为觉得这里再写try catch太奇怪了。
}
然后Sonar报这种错:Change this "try" to a try-with-resources.
然后一直找到这个
https://stackoverflow.com/questions/17739362/java7-try-with-resources-statement-advantage
最后总结:Sonar提示的修改方法还是挺详细的,仔细看全了就不用折腾这么久了。