■

お世話になっております。
協力会社に実装を依頼すると、いつも恐ろしいコードを納品してくださいます。
今回のプロジェクトも発注したのですが、そのソースを読んで驚きました。(Javaです)

• Connectionをfinallyでcloseしていない。
• tryブロックにreturnがある。（上と合わせて）
• try/for/for/if/if/forのようなネストがある。
• 「更新0件は異常だが処理を継続する」というコメントがある。
• try-catchの中で別のメソッドを呼び出すが、その中にもcommit/rollbackがある。
• ifブロックでrollbackし、elseではなくブロックの直後でcommitしている。

上司の指示で発注した（らしい）のですが、誰が面倒みるんですか。
プロジェクトの現状を見る限り、手がかりの仕様書もないことでしょう。
私ですか。私は崩壊したコード体系を吸収するプログラムを書くだけで精一杯です。
以上