リファクタリング入門

はじめに

以下の書籍を読んでリファクタリングを学習します。

外部から見たときのふるまいを保ちつつ、理解や修正が簡単になるようにソフトウェアの内部構造を変化させること

小さな単位で行うこと
- 失敗する(バグを埋め込む)可能性を軽減できる
新規追加の作業とリファクタリングの作業を同じ作業単位(コミット)で行ってはならない。必ず分けること
- バグ発生時、バグ発生個所の特定が難しい (新規追加部分が原因か、リファクタリング部分が原因か・・・)
- コード変更差分がとても見づらくなる (コード差分が新規追加の差分なのか、リファクタリングの差分なのかすぐに判別不可能)

1つのクラスがあまりにも多くの仕事をしている場合は、たいていインスタンス変数を持ちすぎています。
インスタンス変数が多すぎると、重複コードが存在する可能性が高くなります

他のオブジェクトの属性を調べるswitch文処理を書くことは常に間違いである。

switch文処理は、他のオブジェクトについてではなく、自分自身のオブジェクトの情報について行うべきである。

数個のデータがグループとなって、クラスのフィールドやメソッドのシグニチャなど、さまざまな箇所に現れることがあります。

こうした群れをなしたデータはオブジェクトとして、1つにまとめるべき。

属性や操作のパラメータの数を絞り込むことはコードの不吉なにおいをなくすのに役立ちます。

処理の際に、他のオブジェクトのgetメソッドを何度も何度も呼び出している間違ったメソッドがよく見受けられる

処理、および処理に必要なデータを1つにまとめる

本の格言は素晴らしいと思います。