最新要闻

广告

手机

iphone11大小尺寸是多少？苹果iPhone11和iPhone13的区别是什么？

iphone11大小尺寸是多少？苹果iPhone11和iPhone13的区别是什么？

 警方通报辅警执法直播中被撞飞：犯罪嫌疑人已投案

警方通报辅警执法直播中被撞飞：犯罪嫌疑人已投案

家电

  1. 首页
  2. 移动通信
  3. 内容

每日报道：sonar代码扫描bug:Use try-with-resources or close this "FileInputStream" in a "finally" clause.

来源：博客园


(相关资料图)

下面代码

/** * 读取文件到byte数组 * * @param tradeFile * @return */public static byte[] file2byte(File tradeFile) {    try {        FileInputStream fis = new FileInputStream(tradeFile);        ByteArrayOutputStream bos = new ByteArrayOutputStream();        byte[] b = new byte[1024];        int n;        while ((n = fis.read(b)) != -1) {            bos.write(b, 0, n);        }        fis.close();        bos.close();        byte[] buffer = bos.toByteArray();        return buffer;    } catch (Exception e) {        e.printStackTrace();    }    return null;}

上面代码虽然调用了流的close方法，但可能会因中间程序异常致使文件流未关闭。如果应用程序运行过程中这个方法频繁被访问，就有可能导致频繁GC，甚至OOM错误。

SonarQube在扫描到这段代码时，会提示关闭文件流：Use try-with-resources or close this "FileInputStream" in a "finally" clause.

我们用try-with-resources语法糖的方式来改进这段代码，try-with-resouces相当于显式在finally块里调用close方法（从build后的.class文件可以看出来）。为了保证资源及时关闭，我们往往要在finally里写一堆代码，使用这个java语法糖可以让我们专注应用开发，无需关注资源释放问题。如下：

public static byte[] file2byte(File tradeFile) {    try (FileInputStream fis = new FileInputStream(tradeFile);         ByteArrayOutputStream bos = new ByteArrayOutputStream()){        byte[] b = new byte[1024];        int n;        while ((n = fis.read(b)) != -1) {            bos.write(b, 0, n);        }        byte[] buffer = bos.toByteArray();        return buffer;    } catch (Exception e) {        e.printStackTrace();    }    return null;}

关键词： 应用程序 中间程序 可以看出