エラーが出て一度入力すると終了してしまう。

Question

前提・実現したいこと

×が入力されるまで入力を続け、×が入力されたときにそれまでに入力した文字をファイルに上書きしたい。

実行結果

何か入力してください.
234
ファイルに書き込めません
何か入力してください.

該当のソースコード

package ex07;

import java.io.BufferedReader;
import java.io.File;
import java.io.FileWriter;
import java.io.IOException;
import java.io.InputStreamReader;

public class LooplnputWrite{

public static void main(String args[]){
      while(true) {
    try{
      File file = new File("C://java//test01.txt");

      if (checkBeforeWritefile(file)){
        @SuppressWarnings("resource")
		FileWriter filewriter = new FileWriter(file);
          InputStreamReader is = new InputStreamReader(System.in);
        BufferedReader br = new BufferedReader(is);


          System.out.println("何か入力してください.");

          String str = br.readLine();

          filewriter.write(str+"\r\n");


        if (str.equals("×") ){

            System.out.println("終了しました。");
            filewriter.close();
            break;
      }else{
        System.out.println("ファイルに書き込めません");
      }
      }
    }catch(IOException e){
      System.out.println(e);
    }
  }
  }


  private static boolean checkBeforeWritefile(File file){
    if (file.exists()){
      if (file.isFile() && file.canWrite()){
        return true;
      }
    }

    return false;
  }
}

Comments

[m.ts10806]

どこで「×」の入力を受け付けているのでしょうか。

[swordone]

前提が覆る編集をしないでください。この編集により29行目が空白の行になり、少なくとも提示されているエラーにはならないはずです。

[dodox86]

質問の編集によって、前になされた回答が今の質問内容とつながらなくなってしまいます。質問者と回答者以外の、後で読んだ閲覧者の方々にとっても意味不明なものになってしまいますので、そのようなことはお控えください。＞質問者さん

Answer 1

Exception in thread "main" java.lang.NullPointerException
at ex07.LooplnputWrite.main(LooplnputWrite.java:29)

と言われているのですから、そこら辺りのコードを見ると、それはもう、落ちろと言わんばかりではないでしょうか。

...
        filewriter.close();
//!!!!？？？
        Object inputValue = null;
        if (inputValue.equals("×") ){

            System.out.println("終了しました。");
...

Answer 2

やりたかったことはこういうことでしょうか？

import java.io.BufferedReader;
import java.io.File;
import java.io.FileWriter;
import java.io.IOException;
import java.io.InputStreamReader;

public class Main {
    public static void main(String args[]) throws IOException {
        File file = new File("C://java//test01.txt");

        if (checkBeforeWritefile(file)) {
            FileWriter filewriter = new FileWriter(file);
            InputStreamReader is = new InputStreamReader(System.in);
            BufferedReader br = new BufferedReader(is);

            while (true) {
                System.out.println("何か入力してください. ×で終了.");
                String str = br.readLine();
                if (str.equals("×")) {
                    System.out.println("終了しました。");
                    break;
                }

                filewriter.write(str + "\r\n");
            }

            filewriter.close();
        } else {
            System.out.println("ファイルに書き込めません");
        }
    }

    private static boolean checkBeforeWritefile(File file) {
        if (file.exists()) {
            if (file.isFile() && file.canWrite()) {
                return true;
            }
        }

        return false;
    }
}

---

ループは最小限の範囲になるようにすると、トラブルにくくなります。

Comments

[nubba]

ありがとうございます。

Answer 3

        if (str.equals("×") ){

            System.out.println("終了しました。");
            filewriter.close();
            break;
      }else{
        System.out.println("ファイルに書き込めません");
      }

"×"なら終了、それ以外なら「ファイルに書き込めません」なんだから当たり前です。
もっとも、書き込む処理自体はこの前にあるのでほぼ無意味なのですが。