NullReferenceだけは許さない…

Question

Unityで2Dアクションゲームを作っていて、アイテムをステージ上のランダムな地上に生成させるスクリプトを書いたのですが、下記コードでNullReferenceが呼び出されて若干萎えてます。

collider.tagだと処理が重くなるからCompareTagに変えてやったというのに、何がいけないんだーッ！

…何を参照すれば解決するのか教えて下さると助かります。

public class Item : MonoBehaviour
{
    /*　public変数　*/
    public float stageLeft, stageRight, stageHeight;

    /*　private変数　*/
    [SerializeField] GameObject itemPref = null;
    GameObject itemInstance;
    Ray ray;
    RaycastHit2D hit;
    Vector2 tPos;
    int count = 0;
    float distance = 11.0f;

    void Start()
    {
        while(count < 9)
        {
            tPos = new Vector2(Random.Range(stageLeft, stageRight), stageHeight);
            ray = new Ray(tPos, -transform.up);
            Debug.DrawLine(tPos, ray.direction * distance, Color.red);

            if(Physics2D.Raycast(tPos, -transform.up))
            {
                if (hit.collider.CompareTag("Ground"))    //←ここでNullReferenceが起こる
                {
                    itemInstance = Instantiate(itemPref) as GameObject;
                    itemInstance.name = itemPref.name;
                    itemInstance.transform.position = new Vector3(tPos.x, hit.collider.transform.position.y, -3.0f);

                    count++;
                }

                else
                {
                    continue;
                }
            }
        }
    }    

}

Comments

[退会済みユーザー]

> collider.tagだと処理が重くなるからCompareTagに変えてやったというのに、何がいけないんだーッ！

変更はその1点だけでしょうか。
また、「collider.tag」を使用し場合は発生しない、ということでよろしいでしょうか。

Answer 1

hitには何も代入されていません。
なので、hit.collider.CompareTag("Ground")でNullReferenceExceptionになるのは当然です。

以下のようにPhysics2D.Raycast(tPos, -transform.up)の返り値をhitに代入していないのが原因でしょう。

間違い：

            if(Physics2D.Raycast(tPos, -transform.up))
            {
                // …（中略）                
            }

正しい：

            hit = Physics2D.Raycast(tPos, -transform.up);
            if(hit) // 「if(hit != null)」でも同じ
            {
                // …（中略）                
            }

Comments

[kogarashi0917]

ご回答ありがとうございます。
実際に修正して実行してみたところ、固まったまま処理が進まなくなってしまいます。
Start()内で処理を行うのは危険だったりしますか？

[fiveHundred]

現状のコードだと、地面に設置していないと無限ループになります。
どこかでループを抜けない限り、次の処理が行えず、ゲーム全体がフリーズします。
このような書き方は、やってはいけません。
（コルーチンでループ内に「yield return ～;」と書き、適宜抜ける方法もありますが）

[kogarashi0917]

日を跨いでの質問失礼します。
public class Item : MonoBehaviour
{
/*　public変数　*/
public float stageLeft, stageRight, stageHeight;

/*　private変数　*/
[SerializeField] GameObject itemPref = null;
GameObject itemInstance;
Ray ray;
RaycastHit2D hit;
Vector2 tPos;
int count = 0;
float distance = 11.0f;

void Start()
{
StartCoroutine(ItemSetByRay());
}

IEnumerator ItemSetByRay()
{
while (count < 9)
{
tPos = new Vector2(Random.Range(stageLeft, stageRight), stageHeight);
ray = new Ray(tPos, -transform.up);
hit = Physics2D.Raycast(tPos, -transform.up);
Debug.DrawLine(tPos, ray.direction * distance, Color.red);
if (hit)
{
if (hit.collider.CompareTag("Ground"))
{
itemInstance = Instantiate(itemPref) as GameObject;
itemInstance.name = itemPref.name;
itemInstance.transform.position = new Vector3(tPos.x, hit.collider.transform.position.y, -3.0f);

count++;
}

else
{
continue;
}
}
}

yield break;
}
}

このようにコードを書き換えてみたのですが、yieldを差し込む位置としてはどこが良いのでしょうか？

[fiveHundred]

ちょっと勘違いしていたので、コルーチンやyieldのことは一度忘れてください（すみません）。
無限ループになっているのが問題であって、1フレーム内に収まればStart()の中でも特に問題はないです。

この場合、countが9になれば抜けますが、「Physics2D.Raycast(tPos, -transform.up);の結果がnullではないこと」「hit.collider.CompareTag("Ground")がtrueであること」の両方を満たしていなければ、countが増えません。
そのため、これらがどうやっても成立していなければ、無限ループになります。
例えば「位置などの関係でどうやってもGroundに当たらない」「そもそもGroundというタグのものが存在しない」場合がこれに該当するので、そうなっていないか一度確認してください。

また、質問とは関係ないですが、関数内でしか使わない変数は、関数の中で定義したほうがよろしいかと思います。

[kogarashi0917]

> この場合、countが9になれば抜けますが、「Physics2D.Raycast(tPos, -transform.up);の結果がnullではないこと」「hit.collider.CompareTag("Ground")がtrueであること」の両方を満たしていなければ、countが増えません。

なるほどなるほど…
先ほど上記のコードでループの原因を探ろうとしたところ、hit.colliderにCinemachineのCompositeColliderが代入されていました。
CompositeColliderにはタグ付けしていないので無限ループが発生してしまうのはわかるのですが、もしかしてRaycastとCinemachineは共存できなかったりするのでしょうか？

[fiveHundred]

Physics2D.RaycastにlayerMaskを指定すれば、レイが当たるレイヤーを指定することができます。
それを利用して、例えば「Ground」というレイヤーを作っておき、それのみをlayerMaskで指定すればよろしいかと。
（この場合、Groundタグが不要になるかもしれません）

あるいは、どうしてもレイヤーを指定したくないのであれば、Physics2D.RaycastAllで全て取得して、どれかがGroundタグであれば～、といった感じで処理することも出来ます。
（その分処理が多くなるので、前者をおすすめしますが）

[hogefugapiyo]

まずループさせずに1回の処理で作ってみたらどうですか？ それならデバッグも楽ですし。
ちゃんとGroundにアイテムが1個生成されたら、たとえばそれをループではなくて2回連続で処理してみて動くか確認。それも動くならループ文書いて最終的な目的を達成すればよいのではないかと

Answer 2

それが出る場所で実行を止めて、そこに現れる変数の値を片っ端からチェックしよう
nullになってるのはないでしょうか

Comments

[kogarashi0917]

ご回答ありがとうございます。
y_waiwaiさんのおっしゃる通り、hit.colliderがnullになっていました。汗

追加で質問して申し訳ないのですが、
このhit.colliderをnullでなくするには、あらかじめ宣言しておいたほうが良いのでしょうか？

[hogefugapiyo]

fiveHundred さんが答えてたので詳しくはそちらですが。キャストした結果をhitにいれて、hitがnullじゃなければ処理ってすれば、Raycastがヒットしたことを担保できるのでNullになりません

Answer 3

RaycastHit2D.collider

RaycastHit2D を返すいくつかの関数はコライダーを Null としたままにすることに注意してください。これは何も衝突してないことを示します。RaycastHit2D は明示的な変換の演算子を実装しているため、このプロパティーをチェックして衝突があったかどうかを boolean で返します。

上記の通り、「collider」がNullの場合があるため、使用する前に判定を行いましょう。