资源应该被关闭 - 声纳

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; 
} 

问题是声纳仍然抱怨fileIn没有被关闭，虽然它在finally块中的第一资源解决。

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

Answer 1

如果您必须使用Java 7或更高版本，我宁愿您使用try with resources这是在Java 7中引入的新功能。

在Java 7Try-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与否，将supressed (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; 
    }