You don't need the "Else" as the "Player-Weapon==" condition determines which sub event of the condition "On key R pressed" will run. Effectively you are trying to use an Else as an Else/If which isn't quite the same thing but tends to work in C2 as each condition is effectively an "If" condition with a redundant If statement. ie you are saying IF key R is pressed and IF player weapon = Else IF Player weapon = Else IF player weapon =.
It's much better practice to write it:
{If} key R is pressed - condition
{If} weapon = 1=> do something - subevent1
{if} weapon = 2 => do something else - subevent2
ELSE do whatever it is you want to do if weapon doesn't equal either 1 or 2. - subevent3