Eclipse Juno: nicht zugewiesener schließbarer Wert

Question

Ich frage mich, warum ich diese Warnung mit der neuen Eclipse Juno bekomme, obwohl ich denke, dass ich alles richtig gemacht habe. Können Sie mir bitte sagen, warum ich diese Warnung in dem folgenden Codecode bekomme?

public static boolean copyFile(String fileSource, String fileDestination) 
{ 
    try 
    { 
     // Create channel on the source (the line below generates a warning unassigned closeable value) 
     FileChannel srcChannel = new FileInputStream(fileSource).getChannel(); 

     // Create channel on the destination (the line below generates a warning unassigned closeable value) 
     FileChannel dstChannel = new FileOutputStream(fileDestination).getChannel(); 

     // Copy file contents from source to destination 
     dstChannel.transferFrom(srcChannel, 0, srcChannel.size()); 

     // Close the channels 
     srcChannel.close(); 
     dstChannel.close(); 

     return true; 
    } 
    catch (IOException e) 
    { 
     return false; 
    } 
} 

Answer 1

Wenn Sie auf Java laufen 7, können Sie die neuen Try-mit-Ressourcen verwenden Blöcke wie so, und Ihre Streams automatisch geschlossen:

public static boolean copyFile(String fileSource, String fileDestination) 
{ 
    try(
     FileInputStream srcStream = new FileInputStream(fileSource); 
     FileOutputStream dstStream = new FileOutputStream(fileDestination)) 
    { 
     dstStream.getChannel().transferFrom(srcStream.getChannel(), 0, srcStream.getChannel().size()); 
     return true; 
    } 
    catch (IOException e) 
    { 
     return false; 
    } 
} 

Sie werden nicht explizit schließen müssen die zugrunde liegenden Kanäle. Allerdings, wenn Sie nicht Java 7 verwenden, sollten Sie den Code in einer umständlichen alten Art und Weise, mit schließlich Blöcken schreiben:

public static boolean copyFile(String fileSource, String fileDestination) 
{ 
    FileInputStream srcStream=null; 
    FileOutputStream dstStream=null; 
    try { 
     srcStream = new FileInputStream(fileSource); 
     dstStream = new FileOutputStream(fileDestination) 
     dstStream.getChannel().transferFrom(srcStream.getChannel(), 0, srcStream.getChannel().size()); 
     return true; 
    } 
    catch (IOException e) 
    { 
     return false; 
    } finally { 
     try { srcStream.close(); } catch (Exception e) {} 
     try { dstStream.close(); } catch (Exception e) {} 
    } 
} 

Sehen Sie, wie viel besser die Java 7 Version ist :)

Answer 2

Sie sollten immer in der Nähe in finally denn wenn eine Ausnahme steigen, werden Sie nicht die Ressourcen schließen.

FileChannel srcChannel = null 
try { 
    srcChannel = xxx; 
} finally { 
    if (srcChannel != null) { 
    srcChannel.close(); 
    } 
} 

---

Hinweis: auch wenn Sie im catch Block eine Rückkehr setzen, der finally Block durchgeführt werden wird.

Answer 3

Eclipse warnt Sie über die FileInputStream und FileOutputStream, dass Sie nicht mehr verweisen können.