Как реализовать управление машиной по UI кнопкам?

Question

[Bruh_Bruh]

Не могу сделать управление через UI. Проблема именно с осями, как мне кажется. Пробовал определять verticalAxis и transform.localPosition.z, и Vector3.forward.z, и много чего еще, но это не работает. При нажатии кнопки движения вперед машина едет назад, но если ее развернуть, то она будет ехать задом вперед, сама вилять в разные стороны. То, что задокументировано - управление для пк.

spoiler

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

public class Controller : MonoBehaviour
{
    private Rigidbody rb;

    [SerializeField] private Transform frontRightWheelTransform;
    [SerializeField] private Transform frontLeftWheelTransform;
    [SerializeField] private Transform rearRightWheelTransform;
    [SerializeField] private Transform rearLeftWheelTransform;

    [SerializeField] private WheelCollider frontRightWheelCollider;
    [SerializeField] private WheelCollider frontLeftWheelCollider;
    [SerializeField] private WheelCollider rearRightWheelCollider;
    [SerializeField] private WheelCollider rearLeftWheelCollider;

    [SerializeField] private AudioSource engineAudioSourse;

    [SerializeField] private int motorTorque;
    [SerializeField] private int steerAngle;
    private float currentMotorTorque;
    private float currentSteerAngle;

    private float horizontalAxis;
    private float verticalAxis;

    private bool ButtonForwardIsPressed;
    private bool ButtonBackwardIsPressed;
    private bool ButtonRightIsPressed;
    private bool ButtonLeftIsPressed;

    private void Start()
    {
        rb = GetComponent<Rigidbody>();
    }


    private void Update()
    {
        GetAxises();
        HandleMotor();
        HandleSteerAngle();
        EngineHandler();
    }

    private void GetAxises()
    {
        //horizontalAxis = Input.GetAxis("Horizontal");
        //verticalAxis = Input.GetAxis("Vertical");
        verticalAxis = rb.transform.position.z;
        horizontalAxis = rb.transform.position.x;
    }

    private void HandleMotor()
    {
        //currentMotorTorque = motorTorque * verticalAxis;
        //frontRightWheelCollider.motorTorque = currentMotorTorque;
        //frontLeftWheelCollider.motorTorque = currentMotorTorque;
        //rearRightWheelCollider.motorTorque = currentMotorTorque;
        //rearLeftWheelCollider.motorTorque = currentMotorTorque;

        if (ButtonForwardIsPressed == true)
        {
            currentMotorTorque = motorTorque * verticalAxis * -1;
            frontRightWheelCollider.motorTorque = currentMotorTorque;
            frontLeftWheelCollider.motorTorque = currentMotorTorque;
            rearRightWheelCollider.motorTorque = currentMotorTorque;
            rearLeftWheelCollider.motorTorque = currentMotorTorque;
        }
        else
        {
            return;
        }

        if (ButtonBackwardIsPressed == true)
        {
            currentMotorTorque = motorTorque * verticalAxis;
            frontRightWheelCollider.motorTorque = currentMotorTorque;
            frontLeftWheelCollider.motorTorque = currentMotorTorque;
            rearRightWheelCollider.motorTorque = currentMotorTorque;
            rearLeftWheelCollider.motorTorque = currentMotorTorque;
        }
        else
        {
            return;
        }
    }

    private void HandleSteerAngle()
    {
        //currentSteerAngle = steerAngle * horizontalAxis;
        //frontRightWheelCollider.steerAngle = currentSteerAngle;
        //frontLeftWheelCollider.steerAngle = currentSteerAngle;

        if (ButtonRightIsPressed == true)
        {
            currentSteerAngle = steerAngle * rb.transform.right.x;
            frontRightWheelCollider.steerAngle = currentSteerAngle;
            frontLeftWheelCollider.steerAngle = currentSteerAngle;
        }
        else
        {
            return;
        }

        if (ButtonLeftIsPressed == true)
        {
            currentSteerAngle = steerAngle * rb.transform.right.x * -1;
            frontRightWheelCollider.steerAngle = currentSteerAngle;
            frontLeftWheelCollider.steerAngle = currentSteerAngle;
        }
        else
        {
            return;
        }
    }

    private void EngineHandler()
    {
        if (Input.GetKey(KeyCode.W) || Input.GetKey(KeyCode.S))
        {
            if (!engineAudioSourse.isPlaying)
            {
                engineAudioSourse.Play();
            }
        }
        else
        {
            engineAudioSourse.Stop();
        }
    }

    #region UIButtons
    public void OnForwardButtonDown()
    {
        ButtonForwardIsPressed = true;
    }

    public void OnBackwardButtonDown()
    {
        ButtonBackwardIsPressed = true;
    }

    public void OnRightButtonDown()
    {
        ButtonRightIsPressed = true;
    }

    public void OnLeftButtonDown()
    {
        ButtonLeftIsPressed = true;
    }

    public void OnForwardButtonUp()
    {
        ButtonForwardIsPressed = false;
    }

    public void OnBackwardButtonUp()
    {
        ButtonBackwardIsPressed = false;
    }

    public void OnRightButtonUp()
    {
        ButtonRightIsPressed = false;
    }

    public void OnLeftButtonUp()
    {
        ButtonLeftIsPressed = false;
    }
    #endregion

}

Comments

[Dance Macabre]

Выложите скрипт целиком и под спойлер.

[Dance Macabre]

Еще скриншот нужен с иерархией сцены, чтобы понять, кто у кого родитель, и где данный скрипт висит.

[Bruh_Bruh]

Dance Macabre,

spoiler

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

public class Controller : MonoBehaviour
{
    private Rigidbody rb;

    [SerializeField] private Transform frontRightWheelTransform;
    [SerializeField] private Transform frontLeftWheelTransform;
    [SerializeField] private Transform rearRightWheelTransform;
    [SerializeField] private Transform rearLeftWheelTransform;

    [SerializeField] private WheelCollider frontRightWheelCollider;
    [SerializeField] private WheelCollider frontLeftWheelCollider;
    [SerializeField] private WheelCollider rearRightWheelCollider;
    [SerializeField] private WheelCollider rearLeftWheelCollider;

    [SerializeField] private AudioSource engineAudioSourse;

    [SerializeField] private int motorTorque;
    [SerializeField] private int steerAngle;
    private float currentMotorTorque;
    private float currentSteerAngle;

    private float horizontalAxis;
    private float verticalAxis;

    private bool ButtonForwardIsPressed;
    private bool ButtonBackwardIsPressed;
    private bool ButtonRightIsPressed;
    private bool ButtonLeftIsPressed;

    private void Start()
    {
        rb = GetComponent<Rigidbody>();
    }


    private void Update()
    {
        GetAxises();
        HandleMotor();
        HandleSteerAngle();
        EngineHandler();
    }

    private void GetAxises()
    {
        //horizontalAxis = Input.GetAxis("Horizontal");
        //verticalAxis = Input.GetAxis("Vertical");
        verticalAxis = rb.transform.position.z;
        horizontalAxis = rb.transform.position.x;
    }

    private void HandleMotor()
    {
        //currentMotorTorque = motorTorque * verticalAxis;
        //frontRightWheelCollider.motorTorque = currentMotorTorque;
        //frontLeftWheelCollider.motorTorque = currentMotorTorque;
        //rearRightWheelCollider.motorTorque = currentMotorTorque;
        //rearLeftWheelCollider.motorTorque = currentMotorTorque;

        if (ButtonForwardIsPressed == true)
        {
            currentMotorTorque = motorTorque * verticalAxis * -1;
            frontRightWheelCollider.motorTorque = currentMotorTorque;
            frontLeftWheelCollider.motorTorque = currentMotorTorque;
            rearRightWheelCollider.motorTorque = currentMotorTorque;
            rearLeftWheelCollider.motorTorque = currentMotorTorque;
        }
        else
        {
            return;
        }

        if (ButtonBackwardIsPressed == true)
        {
            currentMotorTorque = motorTorque * verticalAxis;
            frontRightWheelCollider.motorTorque = currentMotorTorque;
            frontLeftWheelCollider.motorTorque = currentMotorTorque;
            rearRightWheelCollider.motorTorque = currentMotorTorque;
            rearLeftWheelCollider.motorTorque = currentMotorTorque;
        }
        else
        {
            return;
        }
    }

    private void HandleSteerAngle()
    {
        //currentSteerAngle = steerAngle * horizontalAxis;
        //frontRightWheelCollider.steerAngle = currentSteerAngle;
        //frontLeftWheelCollider.steerAngle = currentSteerAngle;

        if (ButtonRightIsPressed == true)
        {
            currentSteerAngle = steerAngle * rb.transform.right.x;
            frontRightWheelCollider.steerAngle = currentSteerAngle;
            frontLeftWheelCollider.steerAngle = currentSteerAngle;
        }
        else
        {
            return;
        }

        if (ButtonLeftIsPressed == true)
        {
            currentSteerAngle = steerAngle * rb.transform.right.x * -1;
            frontRightWheelCollider.steerAngle = currentSteerAngle;
            frontLeftWheelCollider.steerAngle = currentSteerAngle;
        }
        else
        {
            return;
        }
    }

    private void EngineHandler()
    {
        if (Input.GetKey(KeyCode.W) || Input.GetKey(KeyCode.S))
        {
            if (!engineAudioSourse.isPlaying)
            {
                engineAudioSourse.Play();
            }
        }
        else
        {
            engineAudioSourse.Stop();
        }
    }

    #region UIButtons
    public void OnForwardButtonDown()
    {
        ButtonForwardIsPressed = true;
    }

    public void OnBackwardButtonDown()
    {
        ButtonBackwardIsPressed = true;
    }

    public void OnRightButtonDown()
    {
        ButtonRightIsPressed = true;
    }

    public void OnLeftButtonDown()
    {
        ButtonLeftIsPressed = true;
    }

    public void OnForwardButtonUp()
    {
        ButtonForwardIsPressed = false;
    }

    public void OnBackwardButtonUp()
    {
        ButtonBackwardIsPressed = false;
    }

    public void OnRightButtonUp()
    {
        ButtonRightIsPressed = false;
    }

    public void OnLeftButtonUp()
    {
        ButtonLeftIsPressed = false;
    }
    #endregion

}

как-то так?

[Dance Macabre]

Bruh_Bruh, да, только вопрос то можно редактировать, незачем в комменты такое класть.

Answer 1

[Dance Macabre]

Не знаю, как именно у вас авто построено, кто кому родитель, где коллайдеры и т.п., но думаю причина именно в этом, а не в скрипте. Ведь скрипт только говорит колесам когда крутится и когда поворачивать. А дальше уже физика работает, и работает так, как настроен "физический" объект.

Найдите готовую машину, пример, и изучите от и до структуру. Так будет проще всего понять, что к чему.

По скрипту замечания:

задокументировано - управление для пк

Используйте #ifdef для такого, чтобы работали ОБА варианта в зависимости от платформы, без ручного пекреключения.

Как-то управление криво организовано, вызывается событие, что создаёт условие для проверки по условиям.
Если ГАЗ-НАЖАТ > то ГАЗ-ДА > если ГАЗ-ДА > то СКОРОСТЬ+
можно ведь
Если ГАЗ-НАЖАТ > то СКОРОСТЬ+

else
        {
            return;
        }

Это вы сами похоже не монимаете, зачем вставили, ведь если нажмете кнопку "поворот вправо" и "поворот влево" одновременно, будет только вправо и поворачивать.

rb.transform.position.z и transform.position.z; одно и тоже, в данном случае. И это не "ось", не "axis", а лишь координата объекта по одной из осей. Получается, что чем дальше авто проедет вперед, тем быстрее будет ехать вперед. Чем сильнее сместиться вправо, тем сильнее будет поворачивать вправо. Странно это всё.

Действия с физикой надо проводить в FixedUpdate(), не в Update().

Если один и тот же код пишется дважды или более в разных местах, выводите его в отдельный метод.

Comments

[Bruh_Bruh]

А с verticalAxis что делать? Присвоить ей Input.GetAxis("Vertical") или что-то еще? Оси же надо как-то обозначить.

[Dance Macabre]

Bruh_Bruh, так а зачем вам эти "оси"? Чтобы получить величину движения/поворота. Если кнопка нажата на ПК, то да, берем эту величину из оси берем:
движВперед = ось
или напрямую по кнопке прописываем
update > если нажата ВПЕРЕД > то движВперед = 1
в последнем случае проще будет объеденить с UI кнопкой, та же логика, только не по условию, а по событию
нажата ВПЕРЕД > вызов метода > движВперед = 1
А когда ничего не нажато, то пусть величина прадает постепенно, тормозит, пока до нуля не дойдет.
update > движВперед = движВперед * 0.9

Но это так, грубо, можно еще варианты придумать, но я ммашины не собирал, не подсказу, какой оптимальный, а так то любой вариант сработает.

[Bruh_Bruh]

Dance Macabre, попробовал, вроде работает пока что, как теперь скрипт оцените?

spoiler

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

public class Controller : MonoBehaviour
{
    [SerializeField] private Transform frontRightWheelTransform;
    [SerializeField] private Transform frontLeftWheelTransform;
    [SerializeField] private Transform rearRightWheelTransform;
    [SerializeField] private Transform rearLeftWheelTransform;

    [SerializeField] private WheelCollider frontRightWheelCollider;
    [SerializeField] private WheelCollider frontLeftWheelCollider;
    [SerializeField] private WheelCollider rearRightWheelCollider;
    [SerializeField] private WheelCollider rearLeftWheelCollider;

    [SerializeField] private AudioSource engineAudioSourse;

    [SerializeField] private int motorTorque;
    [SerializeField] private int steerAngle;
    private float currentMotorTorque;
    private float currentSteerAngle;

    private void Update()
    {
        EngineHandler();
    }

    private void HandleMotor(int axisValue)
    {
        currentMotorTorque = motorTorque * axisValue;
        frontRightWheelCollider.motorTorque = currentMotorTorque;
        frontLeftWheelCollider.motorTorque = currentMotorTorque;
        rearRightWheelCollider.motorTorque = currentMotorTorque;
        rearLeftWheelCollider.motorTorque = currentMotorTorque;
    }

    private void HandleSteerAngle(int axisValue)
    {
        currentSteerAngle = steerAngle * axisValue;
        frontRightWheelCollider.steerAngle = currentSteerAngle;
        frontLeftWheelCollider.steerAngle = currentSteerAngle;
    }

    private void EngineHandler()
    {
        if (Input.GetKey(KeyCode.W) || Input.GetKey(KeyCode.S))
        {
            if (!engineAudioSourse.isPlaying)
            {
                engineAudioSourse.Play();
            }
        }
        else
        {
            engineAudioSourse.Stop();
        }
    }

    #region UIButtons
    public void OnForwardButtonDown()
    {
        HandleMotor(1);
    }

    public void OnBackwardButtonDown()
    {
        HandleMotor(-1);
    }

    public void OnRightButtonDown()
    {
        HandleSteerAngle(1);
    }

    public void OnLeftButtonDown()
    {
        HandleSteerAngle(-1);
    }

    public void OnForwardButtonUp()
    {
        StartCoroutine(DecreaseSpeed());
    }

    public void OnBackwardButtonUp()
    {
        StartCoroutine(DecreaseSpeed());
    }

    public void OnRightButtonUp()
    {
        currentSteerAngle = 0;
        frontRightWheelCollider.steerAngle = currentSteerAngle;
        frontLeftWheelCollider.steerAngle = currentSteerAngle;
    }

    public void OnLeftButtonUp()
    {
        currentSteerAngle = 0;
        frontRightWheelCollider.steerAngle = currentSteerAngle;
        frontLeftWheelCollider.steerAngle = currentSteerAngle;
    }
    #endregion

    IEnumerator DecreaseSpeed()
    {
        yield return new WaitForSeconds(5f);
        currentMotorTorque = 0;
        frontRightWheelCollider.motorTorque = currentMotorTorque;
        frontLeftWheelCollider.motorTorque = currentMotorTorque;
        rearRightWheelCollider.motorTorque = currentMotorTorque;
        rearLeftWheelCollider.motorTorque = currentMotorTorque;
    }

}

[Dance Macabre]

Bruh_Bruh, лучше но еще далеко от идеала. По прежнему винегред.
Зачем там корутина? Как её остановить если ГАЗ нажали?
Зачем событие на отпускании кнопки, если именно при НЕ нажатих кнопках машина должна замедлятся (как в жизни)?
Почему сила в int а не float?

Ну и еще много на что можно обратить внимание. Так что дерзайте, учитесь.