診断機能で必要なif文をなるべく減らしたいのですが、どなたか教えて頂けませんか…？

Question

前提・実現したいこと

・現在、javafxで診断メーカーを作成しているのですが、全部で３２通りの結果を作らなければいけません。そのため、EventHandlerよりif文の使用を考えているのですが、それでも３１ものif文が出来てしまうため、なるべく減らす方法は無いのかと考えております。
・また、ファイル(txt)から文を読み込める仕組みと配列の作成の組み合わせも不明確なため、作成の方法をどなたか教えて頂きたいです。

該当のソースコード

	class seEventHandler implements EventHandler<ActionEvent>{
		public void handle(ActionEvent e2) {
			stage3 = new Stage();
			lb = new Label("「暑い」or「寒い」どっちなら我慢できる？");
	        lb2 = new Label("1.暑い");
	        lb3 = new Label("2.寒い");
	        lb4 = new Label("下にある空欄に上の２つのどちらかをご記入ください");
	        tf = new TextField();
	        String str =  tf.getText();
	        if(str == "暑い"){

	        }
	        else if(str == "寒い"){

	        }
	        VBox hb=new VBox(20d);
	        hb.getChildren().add(lb2);
	        hb.getChildren().add(lb3);
	        VBox hb1=new VBox(10d);
	        hb1.getChildren().add(lb4);
	        hb1.getChildren().add(tf);
			BorderPane bp =new BorderPane();
            Scene sc = new Scene(bp, 400, 400);
            hb.setAlignment(Pos.CENTER);
            bp.setTop(lb);
            bp.setCenter(hb);
            bp.setBottom(hb1);
            stage3.setScene(sc);
            stage3.show();
	}

試したこと

ファイル(txt)から、「暑い」、「寒い」、質問文を読み込める仕組み（配列…？）とif文の組み合わせ(forでできるのか…？)などを考えていました

Comments

[退会済みユーザー]

「発生している問題」欄にソースコードを貼ってますね。
質問は慎重に正しく編集した方が、親切な回答がつきやすいですよ。

[nakagen]

ご指摘ありがとうございます。ただいま、訂正致しました。

Answer 1

質問文は固定で、選択のYESの個数で判定するようなものですよね？
質問に重み付けがあったりするともっと本格的ですが、回答は単純に個数で判定しました。
文字で入力するのは面倒なのでボタンにしました。

import java.io.IOException;
import java.nio.charset.StandardCharsets;
import java.nio.file.Files;
import java.nio.file.Paths;
import java.util.Arrays;
import java.util.List;

import javafx.application.Application;
import javafx.event.ActionEvent;
import javafx.event.EventHandler;
import javafx.geometry.Insets;
import javafx.geometry.Pos;
import javafx.scene.Scene;
import javafx.scene.control.Button;
import javafx.scene.control.Label;
import javafx.scene.layout.BorderPane;
import javafx.scene.layout.VBox;
import javafx.stage.Stage;


public class Main extends Application {
    public static void main(String[] args) { launch(args); }

    @Override public void start(Stage primaryStage) {
        BorderPane root = new BorderPane();
        Button button = new Button("心理テストだよ");
        button.setOnAction(new seEventHandler());

        root.setCenter(button);
        primaryStage.setScene(new Scene(root, 500, 400));
        primaryStage.show();
    }
}

class seEventHandler implements EventHandler<ActionEvent> {
    private Label label;
    private Button button1;
    private Button button2;

    private String[][] questions; // 質問の配列の配列（質問個数分の配列の中に「質問文・選択肢１・選択肢２」の配列が入っている）
    private int index; // 今の質問インデックス
    private int dog;

    @Override public void handle(ActionEvent event) {
        Stage stage = new Stage();
        BorderPane root = new BorderPane();
        root.setPadding(new Insets(10, 10, 10, 10));
        stage.setScene(new Scene(root, 400, 300));

        label = new Label();
        root.setCenter(label);

        button1 = new Button();
        button2 = new Button();
        button1.setOnAction(ae -> { dog++; next(); });
        button2.setOnAction(ae -> next());

        VBox box = new VBox(10, button1, button2);
        box.setAlignment(Pos.CENTER);
        root.setBottom(box);

        try {
            createCSV();
            loadCSV();
        } catch (IOException e) {
            e.printStackTrace();
        }
        next();

        stage.show();
    }

    private void next() {
        if (index < questions.length) { // 質問作成
            label.setText(questions[index][0]); // questions[index][0]が質問文
            button1.setText(questions[index][1]); // questions[index][0]は選択肢１
            button2.setText(questions[index][2]);

            index++;
        } else { // 結果判定
            String result;
            switch (dog) {
                case 0: result = "あなたは完全に猫派です！"; break;
                case 1: result = "あなたはどちらかというと猫派です。"; break;
                case 2: result = "あなたはどちらかというと犬派です。"; break;
                default: result = "あなたは完全に犬派です！"; break;
            }

            label.setText(result);
            button1.setVisible(false);
            button2.setVisible(false);
        }
    }

    private void loadCSV() throws IOException {
        List<String> lines = Files.readAllLines(Paths.get("test.csv"), StandardCharsets.UTF_8); // ファイルを1行毎のリストに読み込み

        questions = new String[lines.size()][]; // 行数分配列を確保
        for (int i = 0; i < lines.size(); i++) {
            String[] split = lines.get(i).split(","); // カンマで区切って「質問文・選択肢１・選択肢２」の配列を作成
            questions[i] = split; // questions配列のi番目の中にまた配列を入れた
        }
    }

    // というtest.csvがあるテイで
    private void createCSV() throws IOException {
        List<String> lines = Arrays.asList(
                "休日の過ごし方は？,みんなでワイワイ,ひとりで趣味を",
                "久しぶりに雪が降ったらどうする？,外で駆け回る,コタツでぬくぬく",
                "褒められるとついついしてしまうことは？,よだれを垂らしてしまう,興味ないふりをしてしまう");

        Files.write(Paths.get("test.csv"), lines, StandardCharsets.UTF_8);
    }
}

軽く説明を入れましたが、不明点があればコメントしてください。

---

一刻も早くプログラムを完成させたい気持ちはわかりますが、
Eclipse - ファイルの読み込みのエラーが原因なのでしょうか…？｜teratail

を見る限り到底動くと思えません。ファイルを読み込む以前の問題です。

回答プログラムはtest.csvを作成してから読み込むようになっているので、どこにできているか確認してみてください。

Comments

[TN8001]

こちらもvarを修正しました。
Java8ならこれで動くと思うので、求めているものとどこが違うのかコメントしてください。

[nakagen]

ご回答をありがとうございます!２つ聞きたいのですが、何を見て動かないと思ったのでしょうか…？また、test.csvを用いているのですが、sample.txtを読み込んでやることは可能でしょうか…？無知で申し訳ありません…

[TN8001]

>何を見て動かないと思ったのでしょうか…？

314041のコードでいえば

///診断機能

と書いてある行以降です。
この時点では
String str = tf.getText();
に何も入っていませんから、switchが動くはずありません。

GUIプログラムはCUI（コンソール=黒い画面のやつ）プログラムとは全く作り方が変わってきます（イベントモデル）
そういうこともあり回答ではボタンを使いました（TextFieldでは入力終わりをどう判断しますか？）

>test.csvを用いているのですが、sample.txtを読み込んでやることは可能でしょうか…？

もちろん可能です。
1行1問のほうがシンプルに書けると思ったのでそうしただけですが、そこはもう決定している話なんでしょうか？

[nakagen]

申し訳ありません、こちらがgetTextを空欄に記入した言葉をstrに反映させるものだと誤認してしまいました。そうですよね、、、そう考えましたら、ボタンの方が使いやすいことになりますよね。sample.txtの件につきましては、後々でも文を更新できるようtxtから読み込みたいと考えておりました…。

[TN8001]

勘違いしておられるようですが、test.csvもテキストファイルです。
カンマで区切って表形式のデータを表すという、テキスト形式のファイルです。
[Comma-Separated Values - Wikipedia](https://ja.wikipedia.org/wiki/Comma-Separated_Values

test.txtとしても別に問題はありませんが、この形式のファイルは.csvと拡張子をつけるのが慣例です。

>1行1問のほうがシンプルに書けると思ったのでそうしただけですが、そこはもう決定している話なんでしょうか？

これは1問3行か、1問1行かの話が主題です。

[nakagen]

なるほど、そうでしたか…。このコードをイベントハンドラの中で作動させたいのですが、可能ですかね…？

[TN8001]

まず実行はしていただけて、やりたいこととおおむね合っているということでいいんでしょうか？

>このコードをイベントハンドラの中で作動させたい

メニュー画面のようなものがあって、そこからさらにウィンドウを出しているということでしょうか？

class Main から class seEventHandler に
public void start から public void handle に

でおおむね行けると思います。

[nakagen]

一度、実行したのですが、イメージしていたのと合っていました。「メニュー画面のようなものがあって、そこからさらにウィンドウを出しているということでしょうか？」についてですが、その通りです。一度やってみます！

[nakagen]

すみません、class Mainからclass seEventHandlerに変更するというのは、
class seEventHandler implements EventHandler<ActionEvent>{
public void handle(ActionEvent e2) {
BorderPane root = new BorderPane();
root.setPadding(new Insets(10, 10, 10, 10));
primaryStage.setScene(new Scene(root, 400, 300));

label = new Label();
root.setCenter(label);

button1 = new Button();
button2 = new Button();
button1.setOnAction(ae -> { dog++; next(); });
button2.setOnAction(ae -> next());

VBox box = new VBox(10, button1, button2);
box.setAlignment(Pos.CENTER);
root.setBottom(box);

createCSV();
loadCSV();
next();

primaryStage.show();
}

private void next() {
if (index < questions.length) { // 質問作成
label.setText(questions[index][0]); // questions[index][0]が質問文
button1.setText(questions[index][1]); // questions[index][0]は選択肢１
button2.setText(questions[index][2]);

index++;
} else { // 結果判定
String result;
switch (dog) {
case 0: result = "あなたは完全に猫派です！"; break;
case 1: result = "あなたはどちらかというと猫派です。"; break;
case 2: result = "あなたはどちらかというと犬派です。"; break;
default: result = "あなたは完全に犬派です！"; break;
}

label.setText(result);
button1.setVisible(false);
button2.setVisible(false);
}
}

private void loadCSV() throws IOException {
List<String> lines = Files.readAllLines(Paths.get("test.csv"), StandardCharsets.UTF_8); // ファイルを1行毎のリストに読み込み

questions = new String[lines.size()][]; // 行数分配列を確保
for (int i = 0; i < lines.size(); i++) {
String[] split = lines.get(i).split(","); // カンマで区切って「質問文・選択肢１・選択肢２」の配列を作成
questions[i] = split; // questions配列のi番目の中にまた配列を入れた
}
}

// というtest.csvがあるテイで
private void createCSV() throws IOException {
List<String> lines = Arrays.asList(
"休日の過ごし方は？,みんなでワイワイ,ひとりで趣味を",
"久しぶりに雪が降ったらどうする？,外で駆け回る,コタツでぬくぬく",
"褒められるとついついしてしまうことは？,よだれを垂らしてしまう,興味ないふりをしてしまう");

Files.write(Paths.get("test.csv"), lines, StandardCharsets.UTF_8);
}
}
}
といった感じで大丈夫でしょうか？

[退会済みユーザー]

質問の内容が変わってきてるので、この質問は解決済にして別の質問を立てた方がいいと思います。
コメント欄はマークダウン効かないから、コードが読みづらいですし。

[TN8001]

>といった感じで大丈夫でしょうか？

といった感じです。
が、大丈夫ならコメントしていないでしょうし、だめならエラー文を書いてください。

回答コードをメニュー画面付きに変更しました。

@m-oguraさん
お気遣いありがとうございます。
おそらくこの質問を見ていないと意味が分からない質問になるでしょうから、もう回答してしまいます^^;

[nakagen]

m-oguraさん
そうですよね、最初の質問の主旨から大きく逸れてしまいました…
次回から、そうするよう気を付けます

[nakagen]

TN8001さん
私の様々な質問につき合わせてしまい、大変申し訳ありませんでした。次回から、このようなことが無いよう気を付けたいと思います。また、コードの知識等を頂き本当にありがとうございました。

[nakagen]

TN8001さん
厚かましいようですが、こちらの質問を見ていただくことは出来ないでしょうか…？
https://teratail.com/questions/314155?modal=q-comp
よろしくお願いいたします…

Answer 2

switchと配列で解決することが出来ました

Answer 3

1つ目の質問については、switch文を使うとすっきりします。
【Java】switch文でStringの文字列を扱う方法
2つ目の質問ですが、ググればいくらでも答えが見つかりますので、頑張ってください。

Comments

[nakagen]

ご回答をありがとうございます！switch文を作成してみたのですが…
switch(str) {
case"暑い":
switch(str) {

}
case"寒い":
switch(str) {

}
break;
default:
といった感じで繰り返すのでしょうか…？
配列のほうは、こちらで頑張って探してみます！（どうしてもわからなかったときは、またお聞きしてもよろしいでしょうか…？）

[退会済みユーザー]

まずはリンク先でswitch文の書き方を把握してください。
配列に関しては、ググればサンプルコードがいくらでも転がってます。
頑張って！

[nakagen]

一度、配列と組み合わせて書いてみたのですが…
///
switch(str) {
case sT[1]:
///
lb.setText(sT[1]);

///
switch(str){
case sT[4]:
switch(str) {
case sT[10]:
switch(str) {
case"14":
break;
case"15":
break;
}
break;
case sT[11]:
switch(str) {
case"16":
break;
case"17":
break;
}
break;
}
break;
case sT[5]:
switch(str) {
case sT[13]:
switch(str) {
case"18":
break;
case"19":
break;
}
break;
case sT[14]:
switch(str) {
case"20":
break;
case"21":
break;
}
break;
}
break;
}
break;
///
case に配列を入れたらエラーが起きてしまいました…
この場合は、どうしたらいいのでしょうか…？

[nakagen]

すみません、switc文にswitc文をいれて２択の質問に答えていくことは可能ですよね…？