回答編集履歴
2
リファクタリングを適用するための基準を追記
test
CHANGED
|
@@ -29,3 +29,31 @@
|
|
|
29
29
|
この本を読むとリファクタリングだけでなく、
|
|
30
30
|
|
|
31
31
|
オブジェクト指向プログラミングへの理解が大変深まるのでおすすめです。
|
|
32
|
+
|
|
33
|
+
|
|
34
|
+
|
|
35
|
+
2016/02/20 追記
|
|
36
|
+
|
|
37
|
+
この質問への回答の範囲を超えるのですが、ほかの人がポリシーについて答えられているので追記いたします。
|
|
38
|
+
|
|
39
|
+
|
|
40
|
+
|
|
41
|
+
リファクタリングをすべきか、すべきならどの程度やるべきかは
|
|
42
|
+
|
|
43
|
+
対象のコードの性質によります。
|
|
44
|
+
|
|
45
|
+
|
|
46
|
+
|
|
47
|
+
もし機能追加や改修、不具合修正が頻繁にあり、かつ長期に渡って使われるプロダクトやサービスならば
|
|
48
|
+
|
|
49
|
+
少しずつでもリファクタリングをしていくといいと思います。
|
|
50
|
+
|
|
51
|
+
|
|
52
|
+
|
|
53
|
+
それほど変更がないものなら、リファクタリングのコストがメリットを上回る可能性があります。
|
|
54
|
+
|
|
55
|
+
|
|
56
|
+
|
|
57
|
+
あと正直、業界や会社にもよりますね。
|
|
58
|
+
|
|
59
|
+
先輩社員にリファクタリングをやっているか、まず訊いてみるといいでしょう。
|
1
分かりにくい表現の改善
test
CHANGED
|
@@ -6,13 +6,13 @@
|
|
|
6
6
|
|
|
7
7
|
|
|
8
8
|
|
|
9
|
-
ですので、まず最初にクラスかメソッドごとにJunitなどで単体テスト（できれば自動）を
|
|
9
|
+
ですので、まず最初にクラスかメソッドごとにJunitなどで単体テストのコード（できれば自動）を書いてください。
|
|
10
10
|
|
|
11
11
|
こちらがリファクタリングがうまくいったか（もとのまま動くか）の指標になります。
|
|
12
12
|
|
|
13
13
|
|
|
14
14
|
|
|
15
|
-
次に大きなメソッドを小さなメソッドに分けて
|
|
15
|
+
次に大きなメソッドを小さなメソッドに分けてそのたびテストを実行してください。
|
|
16
16
|
|
|
17
17
|
|
|
18
18
|
|