どうも check_xxx というメソッド名は辞めよう委員会のやましろです。 追記:hoge_managerとかfoo_processorとか辞めよう委員会の委員でもあります ダメやダメや。と言ってたら「なんでダメなん?」って聞かれたので例を書いてみました(ミスってアノニマスで作っちゃった) check_userという名前では、何をチェックするかがわかりません。 一個のメソッドの中で複数のことをするのは辞めましょう 予期しないexitとか… 実は、check_xxx がダメな理由って、最初のだけだけど、 check_xxx 書くやつ、絶対他のこともそのメソッドでやるんだよなー。 まぁ、「絶対的に正しいコード」なんてないですけど、 check_xxx はいただけないですね。 この例だったら、中で余計なことしてなくて、 check_user_x_stateぐらいだったらギリ許す。 でもそれだっ