これまで画面A でのみ使用していた部分を、画面Bでも使う事になったので、その部分を画面Aから切り離して別クラスに切り出すことにした。ついでにその部分で使用していたクラスなどの依存関係を画面Aから分離して構造もすっきりさせようかと。半ば趣味入っていますが、しかし、やるならとことんやらないと!
ここで、昔の人(誰)は大抵の場合画面Bにそのまま既存のコード等をコピペして細部をコンパイルが通る用に書き換えてだましだまし使うなんて事をやっていたようですが(知らない人が昔書いた既存のソースがそんなのばっかり)、そんなことでは後で機能修正などが入った時に修正箇所が増えるばかりですから、やはりコードの重複は完全に潰しておくべきで。コピペコードを量産して後で泣きを見るよりは、いま多少苦労してでも保守しやすい状態にすることを心がけましょう。本当に、引き継ぐほうの身にもなってほしかったよ……。身にしみて実感できてよかったと思うべき?
というか、リファクタリングをもっと浸透させるべきだろうか。こんなの自分しかやっていないし……。
