資源應該被關閉 - 聲納

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