UnityでエラーコードObject reference not set to an instance of an object

Question

https://youtu.be/sPiVz1k-fEs
を参考にオンラインゲームを作っています。
Playerスクリプトに、

void Attack()
    {
      Collider2D[] hitEnemies = Physics2D.OverlapCircleAll(attackPoint.position, attackRadius, enemyLayer);
      foreach(Collider2D hitenemy in hitEnemies)
      {
          hitenemy.GetComponent<Player>().OnDamage(14);
      }
    }

で攻撃するメゾットと、

public void OnDamage(int damagePower)
  {
      healthPoint -= damagePower;
  }

を用意して、Attack()を利用し、オンライン上の敵に攻撃したところ、

NullReferenceException: Object reference not set to an instance of an object>

と、public void OnDamage(int damagePower)
の行にエラーが起きました。
調べたところ hitenemy.GetComponent<Player>().OnDamage(14);
の記述が問題とか、いつの間にか治るとかありますが、解決策と原因を教えてください

Comments

[ayousanz]

単純にPlayer.csをアタッチしているオブジェクトがないだけではないのでしょうか？

[loKA]

あります

[ayousanz]

ではhitenemy.GetComponent<Player>().OnDamage(14);をコメントアウトして，Debug.log(hitenemy.GetComponent<Player>())とかでnullかどうかを確かめてみてください

[loKA]

Debug.Logしたら、
GamePlayer(Clone) (Player)
UnityEngine.Debug:Log(Object)
と出てきました。
GamePlayerはPlayerに設定してるUnity上での名前です

[ayousanz]

では，そのDebugLogとエラー文のどうなっていますか？
DebugLogが確認できる場合はPlayer.csはあるということになります
同じ感じでOnDamage()を呼び出してください

[loKA]

同じ感じとはどういうことですか？
エラー文は出ませんでしたが、.OnDamage(14);を追加すると最初と同じエラーがそのままでます

[ayousanz]

言葉足らずでした．
GetComponentでPlayerが呼べていて，OnDamageのみが呼び出せないということはOnDamageの中がおかしいことになると思います．
その関数が正常に呼べているのかを確認してみてください

[loKA]

foreach(Collider2D hitenemy in hitEnemies)
{
OnDamage(16);
　　}
で相手にじゃなく、攻撃したら自分にくらうようにしたら自殺攻撃できます
やはりGetComponent<>().ondamege()が無理なようで....

[ayousanz]

怪しそうなところがわかったので，その辺で原因を探してみてください
さすがにこれ以上は上記のコード等からでは私にはわかりかねます．．申し訳ないです

Answer 1

取得したhitEnemiesの中に一つでもPlayerがアタッチされていないGameObjectがあると例外が発生します。GetComponentしたあと、nullチェックをしてみてください。

    foreach(Collider2D hitenemy in hitEnemies)
    {
        var player = hitenemy.GetComponent<Player>();
        if(player != null) player.OnDamage(14);
    }

Comments

[loKA]

if(player != null) player.OnDamage(14);
の行に同じエラーがでます...

[takezoux2]

それだとOnDamageメソッドの実装の方に問題ありそうですね
healthPointがプロパティになってて、なにか他のobjectを参照しているとか無いですか？
Playerクラスの実装もすべて書いてもらったほうが良いかと思います。

[loKA]

OnDamage（）の中に
audioSource.PlayOneShot(damageSound);
animator.SetTrigger("Hurt");とサウンドとアニメーターの設定をしていました。
これをコメントアウトして無効化したところうまく行きました。
どうすればaudioSouceやanimatorを設定できるんでしょうか...

[takezoux2]

audioSourceやanimator変数は、自分で定義した定数ですよね？
でしたら、
* GUIのペインのほうで、D&Dで参照を設定
* AwakeやStartなどで、GetComponentして代入しておく
のどちらかが必要かと思います。

Answer 2

GetComponentをCollider2Dから呼び出しているからではないでしょうか。
コライダー.gameObjectから呼び出してみてください。
または地形など他のものとの衝突が発生してそれにPlayerコンポーネントがアタッチされていない
などではないでしょうか。

Comments

[loKA]

hitenemy.gameObject.GetComponent<Player>().OnDamage(16);
にしても、同じようにエラーが起きました...