Jagged Movement

Unity Engine
1

Can anyone tell me why the below is printing all of the directions and giving my enemy jagged movement? Appreciate any advice.

void CheckWandering()
{

if (isWalking)
{
anim.SetBool(“isIdle”, false);
if (Random.Range(0,4) == 0)
{
print(“right”);
transform.position += Vector3.right * moveSpeed * Time.fixedDeltaTime;

}
if (Random.Range(0, 4) == 1)
{
print(“left”);
transform.position += Vector3.left * moveSpeed * Time.fixedDeltaTime;
}
if (Random.Range(0, 4) == 2)
{
print(“up”);
transform.position += Vector3.up * moveSpeed * Time.fixedDeltaTime;
}
if (Random.Range(0, 4) == 3)
{
print(“down”);
transform.position += Vector3.down * moveSpeed * Time.fixedDeltaTime;
}

}
else
{
anim.SetBool(“isIdle”, true);
}
}

2

Hi and welcome!
Please use code tags. It’s explained the first sticky on this subforum. Makes stuff a lot more readable.

Your main problem regarding your question is that you create a new random number in each if statement. So you are basically rolling a dice, saying “do XYZ if result is 1”, then you roll a different other dice and ask again “do ABC only if result is 2” and so on. So technically, all if statements can be true, or none of them. Instead you’ll want to save the result of Random.Range(0,4) in an int variable and compare that number to 0, 1, 2 or 3 instead.

1 Like
3

Thank you for the reply and feedback. I managed to fix the problem and realise my error by watching a tutorial video but this explanation is very good. Sorry about the code block, new to the forum and coding!

4

No problem, that’s why i mentioned it. Keep having fun! :slight_smile: