Не могу понять что не так, почему не работает?

Question

[Герман Коффман]

По плану предмет должен крутиться от минимального до максимального предела, а потом менять направление пока не дойдет до другого и так по кругу. Но что то этот не работает, не могу понять почему.

if (!rightMove)
            {
                if (transform.rotation.z < limitHigh) //90
                {
                    print("limitHigh");
                    transform.Rotate(move * -speed);
                }
                else
                {
                    rightMove = false;
                }
            }
            else 
            {
                if (transform.rotation.z > limitLower) //-90
                {
                    print("limitHigh");
                    transform.Rotate(move * speed);
                }
                else
                {
                    rightMove = !false;
                }
            }

Comments

[Василий Банников]

rightMove = !false;
Интересный код :)

[Eugene]

Василий Банников, надо было !!true

[Василий Банников]

Eugene, ага, в ифе rightMove.ToString().Length == 5

[Eugene]

Василий Банников, это гениально, такого я ещё не видел. Насмешили =)

[Василий Банников]

Eugene, классика с говнокод.ру

Answer 1

[Василий Банников]

rightMove = !false;
Интересный код :)

В целом если у тебя !rightMove (истина, когда rightMove = false) у тебя в итоге код попадает только в ту ветку, где rightMove=false

И наоборот

Comments

[Герман Коффман]

if (rightMove)
            {
                if (transform.rotation.z < limitHigh) //90
                {
                    print("limitHigh");
                    transform.Rotate(move * -speed);
                }
                else
                {
                    rightMove = false;
                }
            }
            else 
            {
                if (transform.rotation.z > limitLower) //-90
                {
                    print("limitHigh");
                    transform.Rotate(move * speed);
                }
                else
                {
                    rightMove = true;
                }
            }

Ну так верно, но оно не меняет направление при достижении придела

[Василий Банников]

А что такое move и speed здесь?
В каком диапазоне изменяется transform.rotation.z?
В мануале ещё пишут, что

If you want to match values you see in the Inspector, use the Quaternion.eulerAngles property on the returned Quaternion.

Тоесть тебе, скорее всего, нужен не transform.rotation.z, а transform.rotation.eulerAngles.z
Ибо свойство z квартерниона возвращает не угол относительно оси Z, а какое-то внутренне представление самого квартерниона.

[Герман Коффман]

Василий Банников, move - вектор куда вращяется все, а speed - скорость вращения

[Василий Банников]

Герман Коффман, в общем я подозреваю больше всего, что проблема в том, что не оттуда читаешь Z

[Герман Коффман]

Василий Банников, А может localRotation - подойдет?

[Eugene]

Герман Коффман, а что значит "куда-то вращаться"? Я знаю только о вращении вокруг осей, что и делает Rotate();
Ты пишешь transform.Rotate(move * speed). Это будет означать, что объект будет вращаться вокруг каждой осей на столько, сколько указано в каждой из координат вектора move. Только исходя из документации я смог сделать вывод, что move - это у тебя вектор. Если в move у тебя ненулевая только ось z, то вращаться будет как и задумано (если я правильно понял задумку). Про eulerAngles тебе уже сказали.

[Eugene]

Герман Коффман, И кстати, вы не сказали, как именно это не работает. Уточняйте пожалуйста, что происходит, когда вы запускаете скрипт.

[Василий Банников]

Eugene, как я понял из диалога - оно крутится, но не меняет направление вращения.

[Герман Коффман]

Василий Банников, да, это так. Должно доходить до одного предела и крутиться в другую сторону пока не достигнет того предела и не пойдет крутиться наоборот и так далее.

[Eugene]

Герман Коффман, так ты ещё не исправил? Тебе сказали выше, используй transform.rotation.eulerAngles.z вместо transform.rotation.z

[Герман Коффман]

Eugene, исправил, но все так же, вертится только в одну сторону.

[Eugene]

Герман Коффман, во время вращения посмотрите в инспекторе значение поворота z. Оно у вас увеличивается или уменьшается? Дождались пока оно дойдёт до значения limitHigh?

[Герман Коффман]

Eugene, вращяется против часовой стрелке по кругу. Доходит, но не с той стороны.

[Eugene]

Герман Коффман, тогда просто поменяйте местами
transform.Rotate(move * -speed);
и
transform.Rotate(move * speed);

[Герман Коффман]

Eugene, все так же

[Василий Банников]

Герман Коффман, какие конкретно значения у move и speed?

[K0TlK]

Ищем костыли 10 часов.
Герман Коффман, Тебе не нужна rightMove, просто скорость изменяй с положительной на отрицательную.

using UnityEngine;

public class Rotation : MonoBehaviour
{
    [SerializeField] private float _minAngle = 0;
    [SerializeField] private float _maxAngle = 90f;
    [SerializeField] private float _speed = 10f;

    private float _angle = 0;

    private void FixedUpdate()
    {
        Rotate();
    }

    private void Rotate()
    {
        var nextAngle = _angle + _speed * Time.deltaTime;

        _angle = Mathf.Clamp(nextAngle, _minAngle, _maxAngle);

        transform.rotation = Quaternion.Euler(0, 0, _angle);

        if (_angle == _minAngle || _angle == _maxAngle) SwitchDirection();
    }

    private void SwitchDirection() => _speed *= -1;
}

Рассичтываешь угол в следующем кадре -> ограничиваешь -> присваиваешь вращение -> меняешь направление. Всё. Никаких миллионов условий.

[Герман Коффман]

K0TlK, Спасибо! Очень классный код, не совсем понимаю что делает Clamp(), почитаю документацию.