Unity C# 次のようなエラーが出ています。 error, ',' expected

Question

Unityを勉強し始めたばかりの初心者です。
被弾したときに左右に振動するというコードを書いてみたのですが、次のようなエラーが出ました。

Assets\Script\Enemy1System.cs(15,40): error CS1003: Syntax error, ',' expected

コードは下記のようなものです。

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
 
public class Enemy1System : MonoBehaviour
{
    public GameObject target;
    [SerializeField] float speed;

    public GameObject effectPrefab;
    public int enemyHP;

    private IEnumerator shakeCoroutine()
    {
        while (OnTriggerEnter(Collider other) | OnTriggerStay(Collider other))
        {
            Vector3 pos = this.gameObject.transform.position;
            this.gameObject.transform.position = new Vector3(pos.x + 1, pos.y, pos.z);


            for (int waitLoop = 0; waitLoop < 1.0f; waitLoop++)
            {
                yield return new WaitForFixedUpdate();
            }

            Vector3 pos = this.gameObject.transform.position;
            this.gameObject.transform.position = new Vector3(pos.x - 1, pos.y, pos.z);


            for (int waitLoop = 0; waitLoop < 1.0f; waitLoop++)
            {
                yield return new WaitForFixedUpdate();
            }
        }
    }

    private void Start()
    {

    }

    private void OnTriggerEnter(Collider other)
    {
        if (other.gameObject.CompareTag("Beam"))
        {
            StartCoroutine(shakeCoroutine());

            enemyHP -= 1;

            if (enemyHP == 0)
            {
                Destroy(transform.root.gameObject);

            }
        }
    }

    private void OnTriggerStay(Collider other)
    {
        if (other.gameObject.CompareTag("Beam"))
        {
            StartCoroutine(shakeCoroutine());

            enemyHP -= 1;

            if (enemyHP == 0)
            {
                Destroy(transform.root.gameObject);
            }
        }
    }

    void Update()
    {
        transform.position = Vector3.MoveTowards(transform.position, target.transform.position, speed);
    }
}

どこを直せばよいか分からず困っています。直すべきところを教えてほしいです。

Comments

[y_waiwai]

エラーが出たなら、エラーメッセージを提示しましょう
エラーメッセージは、いらぬ翻訳省略しないで、出たそのママをコピペで提示してください

Answer 1

前回の質問を理解されなかったみたいですね。
まあ、私が教えるのが下手なのもあるので、威張って言うことではないですが。

        while (OnTriggerEnter(Collider other) | OnTriggerStay(Collider other))

そもそも、OnTriggerEnter()やOnTriggerStay()はこのように呼び出すものではありません。
これらの関数は、接触されたときに自動に呼ばれる関数です。

また、OnTriggerStay()でのStartCoroutine(shakeCoroutine());は不要どころか、正しくありません。
これだと、接触中はOnTriggerStay()が呼び出され続けるので、その度にshakeCoroutine()が開始され、大量のshakeCoroutine()が実行されることになります。

私が前回の回答で意図したところは、以下の通りですが、全く出来ていません。

1. OnCollisionEnter()が呼び出されたときにisCollisionをtrueにしてshakeCoroutine()を開始させる
2. isCollisionがtrueの間、shakeCoroutine()が常に実行され続ける
3. OnCollisionExit()が呼び出されたときにisCollisionをfalseにする
4. isCollisionがfalseになったので、ループから抜け、shakeCoroutine()は終了する

OnCollisionEnter()やOnCollisionExit()については、OnTriggerEnter()などでも同じです。
なので、whileの中身はisCollisionですし、StartCoroutine(shakeCoroutine());もOnCollisionEnter()のみでOnCollisionStay()では呼び出していません。

さらに、waitLoopの指定は秒数ではなく（FixedUpdate()での）フレーム数です。

「コードで書いてあるからコピペでいいや」ではなく、できるだけ中身を理解してから使うようにしてください。

Answer 2

そのエラー解決だけで終わらなそうですが、
該当エラーだけを解決したい場合は15行目の下記の部分を書き換えて下さい。

while (OnTriggerEnter(Collider other) | OnTriggerStay(Collider other))

↓

while (OnTriggerEnter(other) | OnTriggerStay(other))

補足

上記で質問のエラーは解消されると思いますが新たに下記の問題が出てきます

・otherがメンバー・ローカルに定義されていないのでThe name 'other' does not exist in the current contextの様なエラーが出る

・そもそもOnTriggerEnterとOnTriggerStayはboolを返さない上、接触時に自動で実行される関数なのでこういう使い方は出来ない

・OnTriggerEnterとOnTriggerStayでのshakeCoroutine呼び出しが複数回実行される可能性があるので過度の処理負荷とバグが予想される

一つ前の質問の内容を実装されているとは思いますが内容を理解した上で改変していくか、
判らない場合は出来るだけ書き換えないよう実装していくと良いと思います

Comments

[Hawn]

ほぼ同じ回答を書いてしまったので細かい修正があるようならfiveHundredさんに聞いて貰った方が良いです