if/unless 修飾子って、break とか next 以外の一般の文では使わないようにしようと、時々思う。でも、つい使っちゃうんだけど。
なんで使わない方が良いと思うかというと、「ふん、ふん」と読んで右を見て、if とか書いてあると、「なんだ。めったに実行しないのか」と思ったりするのが理由。構文ハイライトのあるエディタを使っていれば、ifやunlessが目立つので良いかとも思うけど、エディタ依存のスタイルというのもなんだかな。それから、時間の順序が逆。こっちのほうが大きいか。
next if とかはそれでイディオムに見えるので使っても良いかなと思う。
あと、
File.delete(file_name) if File.exist?(file_name)
とかはいいかな。あくまで、実行部分と条件部分のバランスの問題。
一般の文の場合、例えば、
# (1) url = a["href"] if a.text == "次を表示"
と、
# (2) if a.text == "次を表示" then url = a["href"] end
と、
# (3) if a.text == "次を表示" url = a["href"] end
と、どれが良いか。やっぱり、(1) は良くないよね。(2)のスタイルは避けられることが多いと思うけど、(3)の かさばり感 がいやだな。
あ、意外と、
# (4) a.text == "次を表示" and url = a["href"]
がしっくり来るかも。少なくとも、シェルスクリプトだとこのスタイルだな。
