UNITY Update内の変数を別のスクリプトから監視したい

Question

シーンが複数ありまして、Rayでクリックされたオブジェクトの処理を共通化したかったので
Aスクリプトにrayの処理
Bスクリプトにヒットしたオブジェクトの実行処理
をわけて作成してみたのですが、Bスクリプト側でオブジェクトの識別ができません。

update自体が一定で呼ばれないということはある程度理解しているので
タイミングがずれて取得できていないということはわかるのですが
確実にBスクリプトからAスクリプトでヒットしたオブジェクトを監視するには
どのようなアイデアがありますでしょうか。

AScript.cs

public GameObject HitGameObject;
void Update(){
 
 var ray = Camera.main.ScreenPointToRay(Input.mousePosition);
 RaycastHit HitObject;

 if (Physics.Raycast(ray, out HitObject))
 {
   Debug.LogFormat("hit with {0}", HitObject);
   HitGameObject = HitObject.collider.gameObject;
   HitGameObject = null;
 }

}

BScript.cs

void Update(){
 
 if (A.HitGameObject != null)
 {
    Debug.Log(A.HitGameObject.name);
    audioSource.PlayOneShot(audioClip);
 }

}

Comments

[sakura_hana]

これでも動きそうな気がしますが「Bスクリプト側でオブジェクトの識別ができません。」というのは具体的にどういうことですか？（どうなってほしいのに、どういう状態になっていますか？）

[hogefugapiyo]

AからHitGameObject = null;を消して
BでHitGameObject = null;をやったらどうなりますか？

[Qoo]

null以外になればの処理に入りません( != null )

AからHitGameObject = null;を消して
BでHitGameObject = null;をやったらどうなりますか？

→ なるほど。これなら動きますね。ありがとうございます。

Answer 1

「スクリプトBで毎フレーム、スクリプトAのオブジェクトをチェックする」とした場合、スクリプト間の実行順制御が問題になります。

単に解決するなら「スクリプトBの処理をスクリプトAより後に実行させる」ために、スクリプトBのUpdateをLateUpdateに変えて解決する方法もあります。

---

さらに発展的な解決方法として、C#の Event を活用して、HitGameObjectが変わったことをを通知する方法はどうでしょうか。こちらは学習コストに見合ったより良い実装スキルが身につくと思いますので是非チャレンジしてみてください。

デリゲート(Delegate)やイベント(Event)とは【C#】
http://kan-kikuchi.hatenablog.com/entry/Delegate

（Event の説明は結構ページの半分以上スクロールしたところにあります。そこまで読み飛ばしてもいいと思いますが、Delegate は Event の基礎になるので出来れば上から読んでいくのもいいと思います。）

Eventの仕組みを使うと「処理すべきタイミングを知ってる側」と「処理したい内容を知ってる側」を分けられることが利点として挙げられると思います。

Eventを使った方法を質問のお話に直して考えた場合、例えば

• スクリプトAが「Rayがオブジェクトにヒットした」というイベントを発行して
• スクリプトBがスクリプトAのイベントを受け取れるようにする

といった実装になるかなと思います。

以下に参考例のコードを示します。

// スクリプトA（マウスと被ったオブジェクトをイベントとして公開する）

public delegate void OnMouseHitEventHanlder(GameObject hitObject);

public event OnMouseHitEventHanlder OnMouseHit;

public GameObject HitGameObject;

void Update()
{
    var hitObject = GetOnMouseHitGameObject();

    HandleOnMouseHit(hitObject);

    HitGameObject = hitObject;
}

void HandleOnMouseHit(GameObject hitObject)
{
    if (hitObject != null)
    {
        Debug.LogFormat("hit with {0}", hitObject);
        // イベントを発行
        OnMouseHit?.Invoke(hitObject);
    }
}

GameObject GetOnMouseHitGameObject()
{
    var ray = Camera.main.ScreenPointToRay(Input.mousePosition);
    return Physics.Raycast(ray, out var hitObject) 
        ? hitObject.collider.gameObject 
        : null
        ;
}

// スクリプトB（スクリプトAのイベントに反応して処理を実行する）

void Start()
{
    A.OnMouseHit += OnMouseHitAction;
}

void OnDestroy()
{
    A.OnMouseHit -= OnMouseHitAction;
}

void OnMouseHitAction(GameObject hitObject)
{
    Debug.Log(hitObject.name);
    audioSource.PlayOneShot(audioClip);
}

Comments

[Qoo]

ありがとうございます！
タミングのずれの懸念点もなくなり、updateのタイミングで呼べるので実行順番的にも管理しやすそうです！
ありがとうございました！

[Y0241-N]

外野から失礼します、EventやDelegateについてよく理解できていなかったので、私にとってもこちらの回答はとても参考になりました。ありがとうございます。

ただ、どうしても調べても分からなかった部分が3点あるので、tor4kichi様にお聞きしたいのですが、
Aスクリプトの[OnMouseHit?.Invoke(hitObject);]の部分なのですが[?]を付けているのは何のためなのでしょうか？試しに消したところ、変化があるように感じなかったので、必要性が気になりました。

Bスクリプトの[OnDestroy]メソッドに関してですが、これはどのタイミングで呼び出されるのでしょうか？RayがHitしているオブジェクトが外的要因で削除された際に呼び出され、OnMouseHitイベントからOnMouseHitActionメソッドを削除する、という認識で合っていますでしょうか？

Bスクリプト側でAのイベントが発行されたかを受信するには、AスクリプトをGetCompomentする必要があると思うのですが、これをstatic eventにするのはダメなのでしょうか？ネットで調べたところ、静的イベントは発行後に破棄されずGCとして残り続けるので良くない、との記述もあったので気になります。

[tor4kichi]

以下、Y0241-Nさんへの返信です。

・?は「OnMouseHitがnullの場合にInvokeの呼び出しをスキップする」という意図でつけています。イベント変数はイベントリスナーが登録されていないとnullを示すのでnull参照回避のためですね。

参考：https://qiita.com/zakuro9715/items/107b9aa60b114beb206a

・OnDestoryを書いているのは、あくまで「イベントリスナーを登録したら解除も面倒を見ないとGCされずに困る可能性がある」という意図です。ゲームロジック的な整合性は考慮していないコードになります。

・静的イベントは使ったことがないのでわかりません。もし必要であれば別件として質問を立ててみてはどうでしょうか。（私が回答するかはわかりませんが）

[Y0241-N]

tor4kichi様

早速のご回答ありがとうございます。
概ね理解いたしました、?を付ける重要性が分かってよかったです。

他の部分も丁寧な回答ありがとうございます、静的イベントにしても良いかどうかは、自分で調査した後に改めて必要であれば別件で質問を建てようと思います。