他のスクリプトのメソッドを呼び出したい

Question

一つのスクリプトで作ったメソッドを、他のスクリプトから呼び出したいのですがうまくいきません。
どこがまずいのか教えてください。

これが最初につくったスクリプトです。

using System.Collections.Generic;
using UnityEngine;

public class Sample : MonoBehaviour
{
    public float xspeed;
    public float yspeed;

    GameObject cir;
    GameObject cup;
    GameObject squ;

    Rigidbody2D rigidcir;
    Rigidbody2D rigidcup;
    Rigidbody2D rigidsqu;


    private void Start()
    {
        cup = GameObject.Find("Capsule");
        cir = GameObject.Find("Circle");
        squ = GameObject.Find("Square");
        Debug.Log(cir);
        rigidcir = cir.GetComponent<Rigidbody2D>();
        rigidsqu = squ.GetComponent<Rigidbody2D>();
        rigidcup = cup.GetComponent<Rigidbody2D>();
    }


      private void Update()
    {
        //Move();
    }


    public void Move()//これを呼び出したい
    {
        cir.transform.Translate(-xspeed, -yspeed, 0);
        squ.transform.Translate(-xspeed, -yspeed, 0);
        Debug.Log(cir);
    }





    private void OnTriggerEnter2D(Collider2D collision)
    {
        Debug.Log("ゴール");
    }


}

コード

次に、上のMove();を呼び出すためのスクリプトです。

using System.Collections.Generic;
using UnityEngine;

public class GameManager : MonoBehaviour
{

    Sample samp = new Sample();

    void Start()
    {
       
    }

    

    void Update()
    {
        samp.Move();
    }
}

コード

エラーは以下のように出ます。
「NullReferenceException: Object reference not set to an instance of an object
Sample.Move () (at Assets/Scripts/Sample.cs:40)
GameManager.Update () (at Assets/Scripts/GameManager.cs:19)」

以前にも似たようなエラーが出たのですが、そのときはスクリプトをオブジェクトにアタッチし忘れていました。
今回はそれも忘れないようにしています。
Debug.Log(cir);のcirには予定通り"circle"が入っていました。

よろしくおねがいします。

Answer 1

Sample samp = new Sample();は「Sampleクラスのインスタンスを新たに作る」という意味です。
欲しいのは「Circleオブジェクトに付与されたSampleクラスのインスタンス」なのでこれでは恐らく正しく動作しません。

なので以下のように書き換えて、
Sample samp = new Sample();→public Sample samp;
インスペクタからCircleオブジェクトをドラッグ＆ドロップしてください。

コードで指定するなら、
Sample samp = new Sample();→Sample samp;書き換え
GameManagerのStart()内にsamp = GameObject.Find("Circle").GetComponent<Sample>();追加
でもできます。（が、GameObject.Findは比較的重い処理なので可能ならインスペクタで指定の方がいいかなと思います。Sampleクラス内についても同様です）

Comments

[退会済みユーザー]

とてもよくわかりました。
これまで教材の本や動画の通りにやっていたのですが、オブジェクトをドラッグ＆ドロップする意味などが全然わかっていませんでした。
本当に助かりました。ありがとうございます。

Answer 2

まえもってsamp.Start() を呼び出しとけばいいんでは

って、Startの内容をコンストラクタに置こう、ってはなしかな？

Comments

[退会済みユーザー]

samp.Start() を呼び出す、というのは Sampleのprivate void Start()の内容をGameManagerに持ってくるということでしょうか。つまりメソッドを使いたい場合は、その度にStart()を呼び出すということでしょうか。
あと、samp.Start();をどこに書いてもエラーになってしまうのですが、どのように呼び出せばいいのでしょうか。