銃弾を数秒ごとに打てるようにしたい

Question

前提・実現したいこと

銃弾を数秒間ごとに発射させ連射打ちさせないようにしたい

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

一発目の銃弾を打った後は設定している時間が経過したら銃弾を打てるがそれ以降組んだプログラムが起動しないことがある。
ほとんどが最初以外起動しない。

inspectorで銃弾は１→０にはなっています。
また０→１にも戻ります。

エラーメッセージ

特になし

該当のソースコード

C#言語
public GameObject item;
public int bulletCount;

public Transform muzzle;

public float speed = 100;

private void Start()
{
    bulletCount = 1;
}

private void Update()
{
    if (Input.GetKeyDown("left ctrl") && bulletCount > 0)
    {
        bulletCount -= 1;

        GameObject items = Instantiate(item) as GameObject;

        Vector3 force;
        force = this.gameObject.transform.forward * speed;

        items.GetComponent<Rigidbody>().AddForce(force);

        items.transform.position = muzzle.position;
    }
    if(bulletCount <= 0)
    {
        StartCoroutine(SetBullet());
    }
}

IEnumerator SetBullet()
{
    yield return new WaitForSeconds(5f);
    bulletCount = 1;
}

試したこと

Tagがないというこれとは関係ないエラーが出ていたのでこれが原因かと思い、解消したが解決しませんでした。

もしかしてif(bulletCountが０)の時のスクリプトの位置に問題があるとかですかね。

よろしくお願いします。

Comments

[退会済みユーザー]

「組んだプログラムが起動しない」
というのは、一発目の銃弾を打った後に設定している時間が経過したら、再び銃弾を打てるようにするプログラムが起動しないということですよね？
連射できてしまうとか

[BRAK]

はい、その通りです。ちょっと文脈が具体性を失っていてすみません。
なにか策ありませんかね。。
よろしくお願いします。

[BRAK]

連射できてしまいます。

Answer 1

処理の流れが、Start()でbulletCountを1にセット、
Update()内で-1して0になった直後に、次のifで0以下が実行されるためではないでしょうか。

0以下でボタンは反応しないのでしたら、先にbulletCountをチェックするようにしてみてはどうですか?

private void Update()
{
     if(bulletCount <= 0)
     {
        if(!IsInvoking("SetBullet")) {
            Invoke("SetBullet",5f);
        }
     } else {
         if (Input.GetKeyDown("left ctrl"))
        {
            bulletCount -= 1;

            GameObject items = Instantiate(item) as GameObject;

            Vector3 force;
            force = this.gameObject.transform.forward * speed;

            items.GetComponent<Rigidbody>().AddForce(force);

            items.transform.position = muzzle.position;
        }
    }

}

void SetBullet() {
   bulletCount = 1;
}

Comments

[BRAK]

ご回答ありがとうございます。
自分が書いたスクリプトですと、leftctrl押した後は０になります。
そのあと一度だけ５秒間打てないというプログラムが起動する限りです。
今、教えていただいたスクリプトを書いて実践してみたいと思います。

[BRAK]

試してみたところ
if (Input.GetKeyDown("left ctrl") && bulletCount > 0)のbulletCount > 0が抜けていたので弾は連射され-1,-2,-3....となっていくだけでした。
if分にbulletCount > 0を入れて試してみたところ１度しか弾は打つことができなく補充はされませんでした。
ちなみにコルーチンの中身はちゃんと書いています。

[madone99]

0以下の場合を少し移動したのですがこれだとどうですか？

[madone99]

すいません。勘違いです。

[madone99]

Update()でやってしまうと、連続でCorutineをセットしてしまうせいかも知れませんね。
Invoke()で実行するとどうでしょう。

https://docs.unity3d.com/ja/current/ScriptReference/MonoBehaviour.Invoke.html

[BRAK]

同じでした。
bulletCountが０になるときとならないときがあるのですがこれはいったい何でしょうか。
そしてさっきは０の時は銃弾が発射されたのですが、countを最初に判定することで発射されないようになりました。
１の時のみ発射できるようになりました。

[BRAK]

ありがとうございます。
ちょっとやってみます。

[BRAK]

やってみたのですが、変わらないですね。。

[BRAK]

if (bulletCount <= 0)
{
Invoke("SetBullet", 5f);
}
public void SetBullet()
{
bulletCount = 1;
}
こんな感じで書きました。

[madone99]

Invokeが呼び出されているかを追加しました。

[BRAK]

ありがとうございます。
５秒間ごとに銃弾を打つことに成功しました。
!IsInvoking("SetBullet")の部分はどういうことなのかを是非教えていただきたいです。
SetBulletが呼び出されていなかったらということですか？

[madone99]

おぉ、よかったですね！
IsInvokingで保留中のものがあればtrueをかえします。

なので、!IsInvokingで、SetBulletがセットしていなければの意味ですね。

https://docs.unity3d.com/jp/460/ScriptReference/MonoBehaviour.IsInvoking.html

[BRAK]

なるほど、初めてこの関数を見ました。また一つ新しい知識が増えてうれしいです。
今度この関数を使ってみたいと思います。
今回は本当にありがとうございました。
ベストアンサー。