条件に名前をつければいいだろ
ifは条件判断してenumを返す
実際の処理はswitchで行う
何でもかんでも一つの関数にごちゃまぜにして書くからややこしくなる