ファイルを作成してそれに書き込む

Question

「test.txt」という名前のファイルを作成した後、入力をファイルに書き込むためにユーザーからの入力を受けました。すべてうまく動作します。プログラムは何のエラーも全く表示しません。ファイルが作成され、プログラムはユーザーからの入力を受け取りますが、ファイルの内容を確認すると空になりました。私のコードで何が間違っているのか誰でも分かりますか？コードは以下の通りです。

package InputOutput; 

import java.io.*; 

public class CharacterFileReaderAndFileWriter{ 

private BufferedReader br = null; 
private BufferedWriter bw = null; 
private PrintWriter pw = new PrintWriter(System.out, true); 

public File createFile() throws IOException{ 
    File f = new File("test.txt"); 
    return f; 
} 

public void writeToFile() throws IOException{ 
    try{ 
     bw = new BufferedWriter(new FileWriter(createFile())); 
    } 
    catch(FileNotFoundException ex){ 
     ex.printStackTrace(); 
    } 

    //take input from the console (user) 
    br = new BufferedReader(new InputStreamReader(System.in)); 
    String s; 

    pw.println("Please enter something"); 
    pw.println("To stop the program, enter 'stop'"); 

    do{ 
     s = br.readLine(); 
     if(s.compareTo("stop")==0) 
      break; 
     s+= "\r\n";//adding an new line to the string s   
     bw.write(s);    
    } 
    while(s.compareTo("stop")!=0); 

    br.close(); 
    bw.close(); 

} 

public static void main(String[] args) throws IOException{ 

    CharacterFileReaderAndFileWriter cfr = new CharacterFileReaderAndFileWriter(); 
    cfr.writeToFile(); 
} 

}

Answer 1

ほとんどのサンプルプログラムは、あなたが.close()前に、あなたのBufferedWriterに.flush()を呼び出す必要があることを示しています。これは必須ではありません。.close()は.flush()に自動的に電話する必要がありますが、怪我をすることはありません。また、すべてのStream/Writerオブジェクトを.close()メソッドと逆の順に呼び出す必要があります。正しく書かれたクラスは、ラップするすべてのオブジェクトに対して.close()を呼び出す必要がありますが、とにかくそれをすることはありません。後であなたをキャッチする可能性がある

他のもの：

if(s.compareTo("stop")==0) 

は

if ("stop".equalsIgnoreCase(s)) 

それは、より効率的であるsにNullPointerExceptionの可能性を排除し、停止のいずれかの場合を扱うする必要があります最も重要なのは、使用するよりも慣用的なJavaです。.compareTo()

s+= "\r\n";//adding an new line to the string s   
    bw.write(s);   

は

bw.write(System.getProperty("line.separator"));   
    bw.write(s);   

s+=を収集する必要がある中間オブジェクトとゴミを作成しなければなりません。ハードコーディングの行末も悪いです。

Answer 2

出力ストリームを閉じる必要があります。 file.close（）;