マップ移動時のマップ名の切り替えについて

Question

マップを移動したときにマップ名を出すようにスクリプトを書きました。
違うマップに移動したときにUIを表示し、また、別のマップに移動したら同じく起動させます。4秒で消えるようにしているのですが、UIが表示され3秒後に別のマップに移動したらマップ名が残り１秒しか表示されないのですが何かいい方法はありませんか

``C#

 public GameObject text;
    public Text placeText; 
    public bool Enter;
    public string placeName;
private void OnTriggerEnter2D(Collider2D col)
    {
        if(col.tag == "Player")
        {
            if (Enter == true)
            {
                StartCoroutine(PlaceNameCo());
            }
        }
    }
    private IEnumerator PlaceNameCo()
    {
        text.SetActive(true);
        placeText.text = placeName;
        yield return new WaitForSeconds(4f);
        text.SetActive(false);
    }

Comments

[nskydiving]

<code>タグが適用されておらず、コードのレイアウトが崩れてしまっています。（改行が必要です）
このままでは回答が付きづらいですので、修正してください。

```C#
// ここにコードを書いてください。
```

Answer 1

コルーチンが起動中ならStopCoroutineで止めてからStartCoroutineしてはどうでしょう？

追記

動作させたわけではないので、下記スクリプトが正常に動作しないかもしれませんが、こんな感じでしょうか。

スクリプトがどのようなオブジェクトにアタッチされているのか分からないので、適切なのか判断できませんが、とりあえず「マップから出た」というのを「OnTriggerExit2D」で判断できるものとしています。

    public GameObject text;
    public Text placeText; 
    public bool Enter;
    public string placeName;

    Coroutine co;

    private void OnTriggerEnter2D(Collider2D col)
    {
        if(col.tag == "Player")
        {
            if (Enter == true)
            {
                co = StartCoroutine(PlaceNameCo());
            }
        }
    }
    private void OnTriggerExit2D(Collider2D col)
    {
        if(col.tag == "Player")
        {
            if (Enter == true)    //Enterが何を示すのもなのか分からなかったのでそのままにしてあります。
            {
                if (co != null)
                {
                    StopCoroutine(co);
                    co = null;
                }
            }
        }
    }

    private IEnumerator PlaceNameCo()
    {
        text.SetActive(true);
        placeText.text = placeName;
        yield return new WaitForSeconds(4f);
        text.SetActive(false);
        co = null;
    }

また別の案ですが、コルーチンの起動に関しては今まで通りとしてコルーチンにて

    private IEnumerator PlaceNameCo()
    {
        text.SetActive(true);
        placeText.text = placeName;
        if(int i = 0;i < 40 ; i++)
        {
            yield return new WaitForSeconds(0.1f);
            if (placeText.text != placeName)
            {
                yield break;
            }
        }
        text.SetActive(false);
        co = null;
    }

といった感じでWaitするのを短時間でループするようにして、テキストが自分のマップ名ではなくなったらコルーチンを終了するようにすればどうでしょう？

ただ、私もUnityはほぼ知らないと言っていいので、もっと適切な書き方があるのではないかと思います。

Comments

[BRAK]

private void OnTriggerEnter2D(Collider2D col)
{
if(col.tag == "Player")
{
if (Enter == true)
{
StopCoroutine(PlaceNameCo()); ★
StartCoroutine(PlaceNameCo());
}
}
}
これでStartCoroutineの中身を数秒遅らせて試してみましたが同じ結果でした。

[YAmaGNZ]

StopCoroutineについて調べると分かると思いますが、その書き方ですとコルーチンは停止しません。
StartCoroutineの戻り値がCoroutineなので、それを渡すようにしてください。

[BRAK]

if (Enter== true)
{
Coroutine coroutine = StartCoroutine(PlaceNameCo());
StartCoroutine(StopWait(coroutine));

StartCoroutine(PlaceNameCo());
}
private IEnumerator StopWait(Coroutine coroutine)
{
yield return new WaitForSeconds(0.1f);
StopCoroutine(coroutine);
}
で書き直してみたのですがだめみたいです。

[YAmaGNZ]

何か勘違いされているようです。
StopCoroutineで検索されることをお勧めします。

[BRAK]

上ので動かなかったのであの後、別のサイト見ながらスクリプトを作ったんですが、うまくいかなかったんです。
それがこれです。
IEnumerator routine;
void Start(){
routine = PlaceNameCo();
}
private void OnTriggerEnter2D(Collider2D col)
{
if(col.tag == "Player")
{
if (Enter== true)
{
StopCoroutine(PlaceNameCo());
routine = null;

routine = PlaceNameCo();
StartCoroutine(PlaceNameCo());
}
}

[YAmaGNZ]

Coroutine co;
を適切な場所で定義しておいて
コルーチン開始時は
if(co!=null)
{
StopCoroutine(co);
}
co = StartCoroutine(PlaceNameCo());

それで、コルーチン側で処理終了時にcoをnullにすればいいかと

[BRAK]

ありがとうございます。
昨日サイトを見て作ったやつも、今教えていただいたやつも恐らく起動しているのですが新たな欠陥があってうまくいかないようでした。

マップが切り替わる際に、スクリプトを同じものを使っているせいでスクリプト内で最初に4秒経過してtext.SetActive(false);があとのものに影響して4秒より先に消えてしまうように感じます。デバックでも見たのですが恐らくそうです。

また、StopCoroutineのタイミングも別のマップに入ったタイミングで行われず同じマップに入ったタイミングで処理が実行されます。

[YAmaGNZ]

すみません、提示部分はどのマップに入った時でも動作するものだと思っていました。
マップ１に入った時に動くスクリプトとマップ２に入った時に動くスクリプトが同一（インスタンス含めて）ではないということですか？
そうなのであれば、先ほど提示したようなStartCoroutineを実行するときにコルーチンが動いたいたらStopCoroutineするのではなく、マップから出たらStopCoroutineだけを行うとかできませんかね？

[BRAK]

すみません。スクリプトは同一で、表示するテキストも同じにしているため処理が同一になって初めの処理があとの同じテキストにかかって4秒以下で消えるっていうことなのかもしれないです。。

マップから出てStopCoroutineにしたら、スクリプトが同一なので新しいマップ名でずっと止まります。

また実験で異なるText(マップ１をText1とマップ２をText2)にしてマップに入って
if(co!=null)
{
StopCoroutine(co);
SetActive(false)
｝
にするというのをやってみたのですが、下に重なる形で失敗しました。
改めてスクリプトは同一のものを使っています。

[YAmaGNZ]

１．マップ１に入った時にコルーチンを開始coに格納（これをco1とします）
２．マップ２に入った時に参照するco（これをco2とします）

現象を聞いているとco1とco2が別物となっているのではないかと思われます。
別物だから、co1にコルーチンが格納されていてもco2はnullのままになっているのではないでしょうか？
マップ１に再度入った時にco1にコルーチンが入っているからtopCoroutineが動くとなっている気がします。

スクリプトが同一というのは、スクリプトのソースが同じだけで、別のオブジェクトにそれぞれアタッチされていたりしませんか？
その場合、ソースが同じでも別インスタンスとなりますので、メンバ変数等もstatic宣言していないかぎりは別物となります。

また、マップから出たタイミングでStopCoroutineする場合は
マップ１→マップ２→マップ１と移動した場合は
１．マップ１に入る（コルーチン開始）
２．マップ１から出る（コルーチンが動いていれば停止）
３．マップ２に入る（コルーチン開始）
４．マップ２から出る（コルーチンが動いていれば停止）
５．マップ１に入る（コルーチン開始）
と動作すると思うのですが、このような動きはしませんか？

[BRAK]

スクリプトは別オブジェクトに付けています。
はい、この動きをしています。
メンバを含めてstatic宣言で実行すれば理想通りの動きが出来るということですか？

[YAmaGNZ]

いまいちスクリプトやTextのオブジェクトが完全に同一の物なのか分からないのでstatic宣言してうまくいくかは分かりません。

一度、object.GetInstanceID（https://docs.unity3d.com/ja/current/ScriptReference/Object.GetInstanceID.html）で本当に同じものなのか確認してみてはどうでしょうか？同じものなのであればGetInstanceIDは同じ値を返すはずです。
Debug.Log(GetInstanceID());
とすればスクリプトのIDがログとして出力されます。
Debug.Log(text.GetInstanceID());
とすればTextのIDが出力されますので、確認してみてください。

これが、例えばマップ１での操作を行っている時は同じ値だが、マップ２での操作を行っている時は違う値が返ってくるとなれば、マップ１で扱っているものとマップ２で扱っているものは別物になります。
こうなると、マップ１に入った時にStartCoroutineして出るときにStopCoroutineするという形がいいのではないかと思います。

動いているスクリプトが同一のものなのか、別の物なのか判断してみてください。
（ここで言う”同一のもの”とはインスタンスIDが同一の物です）

[BRAK]

ありがとうございます。
1度確認してみます。

[BRAK]

すみません。先週はバイトとこれから通う学校のチーム制作の発表がありこっちに手が回らなく回答できませんでした。

今、試してみたところテキストは同じIDだったのですが、スクリプトのインスタンスIDが全く別物だとわかりました。

また、コルーチンをstaticで宣言するとマップ2から１へ移行するときにコルーチンの存続に失敗(エラー名Coroutine continue failure)して理想通りの結果にすることはできました。

これはマップ１のコルーチン内のtext.SetActive(false);がなくなったため
マップ２でのコルーチンを重複させることができました。

本来だとマップ１のコルーチン内のtext.SetActive(false);がマップ２のテキストに影響を与える形になっています。

[BRAK]

それと、＞＞マップ１に入った時にStartCoroutineして出るときにStopCoroutineするという形がいいのではないかと思います。
に関してなのですが、Enterをfalseにするなどしてみたのですが、できなくていまいちよくわからなかったので教えていただけると助かります。

[BRAK]

ありがとうございます。
後で試してみたいと思います。
ちなみにEnterはplayerのtagに触れていたら、もし、trueなら今ではコルーチンをスタートさせるというのを意味しています。

[BRAK]

今、試してみたところ少し変えて無事成功しました。
テキストが自分の名前ではなかったらというスクリプトがかなりよく働いてくれてうまくいきました。
最後までいろいろありがとうございます。

[BRAK]

```C#
private void OnTriggerEnter2D(Collider2D col)
{

if (col.tag == "Player")
{
if (co != null)
{
StopCoroutine(co);
co = null;
}
}
}
private void OnTriggerExit2D(Collider2D col)
{
if(col.tag == "Player")
{
if(Enter == true)
{
co = StartCoroutine(PlaceNameCo());
}
}
}

private IEnumerator PlaceNameCo()
{
text.SetActive(true);
placeText.text = placeName;
for (int i = 0; i < 40; i++)
{
yield return new WaitForSeconds(0.1f);
if (placeText.text != placeName)
{
yield break;
}
}
text.SetActive(false);
co = null;
}
```
入る時と出る時はほぼ同時なので、出る時にStopしてしまうと動かなくなってしまうので出るときにStartさせました。

[YAmaGNZ]

マップ名が異なる時にコルーチンを停止させるというのは、StopCoroutineの変わりにコルーチンを停止させるということなので、StopCoroutineの部分は必要ありません。
多分、質問に載せているコードでうまく動くと思います。

[BRAK]

あ、本当ですね。見やすく書き換え直します。