Unity ゲームデータ セーブ機能 エラー

Question

前提・実現したいこと

Unityでセーブロードのシステムを作っています。

実行前はエラーメッセージはないのですが、SaveLoadFileを関連付けているButtonを押すと、エラーメッセージが出てきます。

どのように解決すれば良いのでしょうか？
教えてください。
よろしくお願いします。

発生している問題・エラーメッセージ

エラーメッセージ
NullReferenceException: Object reference not set to an instance of an object
SaveLoadFile.Save () (at Assets/SaveLoadFile.cs:41)
UnityEngine.Events.InvokableCall.Invoke () (at <ef667cdc82bf4a4e911d1ddea9ff581d>:0)
UnityEngine.Events.UnityEvent.Invoke () (at <ef667cdc82bf4a4e911d1ddea9ff581d>:0)
UnityEngine.UI.Button.Press () (at C:/Program Files/Unity/Hub/Editor/2020.3.9f1/Editor/Data/Resources/PackageManager/BuiltInPackages/com.unity.ugui/Runtime/UI/Core/Button.cs:68)
UnityEngine.UI.Button.OnPointerClick (UnityEngine.EventSystems.PointerEventData eventData) (at C:/Program Files/Unity/Hub/Editor/2020.3.9f1/Editor/Data/Resources/PackageManager/BuiltInPackages/com.unity.ugui/Runtime/UI/Core/Button.cs:110)
UnityEngine.EventSystems.ExecuteEvents.Execute (UnityEngine.EventSystems.IPointerClickHandler handler, UnityEngine.EventSystems.BaseEventData eventData) (at C:/Program Files/Unity/Hub/Editor/2020.3.9f1/Editor/Data/Resources/PackageManager/BuiltInPackages/com.unity.ugui/Runtime/EventSystem/ExecuteEvents.cs:50)
UnityEngine.EventSystems.ExecuteEvents.Execute[T] (UnityEngine.GameObject target, UnityEngine.EventSystems.BaseEventData eventData, UnityEngine.EventSystems.ExecuteEvents+EventFunction`1[T1] functor) (at C:/Program Files/Unity/Hub/Editor/2020.3.9f1/Editor/Data/Resources/PackageManager/BuiltInPackages/com.unity.ugui/Runtime/EventSystem/ExecuteEvents.cs:262)
UnityEngine.EventSystems.EventSystem:Update() (at C:/Program Files/Unity/Hub/Editor/2020.3.9f1/Editor/Data/Resources/PackageManager/BuiltInPackages/com.unity.ugui/Runtime/EventSystem/EventSystem.cs:385)

該当のソースコード

ソースコードusing UnityEngine;
using System.Collections;
using UnityEngine.UI;
using System.IO;
using System.Runtime.Serialization.Formatters.Binary;
using System;

public class SaveLoadFile : MonoBehaviour
{

    //　入力フィールド
    [SerializeField]
    public string inputField;

    [SerializeField]
    public float HPdata;
    [SerializeField]
    public float playerxdata;
    [SerializeField]
    public float prayerydata;

    //　ファイルストリーム
    private FileStream fileStream;
    //　バイナリフォーマッター
    private BinaryFormatter bf;

    private CreateNewData createNewData;

    private player PlayerSclipt;
    private DamageCheckSclipt HPSclipt;


    void Start()
    {
        createNewData = GetComponent<CreateNewData>();
    }

    public void Save()
    {

        HPdata = HPSclipt.HP();
        playerxdata = PlayerSclipt.X();
        prayerydata = PlayerSclipt.Y();


        createNewData.GetSaveData().SetHp(HPdata);

        createNewData.GetSaveData().SetPlayerX(playerxdata);

        createNewData.GetSaveData().SetPlayerY(prayerydata);


        inputField = createNewData.GetSaveData().GetJsonData();


        bf = new BinaryFormatter();
        fileStream = null;

        


        try
        {
            //　ゲームフォルダにfiledata.datファイルを作成
            fileStream = File.Create(Application.dataPath + "/filedata.dat");
            //　クラスの作成
            Data data = new Data();
            //　入力フィールドのテキストをクラスのデータに保存
            data.dataText = inputField;
            //　ファイルにクラスを保存
            bf.Serialize(fileStream, data);
        }
        catch (IOException e1)
        {
            Debug.Log("ファイルオープンエラー");
        }
        finally
        {
            if (fileStream != null)
            {
                fileStream.Close();
            }
        }
    }

    public void Load()
    {
        bf = new BinaryFormatter();
        fileStream = null;

        try
        {
            //　ファイルを読み込む
            fileStream = File.Open(Application.dataPath + "/filedata.dat", FileMode.Open);
            //　読み込んだデータをデシリアライズ
            Data data = bf.Deserialize(fileStream) as Data;
            //inputField.text = data.dataText;
        }
        catch (FileNotFoundException e1)
        {
            Debug.Log("ファイルがありません");
        }
        catch (IOException e2)
        {
            Debug.Log("ファイルオープンエラー");
        }
        finally
        {
            if (fileStream != null)
            {
                fileStream.Close();
            }
        }

    }

    //　保存するデータクラス
    [Serializable]
    class Data
    {
        public string dataText;
    }
}

###試したこと
41行目を消しても42行目に、42行目を消しても43行目にエラーメッセージが出てくるのでその3行がエラーの原因だと思います。

Comments

[1T2R3M4]

調べたこと、試したこと、Debugした結果を質問に追記していただけませんか。

Answer 1

41行目を消しても42行目に、42行目を消しても43行目にエラーメッセージが出てくるのでその3行がエラーの原因だと思います。

HPdata = HPSclipt.HP();
playerxdata = PlayerSclipt.X();
prayerydata = PlayerSclipt.Y();

HPScliptとPlayerScliptにオブジェクトのコンポーネントを和エリアてる必要があります。そのためにはいくつか方法がありますが、例えば29,30行目を下記のように修正し、インスペクター上で対応するオブジェクトのコンポーネントをドラッグアンドドロップすればよいです。

//private player PlayerSclipt;
//private DamageCheckSclipt HPSclipt;
public player PlayerSclipt;
public DamageCheckSclipt HPSclipt;

Comments

[nanakaka]

privateで取得しているスクリプトをすべてpublicで取得したらできました
ありがとうございます！

Answer 2

どのように解決すれば良いのでしょうか？

エラーメッセージに書かれている通り、41行目が実行されたときHPScliptにオブジェクトが設定されていないことが原因です。
HPScliptに何らかの値をセットしてください。
何をセットすればいいのかは、どのような動作を作りたいかによって質問者さんが決める必要があります。

Comments

[nanakaka]

他のスクリプトに下のようなスクリプトがあるのですが、これではHPScliptに値をセットできていないのでしょうか

どのあたりが間違っているのかも教えていただけると幸いです。

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.UI;

public class DamageCheckSclipt : MonoBehaviour
{
private string Damagetag = "enemytag";
public Slider slider;
public float hp;

void Start()
{
hp = 20f;
}

void Heal()
{
if(Input.GetKeyDown("2"))
{
if (hp < 20)
{
hp++;
slider.value = hp;

}
}
}

private void OnTriggerEnter2D(Collider2D collision) //コライダーの中に別のコライダーが入ったら
{
if (collision.tag == Damagetag)
{
if (hp > 0)
{
hp--;
slider.value = hp;

}

}
}

public float HP()
{
return hp;
}

}

[ku__ra__ge]

できません。具体的にはこのスクリプトのなかのどの行でHPScliptに値をセットできると考えたのでしょうか？