else if文やelse文は、不要なものが大半を占めています。
例えばこちらのソースは一見シンプルで綺麗に見えますが、実際に読んでみると、どういうときにメイン処理が実行されるのかが意外と分かりにくいです。
ロジックについて
そのelse文、本当に必要？
ロジックについて
深いネストはバグの温床
深すぎるネストはバグの温床にしかなりません。
できるだけ浅くする努力をしましょう。
例えば、以下のような書き方に慣れてしまっている方が多いように思われます。
この書き方は一見綺麗に見えるのですが、実は意外と読みにくく、修正もしにくいです。
コメントについて
プログラムの翻訳コメントは要らない
プログラムをそのまま日本語に訳したようなコメントは無意味です。
処理の内容をそのままコメントに書いても、後から読むときに何の助けにもなりません。
例えば、このサンプルのコメントは一見丁寧に書かれているように見えますが、よく見ると、プログラムを読めばすぐ分かることしか書かれていません。
ロジックについて
戻り値を返すのは一番最後？
「メソッドのreturn文は一番最後に一個だけ」としなければいけない理由はありません。
戻り値はもっと柔軟に、必要に応じて必要な箇所でその都度返すべきです。
例えばこういう書き方をよく見かけますが、一工夫するだけでかなりシンプルになります。