资源应该关闭 - 声纳

Question

我有以下代码：

public static byte[] readSomeFile(String filePath) {
 byte[] buffer = new byte[FILE_SIZE];
 FileInputStream fileIn = null;
 BufferedInputStream buffIn = null;
 DataInputStream inData = null;

 int size = 0;
 byte[] someArray= null;
 try {
  fileIn = new FileInputStream(filePath);
  buffIn = new BufferedInputStream(fileIn);
  inData = new DataInputStream(buffIn);
  size = inData.read(buffer, 0, FILE_SIZE);
  someArray= new byte[size];
  System.arraycopy(buffer, 0, someArray, 0, size);
 } catch (IOException e) {
  //log(Log.ERROR,"IO ERROR: " + e.toString());
 } finally {
  try {
   if (null != fileIn) {
    fileIn.close();
   }
   if (null != buffIn) {
    buffIn.close();
   }
   if (null != inData) {
    inData.close();
   }
  } catch (Exception exFinally) {
   // some stuff
   someArray= null;
  }
 }
 return someArray;
}

问题是 Sonar 仍然在抱怨 fileIn 没有被关闭，尽管它是 finally 块中处理的第一个资源。

Sonar 在这种情况下如何工作？以及如何解决资源应该关闭规则？

Answer 1

如果你必须使用Java 7及以上，我更喜欢你使用try with resources，它是在Java 7新功能中引入的。

Java 7 中的Try-with-resources 是一种新的exception 处理机制，可以更轻松地正确关闭try-catch block. 中使用的资源

关于你的代码：

finally {
  try {
   if (null != fileIn) {
    fileIn.close();
   }
   if (null != buffIn) {
    buffIn.close();
   }
   if (null != inData) {
    inData.close();
   }
  } catch (Exception exFinally) {
   // some stuff
   someArray= null;
  }
 }

你注意到丑陋的双重尝试了吗？

但是，如果你使用了try with resources，close()会被自动调用，如果它throws和Exception，它会被抑制(as specified in the Java Language Specification 14.20.3)。你的情况也是如此。希望对你有帮助。

因此，您的代码将如下所示：

public static byte[] readSomeFile(String filePath) {
         byte[] buffer = new byte[FILE_SIZE];
         int size = 0;
         byte[] someArray= null;
         try (FileInputStream fileIn = new FileInputStream(filePath);
                 BufferedInputStream buffIn = new BufferedInputStream(fileIn);
                 DataInputStream inData = new DataInputStream(buffIn);) {

          size = inData.read(buffer, 0, FILE_SIZE);
          someArray= new byte[size];
          System.arraycopy(buffer, 0, someArray, 0, size);
         } catch (IOException e) {
          //log(Log.ERROR,"IO ERROR: " + e.toString());
         } 
         return someArray;
    }