診断機能の仕上げをしたい

Question

前提・実現したいこと

現在、診断機能で、診断機能の仕組みは作れたのですが…
・primaryStageを解決することが出来ない
・32通りの答えの辿り着き方
が現在、分かりません…

よろしくお願いいたします。

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

xception in thread "JavaFX Application Thread" java.lang.Error: Unresolved compilation problems: 
	primaryStage を解決できません
	primaryStage を解決できません

	at saisyuukadai.date2$seEventHandler.handle(date2.java:198)
	at saisyuukadai.date2$seEventHandler.handle(date2.java:1)
	at com.sun.javafx.event.CompositeEventHandler.dispatchBubblingEvent(CompositeEventHandler.java:86)
	at com.sun.javafx.event.EventHandlerManager.dispatchBubblingEvent(EventHandlerManager.java:238)
	at com.sun.javafx.event.EventHandlerManager.dispatchBubblingEvent(EventHandlerManager.java:191)
	at com.sun.javafx.event.CompositeEventDispatcher.dispatchBubblingEvent(CompositeEventDispatcher.java:59)
	at com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(BasicEventDispatcher.java:58)
	at com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(EventDispatchChainImpl.java:114)
	at com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(BasicEventDispatcher.java:56)
	at com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(EventDispatchChainImpl.java:114)
	at com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(BasicEventDispatcher.java:56)
	at com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(EventDispatchChainImpl.java:114)
	at com.sun.javafx.event.EventUtil.fireEventImpl(EventUtil.java:74)
	at com.sun.javafx.event.EventUtil.fireEvent(EventUtil.java:49)
	at javafx.event.Event.fireEvent(Event.java:198)
	at javafx.scene.Node.fireEvent(Node.java:8411)
	at javafx.scene.control.Button.fire(Button.java:185)
	at com.sun.javafx.scene.control.behavior.ButtonBehavior.mouseReleased(ButtonBehavior.java:182)
	at com.sun.javafx.scene.control.skin.BehaviorSkinBase$1.handle(BehaviorSkinBase.java:96)
	at com.sun.javafx.scene.control.skin.BehaviorSkinBase$1.handle(BehaviorSkinBase.java:89)
	at com.sun.javafx.event.CompositeEventHandler$NormalEventHandlerRecord.handleBubblingEvent(CompositeEventHandler.java:218)
	at com.sun.javafx.event.CompositeEventHandler.dispatchBubblingEvent(CompositeEventHandler.java:80)
	at com.sun.javafx.event.EventHandlerManager.dispatchBubblingEvent(EventHandlerManager.java:238)
	at com.sun.javafx.event.EventHandlerManager.dispatchBubblingEvent(EventHandlerManager.java:191)
	at com.sun.javafx.event.CompositeEventDispatcher.dispatchBubblingEvent(CompositeEventDispatcher.java:59)
	at com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(BasicEventDispatcher.java:58)
	at com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(EventDispatchChainImpl.java:114)
	at com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(BasicEventDispatcher.java:56)
	at com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(EventDispatchChainImpl.java:114)
	at com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(BasicEventDispatcher.java:56)
	at com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(EventDispatchChainImpl.java:114)
	at com.sun.javafx.event.EventUtil.fireEventImpl(EventUtil.java:74)
	at com.sun.javafx.event.EventUtil.fireEvent(EventUtil.java:54)
	at javafx.event.Event.fireEvent(Event.java:198)
	at javafx.scene.Scene$MouseHandler.process(Scene.java:3757)
	at javafx.scene.Scene$MouseHandler.access$1500(Scene.java:3485)
	at javafx.scene.Scene.impl_processMouseEvent(Scene.java:1762)
	at javafx.scene.Scene$ScenePeerListener.mouseEvent(Scene.java:2494)
	at com.sun.javafx.tk.quantum.GlassViewEventHandler$MouseEventNotification.run(GlassViewEventHandler.java:394)
	at com.sun.javafx.tk.quantum.GlassViewEventHandler$MouseEventNotification.run(GlassViewEventHandler.java:295)
	at java.security.AccessController.doPrivileged(Native Method)
	at com.sun.javafx.tk.quantum.GlassViewEventHandler.lambda$handleMouseEvent$353(GlassViewEventHandler.java:432)
	at com.sun.javafx.tk.quantum.QuantumToolkit.runWithoutRenderLock(QuantumToolkit.java:389)
	at com.sun.javafx.tk.quantum.GlassViewEventHandler.handleMouseEvent(GlassViewEventHandler.java:431)
	at com.sun.glass.ui.View.handleMouseEvent(View.java:555)
	at com.sun.glass.ui.View.notifyMouse(View.java:937)
	at com.sun.glass.ui.win.WinApplication._runLoop(Native Method)
	at com.sun.glass.ui.win.WinApplication.lambda$null$147(WinApplication.java:177)
	at java.lang.Thread.run(Thread.java:748)

該当のソースコード

	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));

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

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

		VBox box = new VBox(10, bt1, bt2);
		box.setAlignment(Pos.CENTER);
		root.setBottom(box);

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

		primaryStage.show();
		}

		private void next() {
		if (index < questions.length) { // 質問作成
		lb.setText(questions[index][0]); // questions[index][0]が質問文
		bt1.setText(questions[index][1]); // questions[index][0]は選択肢１
		bt2.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;
		}

		lb.setText(result);
		bt1.setVisible(false);
		bt2.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);
		}
		}

試したこと

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

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

jdk14

Comments

[TN8001]

[Eclipse - 診断機能で必要なif文をなるべく減らしたいのですが、どなたか教えて頂けませんか…？｜teratail](https://teratail.com/questions/313715

の回答コードをメニュー画面付きに更新しています。確認してください。

急いでいる理由があるのかもしれませんが、落ち着いて一つずつ解決していきましょう。

[nakagen]

ありがとうございます、、、！今作成しているプログラミングだと、メニュー画面には、他の機能に行ける画面となっているので、可能であればイベントハンドラの中でまとめたいのですが、可能でしょうか…？

[TN8001]

メニュー画面がどうなっているのかはわかりませんが、class seEventHandlerに入っていればいいのではないのですか？

>イベントハンドラの中でまとめたい

というのはどういう状態なんでしょうか？？

[nakagen]

すみませんでした、今、意味が分かりました…
私が悩んでいたのは、primarystageの意味で、修正していただいたコードを読んで直したところ診断の画面につなぐことが出来ました。
「イベントハンドラの中でまとめたい」につきましては、primaryStageをイベントハンドラの中で使うものだと思っていました…

[TN8001]

そうですか。よかったです＾＾

この質問ですが、313715とかぶっているように思うので削除依頼をされたらどうでしょうか。

もし削除依頼が通らなかった場合は、自己回答で閉じてください。

[nakagen]

了解です！
最後に一つだけ、お聞きしたいのですが、この診断の結果（４通りの）は、どのようにして出しているのでしょうか…？自分の理想が３２通りなのですが、作れますかね…？

[nakagen]

すみません、もしかしてこれって
---
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++;
}
---
で、質問の数によって、出される答えの数が増えていく意味なのでしょうか？
2の５乗みたいなイメージで…

[TN8001]

>どのようにして出しているのでしょうか…？

回答コードはYESの個数ですので、3*2の6通りの組み合わせがありますが、順番？は気にしない実装です。

>理想が３２通りなのですが、作れますかね…？

３２通りというと16*2ということでしょうかね。
結果の取得が、回答が欲しいのか・どちらを選んだかわかればいいのかがよくわかりません。

>質問の数によって、出される答えの数が増えていく意味なのでしょうか？
>2の５乗みたいなイメージで…

ちょっと意味が分かりませんが、2次元配列を使っています。

質問1,選択肢1,選択肢2
質問2,選択肢1,選択肢2
質問3,選択肢1,選択肢2

行がindexで、列が0,1,2になっています。

// 質問作成
というコメント文はちょっと良くなかったかもしれません^^;
質問内容を画面に反映という意味です。

[nakagen]

「結果の取得が、回答が欲しいのか・どちらを選んだかわかればいいのかがよくわかりません。」についてなのですが、
　　　　　　　　　　〇は質問となります
　　　　　　　回答１　回答２
　　　　　　〇　　　　　　　〇
　　　回答１　回答２　回答１　回答２
　　〇　　　　　　〇　〇　　　　　〇
みたいな感じで進めて、最終的に３２通りの結果に繋げたい感じです…
説明が分かりにくくてすみません、、、

[TN8001]

この図？だと矢印に沿って問題も変わるようなタイプのものに見えてしまいますが、そうではなく各段の問題は同じということでいいんでしょうかね？

恐らくそうだとして、
でしたら文字列を使って〇〇×〇のように、16問分回答を足していくのが簡便かもしれません。

あれ？これを考えているうちに勘違いに気づきましたが、順番を気にするなら、3問だと2^3の8通りですね^^;

ということは一人が答える質問は5問なんでしょうか？（2^5）
ルートによって質問が変わるタイプで16問用意しているということ？？

最初に
>質問文は固定で、選択のYESの個数で判定するようなものですよね？

と確認したのですが、ルートによって質問が変わるとなるとだいぶ話が違ってきます。。。

[nakagen]

＞ということは一人が答える質問は5問なんでしょうか？（2^5）
ルートによって質問が変わるタイプで16問用意しているということ？？

すみません、、、その通りです…
やはり、if文などで、コツコツやるしかないのでしょうか…

[TN8001]

となるとツリー状になりますから、CSVとあまり相性が良くないですねぇ。
何かいいアイデアが浮かべばいいのですが。。。
ちょっと考えさせてください。

[nakagen]

了解です、お手間をおかけしてすみません、、、

[m.ts10806]

あまりやりすぎると「質問と回答」ではなく「作業依頼」になるので、ヒントを得る程度にとどめられたほうが良いかと思います。

[YT0014]

上記で説明した、質問と回答の遷移に関しては、仕様として、質問文に追記してください。

Answer 1

回答がついてしまったのと、追記依頼でのやり取りもあるのでこちらに回答します。

質問のYESNOで矢印に沿っていく「診断チャート」のようなものが、やりたいことだったわけです（質問タイトルはあまり良いとは思えません）

当初質問を読み違えて遠回りになってしまいましたが^^;、このようなものはどうでしょう？

実装の理解のしやすさと、質問の編集のしやすさのバランスを取りました。

本来はjsonやxml等ツリーを扱えるテキスト形式が良いと思いますが、Javaで読むのは割と面倒なのでcsvのままにします。

1行を「質問,選択１,選択２,選択１ジャンプ行番号,選択２ジャンプ行番号」のように表し、ジャンプ先を手書きします^^;（YT0014さんの案と同じ感じです。手書きは面倒ですが仕方ないですね）

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.scene.text.TextAlignment;
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 Question {
    String Text; // 診断結果or質問文
    String Choice1; // 選択肢１
    String Choice2; // 選択肢２
    int Jump1; // 選択肢ジャンプ先インデックス番号
    int Jump2;

    public Question(String line) {
        // インデントがあれば削除しカンマで区切る
        String[] split = line.trim().split(",");

        // 質問or診断中の'↵'は改行に置換
        Text = split[0].replace('↵', '\n');

        if (5 <= split.length) {
            Choice1 = split[1];
            Choice2 = split[2];

            // 編集しやすいように行番号=1始まり と ヘッダー行分のマイナス２
            Jump1 = Integer.parseInt(split[3]) - 2;
            Jump2 = Integer.parseInt(split[4]) - 2;
        }
    }
}

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

    private Question[] questions; // 質問の配列
    private Question current; // 現在の質問

    @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();
        label.setTextAlignment(TextAlignment.CENTER); // 中央揃え
        label.setWrapText(true); // 折り返し
        root.setCenter(label);

        button1 = new Button();
        button2 = new Button();

        // 質問の選択によってジャンプ先を変更
        button1.setOnAction(ae -> jump(current.Jump1));
        button2.setOnAction(ae -> jump(current.Jump2));

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

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

        jump(0); // 最初は先頭の質問

        stage.show();
    }

    private void jump(int index) {
        current = questions[index]; // 現在の質問を指定インデックスの質問に変更

        label.setText(current.Text);

        // Choiceがnull ＝ 診断結果
        if (current.Choice1 == null) button1.setVisible(false);
        else button1.setText(current.Choice1);

        if (current.Choice2 == null) button2.setVisible(false);
        else button2.setText(current.Choice2);
    }

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

        questions = new Question[lines.size() - 1]; // ヘッダーを除いた行数分配列を確保
        for (int i = 1; i < lines.size(); i++) { // i = 1 ヘッダーは飛ばす
            questions[i - 1] = new Question(lines.get(i)); // ヘッダーを除いてるので i - 1
        }
    }

    // というtest.csvがあるテイで
    private void createCSV() throws IOException {
        List<String> lines = Arrays.asList(
                // インデントは単に見やすくするだけで、階層を解析しているわけではない
                // 問１問２問２問３問３問３問３ の順番でも別に構わないが、おそらく見にくいｗ
                // ジャンプ行番号を間違えるとループしたり、IndexOutOfBoundsException が出ます！
                // 質問or診断中の'↵'は、読み込み時に改行に置換
                "質問,選択１,選択２,選択１ジャンプ行番号,選択２ジャンプ行番号",
                "問１：,aaa,bbb,3,10",
                "    問２：,aaa,bbb,4,7",
                "        問３：,aaa,bbb,5,6",
                "            診断↵aaa aaa aaa",
                "            診断↵aaa aaa bbb",
                "        問３：,aaa,bbb,8,9",
                "            診断↵aaa bbb aaa",
                "            診断↵aaa bbb bbb",
                "    問２：,aaa,bbb,11,14",
                "        問３：,aaa,bbb,12,13",
                "            診断↵bbb aaa aaa",
                "            診断↵bbb aaa bbb",
                "        問３：,aaa,bbb,15,16",
                "            診断↵bbb bbb aaa",
                "            診断↵bbb bbb bbb");

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

Comments

[nakagen]

ご回答ありがとうございます…！ファイルの指定ができるように、アレンジしたいと思います…！もし、また分からなくなった場合は、他の質問ページでよろしくお願いいたします

Answer 2

primaryStageは、コードのイベントハンドラの使われ方が不明なので、明確な回答ができません。
新規にStageを作成するか、e2のプロパティなどから取得するかになるかと思いますが。

診断機能の遷移から考えると、データ構造としては、以下をお勧めします。

回答データ
選択肢、次質問コード、診断結果

質問データ
質問コード、質問、回答データ配列

CSVでは、回答データ配列を、回答1、回答2の順番で記載。
質問コードをキーとしたMapで管理。
回答に応じて、次の質問へ。

次質問コードがなければ終了。

ついでに、結果データも、コード込みの結果情報をMapで管理するのも良いかもしれません。