java ファイルからのString格納方法

Question

java初心者です。

以下のようなテキストがあったとして、これをコロン手前の人間の名前だけを集めた配列と
コロン後ろの複数の人間を集めた二次元配列に分ける場合、どういった方法が簡単でしょうか？

私は以下のようなアプローチをとったのですが、不要なものが多いような気がしたのでお聞きした次第です。

１）ファイルの中身をScannerで取ってきて行数を確認➡配列の数字に利用
２）Scannerで拾ってきた以下中身を「：」もしくは改行でSplit。これをString配列へ格納　➡　行数の二倍の数字ができる。
３）奇数は「：」前までの数字、偶数は「：」後の改行までの文字列が入っているので、奇数分をString配列にて格納
４）偶数分については、新たな配列に行ごとに複数のなまえをまるごと格納。
５）１）で行数がわかっているので、二次元配列の前半にその数字を入れて、４）格納した配列から名前をSplitて文字を入れ込む

テキストの中身は以下
Kate: Chris, Joe, Anna, Frank , Nancy
Chris: Kate, Nancy, Peter, Steve, Frank, Tim, Amy
Charles: Sara, Kevin, Peter, Steve
David: Steve, Amy , Bob
Frank: Anna, Kate, Mary , Pop
Max: Kate , Nancy , Tim
Smith: Kevin, Kate, Tim, Bob

Comments

[BluOxy]

そのテキストは何のために配列へ保持するのでしょうか。
どのような形式で配列に保持すればそのプログラムの要件を満たすのでしょうか。

[KttShK]

最終的にはコロン前の名前だけを集めた配列と、コロン以降の名前を入れた二次元配列をぶつけてSerachする目的です。例えばKate以降の５名はKateの友達リストです。
コロン以降の名前の集まりはコロン前の名前の人の友達リストです。
KateとChris（前者の配列の[0],[1]）を比較して後者の二次元配列の中から共通する友人、ここでいえばFrankとNancyを探し当てるようなものを作りたいです。

[BluOxy]

１～５の日本語による説明では動作をイメージするのに限界がありますので、実際のコードを掲示して頂けますか。何をどうするべきかはそのコードをベースに説明できるとより具体的な回答が書けそうです。

[KttShK]

以下のように記載しました。　ぐちゃぐちゃな文で恐縮です。

String[] strBox = listA.toString().split(":|\n");　　//こちらで分割 2)に該当
mainPerson = new String[7];
for (int i = 0; i < strBox.length/2; i++) {
mainPerson[i] = strBox[i*2]; // 名前格納
}
String[] tempName = new String[strBox.length/2];　お友達を格納

for (int i = 1; i <= strBox.length/2; i++) {
tempName[i-1] = strBox[(i*2)-1].toString().replace(" ", "");
}
yujin = new String[7][];
for (int i = 0; i < 7; i++) {
yujin[i] = tempName[i].trim().split(",");
}

[swordone]

質問を編集してコードを記載してください。

Answer 1

追記修正コメント内のコードを見る限り、行数は決まっているのでしょうか？
だとすれば、あらかじめ配列を人数分作っておき、行ごとに読み込んで順次入れていくという手法が一般的ではないでしょうか。

Scanner s = new Scanner(file); // よくわからんので適当
String[] names = new String[7];
String[][] friends = new String[7][]
for(int i = 0; i < 7; i++) {
    String[] temp = s.nextLine().split(":");
    names[i] = temp[0];
    friends[i] = temp[1].split(",");
}

Comments

[KttShK]

ありがとうございます。二次配列の値導入のイメージが分かっておらず、お聞きした次第です。
行数は決まってはいません。質問内容を明確にするために追加修正の中にはあえて数字を固定して入れております。本来は数字は確定しておらず、ファイル内の情報が増えた分、柔軟にとりこめるようにしなければいけません。ここについてはファイルの中の行数を行数チェックでカウントして数字を取りました。

[swordone]

行数チェックって何するんだ？と思いましたが、行数が分かるならどちらでもいいです。

Answer 2

回答

どういった方法が簡単でしょうか？
私は以下のようなアプローチをとったのですが、不要なものが多いような気がした

そのコードが「なぜ不要なものが多い」と感じたかを考えることが大事です。

個人的には不要なものがあるかどうかよりもコードが読みにくいことが気になります。
特に変数の命名やデータの保持方法辺りが気になります。

まずはそういった点を改善し、読みやすいコードを作ると良いでしょう。
ロジック的に不要なものがあるかどうかはその後に判断してはいかがでしょうか。

では、どのようにして読みやすいコードにするか。
改善策を下記に記載します。

改善案

人間の名前と友達一覧を持つクラスを定義して、そのリストを生成して読み込んだデータの管理をするのが良いでしょう。
使うのは配列ではなくリストです。配列よりもリストの方が要素数が不定のときの取り回しが効きます。

また、mainPersonとyujinのように別々の変数で管理するのではなくオブジェクトで両方とも保持しておく方が人に理解しやすいコードを書けます。

例えば下記のようなUserクラスを定義します。

public class User{
    private String name;
    private List<String> friends;

    public User(String name, List<String> friends){
        this.name = name;
        this.friends = friends;
    }
}

プログラム上で下記のように読み取ります。

import java.util.*;
import java.lang.*;

public class Main {
    public static void main(String[] args) throws Exception {
        List<User> users = new ArrayList<User>();

        //input
        String listA = "Kate: Chris, Joe, Anna, Frank , Nancy\nChris: Kate, Nancy, Peter, Steve, Frank, Tim, Amy\nCharles: Sara, Kevin, Peter, Steve\nDavid: Steve, Amy , Bob \nFrank: Anna, Kate, Mary , Pop\nMax: Kate , Nancy , Tim\nSmith: Kevin, Kate, Tim, Bob";
        String[] rows = listA.toString().split("\n");
        for(String row : rows){
            String[] columns = row.split(":");
            List<String> friends = Arrays.asList(columns[1].split(","));
            String name = columns[0];
            User user = new User(name,friends);
            users.add(user);
        }
        
        //output
        for(User user: users){
            System.out.println(
                String.format(
                    "username:%-8s, friends:[%s]",
                    user.getName(),
                    String.join(",",user.getFriends())
                )
            );
        }
    }
}

出力結果

username:Kate    , friends:[ Chris, Joe, Anna, Frank , Nancy]
username:Chris   , friends:[ Kate, Nancy, Peter, Steve, Frank, Tim, Amy]
username:Charles , friends:[ Sara, Kevin, Peter, Steve]
username:David   , friends:[ Steve, Amy , Bob ]
username:Frank   , friends:[ Anna, Kate, Mary , Pop]
username:Max     , friends:[ Kate , Nancy , Tim]
username:Smith   , friends:[ Kevin, Kate, Tim, Bob]

改めて書きますが、修正したポイントは3つです。

• 変数の命名を意味が分かるものにする
• 読み込んだ文字列をオブジェクトとして管理する
• 要素数が不明なときはListを使うと取り回しが効きやすい

Comments

[KttShK]

どうもありがとうございます。命名等、わかりやすく作るように心がけます。
Listはまだ習っておらず、今回は二次元配列と配列のみでどうシンプルに作り上げるのか気になっておりました。