FileInputStreamの誤差はSonarQube

Question

に私は、コード

問題がある以下で私が唯一API LEVEL 19+上記のためであるtry-with-resourcesを使用するように示唆sonarqubeをSonarQube重要な問題を取得していますし、私はminimumSDKとしてLEVEL 16を標的としています。

いずれにしても、私はすでに下のコードで行っているブロック内の閉じるFileInputStreamのブロックを示しています。

protected void copyFile(File sourceFile, File destFile) throws IOException { 
    if (!sourceFile.exists()) { 
     return; 
    } 

    FileChannel source = null; 
    FileChannel destination = null; 
    try { 
     source = new FileInputStream(sourceFile).getChannel(); 
     destination = new FileOutputStream(destFile).getChannel(); 
     if (source != null) { 
      destination.transferFrom(source, 0, source.size()); 
     } 

    } finally { 
     if (source != null && destination != null) { 

      source.close(); 
      destination.close(); 
     } 
    } 

} 

でもアンドロイドスタジオは赤いラインのエラーを与え、finallyブロックにtry-with-resources OR近いストリームを使用するために同じことを示唆しています。

UPDATE

Answer 1

両方がNULLの場合は、2つのチャネルのみを閉じている、とあなたはストリームを閉じていないからです。以下のようなブロックを分離するために近い/チェックを抽出してみてください。sourceまたはdestinationストリームの1つがnullないときのシナリオがありますので

protected void copyFile(File sourceFile, File destFile) throws IOException { 
    if (!sourceFile.exists()) { 
     return; 
    } 

    FileInputStream fis = null; 
    FileChannel source = null; 
    FileOutputStream fout = null; 
    FileChannel destination = null; 
    try { 
     fis = new FileInputStream(sourceFile); 
     source = fis.getChannel(); 
     fout = new FileOutputStream(destFile); 
     destination = fout.getChannel(); 
     if (source != null) { 
      destination.transferFrom(source, 0, source.size()); 
     } 

    } finally { 
     if (fis != null) { 
      try { 
       fis.close(); 
      } catch (IOException e) { 
       System.out.println("Failed to close source input stream."); 
      } 
     } 
     if (source != null) { 
      try { 
       source.close(); 
      } catch (IOException e) { 
       System.out.println("Failed to close source channel."); 
      } 
     } 
     if (fout != null) { 
      try { 
       fout.close(); 
      } catch (IOException e) { 
       System.out.println("Failed to close destination output stream."); 
      } 
     } 
     if (destination != null) { 
      try { 
       destination.close(); 
      } catch (IOException e) { 
       System.out.println("Failed to close destination channel."); 
      } 
     } 
    } 
} 

Answer 2

SonarQubeは重要な問題を与えています。また、FileInputStreamオブジェクトを閉じず、チャンネルを閉じるだけではありません。その場合は、それを閉じることはありませんFileChannel。

1：@fejdが提案されているようのFileChannelのスプリット近いあなたが次の操作を行うことができ、これを避けるために

if (source != null && destination != null) {...} 

：この行を参照してください。

2：ネストされたtry-catchブロック、各finallyブロックはストリームを閉じます。

一般的な注意点として、より複雑なシナリオでは2を使用できますが、指定されたものについては1で十分です。