エラーを回避しながらファイルを開いたり、書き込みしたり、保存したりするにはどうすればよいですか？

Question

私のプログラムでファイルを消去する必要があります。ここ

public static void erase(String string) { 
    FileWriter fw = null; 
    try { 
     fw = new FileWriter(string); 
     fw.write(new String()); 
    } catch (IOException ie) { 
     e.printStackTrace(); 
    } finally { 
     fw.flush(); 
     fw.close(); 
    } 
} 

いくつかの問題：

• fwが正しくファイルが欠落し、何らかの理由（初期設定していない場合は、無効persmissions私のソリューションはそうのようなことをするだろうerase()方法を持っていることでした、など）、ブロックfinallyでブロックしようとすると、NullPointerExceptionが発生します。

• finallyブロックがない場合は、上記の理由でNullPointerExceptionがスローされている可能性があります。

• ファイルをtryブロック内で閉じると、ファイルが正しく開かれてもリソースがリークする可能性がありますが、正しく書き込まれません。

他にどのような問題がありますか、どのようにこの方法を強化できますか？

Answer 1

あなただけのif文で、最終的な機能をラップすることができます。これは、ファイルがこれまでに開かれた場合、それはクローズされるようになります

if(fw != null){ 
    fw.close(); 
} 

。それが最初に開かれなかったなら、それは何もしません。それはあなたが望むものです。

また、ポスティングのようなものかどうかはわかりませんが、一般的にはcatchtブロックでスタックトレースを出力して（例外を「飲み込む」）続けることはお勧めできません。例外がスローされるようにする必要があります。これはバグを隠して、それらを非常に難しくする可能性があるからです。

編集：下記のコメントを参照してください。

Answer 2

メインブロックにflush()を含めて、キャッチ内にclose()のみを含めます。そして、閉じる前にnullをチェック：

finally { 
    if(fw!=null) { fw.close(); } 
    } 

をメインブロックでフラッシュを使用すると、またcloseをキャッチし、エラーを記録したり、無視/試すことができます。

finally { 
    if(fw!=null) { 
     try { fw.close(); } catch(Throwable thr) { log.printError("Close failed: "+thr); thr.printStackTrace(); } 
     } 
    } 

または（一般的にはお勧めしません） ：

finally { 
    try { fw.close(); } catch(Throwable thr) {;} 
    } 

---

EDIT

I/Oを処理するための最高一般的なJavaのイディオムは、IMO、次のとおりです。

FileWriter fw=null; 
try { 
    fw=new FileWriter(string); 
    fw.write(new String()); 
    fw.close(); 
    fw=null; 
    } 
catch(IOException ie) { 
    // do something real here to handle the exception, or don't catch it at all. 
    } 
finally { 
    if(fw!=null) { 
     try { fw.close(); } catch(Throwable thr) { thr.printStackTrace(); } // now we're really out of options 
     } 
    }  

これはcatchはclose()自体によってスローされた例外をキャッチして対処することを可能にする重要な効果を持っています。（あなたが何らかの方法で例外を処理できるかどうcatch句は唯一の存在であるべきである。ませキャッチをし、無視し、あなたは、単にキャッチし、トレースするべきではありません一般）

Answer 3

最高に

FileWriter fw = new FileWriter(string); 
try { 
    fw.write(new String()); 
    fw.flush(); 
} catch (IOException ie) { 
    ie.printStackTrace(); 
} finally { 
    fw.close(); 
} 

説明：

私の知る限りでは、これはあなたのコードを書くための正しい慣用的な方法であります
• new FileWriter()が例外をスローすると、何もクリーンアップする必要はありません。このメソッドはfinallyを実行せずに終了します。
• finallyではなくtryにfw.flush()を記載してください。 2つの理由があります：書き込みが失敗した場合、フラッシュを邪魔する必要はありません。また、にflush()を入れて例外をスローすると、close()はスキップされます。