動くコードと、読みやすいコードは別物です。リファクタリングは、振る舞いを変えずにコードの構造を改善する大切な技法です。

命名で半分が決まる

変数・関数・クラス名は、何を表しているかを過不足なく伝える必要があります。data、info、tmp のような無意味な名前は避けましょう。

関数は1つの責務に

1つの関数で複数のことをやらせると、テストもしづらく再利用もしにくくなります。30行を超えたら分割を検討するサインです。

マジックナンバーを排除

コード中に裸の数字や文字列リテラルがある場合は、定数として名前を付けましょう。意図が読み手に伝わります。

テストとセットで進める

リファクタリングは振る舞いを変えないことが大前提。先にテストを書いてから改善に取り掛かるのが安全です。

まとめ

リファクタリングは大手術ではなく、日々のちょっとした改善の積み重ねです。Boy Scout Rule(来たときよりも綺麗にして帰る)を意識してみてください。