外部クラスから処理を参照した際に、インスタンスに参照値が無いというエラーが発生するので解消したい。

Question

インスタンスに参照値がないというエラーを解消したい

お世話になります。以前、admobの表示/非表示の切り替えについて別クラスから変数をアクセスする方法について質問させて頂いております。そちらについてはおかげさまで解消でき、エラーも無くなったのですが実際に動かしてみると実装した箇所でエラー（インスタンスに参照値がありません）が発生しており困っています。

プログラムの内容としては、表示されているadmobのバナー表示/非表示を切り替える為のコンポーネントとなります。admobバナーの表示は行えており、表示/非表示のクラスもadmobの機能に入っているようですのでそちらを使用しています。

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

エラーメッセージ
NullReferenceException: Object reference not set to an instance of an object

該当のソースコード

//AdbunnerComponent.cs
using UnityEngine;
using System.Collections;
using System.Collections.Generic;
using GoogleMobileAds.Api;

namespace Novel
{
    //バナー表示用処理
    public class AdshowComponent:AbstractComponent
    {
        //Admob.csから呼び出せるようインスペクタから設定しておく
	public AdMob admob;

	public AdshowComponent()
        {
　	　//コンポーネントの変数設定部分。
        }

	public override void start ()
	{
　        //AdMobのインスタンスにあるbannerView変数を使う
　        //admobから処理を呼び出してバナーを表示状態にする。
　        //↓エラー箇所
　         admob.bannerView.Show();

	  //次のシナリオに進む処理
          this.gameManager.nextOrder ();

	}
    }
    //バナー非表示用処理
    public class AdhideComponent:AbstractComponent
    {
      //表示処理と類似処理

    }
}

admob.bannerView.Show();　の所でインスタンスの参照値が無いというエラーが表示されます。
外部のadmobクラスでインスタンスを作成しており、それを引っ張ってきて表示/非表示を切り替えるようにしています。

admobクラスのプログラムは下記の通り。

//AdMob.cs
using System.Collections;
using System.Collections.Generic;
using UnityEngine;

using GoogleMobileAds.Api;

public class AdMob : MonoBehaviour {
    //バナービューに外からアクセス出来るようにしておく（インスペクタには出さない）
    [HideInInspector]
    public BannerView bannerView;

    // Use this for initialization
    void Start () {
     
        // アプリID、 これはテスト用
        string appId = "ca-app-pub-3940256099942544~3347511713";
 
        // Initialize the Google Mobile Ads SDK.
        MobileAds.Initialize(appId);
 
        RequestBanner();
    }

    private void RequestBanner(){

        // 広告ユニットID これはテスト用
        string adUnitId = "ca-app-pub-3940256099942544/6300978111"; 

        // Create a 320x50 banner at the top of the screen.
        //※メンバ変数（Startの前に書いた変数）に入れる
        bannerView = new BannerView(adUnitId, AdSize.Banner, AdPosition.Top);

        // Create an empty ad request.
        AdRequest request = new AdRequest.Builder().Build();
 
        // Load the banner with the request.
        bannerView.LoadAd(request);

        // Create a 320x50 banner at the top of the screen.
        //bannerView = new BannerView(adUnitId, AdSize.Banner, AdPosition.Top);
    }
	
	// Update is called once per frame
	void Update () {
		
	}
}

AdMob.csはオブジェクトにアタッチしており、バナー表示は正常に機能しています。
bannerViewのインスタンスはこちらのクラスで作成しており、作成したコンポーネントで表示/非表示を切り替えるという動作を考えています。

バナーの表示はされている→admobのインスタンスは動作しておりNullではないはず？？
と考えており、詰まってしまっている所です。何が原因でどのようにすれば解消できるのでしょうか。
何かお気づきのことがあればご教示いただけますと幸いです。

試したこと

bannerViewの対象となるオブジェクトが参照されていないのではと考え、オブジェクトの参照を試みましたが改善できず。（そもそもバナー表示はできているのでオブジェクトは参照できている？）

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

　Unity Version 2019.1.13
AdmobSDK GoogleMobileAds-v4.1.0

Admob実装時の参考サイト(AdMob.cs)
https://uni.gas.mixh.jp/unity/android-admob.html

Comments

[Y0241-N]

恐らくですが呼び出しのあとにAdMobの生成が行われているのではないでしょうか？

[PiCo19072018]

追記、ありがとうございます。

呼び出しのあとにAdMobの作成が行われているのではとの事ですが、
http://tsubakit1.hateblo.jp/entry/2017/02/05/003714
こちらのサイトで紹介されているような事でしょうか？

AdMob→AdbunnerComponent　となるようにScript Execution Orderを設定や、メタデータの変更を行ってみたのですが、今のところは変化がないようでした。

Answer 1

別々のスクリプト内のStartの順番は（設定で明示しない限り）保証されません。
つまりAdmobクラスのStartよりも先にAdshowComponentのStartクラスが発動している可能性があります。

Script Execution Orderを設定しても変わらないなら、Admobの内部機能で通信を行っているならばそれを待った上でbannerViewに中身が入るのだと思われます。
（エラーによる中断はスクリプト単位なので、AdshowComponentでエラーが起こってもAdmobクラスは問題無いので動作し続ける＝バナーの描画は正常にされる、だと思います）

最初から表示状態にしたいならAdMobのRequestBannerメソッド内でShowすればいいので、
AdshowComponentのstart内でadmob.bannerView.Show();する必要はそもそも無いのではないでしょうか。

ちなみにAdshowComponentのadmob変数にAdmobクラスをセットし忘れている場合もエラーになるので、ちゃんとセットしているか再確認してください。

Comments

[PiCo19072018]

回答ありがとうございます。Startの順番の説明、ありがとうございました。広告が表示されているので動作していると考えていましたがそのような動きをするんですね。

>AdMobのRequestBannerメソッド内でShowすればいいので～
こちらについてですが、最初から表示状態にしたいというわけではなくコンポーネントよりバナー表示の切り替えを行いたいと考えており、このような形を取っています。

AdshowComponentの内容についてはこちらのサイト
http://jokerscript.jp/adv/next/plugin
を参考に制作をしており、シナリオファイルの中でAdshowComponentで作成した[adshow]、[adhide]のタグを使ってadmobバナーの表示/非表示を切り替えられたらと考えています。

不勉強で申し訳ないのですが、「admob変数にAdmobクラスをセットし忘れている」というのはどういう事でしょうか？　どのような状況なのでしょうか？

[sakura_hana]

とりあえずこのエラーは「admob.bannerView.Show();」の行へ辿り着いた時点で、
「admobの中身がnull」か「admobはnullではないが、bannerViewの中身がnull」という状態になっている為に発生します。

admobは「public AdMob admob;」と宣言されています。
なのでコメントにも書かれている通り「インスペクタから設定しておく」かスクリプトから代入する必要があります。
この時点で意味が分からないのなら（以前から伝えている気もしますが）「別クラスの変数へアクセスする」方法を学んでください。（調べればたくさん情報が出ます）

admobはnullではないがbannerViewがnullの場合、
「admob.bannerView.Show();」へ辿り着いた時点ではまだAdMobクラスのRequestBannerメソッドが完了していないと予想されます。
なので一旦無視して通過する（ if(admob.bannerView != null) {admob.bannerView.Show();} とするなど）か、
完了までコルーチン等で待機するか等の対処をしてください。
（どれが適切かはゲームの仕様によるので自分で決めてください）

[PiCo19072018]

ご回答ありがとうございました。また返信遅れて申し訳ありません。
別の問題があると考えていましたが、原因は前回と同じだったのですね。
変数周りというか、プログラム自体の理解が浅く度々ご面倒おかけして申し訳ありません。
エラーを色々と調べてみた所、どうやらbannerViewがnullのようでした。
回答を頂いた内容を元に修正してみます。ありがとうございました。