As your code stands, if you receive a character, that character is assigned to the variable x, and until another character is received, the action which depends upon x will repeat. You may not want to do this, particularly for the servo position increments and decrements, which appear not to have any end-stops programmed.