Как сделать чтобы метод работал только при первом запуске?

Question

[crowcs]

У меня есть класс который сохраняет и загружает прогресс игры, но при первом запуске из-за того что игра еще не имеет файлы сохранения у меня выдает ошибку. То что я могу сделать это создать отдельную сцену в меню, чтобы при нажатии старта игры у меня реализовывался метод SavePlayer(), но я не хочу создавать сцену ради одного метода. Я сделал в классе сохранения

public void Start()
    {
        SavePlayer();

        LoadPlayer();
    }

Но мне надо чтобы SavePlayer(); отрабатывал только 1 раз, иначе при каждом заходе сохранения будут сбрасываться, если поставлю первым LoadPlayer(); то при первом запуске будет ошибка

Answer 1

[GavriKos]

Дак проверяйте есть ли файлы и если нет - вызывайте сейв (или не вызывайте лоад)

Comments

[crowcs]

Пробовал, так не получится, вызывать метод только 1 раз за все время единственный выход

[GavriKos]

crowcs, у всех получается а у вас не получается? Показывайте код.

Смысл в том что чтобы вызывать метод только первый раз - нужно куда то записать инфу что это первый вызов. А это - тадам - тот же самый файл или плеерПрефс или еще какой сторейдж.

[crowcs]

GavriKos,

public class Player : Monobehaviour
    public void Start()
    {
        SavePlayer();
        LoadPlayer();
    }

 [Header("Player Info Parameters")]
    [Range(0, 250)] public int health = 100;
    [Range(0, 9999)] public int money = 0;
    
    [Range(0, 150)] public int level = 0;

    [Range(0, 150)] public int levelSpeedBust = 0;
    [Range(0, 150)] public int levelStrongBust = 0;
    [Range(0, 150)] public int levelHealthBust = 0;
    public int maxHealth = 100;


    public void SavePlayer()
    {
        SaveSystem.SavePlayer(this);
    }

    public void LoadPlayer()
    {
        GameData data = SaveSystem.LoadPlayer();

        health = data.health;
        money = data.money;

        level = data.level;

        maxHealth = data.maxHealth;
        
        levelSpeedBust = data.levelSpeedBust;
        levelStrongBust = data.levelStrongBust;
        levelHealthBust = data.levelHealthBust;

        Vector3 position;
        position.x = data.position[0];
        position.y = data.position[1];
        position.z = data.position[2];
        transform.position = position;
    }

public class SaveSystem
{
    public static void SavePlayer(Player player)
    {
        BinaryFormatter formatter = new BinaryFormatter();
        string path = Application.persistentDataPath + "/player.save";
        FileStream stream = new FileStream(path, FileMode.Create);


        GameData data = new GameData(player);

        formatter.Serialize(stream, data);
        stream.Close();
    }

    public static GameData LoadPlayer()
    {
        string path = Application.persistentDataPath + "/player.save";
        if (File.Exists(path))
        {
            BinaryFormatter formatter = new BinaryFormatter();
            FileStream stream = new FileStream(path, FileMode.Open);

            GameData data = formatter.Deserialize(stream) as GameData;
            stream.Close();

            return data;
        }
        else
        {
            Debug.LogError("Save file not found in" + path);
            return null; 
        }
    }
}

[System.Serializable]
    public class GameData
    {
       public bool autoSave;

       public int health;
       public int money;
       public int maxHealth;

       public int level;

       public int levelSpeedBust;
       public int levelStrongBust;
       public int levelHealthBust;
       public float[] position;

        public GameData(Player player)
        {
           health = player.health;
           money = player.money;
           maxHealth = player.maxHealth;

           level = player.level;

           levelSpeedBust = player.levelSpeedBust;
           levelStrongBust = player.levelStrongBust;
           levelHealthBust = player.levelHealthBust;

           position = new float[3];
           position[0] = player.transform.position.x;
           position[1] = player.transform.position.y;
           position[2] = player.transform.position.z; 
        } 
    }

[GavriKos]

crowcs, ну так в LoadPlayer обработайте случай если data == null и все.
Если ничего не загрузилось - то ничего никуда не меняем и не присваиваем

[crowcs]

GavriKos, каким образом, нихера не понял

[GavriKos]

crowcs, код писать не буду, поэтому постарайтесь включить голову и понять.
Вот есть участок кода:
GameData data = SaveSystem.LoadPlayer();
Он вызывается всегда, даже при первом старте. Верно? Верно.
Если пойдем во внутрь метода LoadPlayer, то увидим что если файла нет - то метод LoadPlayer вернет null.
соответственно
GameData data = SaveSystem.LoadPlayer();
в data будет null.
Дальше идет вот это:

health = data.health;

Но эта строка вызывает ошибку (ту что вы хотите решить путем костыля с "вызывать один раз сейв). Почему там ошибка? Потому что data - null, и никакого health у нее нет.
Значит что надо сделать? Проверить data на null, и все присвоения из нее делать только если она не null

[crowcs]

GavriKos, благодарю