[Unity] ほぼ同時に発生したテクスチャファイルの読み込みで2回目以降のCoroutineがキャンセルされる

Question

前提・実現したいこと

Unity でゲームを作っているのですが、敵キャラのテクスチャはローカルディスク上のファイルを使います。
テクスチャファイルはその都度読み込むのではなく、キャッシュして使いまわしたいのですが、2回目以降の読み込みを待つうまい方法がわかりません。

吐かれたメッセージを読むと以下の方法ではダメなようですが、生成タイミングがほぼ同時であるような場合、先に発生したファイル読み込みを待つにはどのような方法があるのでしょうか？

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

以下のコードを実行した際のログです。3行目はエラーメッセージではなく通常のログとして吐かれていました。

1行目：作成された敵オブジェクトAの読み込み
2行目：ほぼ同じタイミングで生成された敵オブジェクトBの読み込み
3行目：エラー？メッセージ
4行目：敵オブジェクトAの読み込み完了メッセージ

敵オブジェクトBの読み込み完了メッセージは表示されず、あたかもキャンセルされたような状況です。

load image...D:\Proj\enemy.jpg
wait load image...D:\Proj\enemy.jpg
Another coroutine is already waiting for this coroutine! Currently only one coroutine can wait for another coroutine!
...ready image : D:\Proj\enemy.jpg

該当のソースコード

    public class TextureStoreTest : MonoBehaviour
    {
        //---------------------------------------------------------------------

        class WaitObjectPair
        {
   			public Coroutine Coroutine;
			public IEnumerator Ienumerator;
            public WaitObjectPair(Coroutine co, IEnumerator ie)
            {
                Coroutine = co;
                Ienumerator = ie;
            }
        }

        //---------------------------------------------------------------------

        private static Dictionary<string, WaitObjectPair> _cache = new Dictionary<string, WaitObjectPair>();

        //---------------------------------------------------------------------

		// 敵オブジェクトの初期化で呼ばれる
		// callback は読み込んだテクスチャをマテリアルに設定することを想定している
        public void RequestLoadTexture(string path, Action<Texture2D> callback)
        {
            StartCoroutine(LoadTexture(path, callback));
        }

        //---------------------------------------------------------------------

        private IEnumerator LoadTexture(string path, Action<Texture2D> callback)
        {
            if (!_cache.ContainsKey(path))
            {
                Debug.Log("load image... : " + path);

                var ie = LoadTexture(path);
                var co = StartCoroutine(ie);
                _cache.Add(path, new WaitObjectPair(co, ie));

                yield return co;
            } else {
                Debug.Log("wait load... : " + path);
                yield return _cache[path].Coroutine;
            }

            Debug.Log("...ready image : " + path);
            callback((Texture2D)_cache[path].Ienumerator.Current);
        }

        private IEnumerator LoadTexture(string path)
        {
			Texture2D newTexture = null;
			using(UnityWebRequest uwr = UnityWebRequestTexture.GetTexture("file://" + path, true))
			{
				yield return uwr.SendWebRequest();

				if (uwr.isNetworkError || uwr.isHttpError)
				{
					Debug.LogError("open failed : " + path);
				} else {
					newTexture = ((DownloadHandlerTexture)uwr.downloadHandler).texture;
				}
			}
			yield return newTexture;
        }
    }

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

Windows10 Pro
Unity 2019.3.6f1

Answer 1

多分、これじゃないでしょうか？

• Only one coroutine can wait for another coroutine

おそらく、コルーチン自体をキャッシュして使いまわしているのが原因だと思います。
コルーチン自体を別のタイミングで yield return に用いると出るエラーのようです。
コルーチンをキャッシュするのではなく、ロードするためのクラスなどを別途用意するなど、別の仕組みにしないとダメかもしれません。

Comments

[daidaiiro]

ご回答ありがとうございます。別の仕組みにして回避しました。