ゲームの『つづきから』ボタンを押したら前回のプレイしていたステージから始めるような導入したいです。

Question

マリオのような3Dアクションゲームを作りたいです。ステージがいくつかあり、それぞれ、シーン別にStage1、Stage２のような名前をつけて管理しています。スタート画面を作り続きからという項目を設けて、アプリを一回落としても続きからを押せば前回プレイしていた、ステージからスタートできるような仕組みを作りたいです。その機能を模索し、つづきからボタンに以下のコードをアタッチしたのですが、続きからボタンを押す前に勝手にシーンに移動をしてしまいます。どうすれば続きからボタンを押したあとに続きから始める事が可能になりますか？以下続きからボタンに記述したコードです。```ここに言語を入力
コード

using System.Collections.Generic;
using UnityEngine;

public class ContinuneButton : MonoBehaviour
{
    void Start ()
    {
    }
    public void OnClickStartButton()
    {
        Invoke(nameof(OnApplicationPause),0.5f);
    }

    void OnApplicationPause (bool isPause)
    {
	    DontDestroyOnLoad(gameObject);
        Application.LoadLevel( PlayerPrefs.GetInt("SceneNo", 1));
        if( isPause )
	    {
            PlayerPrefs.SetInt("SceneNo", Application.loadedLevel);
            // ?.
            PlayerPrefs.Save();
	    }

    }
}

Answer 1

OnApplicationPause()は、一時停止時とその解除時に自動で実行される関数です（参考）。
そのため、OnApplicationPause()が自動で呼び出されてシーン切り替えが起こったのでしょう。

今回の場合、そもそも「シーン切り替えを行う処理」と「セーブを行う処理」を同じタイミングで行うこと自体おかしいです。
「シーン切り替えを行う処理」を別の関数にし、それをクリック時に呼び出せばよいかと思います。

Comments

[Abc_.c]

ご回答ありがとうございます！
https://teratail.com/questions/2doq0xz51t5iqj の質問にてコメント頂いたためこちらに書かせていただきました
上の質問とURLの質問は関連性があります。ただ上記のコードを試してもうまくいかずそもそも根本的にPlayerPrefsについて理解しておらず違った角度にて別の質問をさせていただきました

[fiveHundred]

これはPlayerPrefsの使い方の問題ではなく、ロジックの考え方の問題だと思います。

意図を察するに、
- 『ボタンを押下した』ときに『ロードしてシーンを切り替える』
- 『一時停止した』ときに『セーブを行う』
ということだと思いますが、タイミングや処理は全く異なるのに、同じOnApplicationPause()にこの2つの処理を書いているのが問題だということです。
そんなことをしてしまったせいで、「一時停止したなどでOnApplicationPause()が実行されたときにセーブ処理の前にロード処理が行われる」というめちゃくちゃなコードになっています。

また、もう1つの質問についても、GoalCall()はセーブ処理のみを行うべきであって、ロード処理は行う必要はありません。
タイトル画面などでロードする際に処理すればいいだけのことです。

「PlayerPrefsのロードやセーブは同じ関数内でしか使えない」というわけではありません。
PlayerPrefsは同じアプリ内であれば、違う関数やスクリプトでも同じキーで取得できます。
むしろ、前述の通りセーブするタイミングとロードするタイミングが異なるので、その点を理解していれば自然と別々になるはずです。

[Abc_.c]

PlayerPrefsを使って以下のコードの情報をセーブするには具体的にどのように書けば良いのでしょうか？PlayerPrefsについての記事などを色々漁ったのですがみても使い方がイマイチ理解できないんです。　セーブとロードは別でやるということがわかりました。ロードはスタート画面に続きからボタンを設置してあるのでそちらで処理をすれば良いのですよね？
void GoalCall()//次のステージへ繊維
{
goalCall.SetActive(false);
int index = SceneManager.GetActiveScene().buildIndex;//現在のシーンのbuild数を呼び出す。
index++;//現在ステージのインデックスに+１する。
SceneManager.LoadScene(index);//+１したインデックス番号のステージを呼び出す。
}
}

[fiveHundred]

index++;//現在ステージのインデックスに+１する。
と
SceneManager.LoadScene(index);//+１したインデックス番号のステージを呼び出す。
の間にセーブする処理を入れればいいです。

[Abc_.c]

以下にセーブ処理を追記したものを書いたのですが、以下のような記述で問題ないですか？
void GoalCall()//次のステージへ繊維
{
goalCall.SetActive(false);
int index = SceneManager.GetActiveScene().buildIndex;//現在のシーンのbuild数を呼び出す。
index++;//現在ステージのインデックスに+１する。
PlayerPrefs.SetInt("key1", index);
PlayerPrefs.Save ();
SceneManager.LoadScene(index);//+１したインデックス番号のステージを呼び出す。
}

[fiveHundred]

少なくとも私の認識の上ではこれで問題は無いです。

[Abc_.c]

ありがとうございます。以下『続きからボタン』にアタッチしているスクリプトにロードをするコードを以下に書きましたがそれも書き方的に問題はないですか？エラーなく、適当にステージをプレイし、一回ゲームを消して、再度ゲームプレイし、続きからボタンを押したところ問題なく前回のクリアデータが保存されており、続きのステージからプレイできました

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.SceneManagement;

public class ContinuneButton : MonoBehaviour
{
void Start ()
{
}
public void OnClickStartButton()
{
Invoke(nameof(LoadData),0.5f);
}

public void LoadData()
{
int num = PlayerPrefs.GetInt ("key1");
string character = num.ToString();
SceneManager.LoadScene("Stage" + character);//+１したインデックス番号のステージを呼び出す。
}
}

[fiveHundred]

正常だと思うのであれば、問題ありません。

[Abc_.c]

セーブロードの件長くなりましたがありがとうございました！
もう一つの質問は内容がほぼ被ってますのでそちらの方は削除しておきます