you have the same line of code in if/else