unityでポーズ画面を作りたいのですが

Question

ポーズ画面をつくりたいのですが
ポーズ画面の項目が一瞬うつるだけになります

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

public class menu02 : MonoBehaviour
{

    [SerializeField] UnityEngine.UI.Text textbox01;
    [SerializeField] UnityEngine.UI.Text textbox02;
    [SerializeField] UnityEngine.UI.Text textbox03;
    [SerializeField] UnityEngine.UI.Image image01;
    [SerializeField] UnityEngine.UI.Image image02;
    public int Game_01 = 0;

    // Use this for initialization
    void Start()
    {

    }

    // Update is called once per frame
    void Update()
    {
        //入力判定のみ行う。実際の表示・非表示は別メソッドへ
        if (Input.GetButtonDown("メニュー"))
        {
            //非表示中なら表示、そうでない場合は非表示処理を呼ぶ
            if (textbox01.text == "")
            {
                Show();
            }
            else
            {
                Hide();
            }
        }
        if (Input.GetButtonDown("Jump"))
        {
            //問答無用で非表示
            Hide();
        }
    }

    //表示する処理
    private void Show()
    {
        textbox01.text = "メニュー";
        textbox01.enabled = true;
        textbox02.text = "メンバー";
        textbox02.enabled = true;
        textbox03.text = "閉じる";
        textbox03.enabled = true;
        image01.CrossFadeColor(targetColor: Color.red, duration: 0f, ignoreTimeScale: true, useAlpha: true);
        image01.enabled = true;
        image02.CrossFadeColor(targetColor: Color.yellow, duration: 0f, ignoreTimeScale: true, useAlpha: true);
        image02.enabled = true;
        image02.CrossFadeColor(targetColor: Color.red, duration: 1f, ignoreTimeScale: true, useAlpha: true);
    }

    //非表示にする処理
    private void Hide()
    {
        textbox01.text = "";
        textbox01.enabled = false;
        textbox02.text = "";
        textbox02.enabled = false;
        textbox03.text = "";
        textbox03.enabled = false;
        image01.CrossFadeAlpha(0, 0f, true);
        image01.enabled = false;
        image02.CrossFadeAlpha(0, 0f, true);
        image02.enabled = false;
    }
}

if文とfor文もまだ少し理解できてません。
このコードを簡素化か修正できたらおねがいします。

Answer 1

if (Input.GetButtonDown("メニュー"))の使い方の問題だと思われます。

＜現在の挙動＞
初期値：Game_01 = 0
if (Game_01 < 1)：条件満たす
image01.CrossFadeAlpha(0, duration, true);：消去される
if (Input.GetButtonDown("メニュー"))：入力あった場合
Game_01 += 1;：カウント追加
if (Game_01 > 0)：確実に条件満たす（おかしい）
image01.CrossFadeColor(~)：表示される
if (Input.GetButtonDown("メニュー")) { Game_01 -= 1; }：入力があるので再度非表示になる（おかしい）
次フレーム：Game_01 = 0

コードの流れが分かりづらいので、もし私が直すとしたら下記のような感じにするかなと。
（Game_01も不要っぽかったので消してしまっています、必要なら追加・調整お願いします）

void Update()
{
	//入力判定のみ行う。実際の表示・非表示は別メソッドへ
	if (Input.GetButtonDown("メニュー"))
	{
		//非表示中なら表示、そうでない場合は非表示処理を呼ぶ
		if (textbox01.text == "") {
			Show();
		} else {
			Hide();
		}
	}
	if (Input.GetButtonDown("Jump"))
	{
		//問答無用で非表示
		Hide();
	}
}

//表示する処理
private void Show() {
	textbox01.text = "メニュー";
	textbox02.text = "メンバー";
	textbox03.text = "閉じる";
	image01.CrossFadeColor(targetColor: Color.red, duration: 0f, ignoreTimeScale: true, useAlpha: true);
	image02.CrossFadeColor(targetColor: Color.yellow, duration: 0f, ignoreTimeScale: true, useAlpha: true);
	image02.CrossFadeColor(targetColor: Color.red, duration: 1f, ignoreTimeScale: true, useAlpha: true);
}

//非表示にする処理
private void Hide() {
	textbox01.text = "";
	textbox02.text = "";
	textbox03.text = "";
	image01.CrossFadeAlpha(0, 0f, true);
	image02.CrossFadeAlpha(0, 0f, true);
}

---

なお、質問内のコードのインデントが外れて非常に見にくいです。コード部分は「```」で囲んでください。
こちらもご確認ください。→質問するときのヒント｜teratail（テラテイル）

Comments

[manmanJ]

質問を変更しました
回答ありがとうございます！

[manmanJ]

if (Input.GetButtonDown("Jump"))
if (Input.GetButtonDown("メニュー"))
この両方でメニューを閉じたいのですが
書いてもらった
if (Input.GetButtonDown("Jump"))
{
//問答無用で非表示
Hide();
}
の後に
if (Input.GetButtonDown("メニュー"))
でいいですか？
あと最後のshowとhideの文の必要性がわからないのですが
どうゆう役割があるのでしょう？
と質問の消し方はできるのですか？

[sakura_hana]

やりたいことの確認をさせてください。
「"メニュー"ボタンを押した時、ポーズ画面が非表示なら表示する」
「"メニュー"ボタンを押した時、ポーズ画面が表示中なら非表示にする」
「"Jump"ボタンを押した時、ポーズ画面を非表示にする」
でいいですか？
もしそうなら新たにif (Input.GetButtonDown("メニュー"))を追加する必要はありません。
既存のif (Input.GetButtonDown("メニュー"))の中にあるif文でポーズ画面の表示状態を判定、表示非表示の命令を出しているためです。

---

>showとhideの文の必要性がわからない
とりあえず回答のスクリプトを編集しました。これで全文になりますので（コピペ元に誤字等無ければ）そのまま動くと思います。

「void 〜 () { }」となっている領域は「メソッド」と言います。（StartとUpdateもメソッドです）
詳しい話は下記を確認ください。
https://qiita.com/kenta71/items/3629ae1fa47eba9a38d7

超ざっくり言うと、「Show();」と書いてある行に差し掛かったら、「private void Show() { }」の中身が実行され、その後同じ場所に戻ってきます。
つまり

if (textbox01.text == "") {
Show();
}

↑と↓は同じ意味のプログラムです。

if (textbox01.text == "") {
textbox01.text = "メニュー";
textbox02.text = "メンバー";
textbox03.text = "閉じる";
image01.CrossFadeColor(targetColor: Color.red, duration: 0f, ignoreTimeScale: true, useAlpha: true);
image02.CrossFadeColor(targetColor: Color.yellow, duration: 0f, ignoreTimeScale: true, useAlpha: true);
image02.CrossFadeColor(targetColor: Color.red, duration: 1f, ignoreTimeScale: true, useAlpha: true);
}

if文の中にぞろぞろと長い行を書くと見通しが悪くなってしまったり、同じ内容を別の場所に何度も繰り返して書くと後で修正が大変です。
なのでプログラムでは適宜メソッドを分けておくのが推奨されています。
今回は非表示にする処理（Hide）を２箇所で呼び出す為にメソッド化。
ついでに見通しを良くする為に表示する処理（Show）もメソッド化しました。

---

質問の削除は以下をご覧ください。
https://teratail.com/help#delete-question

[manmanJ]

ありがとうございます。

private void Show()
{
textbox01.text = "メニュー";
textbox01.enabled = true;

true、falseを入れることで問題なく動作できるようになりました！

メニューからメンバーやスキルなどを選択することでメニュー中に新しくスキルメニューなどを
作るつもりでGame_01 = 0を作っていたのですが、必要はないですか？

あとtextbox01.text = "";などはまだ初心者なので
覚えるためとこれからメニューからメニューの時に変更するかもとゆうことなのでよろしくお願いします

[sakura_hana]

＞Game_01 = 0を作っていたのですが、必要はないですか？
何が最適かは「どんな風に画面が表示非表示されるのか」によるので一概には言えません。
「スキルメニューが開く時、ポーズ画面は閉じる」とかなら必要無いでしょうし、「現在どの画面が開いているかによって同じボタンでも機能が変わる」とかだと必要かもしれません。

そもそもintなのかboolなのか、変数を何個作るのか、なども考える必要があると思います。
（「nowWindow = 1」のように「１つの変数で今開いている画面を示す（0〜4のどれかが入る）」という方法もあれば、「isPauseWindow = true; isSkillWindow = false;」のように「複数のbool変数でそれぞれの表示状態を示す」という方法もあります）

あくまで私の回答は例でしかないので、お好きに変えてください。
スクリプトの意味を理解しつつ、どう書いたら効率的か・分かりやすいかを考えながらトライ＆エラーを続けると上達が早いかと思います。