unity GameObjectを一定時間だけ表示

前提・実現したいこと

unityでクイズゲームを作っています。
正誤判別をさせてボタンをクリックして正解だったら◯を、不正解だったら✖の表示を一定時間だけ表示させたい。
（不正解だった場合、ハートのＧameObjectの×３を減らす処理をさせたいと思っています。）

ここに質問の内容を詳しく書いてください。
using UnityEngine;
using System.Collections;
using UnityEngine.UI;

public class Judgde : MonoBehaviour{

public GameObject maru;
public GameObject batsu;

    //選択したボタンのテキストラベルと正解のテキストを比較して正誤を判定
    public void JudgeAnswer(){

    // 正解のデータをテキストでセットする
    string answerText = "１０万ｋｍ";
    // 選択したボタンのテキストラベルを取得する
    Text selectedBtn = this.GetComponentInChildren<Text>();

    if (selectedBtn.text == "１０万ｋｍ"){

        // 正解のボタンを押した時の○の表示
        if(Input.GetMouseButtonDown(0){

            Debug.Log("正解");
            maru.SetActive(true);
            
        }else{

            // 不正解のボタンを押した時の✖の表示
            if(Input.GetMouseButtonDown(0)){

                Debug.Log("不正解");
                batsu.SetActive(true);
                
            }
        }
    }
}

}

上記のようにスクリプトを書いたのですが、ボタンにアタッチしても何も起きません。
スクリプトがおかしいとは思うのですがどう修正してよいか分からないので教えてもらいたいです。

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

ボタンにアタッチしても何も起きない。
エラーメッセージ


### 該当のソースコード

```ここに言語名を入力
ソースコード
using UnityEngine;
using System.Collections;
using UnityEngine.UI;

public class Judgde : MonoBehaviour{

    public GameObject maru;
    public GameObject batsu;

        //選択したボタンのテキストラベルと正解のテキストを比較して正誤を判定
        public void JudgeAnswer(){

        // 正解のデータをテキストでセットする
        string answerText = "１０万ｋｍ";
        // 選択したボタンのテキストラベルを取得する
        Text selectedBtn = this.GetComponentInChildren<Text>();

        if (selectedBtn.text == "１０万ｋｍ"){

            // 正解のボタンを押した時の○の表示
            if(Input.GetMouseButtonDown(0){

                Debug.Log("正解");
                maru.SetActive(true);
                
            }else{

                // 不正解のボタンを押した時の✖の表示
                if(Input.GetMouseButtonDown(0)){

                    Debug.Log("不正解");
                    batsu.SetActive(true);
                    
                }
            }
        }
    }
}


### 試したこと
if(Input.GetMouseButtonDown(0)){　のところがおかしいと思い、public void OnClickにかえたりしてみた。


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

行動規範の内容に同意します

回答1件

ベストアンサー

#とりあえず動くようにする

分けて考えていきましょう。まずは提示されたソースを動くようにしていきます。
この書き方だと汎用性が無いので慣れてきたら改善すると良いと思います。ボタン側に正解かどうか判定させるんじゃなくて、クイズマネージャーみたいなのを作ってそこでジャッジするようにするとかですね。

cs
1using UnityEngine;
2using System.Collections;
3using UnityEngine.UI;
4
5public class Judgde : MonoBehaviour {
6
7    public GameObject maru;
8    public GameObject batsu;
9
10    //選択したボタンのテキストラベルと正解のテキストを比較して正誤を判定
11    public void JudgeAnswer() {
12
13        // 正解のデータをテキストでセットする
14        string answerText = "１０万ｋｍ";
15        // 選択したボタンのテキストラベルを取得する
16        Text selectedBtn = this.GetComponentInChildren<Text>();
17
18        if (selectedBtn.text == answerText) {
19            // 正解のボタンを押した時の○の表示
20            Debug.Log("正解");
21            maru.SetActive(true);
22
23        } else {
24            // 不正解のボタンを押した時の✖の表示
25            Debug.Log("不正解");
26            batsu.SetActive(true);
27        }
28    }
29
30}

Input.GetMouseButtonDown(0)は、マウスを左クリックしたときに動作しますが今回はボタンにアタッチするとのことでしたので不要かと思います。なのでそちらを削り、中カッコの数と位置をあわせます。
あとせっかく定義しているanswerTextが使われていなかったので、if文で利用するようにしています。

おそらくこれでボタンをクリックすると正解とか不正解が出るようになるかと思います。

#一定時間後に消す

ここから、一定時間後に消す方法を考えましょう。n秒後になにかするみたいなことはコルーチンを使うと簡単に実装できます。とりあえず時間を使った処理に便利なんだなぐらいの認識でも良いかと思います。詳しくはリファレンスなどを参照してください。

cs
1using UnityEngine;
2using System.Collections;
3using UnityEngine.UI;
4
5public class Judgde : MonoBehaviour {
6
7    public GameObject maru;
8    public GameObject batsu;
9
10    //選択したボタンのテキストラベルと正解のテキストを比較して正誤を判定
11    public void JudgeAnswer() {
12
13        // 正解のデータをテキストでセットする
14        string answerText = "１０万ｋｍ";
15        // 選択したボタンのテキストラベルを取得する
16        Text selectedBtn = this.GetComponentInChildren<Text>();
17
18        if (selectedBtn.text == answerText) {
19
20            // 正解のボタンを押した時の○の表示
21            Debug.Log("正解");
22            //maru.SetActive(true);
23            StartCoroutine(ShowImageSecond(maru, 2f)); // maruを表示させ、２秒後に非表示
24
25
26        } else {
27            // 不正解のボタンを押した時の✖の表示
28            Debug.Log("不正解");
29            //batsu.SetActive(true);
30            StartCoroutine(ShowImageSecond(batsu,2f)); // batsuを表示させ、２秒後に非表示
31        }
32    }
33
34    // 引数で当たられたオブジェクトを表示し、引数の秒数後に非表示にするコルーチン
35    IEnumerator ShowImageSecond(GameObject targetObj, float sec) {
36        targetObj.SetActive(true); //引数で指定されたオブジェクトを表示する
37        yield return new WaitForSeconds(sec); // 引数で指定された秒数待つ
38        targetObj.SetActive(false); //引数で指定されたオブジェクトを非表示にする
39    }
40
41}

SetActiveのかわりに、作成したコルーチンを呼ぶようになっています。
中身は簡単で、引数として渡したゲームオブジェクトをまず表示し、引数で渡した秒数後にまた非表示にするという仕組みです。

投稿2019/05/08 03:02

編集2019/05/08 03:03

hogefugapiyo

総合スコア3302

TIE_Atushi

2019/05/08 05:07

早々に回答ありがとうございます。実際に修正してみたのですが、ShowImageSecondのところでエラーがでました。重大度レベルコード説明プロジェクトファイル行抑制状態エラー CS0103 現在のコンテキストに 'ShowImageSecond' という名前は存在しません。 Assembly-CSharp C:\Users\kurat\Desktop\クイズ脱出ゲーム２\Assets\Judgde.cs 29 アクティブと上記のようになっていたのですがどうすればよいでしょうか？

TIE_Atushi

2019/05/09 02:40

スクリプトのエラーは自己解決できましたが、ボタンをクリックすると◯と✖が表示されますがそのままで消えません。たぶんですが、スタートコルーチンが起動してないように思うのですが、どうなんでしょうか？

行動規範の内容に同意します