code smell リファクタリング[long method編] - kinのリファクタリング日記

記念すべき１つ目は、long methodのリファクタリングを解説をしていきます。 long methodとは なぜcode smellなのか？ long methodになる原因 ソリューション 単純に同じクラス内で、privateメソッドに抽出する。 他のクラスにメソッドを持って行く。 複数の処理をしている場合に、それぞれの処理をメソッドに切り分けてあげる。 まとめ 参考 long methodとは 名前の通り、メソッドに含まれるコードの量が多すぎるということです。 なぜcode smellなのか？ そもそもなぜメソッド内のコードの量が多すぎることが問題なのでしょうか？ 2つの点が考えれると思います。 単純に読みにくいので処理を追いにくい 例えば、同じメソッドを100行ほどのコードが書かれていると、三行目くらいで読むのを諦めてしまいます。 ユニットテスト（UT）がしにくいので、バグの温床

[yuuAn]

僕もこの手順でリファクタリングしてる。メソッドに切り出すときに名前を動詞で始めることに拘らず、主語を書くようにすると、どんなクラスに切り出せばいいかがわかりやすい。