Update関数が一度だけ読み込まれるようにしたいです。

Question

前提・実現したいこと

unity，C#でスペースキーを押したら"public Text"に入れたテキストが削除されるようにしたいと思っています。
一度input.getkeydownが発動したら，"pushed"がfalseとなって、Update関数内のif文が以降発動しないようにしたかったのですが，スペースキーを押してテキストが消えた後も，アクセスしようとしていたみたいです。。支障はないとはいえ，エラーが出ないようにしたいので…。よろしくお願いします。

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

MissingReferenceException: The object of type 'Text' has been destroyed but you are still trying to access it.
Your script should either check if it is null or you should not destroy the object.

該当のソースコード

public class TextDeleteSpace : MonoBehaviour {
	public Text push;
	bool pushed;
	void Start () {
		pushed = true;
	}
	void Update () {
		if ((Input.GetKeyDown("space")) && (pushed))
		{
			Destroy(push);
			pushed = false;
		}
	}
}

Comments

[YAmaGNZ]

Destroyの前にログを出力してみてはどうでしょうか？
このスクリプトが複数アタッチされているとかありませんか？

[yellowtale]

ログ出力してみましたが、スペースキーを押して、テキストが消えてしまった後はif文の中身は実行されていなかったようです。

[Bongo]

MissingReferenceExceptionが発生しているのは本当にこのスクリプトでしょうか?
もし「pushed」によるDestroy再発動回避に失敗していたとしても、そもそも破壊済みのコンポーネントに再度Destroyを行ってもエラーは発生しないんじゃないかと思うのですが...

[yellowtale]

いやぁ…このスクリプトで間違いないんですけどね…。。

[Bongo]

なるほど...念のためおうかがいしますが、そのコンソール上に現れた「MissingReferenceException: The object of ...」のメッセージをクリックして選択すると、コンソールの下の方にごちゃごちゃとしたメッセージの全文が現れるかと思います（もし表示領域が狭いようでしたら、黒い区切り線をドラッグして広げてみてください）。それはどのような内容でしょうか?

Answer 1

多くの方にご相談いただいておいてとても申し訳ない気分なのですが。。。
テキスト自体に付けていた「テキストを点滅させるためのスクリプト」がアクセスしようとしていたようで、このスクリプト内ではきちんと処理されていたようです！

public class TextDeleteSpace : MonoBehaviour {
	public Text push1;
	public Text push2;
	public Text push3;
	public Text push4;
	bool pushed;

	public float speed;
	private float time;
	private enum ObjType{TEXT,};
	private ObjType thisObjType = ObjType.TEXT;

	void Start () {
		pushed = true;
		if (this.gameObject.GetComponent<Text>())
		{
				thisObjType = ObjType.TEXT;
				push1 = this.gameObject.GetComponent<Text>();
				push2 = this.gameObject.GetComponent<Text>();
				push3 = this.gameObject.GetComponent<Text>();
				push4 = this.gameObject.GetComponent<Text>();
		}
	}
	void Update () {
		if ((thisObjType == ObjType.TEXT) && (pushed))
		{
			push1.color = GetAlphaColor(push1.color);
			push2.color = GetAlphaColor(push2.color);
			push3.color = GetAlphaColor(push3.color);
			push4.color = GetAlphaColor(push4.color);
		}
		if ((Input.GetKeyDown("space")) && (pushed))
		{
			Destroy(push1);
			Destroy(push2);
			Destroy(push3);
			Destroy(push4);
			pushed = false;
		}
	}
	Color GetAlphaColor(Color color)
	{
			time += Time.deltaTime * 5.0f * speed;
			color.a = Mathf.Sin(time) * 0.5f + 0.5f;
			return color;
	}
}

（ちょっと長くなってしまいましたが）
こんな感じで点滅させるためのスクリプトをこのスクリプトに組み込んで、"pushed"の正負を条件にして解決しました。
協力してくださった皆様…情報不足のためにご迷惑をおかけしました。
申し訳ありませんでした。。。m(_ _)m

Answer 2

TextDeleteSpace実行してみましたがこちらの環境ではエラーは起こりませんでした
TextDeleteSpaceは問題ないと思います。たぶん原因は他にありそうです

もしDestroyしなくてもいいなら

push.text = "";

で空白にするとか

push.enabled = false;

で非表示にするとかでいいんじゃないでしょうか

Answer 3

該当箇所を一回しか通っていないということであれば、それまでにDestroyされている可能性があるのではないでしょうか？

リファレンスを見た限りですが、

if(push.isDestroyed == false) Destroy(push);

こういう風にできないですかね？

Answer 4

これでいけると思います。

public class TextDeleteSpace : MonoBehaviour {
    public Text push;
    bool pushed;
    void Start () {
        pushed = false;
    }
    void Update () {
        if ((Input.GetKeyDown("space")) && (pushed == false))
        {
            Destroy(push);
            pushed = true;
        }
    }
}

ちなみにnegitamaさんがおっしゃっている指摘は
pushedは「もう押された」という意味なので、
Startでpushed(もう押された)＝true(はい)となって、
Updateでキーを"押したとき"にpushed(もう押された)＝false(いいえ)
となるのは単語から想像される意味から言えばおかしくないですか？
という意味です。

Comments

[akki]

書いてたらかぶってしまいました…

[negitama]

補足ありがとうございます＾＾

Answer 5

####解決案１
&& （AND条件）の箇所を || （OR条件）に変える。
#### 解決案２
Start() 内で pushed = false に、if 文内での処理では pushed = true にする。
解決案２のほうが自然かもしれません。pushed って、「（もう）押された」って意味ですよね。

回答を修正しました。

public class TextDeleteSpace : MonoBehaviour {
    public Text push;
    bool pushed;           // ボタンがすでに押されたかをチェックする bool 型の変数
    void Start () {
        pushed = false;    // 最初は押されていない
    }
    void Update () {
        // スペースキーが押されて、かつ pushed == false（まだ押されていなかった）とき、処理を行う
        if ((Input.GetKeyDown("space")) && (pushed == false))
        {
            Destroy(push);
            pushed = true;  // 押された
        }
    }
}

Comments

[yellowtale]

１：or条件にするとスペースキーを押さなくても消えてしまうんです。。。
２：pushedは私が代名詞的に自分でつけたものなので、意味とかは関係ないと思いますが。。
trueとfalseを逆にするのは意味があるのでしょうか？

[negitama]

回答修正しました。ご確認ください。

[yellowtale]

negitamaさん，akkiさん，YAmaGNZさん，アドバイスありがとうございます。
ですが…やっぱりエラーが出てしまいますね。。エラーメッセージが出ないようにしたいのですが，どうにもなりませんかね…？