thisがインスタンスか否か

Question

前提・実現したいこと

こちらのサイト
https://qiita.com/kenta71/items/1d108e2f329f19077853

のソースコードをコピペしたところうまくいきませんでした。
こちらに沿ってSEを鳴らしたいです。

よろしければunityでのthis,null,インスタンスなど理解の助けとなるサイトなどありましたら教えてください。

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

Debug.Logでいろいろやったところ、
どうやらAwakeのif文でthisがInstanceでないということみたいでした。
ifの中のreturnはAwakeを終わる、という認識でいいのでしょうか。

該当のソースコード

~中略~
  public static AudioManager Instance = null;

    //=================================================================================
    //初期化
    //=================================================================================

    private void Awake ()
    {
        if (this != Instance) {
            Destroy (this);
            return;
        }
~後略~

試したこと

Debug.Logでいろいろと試しました。
Singletonとやらも必要だったみたいでそれを記述して
SingletonMonoBehaviour.csというスクリプトにしました。
一番最初のclass宣言で
AudioManager:AudioManagerとしたらエラーが返ってきたので
AudioManager:SingletonMonoBehavior<AudioManager>
と書き替え、エラーはなくなりました。

補足情報（FW/ツールのバージョンなど）

unity 2018.1.0f2
VisualStudio2017

Answer 1

どうやらAwakeのif文でthisがInstanceでないということみたいでした。

thisは自身のインスタンスを指します。
よって、thisがインスタンスでは無いということはありません。
ただし、インスタンスは複数存在できるので、thisのインスタンスとInstanceのインスタンスが異なることはありえます。

この場合、すでに別のAudioManagerのインスタンスが存在している場合、AudioManagerが2つになってしまうので、古い方を残して新しい方を削除するという意味で書いたのだと思います。
ただ、このソースだと、Instanceがnullの時でも削除されてしまいそうな気がしますし、そもそもInstanceを代入している箇所もありませんでした。
元のソースではSingletonMonoBehaviourを使っているので、Instanceはそこで定義されているのでしょう。

AudioManager:AudioManagerとしたらエラーが返ってきたので

ClassA:ClassBと書いた場合、「ClassBを継承したClassA」という意味になります。
AudioManager:AudioManagerと書くと、元のAudioManagerはありませんので当然エラーになります。

結局のところ、その記事のソース自体がおかしいので、元のソースを使うのがよろしいかと思います。

Comments

[kotowari_yasusi]

時間を空けずに回答ありがとうございます。
元のソースをコピペしただけでは新たにエラーが発生したので、その対処策を書いておきます。
本の記事「僕が考えたさいきょうのAudioManager２」
http://kan-kikuchi.hatenablog.com/entry/AudioManager2
のコードすべてではなく
最初の
using UnityEngine; から
Awake()の終わりの "}"　　までを
https://qiita.com/kenta71/items/1d108e2f329f19077853
のソースの該当箇所に置き換えればよいようです。
VisualStudio側に警告CS0114が出ますが、実行には今のところ問題ありません。
また、Awake()内でAudioListenerコンポーネントを生成しており、実行時、シーン内のAudioListenerコンポーネントは1つだけにしろというDebugがものすごい勢いで増えました。
(メインカメラには元からAudioListenerが付いている)
※実行には今のところ影響はありません
なので、カメラのAudioListenerを外すか、スクリプトのその部分を外すかしたほうがよいようです。

Answer 2

下でコメントしたことをこっちにも書いておきます(こっちの方が読みやすいかな)
元のソースをコピペしただけでは新たにエラーが発生したので、その対処策を書いておきます。
元の記事「僕が考えたさいきょうのAudioManager２」
http://kan-kikuchi.hatenablog.com/entry/AudioManager2
のコードすべてではなく
最初の

using UnityEngine; //から



//~略~

 private void Awake ()
    {
     //~略~
    }//Awakeの終わりまで

を

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

/// <summary>
/// BGMとSEの管理をするマネージャ。シングルトン。
/// </summary>
public class AudioManager : SingletonMonoBehaviour<AudioManager>
{
    //ボリューム保存用のkeyとデフォルト値
    private const string BGM_VOLUME_KEY = "BGM_VOLUME_KEY";
    private const string SE_VOLUME_KEY  = "SE_VOLUME_KEY";
    private const float  BGM_VOLUME_DEFULT = 1.0f;
    private const float  SE_VOLUME_DEFULT  = 1.0f;


    //オーディオファイルのパス
    private const string BGM_PATH = "Audio/BGM";
    private const string SE_PATH  = "Audio/SE";


    //BGMがフェードするのにかかる時間
    public const float BGM_FADE_SPEED_RATE_HIGH = 0.9f;
    public const float BGM_FADE_SPEED_RATE_LOW  = 0.3f;
    private float _bgmFadeSpeedRate = BGM_FADE_SPEED_RATE_HIGH;

    //次流すBGM名、SE名
    private string _nextBGMName;
    private string _nextSEName;

    //BGMをフェードアウト中か
    private bool _isFadeOut = false;

    //BGM用、SE用に分けてオーディオソースを持つ
    private AudioSource       _bgmSource;
    private List<AudioSource> _seSourceList;
    private const int SE_SOURCE_NUM = 10;

    //全AudioClipを保持
    private Dictionary<string, AudioClip> _bgmDic, _seDic;


    //=================================================================================
    //初期化
    //=================================================================================

    private void Awake ()
    {
        if (this != Instance) {
            Destroy (this);
            return;
        }

と置き換えればよいようです。
VisualStudio側に警告CS0114が出ますが、実行には今のところ問題ありません。
また、Awake()内でAudioListenerコンポーネントを生成しており、実行時、シーン内のAudioListenerコンポーネントは1つだけにしろというDebugがものすごい勢いで増えました。
(シーン作成時のメインカメラには元からAudioListenerが付いている)
※実行には今のところ影響はありません
なので、カメラのAudioListenerを外すか、スクリプトのその部分を外すかしたほうがよいようです。
シングルトンに関しては自分もよくわかってないのでご自分で調べてください。