Sonar检查出现 流关闭和try catch重用 的问题

在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提示的修改方法还是挺详细的,仔细看全了就不用折腾这么久了。

  • 2
    点赞
  • 9
    收藏
    觉得还不错? 一键收藏
  • 2
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论 2
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值