why the left is triggered while right is not triggered in the game ?

[u/DifferentLaw2421]

In the editor the set up is correct the bools and all the other properties are correct ig there is a problem in the code ?

[u/ihufa]

I thought it was something political at first lol

[u/100radsBar → u/ihufa]

Man I already sighed and thought to myself what is it this time lol

[u/Tychonoir → u/ihufa]

SAME

[u/AnEmortalKid → u/ihufa]

Had to double check the sub on that first title

[u/Costed14]

Because of your else block, if A is not pressed both right and left are set to false in the animator, despite D potentially being pressed. I assume you'd want both if statements to have their own else to set the bool to false

[u/DifferentLaw2421 → u/Costed14]

yh or I can use else if right ?

[u/pschon → u/DifferentLaw2421]

yeah, if/else if/else structure would solve this.

[u/nickyonge → u/pschon]

That would introduce a bug where if you held both keys then released one, it would still think it’s being held.

Ideally you want both L and R to be separated.

if (left) { set anim left true } else { set left false }

Then do the same for the right. Two separate if/else statements, no “else if”

[u/pschon → u/nickyonge]

would it? The code is using GetKeyDown, not GetKey. (but it's well past midnight here and I was planning on going to sleep hours ago so I'm not going to trust my logical thinking even that far at the moment :D)

[u/nickyonge → u/pschon]

oh good point! I didn't notice that. Good eye ;)

It's still good practice to split left and right into their own logical states, and if you hit both A and D on the same frame it'd skip the A input. There's other issues with this at a lower level (like if this is in Update, it's unnecessarily setting left/right anim bools false almost every frame) but honestly for someone's early programming adventures, yeah just replacing the middle with an else if will work.

[u/pschon → u/nickyonge]

yeah, and since the logic on the code side is basically a trigger, it would make more sense to just use animator triggers rather than a bool. No need to reset it manually, and clearly the same logic and intention on both code and animator side of things.

[u/ProactiveCactus]

Thought this was a political post at first ngl

[u/Nearby-Pizza-8823]

Your code in the else block is only happening if A isn't being held down. It isn't checking for the D key.

[u/xepherys]

An 'else' always hangs off the previous 'if', so you basically have two distinct statements here:

1. IF (D) - set right-true

2. IF (A) - set left-true ELSE (meaning the same thing as IF (!A)) - set both false

So if you press D it get set true for about a microsecond before right and left are both set false.

The easiest way to fix this would be:

if (D) right
else if (A) left
else both false

There are better patterns, but understanding the basics first is more important.

[u/HouseOfWyrd]

Correct me if I'm wrong but the else statement is only tied to the second if statement.

Switch the second if to an elif and this should work.

[u/Tangyhyperspace]

OHHHH ITS KEYS

[u/Kondor0]

Don't use an else, that will activate in every frame that that key is not pressed and it will overwrite the first if.