[SOVLED] Diagonally movement too fast

Diagonally movmement is too fast:

Code (CSharp):

1.        animator.SetFloat("Horizontal", Input.GetAxis("Horizontal"));
2.        animator.SetFloat("Vertical", Input.GetAxis("Vertical"));
3. ...
4.             horizontal = JPE.JoystickHor;
5.             vertical = JPE.JoystickVer;
6.             animator.SetFloat("Horizontal", JPE.JoystickHor);
7.             animator.SetFloat("Vertical", JPE.JoystickVer);
8.             Vector3 move = GetComponent<Rigidbody2D>().velocity = new Vector3(horizontal * speed, vertical * speed, 0);
9.             move.Normalize();

This doesn't work. How can I make it work?

 

https://forum.unity.com/threads/diagonal-movement-speed-to-fast.271703/

 

Yes, I have seen this before I started this thread, but is there any other way, because I don't want to change variables if it's not nessesary.

 

Line 8 isn't doing what you think it is.

move

isn't a reference to rigidbody.velocity -- it's another copy. So, normalizing move later on has no effect on the velocity. Try rearranging the code like this:

Code (CSharp):

1. Vector3 move = new Vector3(horizontal * speed, vertical * speed, 0);
2. move.Normalize();
3. GetComponent<Rigidbody2D>().velocity = move;

 

It's working but it's very very slow now and changing speed doesn't make any difference :/

 

Show what your code looks like now

 

GameController:

Code (CSharp):

1. ...
2. public static float moveSpeed = 15f;
3. ...

PlayerController:

Code (CSharp):

1. speed = GameController.MoveSpeed;
2. ...
3. float horizontal = Input.GetAxis("Horizontal");
4. float vertical = Input.GetAxis("Vertical");
5. ...
6. horizontal = JPE.JoystickHor;
7.             vertical = JPE.JoystickVer;
8.             animator.SetFloat("Horizontal", JPE.JoystickHor);
9.             animator.SetFloat("Vertical", JPE.JoystickVer);
10.             Vector3 move = new Vector3(horizontal * speed, vertical * speed, 0);
11.             move.Normalize();
12.             GetComponent<Rigidbody2D>().velocity = move;

and this is all.

 

Normalize is giving you a unit vector, so speed is having no effect

Code (CSharp):

1.  
2. Vector3 move = new Vector3(horizontal, vertical, 0);
3. move.Normalize();
4. GetComponent<Rigidbody2D>().velocity = move * speed;
5.  

 

That's what I suspected, you're multiplying your speed before you normalize it. Multiply move by speed after you normalize it, it should work fine.

 

You're right. Thanks

This is correct code:

Code (CSharp):

1. Vector3 move = new Vector3(horizontal, vertical, 0);
2.             move.Normalize();
3.             GetComponent<Rigidbody2D>().velocity = move * speed;