unityクイズゲームで正解数を正しく表示させたい

Question

前提、実現したいこと

unityを使い4択のクイズゲームを作成しています。

クイズの正解数をカウントし最後に何問正解したかを表示したいです。
ですが、得点が2点づつカウントされてしまいます。
これを1点づつ増やしたいです。

なぜこうなるのか、解決のヒントなどを教えてくださると幸いです。

https://engineer.blog.lancers.jp/ios/ugui_quiz_unity/
こちらのサイトを参考に作っていました。

エラーなどは出ていないです。

発生している問題

1
UnityEngine.Debug:Log (object)
ResultMgr:Start () (at Assets/Scripts/ResultMgr.cs:33)

2
UnityEngine.Debug:Log (object)
ResultMgr:Start () (at Assets/Scripts/ResultMgr.cs:33)

1問正解するごとに2回出てきます。

該当のソースコード

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.UI;

public class ResultMgr : MonoBehaviour
{
    //他のスクリプトからも参照可能な変数宣言
    public static string g_judgeData;
    public static int g_scoreData;


    void Start()
    {
        //デフォルトは正解、不正解なら画像と文言を切り替える
        if (g_judgeData == "不正解")
        {
            //現在描画している画像を取得
            SpriteRenderer judgeImage = GameObject.Find("JudgeUI/JudgeImage").GetComponent<SpriteRenderer>();
            //Resourcesから指定した名前の画像データをロード
            Sprite loadingImage = Resources.Load<Sprite>("batsu");
            //画像を置換
            judgeImage.sprite = loadingImage;
            //表示テキストを取得して置換
            Text judgeLabel = GameObject.Find("JudgeUI/JudgeLabel").GetComponent<Text>();
            judgeLabel.text = "不正解";
        }
        else if (g_judgeData == "正解")
        {
            //正解であればScoreを足す
            g_scoreData++;
            Debug.Log(g_scoreData);
        }
    }

    //他のスクリプトからも参照可能な関数宣言
    public static void SetJudgeData(string judgeData)
    {
        g_judgeData = judgeData;
    }

    public static int GetScoreData()
    {
        return g_scoreData;
    }

    public static int SetScoreData(int scoreData)
    {
        g_scoreData = scoreData;
        return g_scoreData;
    }
}

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.UI;

public class Score : MonoBehaviour
{
    // Use this for initialization
    void Start()
    {
        //スコア表示用のゲームオブジェクトを取得
        Text scoreLabel = GameObject.Find("Canvas/Score").GetComponent<Text>();
        scoreLabel.color = Color.red;
        //グローバルに宣言したスコアをResultMgrのスクリプトから読み込む
        int Score = ResultMgr.GetScoreData();
        scoreLabel.text = Score.ToString() + " 問正解";
    }
}

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.UI;
using System.IO;

public class Judge1 : MonoBehaviour
{

    QuizMgr quizMgr;
    string answerText;

    void Start()
    {
        quizMgr = GameObject.Find("Main Camera").GetComponent<QuizMgr>();
        answerText = quizMgr.AnswerText;
    }

    //選択したボタンのテキストラベルと正解のテキストを比較して正誤を判定
    public void Answer()
    {
        //選択したボタンのテキストを取得する
        Text selectedBtn = this.GetComponentInChildren<Text>();
        Debug.Log("セレクト'" + selectedBtn.text + "'");
        Debug.Log("アンサー'" + answerText + "'");

        if (answerText == selectedBtn.text){
            ResultMgr.SetJudgeData("正解");
            Debug.Log("正解");
            Application.LoadLevel("Result1");
        } else {
            ResultMgr.SetJudgeData("不正解");
            Debug.Log("不正解");
            Application.LoadLevel("Result1");
        }
    }
}

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.UI;
using System.IO;
using System.Linq;

public class QuizMgr : MonoBehaviour
{
    public string dataName;
    public string title;
    public TextAsset csvFile;
    public List<string[]> csvDatas = new List<string[]>();
    public int height = 0;
    public int i, j = 0;
    public int k = 0;
    internal Text ansLabel;
    public string AnswerText;
    public string Kaisetsubun;
    const int size = 5;


    void Start()
    {
        title = "theme_";
        csvFile = Resources.Load("CSV/" + title + dataName) as TextAsset;
        StringReader reader = new StringReader(csvFile.text);

        while (reader.Peek() > -1)
        {
            string line = reader.ReadLine();
            csvDatas.Add(line.Split(','));
            Debug.Log("reading:" + height);
            height++;
        }

        for (i = 0; i < height; i++)
        {
            for (j = 0; j < size; j++)
            {
                Debug.Log("csvDatas[" + i + "][" + j + "]:" + csvDatas[i][j]);
            }
        }

        QuestionLabelSet();
        AnswerLabelSet();
        AnswerSet();
    }


    public void QuestionLabelSet()
    {
        csvDatas[k] = csvDatas[Random.Range(0, 9)];
        
        //特定の名前のオブジェクトを検索してアクセス
        Text qLabel = GameObject.Find("Quiz/Image/QLabel").GetComponentInChildren<Text>();
        //データをセットすることで、既存情報を上書きできる
        qLabel.text = csvDatas[k][0];
    }

    public void AnswerLabelSet()
    {
        //問題文に対応した答えをそれぞれのuGUIボタンにセット
        string[] array = new string[] { csvDatas[k][1], csvDatas[k][2], csvDatas[k][3], csvDatas[k][4] };

        //問題文をシャッフル
        array = array.OrderBy(x => System.Guid.NewGuid()).ToArray();


        //ボタンが4つあるのでそれぞれ代入
        for (int i = 1; i <= 4; i++)
        {
            Text ansLabel = GameObject.Find("Quiz/AnsButton" + i).GetComponentInChildren<Text>();
            ansLabel.text = array[i - 1];
        }
    }

    public void AnswerSet()
    {
        //答えとなるcsvデータを変数として型に代入する
        AnswerText = csvDatas[k][1];
        Debug.Log ("アンサーセット'" + AnswerText + "'");

        Kaisetsubun = csvDatas[k][5];
        Debug.Log(Kaisetsubun);
    }

    void Update() { }
}

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.UI;
using System.IO;
using System.Linq;

public class Game1 : MonoBehaviour
{

	public static int qCount;

	public void NextQuiz()
	{
		if (Application.loadedLevelName == "Result1")
		{
			if (qCount < 5)
			{
				qCount++;
				Application.LoadLevel("1F");
			}
			else
			{
				qCount = 0;
				Application.LoadLevel("Score1");
			}
		}
	}

	
}

補足情報

unityは2021.2.2f1です。

勉強し始めて間もなく、知識不足なのですがよろしくお願いします。

Answer 1

実行するスクリプトがほかの場所にもアタッチされており2回動作していました。

とても拙い質問でお2人にはご迷惑をおかけしました。
本当にありがとうございます。

Comments

[fiveHundred]

低評価は私ではないですが、「同じ関数が（理由は不明だが）2回呼ばれている」ということになるので、対処法として間違っていると思います。

[taaiii]

確かにそうですね。
ご指摘ありがとうございます。
どこか間違えてそうな箇所があれば教えていただきたいのですが分かりますでしょうか？

[fiveHundred]

分かりませんが、とりあえずデバッガでどこから呼ばれているのか確認するといいと思います。

[fana]

Unity は知らんのですが，「MonoBehaviour　Start」でググると

> Start はスクリプトが有効で、Update メソッドが最初に呼び出される前のフレームで呼び出されます

とのこと．
文言の雰囲気的に「各インスタンス毎に１回だけ呼ばれる」っぽい感じを受けますが，
ResultMgr型のインスタンスはいつどうやって生成されたり破棄されたりするのでしょう？
同時に２つ生成されてたりとか，あるいは　生成→破棄→生成　みたいなことになってたりしないのでしょうか？

[taaiii]

ネットの記事を参考にプログラムを組んでおり、自身も詳しくはないのでインスタンスがどのようになっているのかは分かりません。

インスタンスの生成や破棄はどのように確認するのでしょうか？
デバッグで出来ますか？

教えていただけると嬉しいです。

[fana]

前述のように，私はUnityを知らないので助力できませんが……

個人的な感覚ではインスタンスがいつどうやって生成されるのかを把握してない状態でプログラム作るっていうのが考えられないです．
そのまま進めても同じようなことが今後も起きそうですし，一度，そこらへんのルールというか仕組みについて調べてみては？

[fiveHundred]

この場合、着目してほしいのはResultMgrですから、ResultMgrはどのシーンに配置されているのか、それが初期状態で有効なのか無効なのか、それを生成・破棄・有効化・無効化しているコードがどうなっているのか、という点が分からないので、答えられない感じですね。

完全な推測ですが、シーンAのResultMgrのStart()→シーンチェンジ→シーンBのResultMgrのStart()となっているのではないか、というのが怪しいですね。

[fiveHundred]

もしかしたら、最初の選択肢を選択→g_judgeDataに「正解」をセット→スコア表示シーンでResultMgrのStart()が実行→次の問題のシーンでResultMgrのStart()が実行、となっているような気がします。

[taaiii]

>ResultMgrはどのシーンに配置されているのか、それが初期状態で有効なのか無効なのか、それを生成・破棄・有効化・無効化しているコードがどうなっているのか、という点が分からないので、答えられない感じですね

今あるスクリプトを追加しました。
使用しているシーンは、問題と選択肢がある「1F」と〇×の画像を切り替え正解不正解を表示する「Result1」、これらを何回か繰り返し、最終スコアを表示する「Score1」があります。

ResultMgrはResult1の〇のイメージにアタッチしています。
これらは参考になりますでしょうか？

[fiveHundred]

本当にResultMgrはResult1の〇のイメージ「のみ」にアタッチしていますか？

あと、解決していないのであれば、ベストアンサーは解除したほうがいいと思います。

また、参考のURL自体、初心者が書いているのではないかと疑いたくなるものなので、他を参考にしたほうがいいかもしれません。
例えば、「他のスクリプトからも参照可能な変数(関数)宣言」としてstaticにしていますが、これは全くのデタラメで、static以外でも参照する手段はあります。
staticは「クラスに一つしかない共通の変数」として運用すべきです。
今回の場合、「ResultMgrのインスタンスが存在しないのにg_judgeDataとg_scoreDataが存在する」状況にもなっていますし、これが意図的でなかったとしたら、適切でないことは明確です。
もっとも、シーン切り替え時に全てのゲームオブジェクトは（デフォルトでは）消えてしまうので、結局使うことになるかもしれませんが、ざっと読んだ限り記事のどこにもそのような記載は見当たりませんでした。

[taaiii]

Result1が背景のpanelにもアタッチされていました。
それを削除すると正解がちゃんと増えていくようになりました！

こちらの確認ミスです！
本当に申し訳ないです。

参考のURLは他のも調べながら進めていこうと思います。

このような自分に丁寧に教えてくださり本当にありがとうございます！
今後もっと自分で勉強していきます。