teratail header banner
teratail header banner
質問するログイン新規登録

回答編集履歴

2

追記

2017/05/23 09:09

投稿

haru666
haru666

スコア1593

answer CHANGED
@@ -12,6 +12,6 @@
12
12
  質問がでっかすぎて小さなレスポンスが僕の脳みそだと書けません。
13
13
  こういうの熱く長文書くと間違ってたら恥ずかしいから嫌だなぁって思いますが…
14
14
  コードこみこみ100行超過して回答をレスしていいならコメントくれれば追記します。
15
- とりあえず、それが社内ルールだとどうしようもないんですが、そうじゃない場合、ユニットテストのためにコードを捻じ曲げる前に正しい方法が見つかると思います。
15
+ とりあえず、実装したメソッドのテストカバレッジ100%を目指すとか、〇〇を使うが社内ルールだとどうしようもないんですが、そうじゃない場合、ユニットテストのためにコードを捻じ曲げる前に正しい方法が見つかると思います。
16
16
 
17
17
  着目してることは正しいと思います。問題はたぶん、クラスの粒度が荒すぎるんです。

1

追記

2017/05/23 09:09

投稿

haru666
haru666

スコア1593

answer CHANGED
@@ -2,7 +2,7 @@
2
2
 
3
3
  回答1.static メソッドをわざわざ instance メソッドにすることはありません。
4
4
  あるクラスに対して固有なものであっても、staticメソッドは混在しないようにします。
5
- ファクトリメソッドを例に挙げていますが、ファクトリメソッドはファクトリクラスに纏めて外部化すべきです。
5
+ 他の方の回答へのレスでファクトリメソッドを例に挙げていますが、ファクトリメソッドはファクトリクラスに纏めて外部化すべき部分です。
6
6
  そのようにしていくと、最終的にstaticメソッドとinstanceメソッドの共存は無くなるはずです。
7
7
 
8
8
  回答2.virtualが必要なMockライブラリを使いません。