Mouse Look (Script)

Scripting
1

First of all I would like to apologise if this thread has been made already. However I have been looking for a good 1.5-2 hours now with no luck.

Basically I have done my research and discovered that the Mouse Look (Script) in Unity is bugged in the fact that the development team haven’t implemented the “X” axis into the script. I have opened it up and knowing nothing about scripting I can see myself where the error is. However, all fixes I find on the internet tell you to change code or copy and paste this in the right place and like I say, I haven’t got a clue when it comes to scripting so no idea which bits need changing.

I have tried changing parts of the code myself with no luck.

I was hoping someone could go into the code and change the relevant bit for me and copy and paste the whole code here so basically I can just copy and paste the whole code into Unity.

I will be very grateful if someone who knows what they are doing could do this for me. It will only be a 5 minute job.

Many Thanks

Dean Hillary

[Current Code] - I have highlighted the bit I believe to be bugged in red

using UnityEngine;
using System.Collections;

/// MouseLook rotates the transform based on the mouse delta.
/// Minimum and Maximum values can be used to constrain the possible rotation

/// To make an FPS style character:
/// - Create a capsule.
/// - Add the MouseLook script to the capsule.
/// → Set the mouse look to use LookX. (You want to only turn character but not tilt it)
/// - Add FPSInputController script to the capsule
/// → A CharacterMotor and a CharacterController component will be automatically added.

/// - Create a camera. Make the camera a child of the capsule. Reset it’s transform.
/// - Add a MouseLook script to the camera.
/// → Set the mouse look to use LookY. (You want the camera to tilt up and down like a head. The character already turns.)
[AddComponentMenu(“Camera-Control/Mouse Look”)]
public class MouseLook : MonoBehaviour {

public enum RotationAxes { MouseXAndY = 0, MouseX = 1, MouseY = 2 }
public RotationAxes axes = RotationAxes.MouseXAndY;
public float sensitivityX = 15F;
public float sensitivityY = 15F;

public float minimumX = -360F;
public float maximumX = 360F;

public float minimumY = -60F;
public float maximumY = 60F;

float rotationY = 0F;

void Update ()
{
if (axes == RotationAxes.MouseXAndY)
{
float rotationX = transform.localEulerAngles.y + Input.GetAxis(“Mouse X”) * sensitivityX;

rotationY += Input.GetAxis(“Mouse Y”) * sensitivityY;
rotationY = Mathf.Clamp (rotationY, minimumY, maximumY);

transform.localEulerAngles = new Vector3(-rotationY, rotationX, 0);
}
else if (axes == RotationAxes.MouseX)
{
transform.Rotate(0, Input.GetAxis(“Mouse X”) * sensitivityX, 0);
}
else
{
rotationY += Input.GetAxis(“Mouse Y”) * sensitivityY;
rotationY = Mathf.Clamp (rotationY, minimumY, maximumY);

transform.localEulerAngles = new Vector3(-rotationY, transform.localEulerAngles.y, 0);
}
}

void Start ()
{
// Make the rigid body not change rotation
if (rigidbody)
rigidbody.freezeRotation = true;
}
}

1 Like
2

How is it that nobody commented this thread?

Updated version:

Mouse Look script

using UnityEngine;
using System.Collections;

[AddComponentMenu("Camera-Control/Mouse Look")]
public class MouseLook : MonoBehaviour
{

    public enum RotationAxes { MouseXAndY = 0, MouseX = 1, MouseY = 2 }
    public RotationAxes axes = RotationAxes.MouseXAndY;
    public float sensitivityX = 15F;
    public float sensitivityY = 15F;

    public float minimumX = -360F;
    public float maximumX = 360F;

    public float minimumY = -60F;
    public float maximumY = 60F;

    private Rigidbody rb;
    float rotationY = 0F;

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

        // Make the rigid body not change rotation
        if (GetComponent<Rigidbody>())
            rb.freezeRotation = true;
    }

    void Update ()
    {
        if (axes == RotationAxes.MouseXAndY)
        {
            float rotationX = transform.localEulerAngles.y + Input.GetAxis("Mouse X") * sensitivityX;

            rotationY += Input.GetAxis("Mouse Y") * sensitivityY;
            rotationY = Mathf.Clamp (rotationY, minimumY, maximumY);

            transform.localEulerAngles = new Vector3(-rotationY, rotationX, 0);
        }
        else if (axes == RotationAxes.MouseX)
        {
            transform.Rotate(0, Input.GetAxis("Mouse X") * sensitivityX, 0);
        }
        else
        {
            rotationY += Input.GetAxis("Mouse Y") * sensitivityY;
            rotationY = Mathf.Clamp (rotationY, minimumY, maximumY);

            transform.localEulerAngles = new Vector3(-rotationY, transform.localEulerAngles.y, 0);
        }
    }
}

It works rather well when attached directly to the player; except that the mouse pointer is still visible on the screen and that “Mouse x” and “Mouse y” are independent from each other.

It doesn’t work well at all when attached to a camera child of the player.

I would be grateful is someone was willing to help in the matter, thanks! :slight_smile:

1 Like
3

See:

Here is a simplified version of a FPS mouse look script that will prevent rotation when looking 90 degrees up or down.

private float X;
private float Y;

public float Sensitivity;

void Awake()
{
    Vector3 euler = transform.rotation.eulerAngles;
    X = euler.x;
    Y = euler.y;
}

void Update()
{
    const float MIN_X = 0.0f;
    const float MAX_X = 360.0f;
    const float MIN_Y = -90.0f;
    const float MAX_Y = 90.0f;

    X += Input.GetAxis("Mouse X") * (Sensitivity * Time.deltaTime);
    if (X < MIN_X) X += MAX_X;
    else if (X > MAX_X) X -= MAX_X;
    Y -= Input.GetAxis("Mouse Y") * (Sensitivity * Time.deltaTime);
    if (Y < MIN_Y) Y = MIN_Y;
    else if (Y > MAX_Y) Y = MAX_Y;

    transform.rotation = Quaternion.Euler(Y, X, 0.0f);
}

You can lock the cursor with Cursor.lockState and hide the cursor with Cursor.visible.

4 Likes
4

Thank you! :slight_smile:

5

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

public class MouseControll : MonoBehaviour {

private Vector3 Rot;
private float xRot;
private float yRot;
private float xCurrRot;
private float yCurrRot;
[SerializeField]
private Camera fpcCamera;
[SerializeField]
private GameObject fpcObject;
public float mouseSensetive;
private float xRotVelocity;
private float yRotVelocity;
[SerializeField]
private float smoothDampTime = 0.1f;

// Use this for initialization
void Start () {
mouseSensetive=1f;
}

// Update is called once per frame
void Update () {
MouseMove();
}

private void MouseMove()
{
xRot+=Input.GetAxis(“Mouse Y”)*mouseSensetive;
yRot+=Input.GetAxis(“Mouse X”)*mouseSensetive;
xRot=Mathf.Clamp(xRot,-90,90);

xCurrRot=Mathf.SmoothDamp(xCurrRot, xRot, ref xRotVelocity, smoothDampTime);
yCurrRot=Mathf.SmoothDamp(yCurrRot, yRot, ref yRotVelocity, smoothDampTime);

fpcCamera.transform.rotation=Quaternion.Euler(xCurrRot,yCurrRot,0f);
fpcObject.transform.rotation=Quaternion.Euler(0f,yCurrRot,0f);

}
}

1 Like
6

why is it inverted

2 Likes
7

thanks man

8

Bad Code Practice

9

I am really late to the thread, but if someone could help me that would be great.
I used the code in this thread for my look script but and it works great except when my character turns ‘w’, which moved the character forwards, is now moving my character forwards. I used ladyonthemoon’s code and this is my movement code.

using UnityEngine;
using System.Collections;

// This script moves the character controller forward
// and sideways based on the arrow keys.
// It also jumps when pressing space.
// Make sure to attach a character controller to the same game object.
// It is recommended that you make only one call to Move or SimpleMove per frame.

public class movplayer : MonoBehaviour
{
CharacterController characterController;

public float speed = 6.0f;
public float jumpSpeed = 8.0f;
public float gravity = 20.0f;

private Vector3 moveDirection = Vector3.zero;

void Start()
{
characterController = GetComponent();
}

void Update()
{
if (characterController.isGrounded)
{
// We are grounded, so recalculate
// move direction directly from axes

moveDirection = new Vector3(Input.GetAxis(“Horizontal”), 0.0f, Input.GetAxis(“Vertical”));
moveDirection *= speed;

if (Input.GetButton(“Jump”))
{
moveDirection.y = jumpSpeed;
}
}

// Apply gravity. Gravity is multiplied by deltaTime twice (once here, and once below
// when the moveDirection is multiplied by deltaTime). This is because gravity should be applied
// as an acceleration (ms^-2)
moveDirection.y -= gravity * Time.deltaTime;

// Move the controller
characterController.Move(moveDirection * Time.deltaTime);

}
}

10

No, you’re not late but you are necroing/hijacking an existing thread. Please create your own thread instead.

Also, please use code-tags when posting code and not plain text.

Thanks.