How do I stop the flickering that occurs when the character stops moving?

Question

0.00/5 (No votes)

See more:

I followed this tutorial on youtube: Unity3D. Top-down 8 directions movement - YouTube[^]. It took me a while but I've almost converted from movement by arrows to mouse touch . but now I have this problem, there's flickering that occurs when the character stops moving. I looked at the link he sent me but it doesn't work. Please look at this video to show you what I am talking about: My unity problem - YouTube[^]. Does anyone knows how to stop the flickering that occurs when the character stops moving? This is my code:

C++

private Animator anim;
 public float speed = 15f;
 private Vector3 target;
 private bool touched;


 void Start () {
     target = transform.position;
     anim = GetComponent<Animator> ();
 }

 void Update () {
     touched = true;
     if (Input.GetMouseButtonDown (0)) {
         Vector3 mousePosition = Input.mousePosition;
         mousePosition.z = 10; // distance from the camera
         target = Camera.main.ScreenToWorldPoint(mousePosition);
         target.z = transform.position.z;
     }



         var movementDirection = (target - transform.position).normalized;

         if (movementDirection.x != 0 || movementDirection.y != 0) {
             anim.SetBool("walking" , true);
             anim.SetFloat("SpeedX" , movementDirection.x);
             anim.SetFloat("SpeedY" , movementDirection.y);

             Vector2 movement = new Vector2(
                 speed * movementDirection.x ,
                 speed * movementDirection.y);
             movement *= Time.deltaTime;
             transform.Translate(movement);

             if (movementDirection.x < 0) {
                 anim.SetFloat("LastMoveX" , -1f);
             }
             else if (movementDirection.x > 0) {
                 anim.SetFloat("LastMoveX" , 1f);
             }
             else {
                 anim.SetFloat("LastMoveX" , 0f);
             }
             if (movementDirection.y > 0) {
                 anim.SetFloat("LastMoveY" , 1f);
             }
             else if (movementDirection.y < 0) {
                 anim.SetFloat("LastMoveY" , -1f);
             }
             else {
                 anim.SetFloat("LastMoveY" , 0f);
             }
         } else {
         touched = false;
         anim.SetBool("walking" , false);
     }
 }

What I have tried:

I have tried the following:
https://josejimenez.info/blog/unity3d-top-down-8-direction-movement.html

I've Tried checking out what happens if I add something like

if (movementDirection.x != 0 || movementDirection.y != 0) {
anim.SetFloat("SpeedX", movementDirection.x);
anim.SetFloat("SpeedY", movementDirection.y);
}

and

if (movementDirection.x != 0 || movementDirection.y != 0) {
float rot_z = Mathf.Atan2 (movementDirection.y, movementDirection.x) * Mathf.Rad2Deg;
transform.rotation = Quaternion.Euler (0f, 0f, rot_z);
}

as well as
transform.rotation = Quaternion.Euler (0f, 0f, rot_z - 90)

Posted 17-Apr-16 0:34am

wasicool2

Updated 17-Apr-16 3:23am

Add a Solution

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

leon de boer · Answer 1 · 2016-04-17T03:23:00

You need a dead zone on your Animation transitions make them > 0.1f NOT > 0f you are getting jitter. Hold focus on the character in the object inspector and watch the animator editor transitions.

This is the patch to your code but this is not the proper fix which I will explain next

if (movementDirection.x < 0.1f) {
  anim.SetFloat("LastMoveX" , -1f);
} else if (movementDirection.x > 0.1f) {
  anim.SetFloat("LastMoveX" , 1f);
} else {
  anim.SetFloat("LastMoveX" , 0f);
}
 
if (movementDirection.y < 0.1f) {
  anim.SetFloat("LastMoveY" , -1f);
} else if (movementDirection.y > 0.1f) {
  anim.SetFloat("LastMoveY" , 1f);
} else {
  anim.SetFloat("LastMoveYX" , 0f);
}

You can do what this code is doing properly just sending it out to the animator control untouched

anim.SetFloat("LastMoveX" , movement.x);
anim.SetFloat("LastMoveY" , movement.y);

Now on the transitions (the "Arrows" between the states) in the animator control change to things like >0.1f and <-0.1f

There is no need to restrict yourself to 0-1 the value passed out to the animator is a float and it can be negative. Basically you have code doing what is much better and simpler to do out at the animator.

So out at the animator editor on those arrows between states change them

Idle state transitions
LastMoveX < -0.1f transition to walking left
LastMoveX > 0.1f transition to walking right

Walk left state transition dropback to idle
LastMoveX > -0.1f transition to idle

Walk right state transition dropback to idle
LastMoveX < 0.1f transition to idle

Organize that on the animator like you are supposed to and your code becomes totally redundant and is not needed.