リファクタリングとコードの臭い – きれいなコードへの旅より
コードの臭いはいくつかのカテゴリに分類できます。(中略)長いメソッドやたくさんのパラメータなどがこれに該当します。
コードの臭いは、視覚的なサイン、あるいは、物理的なサイン、あるいは、抽象的なサインでわかります。視覚的なサインとは、長いメソッド、長いパラメータリスト、同じパターンで何度も現れる変数などは責務が混ざっており、抽象が間違っている兆候です。
コードベース全体に価値の乏しいリファクタリングをして"完璧なコード"を目指すのは金めっきと呼ばれます。これはコードをきれいにするのに有害です。リファクタリングのためのリファクタリングによって本当のリファクタリングのニーズを希薄にしてしまうのです。コードが問題なく動いており、変更する必要がないなら触らないほうがいいでしょう。