$shibayu36->blog;スマートフォン用の表示で見る$shibayu36->blog;株式会社はてなでエンジニアをしています。プログラミングや読書のことなどについて書いています。2013-03-08修正時に一箇所リファクタリングするtech最近仕事でバグ修正をするときには、必ずその周辺のリファクタリングを一箇所行うという事をしている。今日はそのことについて書いてみる。時間が経つとコードはレガシーになるこれは自分の感覚でしかないのだが、コードというのはどんなに素晴らしい人が書いたとしても、時間が経つと必ずレガシーコードになると思っている。これには例えば以下の理由などがある。するそれでリファクタリングをしたいのだけど、ずっとリファクタリングをするのもモチベーションを維持できないし、リファクタリングをしすぎることによって機能開発やバグ修正が遅れてしまったら本末転倒なので、最近はバグ修正時に一つリファクタリングをするということを心がけている。編集機能にバグが見つかりましたそのバグ周辺のテストを書く周辺のテストが増えたので、そのバグ周辺のリファクタリングを一箇所する一箇所の定義は難しいけど、わかりづらい部分の一部分をモジュールもしくはメソッド化とかみたいな感じ。大きいリファクタリングをしないといけないことが分かった時には、その時にはせずに、タスク化してしまって別の機会にやるようにしている。バグ修正なのにリファクタリングしているせいでリリースが遅れるとか、レビューの時に見る範囲が増えるとか、色々理由があるけどそういうふうにしてる。リファクタリングは一気にやるべきみたいな意見もあると思うし、そうしたほうがよい部分もあると思うけど、全部が全部そうできるとは限らないと思うし、それであまりにもモチベーション低下や開発速度*1が下がるのではあまり意味が無い。直してくっていうのと、一気に直してくというのを合わせて使ったほうがいいと思った。まとめリファクタリングについて書いたけど、自分はリファクタリングの書籍をまだちゃんと読んだこと無いので、ほとんどが自分の感覚になってしまっている。これは良くないので、ちゃんとその辺の本も読まないとなと思った。最近はコードコンプリートとかを読んでる。この中にはリファクタリングの話も書いてあって参考になる。Series)を読んだりしたい。月別アーカイブはてなブログをはじめよう!shiba_yu36さんは、はてなブログを使っています。ブログを報告するスターをつけました引用をストックしましたストック一覧を見る閉じる引用するにはまずログインしてくださいログイン閉じる引用をストックできませんでした。再度お試しください閉じる限定公開記事のため引用できません。