Unity スクリプトで、GUIボタンから音声を取得し、再生するやり方がうまくいかない

Question

Unityでスクリプトを使い、アプリケーションパスにある音声ファイルを取得し、GUIのボタンで押すと再生出来る様にしたいと
考えていますが、うまくいきません。

再生するだけなら他に方法はあると思いますが、このやり方を選ぶ理由として
今回のプログラムの前に音声をマイクで取得し、保存してから、再び呼び出して音声解析まで
行えるアプリを作りたいと考えているためです。

ファイルを取得し再生するまでのコードは以下のようにしています。

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

public class AudioRead
{
   public void  ReadAudio()
    {
	    using (WWW www = new WWW("file:///" + Application.dataPath + "myvoice.wav"))
            {
                yield return www;

                AudioClip audioTrack = www.GetAudioClip(false, true);
                AudioSource source = gameObject.AddComponent<AudioSource>();
                source.clip = audioTrack;
                source.Play();
            }
    }
}

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

public class Start_analyze_action : MonoBehaviour
{
    AudioRead audioreads;
    // Use this for initialization
    void Start() { }
    // Update is called once per frame
    void Update() { }
    void OnGUI()
    {
        if (GUI.Button(new Rect(Screen.width / 2 - 100, Screen.height / 2 - 80, 100, 50), "Start Analyze"))
        {
            audioreads.ReadAudio();
        }
    }
}

AudioRead.csのReadAudioの赤線があり、カーソルを合わせると以下のような表示がでます。

以下のサイトを参考に作成しました。
リンク内容

C#での記述にも慣れておらず、Unity初心者で、
Unity独自の書き方、ソフトの使い方に苦労しております。

どうすれば解決できそうか教えていただきたいです。

また、他のやり方でも、アプリ起動後にGUIのボタンを使って、
保存した音声ファイルを再度取得できるやり方でこちらの方がいい
というのがあれば是非そちらも教えていただければなと思います。

どなたかご教授お願いいたします。

Answer 1

yield return www;は普通の関数で使うことは出来ません。
「コルーチン」の中で使用することになります。

---

今回のエラーとは関係ないですが、以下の問題点もあります。

・AudioReadクラスにMonoBehaviourを継承していないのですが、それだとgameObjectを参照することはできません。
なぜなら、gameObjectはMonoBehaviourのプロパティだからです。

・Application.dataPathはエディタ上とビルドしたゲーム上では参照されるディレクトリは違います。
特に、ビルドしたゲーム上ではAssetsディレクトリの内容を直接読み込むことは出来ないのですが、そこは考慮されておりますでしょうか？

今回のプログラムの前に音声をマイクで取得し、保存してから、再び呼び出して音声解析まで
行えるアプリを作りたいと考えているためです。

ということであれば、Application.temporaryCachePath（一時的）かApplication.persistentDataPath（ゲームが終了しても残る）を使うべきなのではないかと思うのですが、どうでしょうか？

・Start_analyze_actionクラスのaudioreadsにオブジェクトを代入する処理がどこにもありません。
このままだとaudioreadsはnullのままなので、audioreads.ReadAudio();でエラーが発生してしまいます。

・OnGUI()は古いものだそうです。
できれば、新しいGUI（uGUIと呼ばれることがある）を使う方がよろしいかと思います。

---

追記：
www.GetAudioClip()の第2引数がtrueとなっておりますが、もしかしてストリーミング再生を行いたいということでしょうか？
もし、そうなのであれば、yield return www;はロード完了するまで待機するという意味なので、違うのではないかなと思います。

そのため、スクリプトを作り直してみました。
同じファイルを2回ロードしようとするとエラーが出たので、そうならないように変更を加えたコードになります。

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

public class AudioRead : MonoBehaviour
{
    AudioSource source;

    void Start()
    {
        // AudioSourceは1度だけ生成するようにする
        source = gameObject.AddComponent<AudioSource>();
        source.playOnAwake = false;
    }

    public void ReadAudio()
    {
        if (source.clip == null || source.clip.loadState == AudioDataLoadState.Failed)
        {
            // コルーチンを開始
            StartCoroutine(LoadAudioCoroutine());
        }
        else if (source.clip.loadState != AudioDataLoadState.Unloaded)
        {
            // ロード開始後のみ再生
            source.Stop();
            source.Play();
        }
    }

    IEnumerator LoadAudioCoroutine()
    {
        // パスは適宜変更してください
        using (WWW www = new WWW("file://" + Application.streamingAssetsPath + "/myvoice.wav"))
        {
            source.clip = www.GetAudioClip(false, true);

            // ロード開始まで待機
            while (source.clip.loadState == AudioDataLoadState.Unloaded)
            {
                yield return null;
            }

            source.Play();

            // ロード完了まで待機
            yield return www;
        }
    }
}

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

public class Start_analyze_action : MonoBehaviour
{
    // Inspector上からAudioReadをアタッチしたオブジェクトを指定
    public AudioRead audioreads;

    // uGUIのボタンを作成し、そのボタンのOn Click()に登録する
    public void OnClickButton()
    {
        audioreads.ReadAudio();
    }
}

Comments

[kazuh1r010]

上記のエラーを無事に解決することができ、ご丁寧に他の部分もご指摘いただきありがとうございました。
ただ、fiveHundredさんがおっしゃる通り、
audioreads.ReadAudio();で発生するエラーの解決方法が分からず、
コルーチンに対応させつつエラーを解決するために実行部分のコードを
m_coroutineObject.StartCoroutine(audioreads.ReadAudio());に
変更したのですが、ビルドした後以下のような、エラーが出ました。
「NullReferenceException: Object reference not set to an instance of an object」
おそらく、おっしゃっていたオブジェクトの代入のことだと思いますが、
コルーチンを考慮しつつ加える方法が分かりませんでした。
よろしければ、教えていただけないでしょうか？

[fiveHundred]

AudioReadのオブジェクトを取得して格納すればよいです。
あるいはaudioreadsにpublicをつけると、Inspector上から設定できるようになるので、それで指定してもよいです。
これらはコルーチンとは関係はありません。

それと、後で気づいた点があるので、回答に追記したので、それも確認してください。

[fiveHundred]

すいません。追記したスクリプトを若干修正しましたので、確認をお願いします。

[kazuh1r010]

スクリプトまで記述していただき、ありがとうございます。
自分の環境で試してみた結果以下のエラーが返ってきてしまうのですが
考えられる原因が突き止められませんでした。
Error: Cannot create FMOD::Sound instance for resource iBC
, (Operation could not be performed because specified sound/DSP connection is not ready. )
エラーか所は、
public void OnClickButton()　の　audioreads.ReadAudio();
ReadAudio()　　　　　　　　の　StartCoroutine(LoadAudioCoroutine());
IEnumerator LoadAudioCoroutine()　の
source.clip = www.GetAudioClip(false, true);
と
while (source.clip.loadState == AudioDataLoadState.Unloaded)
重ねてになりますが教えていただけませんでしょうか？

[fiveHundred]

パスを「Application.streamingAssetsPath」に変えてますので、それが原因かもしれません。
パスを変更するか、Assetsディレクトリ内に「StreamingAssets」という名前のディレクトリを作ってそれにmyvoice.wavを入れてください。

[kazuh1r010]

新たにディレクトリを作成する方法でエラーが解決しました！
まだまだ、先はありますが頑張ろうと思います。

何度も回答いただきありがとうございました！