【unity】何故このスプリクトが2段回ジャンプになるのか？

Question

プレイヤーのジャンプのスクリプトについて質問です。
プレイヤーを1段階ジャンプと2段階ジャンプで分けようと思って、1段階ジャンプのスクリプトを書いたつもりなのですが、何故だか2段階ジャンプします。

【スクリプト】
int count = 1;

void Update () {
if (Input.GetKeyDown(KeyCode.Space) && count != 0)
{
rigidbody.AddForce(transform.up * jump);
count -= 1;
}

        if (rigidbody.velocity.y == 0)
        {
            count = 1;
        }

}

ジャンプをしたら、countをマイナス１して0にして、地面に着地しYの加速度が０になったら、countを回復しているつもりなのですが、このスプリクトが、どうして1段階になるのでしょうか？

Answer 1

rigidbody.AddForce()が反映される前に、if (rigidbody.velocity.y == 0) { ～ }を行っているため、回数が減った直後に（同じフレーム中に）リセットされているからです。

また、質問と直接関係ないですが、AddForce(～);のような物理関係の処理は基本的にFixedUpdate()内で行います。
この場合、FixedUpdate()内でInputクラスの処理を行えない点にも注意してください。

これらを踏まえて、修正すると以下のようになります。
（※修正：判定を当たり判定に変更しました）

    int count = 1;

    // Spaceが押されたかどうか
    bool spaceKeyDown = false;

    void Update()
    {
        if (Input.GetKeyDown(KeyCode.Space))
        {
            spaceKeyDown = true;
        }
    }

    void FixedUpdate()
    {
        if (spaceKeyDown && count != 0)
        {
            rigidbody.AddForce(transform.up * jump);
            count -= 1;
        }

        spaceKeyDown = false;
    }

    // 何かにぶつかったときに実行される関数
    void OnCollisionEnter(Collision collision)
    {
        // ここではそのままですが、必要に応じて「ぶつかったのが床であるとき」のような処理を入れてください
        count = 1;
    }

---

なお、ソースコードは「```」×2で囲うと、上記のように見やすくなります。

Comments

[fiveHundred]

ああ「rigidbody.velocity.y == 0」は良くないのか、失念してました。
これでも（少なくともこちらの環境では）動きましたが、rucchoさんの言っているような問題が起きる可能性があるので、当たり判定で判定した方がよいですね。

[fiveHundred]

ソースを修正しました。

Answer 2

Rigidbodyのvelocityは小数(float)です。このスクリプトだとvelocityのYがピッタリ0.0になった瞬間しかcount = 1;が実行されません。Unityの物理エンジンは静止しているように見える物体でもごくわずかな速度を持っている場合があり、これがvelocityがピッタリ0.0にならない原因かと思います。
これを愚直に改善するならば、velocityの判定に閾値を持たせてあげれば良いのですが…
そもそもジャンプの最高点にもvelocity.yが約0になるので、タイミングによっては2段ジャンプが可能になってしまいます。
結論を言うと、接地判定にはvelocityではなく当たり判定(ColliderやRaycastやPhysics.OverlapSphereなど)を使って地面との接触判定をするべきです。このような方法は各所で紹介されているので調べてみてください。

Comments

[fiveHundred]

問題になっているのは「2段階ジャンプができてしまう」なので、原因はそこではないですね。
もし、「rigidbody.velocity.y == 0」が真にならないのであれば、「1度しかジャンプできない」ということになります。

[tukituki]

悩みがスッキリしました！
みなさんありがとうございます！