unity c# コルーチン IEnumerator

Question

現在unityで開発をしています。

コルーチンを使用してphpと通信を行う処理を実装しているのですがそれぞれのクラスで実装するとうまくいくのですが下記のようにインスタンス化して使用するとエラーが起きます。

原因がわからず困っています。インスタンス化すると使えないのでしょうか？

コード

get.cs

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

public class get : MonoBehaviour
{
    // Start is called before the first frame update
    void Start()
    {
        var ins = new common();
        ins.getTalkCatalog();
    }

    // Update is called once per frame
    void Update()
    {

    }

}

コード

common.cs

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

public class common : MonoBehaviour
{
    private string SeverAddress = "http://localhost/sample.php";

    // Start is called before the first frame update
    void Start()
    {

    }

    // Update is called once per frame
    void Update()
    {

    }

    public void getTalkCatalog()
    {
        StartCoroutine("Sample");　　　　←ここでエラーが起きる
    }

    private IEnumerator Sample()
    {
        Dictionary<string, string> dic = new Dictionary<string, string>();

        StartCoroutine(Post(SeverAddress, dic));

        yield return 0;
    }

    private IEnumerator Post(string url, Dictionary<string, string> post)
    {
        WWWForm form = new WWWForm();

        foreach (KeyValuePair<string, string> post_arg in post)
        {
            form.AddField(post_arg.Key, post_arg.Value);
        }

        WWW www = new WWW(url, form);

        yield return StartCoroutine(CheckTimeOut(www, 3f));

        if (www.error != null)
        {
            Debug.Log("つながらないよー");

        }
        else if (www.isDone)
        {
            Debug.Log("つながったよー");

            Debug.Log(www.text);

        }
    }

    private IEnumerator CheckTimeOut(WWW www, float timeout)
    {
        float requestTime = Time.time;

        while (!www.isDone)
        {
            if (Time.time - requestTime < timeout)
            {
                yield return null;
            }
            else
            {
                Debug.Log("TimeOut");

                break;
            }
        }

        yield return null;
    }
}

Comments

[y_waiwai]

エラー出るならエラーメッセージを提示しましょう

[sashiburi]

すみませんエラーではなくここで処理が止まりコルーチンの処理までたどりつきませんでした。

Answer 1

        var ins = new common();

上記ですが、MonoBehaviourを継承したクラスはnewしてはいけません。
（というかその旨のwarningが出ていたと思いますが、ちゃんと読んだのでしょうか？）

gameObject.AddComponent<common>()でgetのゲームオブジェクトにcommonをアタッチするか、どこかのゲームオブジェクトにcommonをアタッチしてGetComponent<common>()やFindObjectOfType<common>()で取得してください。
GameObjectのほうはnewできるので、併用すれば「空のゲームオブジェクトを作成し、それにcommonをアタッチする」ということも可能です。

---

あと、本題と逸れますが、以下気になった点があります。

• この内容（おそらく参考にしているサイトの情報）は少し古いです。

今では、WWWクラスは非推奨ですし（代わりにUnityWebRequestを使えとのこと）、yield return StartCoroutine(CheckTimeOut(www, 3f));は単にyield return CheckTimeOut(www, 3f);としても動作するようになりました。

• クラス名（≒スクリプト名）の先頭は大文字にするのが慣例です。
• クラス名がgetだと「何を取得するのか」が分からないので、もっと分かりやすい名前にしたほうがいいでしょう。
• これは個人的な意見ですが、StartCoroutine("Sample");よりもStartCoroutine(Sample());のほうが名称変更しやすいのでいいと思っています。

Comments

[sashiburi]

細かい点まで指摘していただきありがとうございます。

改善頑張ります。