一定の場所までまで上がり、その後0に戻したい。

Question

こんにちは、只今ゲーム制作をしております。
そこで詰まってしまったので、質問させていただきます。

現在スペースを押したら、ジャンプしたようにしたいのですが、自分なりにコードを書いてみました

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

public class CarContrl : MonoBehaviour
{
   

    // 車のモデル　　注意　　子オブジェクトです。
    [SerializeField] private Vector3 modelPos;

    void Update()
    {
        if (Input.GetKeyDown(KeyCode.Space))
        {
            print("スペースを押した");
            StartCoroutine(CarJump());
        }
    }
   

       
    }
    /// <summary>
    /// 車をジャンプする
    /// </summary>
    private IEnumerator CarJump()
    {
        print("ジャンプ");
        var CarJumpPos = 0;
        var CarJumpMax = 10;
        var CarJumpPower = 2;

        while (CarJumpPos < CarJumpMax)
        {
            CarJumpPos += CarJumpPower;
            modelPos =
                new Vector3(modelPos.x, modelPos.y + CarJumpPos, modelPos.z);
            print("ジャンプしている");
            yield return null;
        }

        while (CarJumpPos >= 0)
        {
            CarJumpPos -= CarJumpPower;
            modelPos -= new Vector3(modelPos.x, modelPos.y - CarJumpPos, modelPos.z);
        }
    }
}

やった事
ジャンプするのは、コルーチンで実行しております。
コードの通り、スペースを押したら、一定値まで上がり、そのあと０に戻るというものです。
注意なのですが、フィールドの　modelPos というのは子オブジェクトなためこう言うコードにしております。（親に車を動かすコードが書いてあります。)
スペースを押してみても、動きませんが、コンソールにはスペースを押したしかでません。
コードが見にくいのですが、あまりいいやり方ではないです。うまいコードが欠けていないのはわかっております。
なので、お力をお貸しいただけないでしょうか？
解説やコメントなど加えてくださるとありがたいです。

ーーーーーーーーーーーーーーー　修正後---------------------------------

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

public class CarContrl : MonoBehaviour
{
   

    // 車のモデル　　注意　　子オブジェクトです。
     [SerializeField] private Transform modelPos;
    void Update()
    {
        if (Input.GetKeyDown(KeyCode.Space))
        {
            print("スペースを押した");
            StartCoroutine(CarJump());
        }
    }
   

       
    }
    /// <summary>
    /// 車をジャンプする
    /// </summary>
    private IEnumerator CarJump()
    {
        print("ジャンプ");
        var CarJumpPos = 0;
        var CarJumpMax = 50;
        var CarJumpPower = 2;

        while (CarJumpPos < CarJumpMax)
        {
            CarJumpPos += CarJumpPower;
            modelPos.transform.localPosition
                += new Vector3(transform.localPosition.x, transform.localPosition.y + CarJumpPower, transform.localPosition.z);
            print("ジャンプしている");
            yield return null;
        }

        while (CarJumpPos > 0)
        {
            CarJumpPos -= CarJumpPower;
            modelPos.transform.localPosition -= new Vector3(transform.localPosition.x, transform.localPosition.y + CarJumpPower, transform.localPosition.z);
        }
    }
}

これで動くようにはなったのですが、Z軸方向に飛んでしまいます。
何故でしょうか？ご意見ください。他にいいコードの書き方を教えてくだされば幸いです。

Answer 1

回答依頼ありがとうございます。

スペースを押してみても、動きませんが、コンソールにはスペースを押したしかでません。

とのことですが、コルーチンを呼び出す処理がコメントアウトされているのは大丈夫ですか？

if (Input.GetKeyDown(KeyCode.Space))　{
    print("スペースを押した");
    // StartCoroutine(CarJump());
}

追記: コルーチンでのジャンプ処理

一応コピペしてから改造したので仕様が大きく変わってるとは思いませんが、もし違っていたら適宜修正をお願いします。

using System.Collections;
using UnityEngine;

public class JumpTest : MonoBehaviour
{

    // 車のモデル（子オブジェクト）
    // 型をTransformに変更
    [SerializeField] private Transform model_t;

    [SerializeField] private float jumpHeight = 0.4f;
    [SerializeField] private float jumpSpeed = 5.0f;

    // スペースを連打するとコルーチンが何回も走るので防止用
    private bool jumping = false;

    void Update() {
        if (Input.GetKeyDown(KeyCode.Space) && jumping == false) {
            print("スペースを押した (jumping == false)");
            StartCoroutine(CarJump());
        }
    }

    /// <summary>
    /// 車をジャンプする
    /// </summary>
    /* private IEnumerator CarJump() {
        print("ジャンプ");
        var CarJumpPos = 0;
        var CarJumpMax = 10;
        var CarJumpPower = 2;

        while (CarJumpPos < CarJumpMax) {
            CarJumpPos += CarJumpPower;
            modelPos =
                new Vector3(modelPos.x, modelPos.y + CarJumpPos, modelPos.z);
            print("ジャンプしている");
            yield return null;
        }

        while (CarJumpPos >= 0) {
            CarJumpPos -= CarJumpPower;
            modelPos -= new Vector3(modelPos.x, modelPos.y - CarJumpPos, modelPos.z);
        }
    }*/

    private IEnumerator CarJump () {

        // モデルがセットされていなければ、エラー吐いてコルーチンが止まってしまうので終了
        if (model_t == null) yield break;

        print("ジャンプ開始");
        jumping = true;

        // 子オブジェクト（車のモデル）のローカル座標(y)が親に対して必ずゼロとは限らないので一応格納。
        float yOffset = model_t.localPosition.y;
        // オフセットにジャンプの高さを足したものをジャンプの最終地点とする。
        float localizedJumpHeight = yOffset + jumpHeight;

        // 目標の高さより低ければ、ポジションを上げていく。
        while (model_t.localPosition.y < localizedJumpHeight) {
            // *** yだけいじる *** //
            // ▼ model_t.localPosition.y = 1　のように直接はいじれないので変数に一度格納。
            Vector3 localPos = model_t.localPosition;
            localPos.y += jumpSpeed * Time.deltaTime;   // yに足していく
            localPos.y = Mathf.Clamp(localPos.y, yOffset, localizedJumpHeight);   // yが最低値と最大値からはみ出ないようにClamp
            model_t.localPosition = localPos;   // 代入
            // ******************* //

            yield return null;   // ループの場合、この記述がないとフリーズする
        }

        // 一応、最大値で代入しておく（たぶんなくてもいい）
        model_t.localPosition = new Vector3(model_t.localPosition.x, localizedJumpHeight, model_t.localPosition.z);

        // オフセット（始めの位置）より高ければポジションを下げる。
        while (model_t.localPosition.y > yOffset) {
            // *** yだけいじる *** //
            Vector3 localPos = model_t.localPosition;
            localPos.y -= jumpSpeed * Time.deltaTime;   // yから引いていく
            localPos.y = Mathf.Clamp(localPos.y, yOffset, localizedJumpHeight);
            model_t.localPosition = localPos;
            // ******************* //

            yield return null;
        }

        // こっちも一応最低値で代入しておく（たぶんなくてもいい）
        model_t.localPosition = new Vector3(model_t.localPosition.x, yOffset, model_t.localPosition.z);

        print("ジャンプ終了");
        jumping = false;

        // コルーチン終了
        yield break;
    }
}

余談

大きなお世話かもしれませんが、コードをコピペした時、手を加えていないにも関わらずエラーが出ていました。
上のコードはそのエラーを修正したのち、改造したものになります。

原因は {} の数が合っていないという小さいものでしたが、エラーはエラーです。
スクリプトの記述について、今一度ご確認ください。

Comments

[Um_kok]

回答ありがとうございます。
これは、1回無限ループになってしまったことがあったので、その時コメントアウトしました。ミスです。
コメントを消しても、動きません。

[Um_kok]

コメントを消しておきますすみません。

[Um_kok]

質問を修正いたしました。お時間合うときにお願いします。

[PinoMatcha]

回答に追記しました。

[PinoMatcha]

ごめんなさい、修正前のコードに手を加えてしまいました…。
動作は確認済みですが、私のコードは参考程度にでもしてください。

[Um_kok]

回答ありがとうございます。
いえいえ、わかりにくいところに書いてしまって申し訳ありません。
ミスしていたのは私なので、、
貴方様の書いてもらったコードを読みながら一文字ずつ打ってみたところ、
すごい分かりやすかったです。
いつもコメントをたくさん書いてくださってとても感謝しております。
実行してみると思い通りの動きをしてくれました！
なるほど、Clamp で高さと低さの範囲を制限するのですね。
あとジャンプ中のことを考えておりませんでした。
やはりまだ私のコードの書き方が余り良くなかったですね。
いい勉強になりました。コードを書くときにはもうちょっと深く考えてみようと思います。
ありがとうございました。困ったことがあったら、またお願いします。

[PinoMatcha]

実装したい処理について、コードを書く前にどういうフローで処理を走らせ、どこでエラーが想定されるのかを考えてみると良いかもしれません。
まぁ別に私もプロではないので、心に留めておいて貰えれば……。
頑張ってください。

[Um_kok]

そうですね。実装前にきちんとエラーのことやその実装に何か抜けがないか、紙にでも書いてやってみます。
貴重なご意見ありがとうございます。
頑張ります。