javaでの表の作成

Question

前提・実現したいこと

テキストファイルの情報を読み込みそれを基に、表を作成。

・テキストファイル
田中ゆうき,国語,80
佐藤かつひこ,数学,50

発生している問題・エラーメッセージ

エラーメッセージ

該当のソースコード

class Sample {
	
	
	
	public static void main(String[] args)throws IOException {
		
		File file = new File("Sample1.txt");
       	ArrayList<String[]> lineList = new ArrayList<>(); 
		String az= "名前,国語,数学";
		lineList.add(az.split(","));
    	
        try(BufferedReader br= Files.newBufferedReader(file.toPath(), StandardCharsets.UTF_8)){
            String line;
            while ((line = br.readLine()) != null) {
            	lineList.add(line.split(","));
            }
        } catch (IOException e) {
            e.printStackTrace();
        }
		
		String[][] data = lineList.toArray(new String[lineList.size()][]);
		
		
	}
	
	
	 
}

試したこと

ここに問題に対して試したことを記載してください。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Comments

[BeatStar]

Markdownの```がズレています。ご自身の質問を読んでみてください。

[taka0145]

失礼しました。
訂正させていただきます。

[BeatStar]

後、タイトルも変えた方がいいです。内容と一致していません。
確かに表を作ろうとしているようですが、『表の作り方』というより、
『どのようにテーブルに合わせてハイフン(-)を連続させるか』のように見えます。

内容とタイトルが乖離(かいり)していると回答者は戸惑いますよ。

[taka0145]

分かりにくいタイトルで申し訳ありません。
訂正しました。

[jimbe]

ご質問とコードの出来に差があるように感じられますが、このコードはご自身で作ったものでしょうか。

[taka0145]

ネットに載っていたコードをアレンジしたもので、一から私が作ったものではないです。

[dodox86]

結構これは難しい（面倒くさい）問題のはずなのですけど、count()メソッドで1バイト文字で行けるかどうか判定してそうなので、ここまで意識しようとしている割りには質問内容とレベルが合ってないかんじがしますね。（コードは追っていませんが）

[jimbe]

なるほど。
コードをお読みになれば、各列の表示幅は maxWidth メソッドによって max 配列に入れられることがお分かりになると思います。
"-" の必要数は、各列の表示幅と縦線の数を足すことで計算できるのではないでしょうか。

[taka0145]

ご返信ありがとうございます。
ご教授頂い方法でやってみたいと思います。

Answer 1

点数のみを該当セルに入れる方法は主に二つ、データを読んだ時に編集するか、表示する時に編集するか、です。

読んだ時に編集する場合は、 lineList.add(line.split(",")); で直接 lineList に入れているのを変えて、必要な構造で add するようにします。

表示する時に変数する場合は、 writeLine の //2列目以降を右寄せ の中での k のループの時点で、line[j] を出力するか空白を出力するかを判断するようにします。
が、List をわざわざ 2次元配列にしているなどの構造から見て、読み込み時に変換したほうが良いと思います。

---

Sample クラスを活用するようにしてみました。
科目の表示順・追加等は Sample のコンストラクタで行ってください。
ex) Sample sample = new Sample("名前","国語","外国語","数学");

同一名のデータは、纏めます。
ex)

田中ゆうき,国語,80
佐藤かつひこ,数学,50
田中ゆうき,数学,30
田中たくや,国語,60

|    名前    |国語|外国語|数学|
+-----------------------------+
|田中ゆうき  |  80|      |  30|
+-----------------------------+
|佐藤かつひこ|    |      |  50|
+-----------------------------+
|田中たくや  |  60|      |    |
+-----------------------------+

package teratail_java.q363724;

import java.io.*;
import java.nio.charset.StandardCharsets;
import java.nio.file.Files;
import java.util.*;

public class Sample {
  public static void main(String[] args) throws IOException {
    Sample sample = new Sample("名前","国語","外国語","数学");
    File file = new File("Sample1.txt");
    try(BufferedReader br = Files.newBufferedReader(file.toPath(), StandardCharsets.UTF_8); ) {
      for(String line; (line=br.readLine()) != null; ) {
        String[] array = line.split(",");
        sample.put(array[0], array[1], Integer.parseInt(array[2]));
      }
      sample.drawSheet();
    } catch (IOException e) {
      e.printStackTrace();
    }
  }

  private List<String> headerList;
  private int nameIndex;
  private int[] maxWidth;
  private Map<String,Student> studentMap = new HashMap<>();

  private static class Student {
    final int number;
    final String name;
    private final Map<String,Integer> scoreMap = new HashMap<>();
    Student(int number, String name) {
      this.number = number;
      this.name = name;
    }
    void putScore(String subject, int score) {
      scoreMap.put(subject, score);
    }
    boolean hasScore(String subject) {
      return scoreMap.containsKey(subject);
    }
    int getScore(String subject) {
      return scoreMap.get(subject);
    }
  }

  Sample(String... header) {
    headerList = Arrays.asList(header);

    nameIndex = headerList.indexOf("名前");
    if(nameIndex < 0) throw new IllegalArgumentException("'名前' がありません");

    maxWidth = new int[headerList.size()];
    for(int i=0; i<headerList.size(); i++) {
      maxWidth[i] = count(headerList.get(i));
    }
  }

  void put(String name, String subject, int score) {
    Student s = studentMap.get(name);
    if(s == null) {
      s = new Student(studentMap.size()+1, name); //number は出現順にしておく
      studentMap.put(name, s);
      maxWidth[nameIndex] = Math.max(maxWidth[nameIndex], count(s.name));
    }
    s.putScore(subject, score);
    int i = headerList.indexOf(subject);
    if(i >= 0 && i != nameIndex) maxWidth[i] = Math.max(maxWidth[i], count(""+score));
  }

  void drawSheet() {
    drawHeader();
    drawRowSeparator();
    studentMap.values().stream()
      .sorted(Comparator.comparingInt(s -> s.number)) //number順に並び替え
      .forEach(s -> { drawStudent(s); drawRowSeparator(); });
  }

  private void drawHeader() {
    StringJoiner sj = new StringJoiner("|","|","|");
    for(int i=0; i<headerList.size(); i++) {
      sj.add(getCenteringString(headerList.get(i), maxWidth[i]));
    }
    System.out.println(sj.toString());
  }

  private String getCenteringString(String str, int width) {
    int space = width - count(str);
    int pre = space / 2;
    int post = space - pre;
    return " ".repeat(pre)+str+" ".repeat(post);
  }

  private String rowSeparator;
  private void drawRowSeparator() {
    if(rowSeparator == null) {
      StringJoiner sj = new StringJoiner("-","+","+");
      for(int i=0; i<maxWidth.length; i++) {
        sj.add("-".repeat(maxWidth[i]));
      }
      rowSeparator = sj.toString();
    }
    System.out.println(rowSeparator);
  }

  private void drawStudent(Student s) {
    StringJoiner sj = new StringJoiner("|","|","|");
    for(int i=0; i<maxWidth.length; i++) {
      if(i == nameIndex) {
        sj.add(s.name + " ".repeat(maxWidth[i]-count(s.name))); //名前=左詰め
      } else if(s.hasScore(headerList.get(i))) {
        sj.add(String.format("%"+maxWidth[i]+"d", s.getScore(headerList.get(i)))); //得点=右詰め
      } else {
        sj.add(" ".repeat(maxWidth[i]));
      }
    }
    System.out.println(sj.toString());
  }

  //英数字、カタカナ、ひらがな、漢字の文字コードを分類する
  public static int count(String str) {
    int count = 0;
    for(char c : str.toCharArray()) {
      count += Character.UnicodeBlock.of(c) == Character.UnicodeBlock.BASIC_LATIN ? 1 : 2;
    }
    return count;
  }
}

Comments

[taka0145]

splitで分割する方法しか存じ上げないのですが、「必要な構造でaddする」についてもう少しヒントをいただけないでしょうか？
勉強不足で申し訳ありません。

[jimbe]

lineList.add(line.split(","));
は
String[] a = line.split(",");
lineList.add(a);
と分割出来ます。これは具体的なデータから考えますと、
String line = "田中ゆうき,国語,80" の時、String[] a には
a[0]="田中ゆうき", a[1]="国語", a[2]="80"・・・(A)
と入り、lineList にそのまま入って表示されます。
lineList に入れてから以降は現状のままとすると、もし String[] a に
a[0]="田中ゆうき", a[1]="80", a[2]=""・・・(B)
と入れられれば、ご希望の表示になるはずですね。

つまり、
lineList.add(line.split(","));
を split と add に分割したうえで、split した結果 (A) を必要な構造 (B) になるように編集等してから add すれば良いのではないでしょうか。

[taka0145]

ご返信ありがとうございます。
ご教授頂いたことを基に作成しました。
正しいのかわかりませんが...

String line;
while ((line = br.readLine()) != null) {
String[] array = line.split(",");
String num1 = array[1];
String num2 = array[2];

if(num1.equals("国語")){
array[1] = num2;
array[2] = num1;
array[2] = "";
}else if(num1.equals("数学")){
array[1] = "";
}
lineList.add(array);
}

Answer 2

タイトルの「ハイフンを連続して表示させたい」についての回答です。
次のコードのどこにあるか見つけてください。

import java.util.ArrayList;
import java.io.*;  // IOExcepton, File, BufferedReader
import java.nio.file.Files;
import java.nio.charset.StandardCharsets;

class Sample {
    public static void main(String[] args) throws IOException {
        File file = new File("Sample1.txt");
        ArrayList<String[]> lineList = new ArrayList<>(); 
        lineList.add("名前,国語,数学".split(","));
		int[] width = { 4, 4, 4 };
        String line;
        try (BufferedReader br = Files.newBufferedReader(file.toPath(),
				StandardCharsets.UTF_8)) {
            while ((line = br.readLine()) != null) {
				String[] s = line.split(",");
				String[] t = { s[0], "", "" };
				if (s[1].equals("国語")) t[1] = s[2];
				if (s[1].equals("数学")) t[2] = s[2];
				for (int n, i = 0; i < 3; i++)
					if ((n = count(t[i])) > width[i]) width[i] = n;
        		lineList.add(t);
            }
        } catch (IOException e) {
            e.printStackTrace();
			return;
        }
		line = "+" + "-".repeat(width[0] + width[1] + width[2] + 2) + "+";
        drawHeader(lineList, width, line);
        drawTable(lineList, width, line);
    }

     public static void drawHeader(ArrayList<String[]> a, int[] width, String line) {
		String[] s = a.get(0);
		System.out.println(line);
		for (int i = 0; i < 3; i++) {
			int w = count(s[i]);
			int j = (width[i] - w) / 2, k = width[i] - j - w;
			System.out.print("|" + " ".repeat(j) + s[i] + " ".repeat(k));
		}
		System.out.println("|");
		System.out.println(line);
    }

	public static void drawTable(ArrayList<String[]> a, int[] width, String line) {
		for (int i = 1; i < a.size(); i++) {
			String[] s = a.get(i);
			System.out.print("|" + s[0] + " ".repeat(width[0] - count(s[0])));
			System.out.print("|" + " ".repeat(width[1] - count(s[1])) + s[1]);
			System.out.print("|" + " ".repeat(width[2] - count(s[2])) + s[2]);
			System.out.println("|");
			System.out.println(line);
		}
	}

    public static int count(String str) {
        int count = 0;
		for (int i = 0; i < str.length(); i++) {
			var b = Character.UnicodeBlock.of(str.charAt(i));
			if (b.equals(Character.UnicodeBlock.BASIC_LATIN))
				count++;
			else if (b.equals(Character.UnicodeBlock.KATAKANA) ||
			         b.equals(Character.UnicodeBlock.HIRAGANA) ||
			         b.equals(Character.UnicodeBlock.CJK_UNIFIED_IDEOGRAPHS))
				count += 2;
		}
        return count;
    }
}

Comments

[taka0145]

素晴らしい回答をありがとうございます。
lineのところで表示させているという認識で正しいでしょうか？
まとめ方がきれいで大変勉強になります。

１つ質問させていただきたいのですが、
仮にテキストファイルに情報が追加された場合はどうなるのでしょうか？

[kazuma-s]

追加された情報の具体例を示してください。

[taka0145]

田中たくや,国語、60

テキストファイルにこのような情報が追加された際、特に処理を追加する必要はないのでしょうか？

[kazuma-s]

「,」以外に「、」も区切りにしたいということでしょうか？
「田中たくや,国語,60」の間違いだったとして、なぜ自分で試してみないのですか？

[taka0145]

申し訳ありません。私のミスです。
田中たくや,国語,60で合っております。

先ほどお伝えした情報や、科目数を増やしたりなどしているのですが、
エラーが出るので確認させていただきました。

Answer 3

まず気になったのは『例外処理』関係。

        try(BufferedReader br= Files.newBufferedReader(file.toPath(), StandardCharsets.UTF_8)){
            String line;
            while ((line = br.readLine()) != null) {
                lineList.add(line.split(","));
            }
        } catch (IOException e) {
            e.printStackTrace();
        }

        String[][] data = lineList.toArray(new String[lineList.size()][]);

確かに動きはしますが、『なんのための例外処理』なのでしょうか。

例外処理は『ファイルが無かった場合』とかのように例外的な、『想定されていない』場合を対処するためのはずです。

でも上記コードだと、『ファイルが無い場合でも、リストを配列に変換する』とかの処理をしてしまいます。

ただし、今回の問題はここじゃないので中断します。

次に、質問にあるコードは絶対にダメだとは言いませんが、あまり良い傾向ではありませんね。
(別に悪く言うつもりはまったくありません)

参考1: なぜ読みやすいコードが必要なのか - コードの可読性を高める手法をサンプルで学ぶ
参考2: 良いコードとは slideshare
参考3: 他の人のコードが読みづらい理由でのmrasuさんのご回答

参考3でのmrasuさんのご回答で、

コードが読める人向け

コードが何をしたいかわからない。
直観に反した(「自分だったらこう書く」という物と違う)動きをしていて、動きが把握しづらい
データ変更と描画が同時に行われているなど、多数の状況を把握し続ける必要がある
というのがあるかなと思います。

と書かれています。これが私が初見で思ったことですねぇ。

ダメとは言いませんが、今回の場合、もっとシンプルに書けるはずです。
完璧なロジックはまだ思い浮かんでいませんが、ざっくりとしたロジックが思い浮かんでいます。(私の中で)

プログラミングっていうのは、出鱈目に書けばいいってものではありません。
プログラミングは現実世界のシミュレーションです。

なのでとりあず、プログラミングがとかは置いといて、現実世界で考えてみましょう。

田中ゆうき,国語,80
佐藤かつひこ,数学,50

というのを、

|    名前    |国語|数学| 
+----------------------+
|田中ゆうき  | 80|     | 
+----------------------+
|佐藤かつひこ|   |   50|      
+----------------------+

の状態にするにはどうしますか? (なんかズレがあるようなのでこちらで適当に修正しています。違う所があるなら読みかえてください)

私なら、

1. 何を書くかを考える( 国語の方に書くのか、数学の方に書くのかとか )
2. それぞれのセル(?)の長さはどういう風にするか(長さの基準等)
3. 文字はどのように並んでいるか(右揃え? 左揃え? センタリング? )
4. 数字(国語の点数等)はどのように並んでいるか(〃)
5. 区切り線(?)の"+----+"のやつの長さはどうするか
6. 区切り線(?)の"|"はどのように入れるか(位置とか)

をそれぞれ考える。

つまり、規則性を見付けるのです。

何を書くか(= 1)は元のデータ "田中ゆうき,国語,80" とかの二番目から考える。

それぞれのセル云々(= 2) は『その列の最大行数から考える』。

文字や数字はどのように並んでいるか(= 3, 4)は『項目名は中央揃え、名前は左揃え、点数は右寄せ』。

"+---+" の長さ(= 5) は『レコード(*1)の長さ分』。
*1: ここでいうレコードは、一行分のデータ列(ただし"|"等も含めている)

6は2と一緒ですね。

次に実際にノートかホワイトボードかなんかに上記の表を書いてみましょう。

[Flow]:
1. Excelでいう列のそれぞれの最大行数を求める
2. (1)のそれぞれの長さを『最大値』として"名前"や"国語"等が中心に来るように書く
3. 以下を『書くものが無くなるまで』繰り返す
    3.1. (2)で出来た行の最大の長さ分、"+---+" 的なものを書く
    3.2. (2)と同様に『名前を左揃え』、『数字を右揃え』で書く
4. (3.1)と同様に線を引く

みたいな流れになるはずです。(まあ、実際には(3.1)と(3.2)が逆になるかもしれないが )

後はこれを実装すればいいだけです。

まだわかりにくい部分があれば、同じように流れを考える。

で、(3.1)での"+---+" のやつは、どの行?でも同じ長さですね。完全固定。
なので、一度定義すれば単に描画すればいいだけです。

と考えると、

[Flow]
1. 列のそれぞれの最大行数を計算する
2. ヘッダに相当する、"名前"等の部分を(1)を最大数としてセンタリングして表示
3. (2)の長さ分、"+---+"のような文字列を生成
4. 以下を『書くものが無くなるまで』繰り返す
    4.1. (3)で生成した文字列を表示
    4.2. (2)の要領で『左揃え』や『右揃え』で表示していく
5. (3)で生成した文字列を表示

のような感じになりますね。(順番が変わる可能性があるが)

[コメントにて]

１番の何を書くかを考えるについてですが、...

あ、それでもいいですよ。

とりあえず、一気に作るのではなく、少しづつ作っていってください。
自分の考えが正しいかどうかもそれでわかります。

Comments

[taka0145]

勉強不足で申し訳ありません。

１番の何を書くかを考えるについてですが、
for文のcontinueなどを用いて処理すると考えたのですが、実現可能でしょうか？