unityにおける一人称視点のカメラの角度制限について

Question

前提・実現したいこと

私はunityとC#を使い一人称視点のゲームを作っています。
そこでカメラの角度制限をつけようとしたところ以下の二つの問題が発生しています。

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

1.上方向のカメラの角度を90度で制限したいが90度を超えて駆動してしまう
2.下方向にカメラを動かすと指定した40度で止まるがそれ以上動かそうとするとカメラのY軸が回転してしまう

該当のソースコード

//プログラム上から関係のあるところのみ抜粋
　  public Transform verRot;
    public Transform horRot;

	// Use this for initialization
	void Start () {
        verRot = transform.parent;
        horRot = GetComponent<Transform>();
	}

    // Update is called once per frame
    void Update() {
        float X_Rotation = Input.GetAxis("Mouse X");
        float Y_Rotation = Input.GetAxis("Mouse Y");
        float mouseLogY = 0;
        verRot.transform.Rotate(0, -X_Rotation, 0);
        if (horRot.rotation.eulerAngles.x < 40 || horRot.rotation.eulerAngles.x > -90)
        {
            horRot.transform.Rotate(-Y_Rotation, 0, 0);
            mouseLogY = Y_Rotation;
        }
        else if (horRot.rotation.eulerAngles.x > 0)
        {
            if (Y_Rotation < mouseLogY)
            {
                horRot.transform.Rotate(-Y_Rotation, 0, 0);
            }

        }
        else if (horRot.rotation.eulerAngles.x < 0)
        {
            if (Y_Rotation > mouseLogY)
            {
                horRot.transform.Rotate(-Y_Rotation, 0, 0);
            }
        }
        if (horRot.rotation.eulerAngles.x > 40 && horRot.rotation.eulerAngles.x < 180 || horRot.rotation.eulerAngles.x < -90)
        {
            if(horRot.rotation.eulerAngles.x > 0)
            {
                horRot.transform.eulerAngles = new Vector3(40, 0, 0);
            }
            else if(horRot.rotation.eulerAngles.x < 0)
            {
                horRot.transform.eulerAngles = new Vector3(-90, 0, 0);
            }
        }

    }

補足情報

Unity 2017.3.1f1 (64-bit)を使いC#はvisual studioを使用
また、上記のプログラムはウェブサイトを参照しています。

どうかご教授いただけると幸いです。

Answer 1

上90°で止まらなかったのは条件が適切でなかったため、下40°でY軸回転が発生したのはhorRot.transform.eulerAngles = new Vector3(40, 0, 0);の部分でeulerAnglesによるワールド空間基準での回転設定を行ったためではないかと思います。

下記のようにコードを改変してみましたが、いかがでしょうか。
なお、上下方向の回転にはRotateに代わってlocalEulerAnglesへの角度代入を使うことにしました。というのも、オイラー角を監視して上下方向回転角を補正する方法だと、上90°に到達したときに左右方向回転角が不定になり、おかしな挙動をする可能性があったためです（一旦90°まで見上げると視点を下へ戻せなくなるなど）。

    public Transform verRot;
    public Transform horRot;

    private float horAngle; // 現在の上下回転角を保持するフィールドを追加

    // Use this for initialization
    void Start()
    {
        verRot = transform.parent;
        horRot = GetComponent<Transform>();
        horAngle = horRot.localEulerAngles.x; // 上下方向初期回転角を取得
        // localEulerAnglesからの値の取得は最初の一回だけとし、Update内で毎回取得するのは避けた方がいいでしょう
        // やってみるとお分かりかと思いますが、意図しない値にころころ変わってしまっている場合があります
    }

    // Update is called once per frame
    void Update()
    {
        float X_Rotation = Input.GetAxis("Mouse X");
        float Y_Rotation = Input.GetAxis("Mouse Y");
        // float mouseLogY = 0; // この変数は特に使う必要がないように思えたため削除しました

        verRot.transform.Rotate(0, -X_Rotation, 0);
        
        // 以下のコメントアウト部分は意図が不明だったため削除しました
        /*
        if (horRot.rotation.eulerAngles.x < 40 || horRot.rotation.eulerAngles.x > -90) // 「40未満または-90超」ということは、この条件は常にtrueになる?
        {
            horRot.transform.Rotate(-Y_Rotation, 0, 0);
            mouseLogY = Y_Rotation; // ここで代入されたmouseLogYは以降のコードで（引き続くelse節でも）使われていない
        }
        else if (horRot.rotation.eulerAngles.x > 0)
        {
            if (Y_Rotation < mouseLogY) // mouseLogYは常に0
            {
                horRot.transform.Rotate(-Y_Rotation, 0, 0);
            }

        }
        else if (horRot.rotation.eulerAngles.x < 0)
        {
            if (Y_Rotation > mouseLogY) // mouseLogYは常に0
            {
                horRot.transform.Rotate(-Y_Rotation, 0, 0);
            }
        }
        */

        // 回転角制限方法を変えたため以下の部分は削除しました
        /*
        if (horRot.rotation.eulerAngles.x > 40 && horRot.rotation.eulerAngles.x < 180 || horRot.rotation.eulerAngles.x < -90)
        {
            if (horRot.rotation.eulerAngles.x > 0)
            {
                horRot.transform.eulerAngles = new Vector3(40, 0, 0);
            }
            else if (horRot.rotation.eulerAngles.x < 0)
            {
                horRot.transform.eulerAngles = new Vector3(-90, 0, 0);
            }
        }
        */

        horAngle = Mathf.Clamp(horAngle - Y_Rotation, -90, 40); // horAngleを増減、さらに-90~40にクランプ
        horRot.localEulerAngles = new Vector3(horAngle, 0, 0); // ワールド回転ではなく、ローカル回転をhorAngleに設定する

        if (Y_Rotation != 0)
        {
            // localEulerAngles.xをコンソールに表示してみるとお分かりかと思いますが
            // オイラー角は(ほぼ)0～360になるよう整形されて返ってきます
            // ですので、真上を見上げたときは-90でなく270になるはずです
            Debug.Log(horRot.localEulerAngles.x);
        }
    }

Comments

[yaira]

わざわざコードの全文を改変していただき、その上で説明のコメントまでつけていただきありがとうございます。このコードを実行したところ正常に作動しました。