条件を指定し、別のファイルで出力する方法

Question

出力されるファイルの各行にブロック開始（｛）、終了（｝）、開始終了（｛｝）があれば、それぞれ：の後に＋、ー、＊を出力させるプログラムを書きたいです。

やったこと

import java.io.*;
import java.util.ArrayList;

public class AddLineNumber {

    public static void main(String[] args) {
        try {
            FileReader fr = new FileReader("AddLineNumber.java");
            BufferedReader br = new BufferedReader(fr);

            FileWriter fw = new FileWriter("Output.txt");
            BufferedWriter bw = new BufferedWriter(fw);

            String s;
            int lineNumber = 1;
            while ((s = br.readLine()) != null) {
                ArrayList<String> strs = new ArrayList<String>();
                strs.add(s);
                if (strs.contains("{")) {
                    bw.write(String.format("%04d", lineNumber) + ":" + "+" + " " + s + "\r\n");
                }
                if (strs.contains("}")) {
                    bw.write(String.format("%04d", lineNumber) + ":" + "-" + " " + s + "\r\n");
                }
                if (strs.contains("{" + "}")) {
                    bw.write(String.format("%04d", lineNumber) + ":" + "*" + " " + s + "\r\n");
                } else {
                    bw.write(String.format("%04d", lineNumber) + ":" + " " + s + "\r\n");
                }
                lineNumber++;
            }
            br.close();
            bw.close();
        } catch (IOException e) {
            System.out.println(e);
        }
    }
}

リストを作りファイルを一行ずつ読み取りcontainsメソッドでその行に含まれているか判定。含まれていたら書式を指定し出力

出力結果

0001: import java.io.*;
0002: import java.util.ArrayList;
0003: 
0004: public class AddLineNumber {
0005: 
0006:     public static void main(String[] args) {
0007:         try {
0008:             FileReader fr = new FileReader("AddLineNumber.java");
0009:             BufferedReader br = new BufferedReader(fr);
0010: 
0011:             FileWriter fw = new FileWriter("Output.txt");
0012:             BufferedWriter bw = new BufferedWriter(fw);
0013: 
0014:             String s;
0015:             int lineNumber = 1;
0016:             while ((s = br.readLine()) != null) {
0017:                 ArrayList<String> strs = new ArrayList<String>();
0018:                 strs.add(s);
0019:                 if (strs.contains("{")) {
0020:                     bw.write(String.format("%04d", lineNumber) + ":" + "+" + " " + s + "\r\n");
0021:                 }
0022:                 if (strs.contains("}")) {
0023:                     bw.write(String.format("%04d", lineNumber) + ":" + "-" + " " + s + "\r\n");
0024:                 }
0025:                 if (strs.contains("{" + "}")) {
0026:                     bw.write(String.format("%04d", lineNumber) + ":" + "*" + " " + s + "\r\n");
0027:                 } else {
0028:                     bw.write(String.format("%04d", lineNumber) + ":" + " " + s + "\r\n");
0029:                 }
0030:                 lineNumber++;
0031:             }
0032:             br.close();
0033:             bw.close();
0034:         } catch (IOException e) {
0035:             System.out.println(e);
0036:         }
0037:     }
0038:- }
0038: }

　その行に空白が含まれると空白以前までしか判定されない？のか３８行目しか思い通りの結果にならなかった。
どなたか解決方法教えてください。

Answer 1

一行入力に対し処理をして一行出力するだけですので、ArrayList に保存する意味はありません。
RiaFeed さんがご指摘のように、判断と処理を一つずつ書く方法では、複数の条件に一致する可能性がある場合は注意が必要です。
以下のコードでは "{","}" の有無判断を変数 c に 0～3 として一旦保存し、それから文字化しています。
ついでに、クローズする必要のある処理は、可能であれば try-with-resources で書いたほうが close を並べる必要無くかつ確実に close されます。

package teratail_java.q_6hgy0u9a4yg0a0;

import java.io.*;

public class AddLineNumber {
  public static void main(String[] args) {
    try(BufferedReader br = new BufferedReader(new FileReader("AddLineNumber.java"));
        PrintWriter pw = new PrintWriter(new FileOutputStream("Output.txt"));) {
      String line;
      for(int no=1; (line=br.readLine()) != null; no++) {
        int c = (line.contains("{") ? 1 : 0) + (line.contains("}") ? 2 : 0);
        pw.printf("%04d:%c %s\n", no, " +-*".charAt(c), line);
      }
    } catch(IOException e) {
      e.printStackTrace();
    }
  }
}

実行結果(Output.txt)

0001:  package teratail_java.q_6hgy0u9a4yg0a0;
0002:  
0003:  import java.io.*;
0004:  
0005:+ public class AddLineNumber {
0006:+   public static void main(String[] args) {
0007:      try(BufferedReader br = new BufferedReader(new FileReader("AddLineNumber.java"));
0008:+         PrintWriter pw = new PrintWriter(new FileOutputStream("Output.txt"));) {
0009:        String line;
0010:+       for(int no=1; (line=br.readLine()) != null; no++) {
0011:*         int c = (line.contains("{") ? 1 : 0) + (line.contains("}") ? 2 : 0);
0012:          pw.printf("%04d:%c %s\n", no, " +-*".charAt(c), line);
0013:-       }
0014:*     } catch(IOException e) {
0015:        e.printStackTrace();
0016:-     }
0017:-   }
0018:- }

Comments

[tuka]

回答ありがとうございます。なるほど、文字の有無判断を数字としていったん保存する方法は思いつきませんでした。try-with-resources文でリソースを宣言することができることも初めて知りました。参考にさせていただきます！

[jimbe]

条件から N 個の何かを選択する場合、条件から０~N-1 の値を生成できれば、その後の処理に配列(本件では文字が必要なので文字配列としての文字列)が利用できます。
"条件"と"処理"の間にどのような関係があるか、そしてそれを簡潔に表す値はどのようなものがあるかを気にしながらプログラムを作れるようになると、かなりコードが変わると思います。

Answer 2

上手くいかない理由は提示のコードでは文字列に対してcontainsしているのではなく、ArrayList<String>に対してcontainsしているからです。
提示のコードの範囲では読み込んだコードをArrayList<String>に格納する必要性が全くありませんので、
1行読み込んだ文字列が入っているsに対してcontainsすればいいんじゃないでしょうか。

とはいえそもそも+-*を付与するロジックも間違ってまして、このままだと
片方だけある行は+または-のついたコードと何もついてないコードが二重に表示され。
両方ある行は+と-が付いたコードと何もついてないコードが三重に表示され。({}がある行を除く)
{}のある行は+と-と*がついたコードが三重に表示されます。
{ }のあるなしの判定結果をboolean変数で記憶して後でその変数を基に行を出力するコードにしてみては。

Comments

[tuka]

回答ありがとうございます。「{ }のあるなしの判定結果をboolean変数で記憶して後でその変数を基に行を出力するコード」とはどのように書けばよいのでしょうか。boolean変数でboolean b1 = s.contains("{");のようにしてif文で行を出力するコードを書いてみましたが、経験が浅くRiaFeedさんの言う通り、二重三重で表示されてしまいました。

[RiaFeed]

こんな感じですかね

String s;
String addstr;
int lineNumber = 1;
while ((s = br.readLine()) != null) {
boolean left = s.contains("{");
boolean right = s.contains("}");
if (left && right) {
addstr = "*";
}else if (left) {
addstr = "+";
}else if (right) {
addstr = "-";
}else{
addstr = "";
}
bw.write(String.format("%04d", lineNumber) + ":" + addstr + " " + s + "\r\n");

lineNumber++;
}
br.close();
bw.close();

[tuka]

ありがとうございます、うまくいきました！