配列を使って、指定したキーを押すとそこに格納されたオブジェクトだけを表示したい

Question

配列を使って、指定したキーを押すとそこに格納されたオブジェクトだけを表示したい

配列を使い、インスペクター上でキーとオブジェクトを数パターン指定しました。
そのキーを入力した際だけそのElementに格納したオブジェクトを表示したいのですが、どうコードを書けばよいのかわかりません。

例えば
Size 3　だとして
○Element 0
[Key] a
[Gameobject] cube1

○Element 1
[Key] s
[Gameobject] cube2

○Element 2
[Key] d
[Gameobject] cube3

と配置したときに
「a」を押すと　cube1がsetActive(true)になって、cube2とcube3がsetActive(false)になるイメージです。

どなたかおしえていだだけませんか。。。

現状はここまでしかできていません

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

public class test : MonoBehaviour
{
    [System.Serializable]
    public class ActiveObjects
    {
        public KeyCode KEY;
        public GameObject GameObject;
    }

    public ActiveObjects[] activeObjects;

}

Answer 1

変な回答してしまいました、申し訳ありません。
この書き方ならば動くはずです。

ですが、その前にメンバ名が気になったので訂正させてください。

[System.Serializable]
    public class ActiveObjects
    {
        public KeyCode key; // C#はフィールド(変数)は小文字スタートが推奨されていたはずです.
        public GameObject gmObj; // gameObjectはUnity内部で利用されているので, 混乱しないよう避けるべきです.
    }

private void Update()
{
	// キー監視
	for(var i=0;i<activeObjects.Length;i++)
	{
		// 対応するキーが押されたら
		if(Input.GetKeyDown(activeObjects[i].key))
		{
			ToggleObjects(i); //トグル開始
		}
	}
}

// 指定したオブジェクトを有効, ソレ以外を無効化するメソッド.
private void ToggleObjects(int idx)
{
	for(var i=0;i<activeObjects.Length;i++)
	{
		if(idx==i)
			activeObjects[i].gmObj.SetActive(true);
		else
			activeObjects[i].gmObj.SetActive(false);
	}
}

Comments

[yuuuuun]

すごい！コメントまでつけて丁寧に教えていただきありがとうございます；；
完璧に動作しました！
また、私が書いた変数まで訂正していただきありがとうございます；；
書き方もわかりやすくて、めちゃめちゃ助かりました。
これを元に勉強させていただきます！！

[Sado]

恐れ入ります。

Answer 2

キー押下時にactiveObjectsをひとつずつ比較するしか……。
(activeObjects[i].KEY == nowPressed)のような条件式で、trueならSetActive(true)、falseならその逆と、全ての要素に対して行えばよいかと思います。

これだけですと、非対応のキーが押された場合すべて消えてしまいますので、お望みの挙動をするよう工夫する必要があります。

Comments

[yuuuuun]

素早いアドバイスありがとうございます！
大変嬉しいのですが、unity初心者なもので、条件式とかあまり良くわかっておらず、実際にどうコードを書けばよいかわかりません、、、ごめんなさい；；

Sadoさんのやり方ですと間違って指定外のキーを押してしまうとすべて消えてしまうんですね、、、

[Sado]

そうですね、手っ取り早く書くと以下のようになります。
private void Update()に書く内容になります。
//
for(var i=0;i<activeObjects.Length;i++){
var isBtnDown= Input.GetKeyDown(activeObjects[i].KEY); //今押されたキーがソレか
activeObjects[i].gameObject.SetActive(isBtnDown);
}
//
配列内のすべてを比較し「キーが押されていたら有効化、押されていなければ無効化」のように書いています。
これは、全てのオブジェクトで「キーが押されていなければ無効化」が通ってしまうif文の書き方ですので、有効無効を切り替える前に「押されたキーが想定範囲内かどうか」を確認するべきでしょうね。

[Sado]

すみません、あまり深く考えずに書きました。
これでは色々穴がありますね。
押された瞬間だけ、有効無効をトグルする処理を書くべきでした。

[yuuuuun]

詳しくかいてくださり、ありがとうございます！
unityで実際に書いてみました。

そして。。。すみません！説明不足だったかもしれないのですが、キーを押してる間だけではなくて、イメージ的には、キーを押したら表示です；；