为什么即使我在特定度数范围之间随机限制旋转，变换也会旋转 360 度？

Question

The guns is rotating 360 degrees and not between -40 and 0

This screenshot is the default before running the game the rotation on x is 0 :

and this screenshot is after I changed the guns x rotation value to -40

I want the guns to rotate up down between 0 and -40 but when I'm running the game the guns rotating 360 degrees.

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

public class MissleLauncherRotation : MonoBehaviour
{
    public Transform body;
    public Transform guns;
    public Vector3 spinAxisBody;
    public Vector3 spinAxisGuns;
    public float timeToSpin = 5f;
    public float spinSpeed = 20f;
    public bool randomSpin = false;

    private void Start()
    {
        StartCoroutine("Spin");
    }

    private void Update()
    {

    }

    IEnumerator Spin()
    {
        float spinTimer;
        while (true)
        {
            if (randomSpin == true)
            {
                spinAxisBody = new Vector3(0,Random.Range(-180, 180),0);

                spinAxisGuns = new Vector3(Random.Range(-40, 0), 0, 0);
            }

            spinTimer = timeToSpin;
            while (spinTimer > 0f)
            {
                body.transform.Rotate(spinAxisBody, Time.deltaTime * spinSpeed);
                guns.Rotate(spinAxisGuns, Time.deltaTime * spinSpeed);
                spinTimer -= Time.deltaTime;
                yield return null;
            }
        }
    }
}

I tried this :

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

public class MissleLauncherRotation : MonoBehaviour
{
    public Transform body;
    public Transform guns;
    public Vector3 spinAxisBody;
    public Vector3 spinAxisGuns;
    public float timeToSpin = 5f;
    public float spinSpeed = 20f;
    public bool randomSpin = false;

    Quaternion rot;

    private void Start()
    {
        rot = guns.rotation;
    }

    private void Update()
    {
        guns.Rotate(guns.up, (spinSpeed * Time.deltaTime));
        var rot = guns.rotation.eulerAngles;
        rot = new Vector3
                                        (
                                            guns.eulerAngles.x,
                                            Mathf.Clamp(rot.x, -40, 0),
                                            guns.eulerAngles.z
                                        );
        guns.rotation = Quaternion.Euler(rot);
    }
}

The only thing is working now is the clamping but the clamping is not only on the x and I want to use coroutine or somehow to make that it will rotate ever x seconds.

I did it this way to test the clamping if it's working at all now how do I use it with my first code ?

Answer 1

As already mentioned Rotate adds a rotation to the current one.

I would rather precalculate the target rotation and use eg

public Transform body;
public Transform guns;
public Vector3 spinAxisBody;
public Vector3 spinAxisGuns;
public float timeToSpin = 5f;
public float spinSpeed = 20f;
public bool randomSpin = false;

IEnumerator Start()
{
    while (true)
    {
        if (randomSpin == true)
        {
            spinAxisBody = new Vector3(0,Random.Range(-180, 180),0);

            spinAxisGuns = new Vector3(Random.Range(-40, 0), 0, 0);
        }

        var startBodyRotation = body.localRotation;
        var targetBodyRotation = startBodyRotation * Quaternion.Euler(spinAxisBody);   

        var startGunRotation = guns.localRotation;
        var targetGunsRotation = startGunRotation * Quaternion.Euler(spinAxisGuns);  

        for(spinTimer = 0; spinTimer < timeToSpin; spinTimer += Time.deltaTime)
        {
            // Factor linear moving from 0 to 1 within timeToSpin seconds
            var factor = spinTimer / timeToSpin;
            // optional add ease-in and -out
            //factor = Mathf.SmoothStep(0, 1, factor);

            body.transform.localRotation = Quaternion.Lerp(startBodyRotation, targetBodyRotation, factor);
            guns.localRotation(startGunRotation, targetGunsRotation, factor);

            yield return null;
        }
    }
}