歩いているときのカメラの揺れが調整出来ない...

Question

unity初心者です
WASDキーのどれかを押すカメラが大きく揺れて何も押してない状態だと小さく揺れるスクリプトを書いたんですが、小さい揺れから大きい揺れになる間をいい感じ調節することができません。
CurveControlledBob.csが動作を調整するスクリプト、Move.csが実行するスクリプトです。

using System;
using UnityEngine;

[Serializable]
public class CurveControlledBob
{
    public AnimationCurve Bobcurve = new AnimationCurve(new Keyframe(0f, 0f), new Keyframe(0.5f, 1f),
                                                        new Keyframe(1f, 0f), new Keyframe(1.5f, -1f),
                                                        new Keyframe(2f, 0f)); // sin curve for head bob
    public float VerticaltoHorizontalRatio;

    private float m_CyclePositionX;
    private float m_CyclePositionY;
    private float m_BobBaseInterval;
    private Vector3 m_OriginalCameraPosition;
    private float m_Time;
    private float kWalkHorizontalBobRange = 0.05f;
    private float kWalkVerticalBobRange = 0.05f;
    private float kStopHorizontalBobRange = 0.05f;
    private float kStopVerticalBobRange = 0.05f;
    private float horizontalBobRange = 0.02f;
    private float verticalBobRange = 0.02f;
    private float kChangeBobSpeed = 0.04f;


    public void Setup(Camera camera, float bobBaseInterval)
    {
        m_BobBaseInterval = bobBaseInterval;
        m_OriginalCameraPosition = camera.transform.localPosition;

        // get the length of the curve in time
        m_Time = Bobcurve[Bobcurve.length - 1].time;
    }


    public Vector3 DoHeadBob(float speed, bool isWalk)
    {
        if (isWalk)
        {
            if (horizontalBobRange < kWalkHorizontalBobRange)
            {
                horizontalBobRange += kChangeBobSpeed;
            }
            if (verticalBobRange < kWalkVerticalBobRange)
            {
                verticalBobRange += kChangeBobSpeed;
            }
        }
        else
        {
            if (horizontalBobRange > kStopHorizontalBobRange)
            {
                horizontalBobRange -= kChangeBobSpeed;
            }
            if (verticalBobRange > kStopVerticalBobRange)
            {
                verticalBobRange -= kChangeBobSpeed;
            }
        }

        float xPos = m_OriginalCameraPosition.x + (Bobcurve.Evaluate(m_CyclePositionX) * horizontalBobRange);
        float yPos = m_OriginalCameraPosition.y + (Bobcurve.Evaluate(m_CyclePositionY) * verticalBobRange);

        m_CyclePositionX += (speed * Time.deltaTime) / m_BobBaseInterval;
        m_CyclePositionY += ((speed * Time.deltaTime) / m_BobBaseInterval) * VerticaltoHorizontalRatio;

        if (m_CyclePositionX > m_Time)
        {
            m_CyclePositionX = m_CyclePositionX - m_Time;
        }
        if (m_CyclePositionY > m_Time)
        {
            m_CyclePositionY = m_CyclePositionY - m_Time;
        }

        return new Vector3(xPos, yPos, 0f);
    }
}

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

public class Move : MonoBehaviour
{
    [SerializeField] private CurveControlledBob headBob_ = new CurveControlledBob();

    private Camera mainCamera_;
    private Transform cameraTransform_;
    // Start is called before the first frame update
    void Start()
    {
        mainCamera_ = this.GetComponent<Camera>();
        headBob_.Setup(mainCamera_, 1.0f);
        cameraTransform_ = GetComponent<Transform>();
    }

    // Update is called once per frame
    void Update()
    {
        if (Input.GetKey(KeyCode.W)
            || Input.GetKey(KeyCode.A)
            || Input.GetKey(KeyCode.S)
            || Input.GetKey(KeyCode.D))
        {
            Vector3 handBob = headBob_.DoHeadBob(0.9f,true);
            cameraTransform_.localPosition = handBob;
        }
        else
        {
            Vector3 handBob = headBob_.DoHeadBob(0.4f, false);
            cameraTransform_.localPosition = handBob;
        }
        
    }
}

Comments

[bboydaisuke]

スクリプトでやるんじゃなくてアニメーションで作ってBlendで滑らかに繋げればいいです。
普通でイージーなやり方をするなら、Cinemachine で仮想カメラを２つ作って、一つは Noise の設定で揺れを小さく、一つは大きくしておき（つまりカメラを揺らすの機能は最初から Unity にあり、コードを書く必要はない）、その２つの仮想カメラをブレンドで切り替えればスムーズにつながります。

[lunayuuta]

なるほど！確かにそのやり方の方がいいですね！
ありがとうございます！

Answer 1

Unity の機能を使わずにスクリプトで何でも解決してしまおうとする癖がついていると、追記修正依頼の所に書いたようなことは見当がつかないかもしれないと思ったので見本を示しておきます。

パッケージ: Blend.unitypackage

スクリプトでやっているのは移動速度を Animator.SetFloat() で Animator Controller に渡しているだけで、Idle/Walk/Run の状態管理は Animator Controller のステートマシンでやっています。
Idle/Walk/Run それぞれの時の仮想カメラ（つまり揺れの違うカメラ）を３台用意して、Cinemachine の State Driven Camera の機能を使って各状態のカメラを（Blend しながら）切り替えています。

つまりスクリプトは Animator.SetFloat() の一行のみで、残りのカメラ制御は全て設定でやっています。FPS 用のマウスでのカメラ操作もコードは一行も書いていません。これらの機能は最初から Unity にあるのでコードを描く必要がありません。

元としたものは Unity Cinemachine 入門2 - FPS カメラを作る の動画からダウンロードできる FPS サンプルです。

// そもそも Head Bob なんて酔うから私はやりませんし、やろうと言われても反対しますけれども