if (hoge.equals("foo")) {
もう、このように書くのはやめてもらいたい。
hogeの内容がNULLである場合にNullPointerExceptionが発生します。
if ("foo".equals(hoge)) {
このように書いてもらいたい。
この場合、"foo"がNULLであることはあり得ず、hogeがNULLであっても存在すればelse側の処理が実行されるだけの話である。
DBアクセスをするプログラムでDBConnectionの解放はfinallyで行いましょう。DBアクセス時などに例外が発生した場合、finallyにないとDBConnectionの解放が行われない可能性があります。
なぜか、オラクルのJDBCドライバとかガベージコレクションしてくれないからリソースがいつまでも残ることになり動作不良の原因になります。