自機弾丸発射のコード

Question

前提・実現したいこと

ここに質問の内容を詳しく書いてください。

C# unityで2Dシューティングを作っており、　
自機の弾丸を数フレーム間隔で発射を繰り返すコードを組んでいるのですが、　
想定通りに動きません。
どなたかアドバイスをいただけると嬉しいです。

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

スペースキーを押した際に、for文以下を繰り返して、　
等間隔に発射したいのですが、　
スペースを押してもinput.Getkey以下の文章が起動しません。　
またstart関数にあるコルーチンは始動しており、　
コンソールにデバッグログの内容が出力しております。　

該当のソースコード

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

public class BulletController : MonoBehaviour
{
    //弾の速さ
    private const int SPEED = 10;

    //自機弾丸の生成
    public GameObject Bullet0Prefab; 

    // 画面の一番上のy座標。画面外かどうかの判定に使用
    private float ScreenTop; 

    private Rigidbody2D myRigidbody2d;
    private Transform myTransform;

    //弾を連続で発射する際の間隔
    public float interval = 0.1f;

    void Start()
    {

        StartCoroutine(enumerator());

    }


    void Awake()
    {
        myRigidbody2d = GetComponent<Rigidbody2D>();
        myTransform = GetComponent<Transform>();

        // 画面の一番上のy座標を取得
        ScreenTop = Camera.main.ViewportToWorldPoint(new Vector2(0, 1)).y;
        // 弾を上に移動させる
        myRigidbody2d.velocity = myTransform.up.normalized * SPEED;

        


    }

    void Update()
    {
        // Rigidbody2Dのsimulatedがfalse(弾が使われていない状態)であれば何もしない
        if (myRigidbody2d.simulated == false)
            return;

        // ここからはRigidbody2Dのsimulatedがtrueの場合(=弾が動いている場合)
        // 画面外に弾が出ていたらRigidbody2Dのsimulatedをfalseにして物理演算を止める(弾をストップする)
        // ＋１しているのは余裕を持っているだけです。
        if (myRigidbody2d.position.y > ScreenTop + 1)
        {
            myRigidbody2d.simulated = false;
        }
    }



    IEnumerator enumerator()
    {
        Debug.Log("あいうえお");

        
        while (Input.GetKey(KeyCode.Space))
        {
            

            for (int Bulletlimit = 0; Bulletlimit < 10; Bulletlimit += 1);

            yield return new WaitForSeconds(interval);

            GameObject Bullet0 = Instantiate(Bullet0Prefab) as GameObject;
            

            if (Bullet0Prefab.transform.position.y > ScreenTop)
            {
                //画面外の弾丸を破棄する
                Destroy(this.gameObject);
            }




        }

    }    

}

試したこと

debug.logを用いてIEnumerator enumerator()まで指示が行き届いていることを
確認した。　

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

ここにより詳細な情報を記載してください。

Answer 1

コルーチンは自動で無限ループしてくれるわけではないので、
下までいったら終了してしまいます。
なので、while(true)で囲ってください。

    IEnumerator enumerator()
    {
        while (true)
        {
            while (Input.GetKey(KeyCode.Space))
            {
                for (int Bulletlimit = 0; Bulletlimit < 10; Bulletlimit += 1) ;

                yield return new WaitForSeconds(interval);

                GameObject Bullet0 = Instantiate(Bullet0Prefab) as GameObject;

                /*画面外の弾丸を破棄する処理は弾丸につけたスクリプトで弾丸自身が行ったほうが良いと思います
                if (Bullet0Prefab.transform.position.y > ScreenTop)//Bullet0PrefabじゃなくてBullet0では？
                {
                    //画面外の弾丸を破棄する
                    Destroy(this.gameObject);
                }
                */
            }

            yield return null;
        }
    }

Answer 2

while (Input.GetKey(KeyCode.Space))
{
    // 省略
}

このコードでは、一瞬でもスペースキーが押されていないと、一瞬でもループ処理が直ちに終了してしまいます。

スペースキーを押した際に、for文以下を繰り返して、　
等間隔に発射したいのですが、　

コルーチンではなく Update メソッド内でやった方が良いのではないでしょうか。