同じキー入力でSetActiveの切り替えの仕方

Question

前提

スペースキーで宝箱を開けた際に最初はプレイヤーを未表示にしてそこから同じキーを押したらプレイヤーを再度表示するようにしたいです

※trueにする時のキーは正直何でもいいです。

実現したいこと

・プレイヤーを一回目未表示で2回目は表示させるようにしたいです

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

Playerをfalseにはできるのですが再度trueにすることが出来ないです。

該当のソースコード

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using KanKikuchi.AudioManager;

public class aksmKey : MonoBehaviour
{
    public GameObject Player;
    public GameObject Item;
    public Sprite takarabako;
    SpriteRenderer sr;
    public bool isPushed = false;

    int count = 0;

    void Start()
    {
        sr = gameObject.GetComponent<SpriteRenderer>();
        Item.SetActive(false);
        Player.SetActive(true);
    }

    void Update()
    {

    }

    private void OnCollisionStay2D(Collision2D other)//これが原因？
    {
        if (isPushed) return;

        if (other.gameObject)
        {
            if (Input.GetKey(KeyCode.Space))
            {
                SEManager.Instance.Play(SEPath.ITEM_GET);
                SEManager.Instance.Play(SEPath.TAKARABAKO_OPEN);
                sr.sprite = takarabako;
                isPushed = true;
                Item.SetActive(true);
                count += 1;

                Player.SetActive(false);//もしかしたらこの書く場所も悪いかも？
            }
        }
    }
}

試したこと

elseを使って再度表示させようかと思ったんですけどうまくいきませんでした

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

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

ここにより詳細な情報を記載してください。

Answer 1

if (isPushed) return;が一番頭にきているので、isPushed = trueになった後にfalseに戻す方法がこのスクリプト内に無いので、永久にreturnし続けます。
入れるならif (Input.GetKey(KeyCode.Space))の下にif (isPushed)なら isPushed = falseとプレイヤー表示、else でisPushed = trueとプレイヤー非表示という形にすればいいと思います。

Comments

[P0NZUU_]

回答ありがとうございます。
今現在試してみましたが、未表示にはできるのですが再度表示がどうしてもできないです。

今の状況はスペースキーを押すと”プレイヤーを未表示”というログは流れますが、その後もう一度押してもチェックは消えずログもでない感じです。
このコードだとどこが悪いのでしょうか？

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using KanKikuchi.AudioManager;

public class aksmKey : MonoBehaviour
{
public GameObject Player;
public GameObject Item;
public Sprite takarabako;
SpriteRenderer sr;
public bool isPushed;

int count = 0;

void Start()
{
sr = gameObject.GetComponent<SpriteRenderer>();
Item.SetActive(false);
Player.SetActive(true);

isPushed = false;

}

void Update()
{

}

private void OnCollisionStay2D(Collision2D collision)
{
if (isPushed) return;

if (collision.gameObject.tag == "Player")
{

if (Input.GetKey(KeyCode.Space))
{
SEManager.Instance.Play(SEPath.ITEM_GET);
SEManager.Instance.Play(SEPath.TAKARABAKO_OPEN);
sr.sprite = takarabako;
Item.SetActive(true);
count += 1;

isPushed = false;

if (isPushed == false)
{
isPushed = true;
Debug.Log("プレイヤー未表示");
}
else if (isPushed == true)
{
isPushed = false;
Debug.Log("プレイヤーを表示");
}

}
}
}
}

[Y0241-N]

private void OnCollisionStay2D(Collision2D collision)
{
if (isPushed) return;　←これが原因です、これがあるとisPushedがtureになったあとはOnCollisionStay2Dメソッド以外でisPushedをfalseにしない限りtrueのままですよね？なのでtrueになっている限り、returnされ、return以下の処理が実行されることはありません。

あとif (Input.GetKey(KeyCode.Space))内のisPushed = false;の下にif (isPushed == false)、else if (isPushed == true)を書かれていますが、isPushed = falseにした後では必ずif (isPushed == false)の結果しか得られません。判定するならisPushed = false;する前に判定しましょう。

思うに基本的なC#の知識が不足されていると思うので、まず基礎知識を学ばれるのも良いかと思います。

[P0NZUU_]

細かくありがとうございます！
無事解決出来ましたが自分もまだまだ知識が浅いのでこれからも勉強します！