保存した音量を他シーンで共有できない。

Question

前提・実現したいこと

メニューシーン保存した音量を他シーンで共有させたい。

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

メニューシーンにスライダーを配置し音量と同期させ同シーンにあるタイトルに戻るボタンで音量を保存し、ステージシーンで保存した音量の値を受け取るまではできました。しかしその取得した音量をミキサーにセットできません。

該当のソースコード

メニューシーンにあるスライダーのスクリプト

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


public class VSlider : MonoBehaviour
{
    public UnityEngine.Audio.AudioMixer mixer;

    public Slider VolumeSlider;

    public static float GetVolume;



    void Awake()
    {
        GetComponent<Slider>().value = PlayerPrefs.GetFloat("master", 0);

       float GetVolume = VolumeSlider.value;

        PlayerPrefs.SetFloat("GetVolume", VolumeSlider.value);

        Debug.Log("ステージに渡す"+PlayerPrefs.GetFloat("GetVolume", GetVolume));
        Debug.Log("スライダー値" + VolumeSlider.value);
    }

    public void masterVol(Slider VolumeSlider)
    {
        if (PlayerPrefs.HasKey("master"))
        {
            PlayerPrefs.GetFloat("master", VolumeSlider.value);
         
            mixer.SetFloat("master", VolumeSlider.value);     
        }
        else
        {
            mixer.SetFloat("master",-20);
        }
    }
        void Update()
    {
            Slider VolumeSlider = gameObject.GetComponent<Slider>();
              
    }
}

メニューシーンにあるタイトルに戻るボタン、PlayerPrefs.Save();は負荷が大きくボタンに置くといいと読んだのでここにおきました。

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

public class buttonmenu : MonoBehaviour
{
    public Slider VolumeSlider;

    public void OnClick()
    {
        FadeManager.Instance.LoadScene("Title", 0.3f);

      

        PlayerPrefs.SetFloat("master", VolumeSlider.value);
        PlayerPrefs.SetFloat("GetVolume", VolumeSlider.value);
        PlayerPrefs.Save();
        Debug.Log("ステージに渡す" + PlayerPrefs.GetFloat("master", 0));
        Debug.Log("masterセーブ値" + PlayerPrefs.GetFloat("master", 0));
        Debug.Log("GetVolumeセーブ値" + PlayerPrefs.GetFloat("GetVolume", 0));
    }

   

}

ステージにあるAudio Sourceにアタッチしてあるスクリプト。ここのデバッグには保存した数値がでてます。
しかしmixer.SetFloat("master", GetVolume);が機能しないです。

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

public class SetVolume : MonoBehaviour {

    public UnityEngine.Audio.AudioMixer mixer;

    public static float GetVolume;

    void Start()
    {
        
            PlayerPrefs.GetFloat("master", GetVolume);

            mixer.SetFloat("master", GetVolume);//ここが機能しません。

            Debug.Log("メニューからもらうStart" + PlayerPrefs.GetFloat("master", GetVolume));//ここはセーブした値が出ます。
        
    }
}

重複しますがSetVolumeのデバッグは保存した値がでます。
どうか知恵をお貸しください。
基本的なコードの書き方も理解していない素人ゆえコードで説明があるとすごく助かります。よろしくお願いします。

Comments

[bochan2]

Playerprefsはメモリ上にキャッシュされてるので、実は（パフォーマンスの話のみなら）ガンガン呼んでしまって大丈夫だと思います。

Answer 1

質問頂きありがとうございます。

PlayerPrefs.GetFloat("master", GetVolume);を
GetVolume=PlayerPrefs.GetFloat("master", 1);にしたら動くと思います。

public static float GetFloat(string key, float defaultValue);

と公式ドキュメントに書いてある通り、第２引数は値が無かったときの既定値です。

ここに公式ドキュメントのリンクを貼っておきますので読んでみてください。

https://docs.unity3d.com/ScriptReference/PlayerPrefs.GetFloat.html

Comments

[bochan2]

関係ないといえば関係ないのですがstatic変数を使うべきかはよく考えた方が良いと思います

[giftVS]

回答ありがとうございます。うまく作動しました。
申し訳ないのですがなぜよく考えたほうがいいのか厳しめな表現でも構いません勉強したいので教えていただけませんか。

[bochan2]

クラスを複数生成したとき（同じScriptを複数のゲームオブジェクトに貼り付けるなどしたとき）に干渉するからです。
音量を複数種類用意することはあんまり無さそうですが、staticを外しても動くものは外してしまっておいた方が後で拡張するときに楽だと思います。

[giftVS]

public static float GetVolume;は出す側でももらう側でも不要ということでしょうか。
そしてこれ（音量関係）以外でこのような書き方は不具合の元といことですかね。

[bochan2]

仰る通り、もらう側では不要ということです。
あと、playerprefsで共有している以上変数は渡す必要が無いのでstaticでない方が冗長でなくなります。
playerprefsでしか受け渡ししていないのにstaticがあると他の人や未来の自分がどっちを使えば良いか分からなくなってしまうという問題もあります。

[giftVS]

ありがとうございます。自分の理解が少し深まったと思います。
今後のためにもいろいろ修正してみます。本当にありがとうございました。