やめてほしいプログラムたち(Java)

if (hoge.equals("foo")) {

もう、このように書くのはやめてもらいたい。
hogeの内容がNULLである場合にNullPointerExceptionが発生します。

if ("foo".equals(hoge)) {

このように書いてもらいたい。
この場合、"foo"がNULLであることはあり得ず、hogeがNULLであっても存在すればelse側の処理が実行されるだけの話である。

DBアクセスをするプログラムでDBConnectionの解放はfinallyで行いましょう。DBアクセス時などに例外が発生した場合、finallyにないとDBConnectionの解放が行われない可能性があります。
なぜか、オラクルのJDBCドライバとかガベージコレクションしてくれないからリソースがいつまでも残ることになり動作不良の原因になります。