汚いソースコード

帰国してから、ぼちぼち職場で過去１年間に書かれたコードを読み始めたのですが、すごいことになってました。デザパタとかコーディングに関する本って、山ほどあるような気がするのですが、読んでないのでしょうか？驚きのソースコードです。Rational Roseなんか使って、クラス図やシーケンス図を書いても、こうなるものなのか...
けっこう目につくのが、似たようなコードの重複。コメントまで、御丁寧にコピペされているのだから、きっと、似たような関数作るのに、コピーしてから改造にとりかかっているんでしょう。凡化とか再利用というアイディアのかけらも見あたらない...だからフットプリントがでかくなるんだよな...
他にも、マジックナンバーの使用とかconstにすべきものがなっていないとか、初歩的なものもちらほら。あえて、何かを褒めるとしたら、さすがに職業プログラマー、エラー処理だけはしっかりやっている。組み込みだもの、セグメンテーションフォルトなんかはさすがにまずい。

とはいえ、一度ソースコードレビューをしなければなるまい。