回答編集履歴
4
てにをは補足
test
CHANGED
|
@@ -8,7 +8,7 @@
|
|
|
8
8
|
|
|
9
9
|
|
|
10
10
|
|
|
11
|
-
データ変換に失敗することが普段から十分ありえる（織り込み済み）ならば、例外送出はせず結果用クラスに情報を入れて呼び出し側が判断、特別な対処をユーザーに要求したいほど滅多におきないことであれば例外を送出して処理を中断させる、と私ならばします。
|
|
11
|
+
また、データ変換に失敗することが普段から十分ありえる（織り込み済み）ならば、例外送出はせず結果用クラスに情報を入れて呼び出し側が判断、特別な対処をユーザーに要求したいほど滅多におきないことであれば例外を送出して処理を中断させる、と私ならばします。
|
|
12
12
|
|
|
13
13
|
|
|
14
14
|
|
|
@@ -42,4 +42,4 @@
|
|
|
42
42
|
|
|
43
43
|
|
|
44
44
|
|
|
45
|
-
TestクラスのDoメソッドが、3つの処理を可能な限り全うさせることを責務としているのであれば、ログを書くなどは Do メソッドのなかでやってはまずいでしょう（単一責任の原則に反します）
|
|
45
|
+
なお、TestクラスのDoメソッドが、3つの処理を可能な限り全うさせることを責務としているのであれば、ログを書くなどは Do メソッドのなかでやってはまずいでしょう（単一責任の原則に反します）
|
3
コードが不足していたので修正
test
CHANGED
|
@@ -17,6 +17,10 @@
|
|
|
17
17
|
|
|
18
18
|
|
|
19
19
|
```C#
|
|
20
|
+
|
|
21
|
+
var ping = new System.Net.NetworkInformation.Ping();
|
|
22
|
+
|
|
23
|
+
var reply = ping.Send("www.example.com");
|
|
20
24
|
|
|
21
25
|
if (reply.Status == System.Net.NetworkInformation.IPStatus.Success)
|
|
22
26
|
|
2
solid原則に
test
CHANGED
|
@@ -37,3 +37,5 @@
|
|
|
37
37
|
```
|
|
38
38
|
|
|
39
39
|
|
|
40
|
+
|
|
41
|
+
TestクラスのDoメソッドが、3つの処理を可能な限り全うさせることを責務としているのであれば、ログを書くなどは Do メソッドのなかでやってはまずいでしょう（単一責任の原則に反します）
|
1
コードが反映されていなかったので再編集
test
CHANGED
|
@@ -4,11 +4,15 @@
|
|
|
4
4
|
|
|
5
5
|
[Ping.Send](https://msdn.microsoft.com/ja-jp/library/system.net.networkinformation.ping.send.aspx)メソッドの結果を[PingReply](https://msdn.microsoft.com/ja-jp/library/system.net.networkinformation.pingreply.aspx)クラスで返す実装が、標準のフレームワークには存在していますね。
|
|
6
6
|
|
|
7
|
+
すなわち、Do メソッドは、結果用のクラスを生成して、そこにどのような処理となったかステータスを入れます。
|
|
7
8
|
|
|
8
9
|
|
|
10
|
+
|
|
9
|
-
|
|
11
|
+
データ変換に失敗することが普段から十分ありえる（織り込み済み）ならば、例外送出はせず結果用クラスに情報を入れて呼び出し側が判断、特別な対処をユーザーに要求したいほど滅多におきないことであれば例外を送出して処理を中断させる、と私ならばします。
|
|
10
12
|
|
|
11
13
|
|
|
14
|
+
|
|
15
|
+
ご参考 - http://dobon.net/vb/dotnet/internet/ping.html
|
|
12
16
|
|
|
13
17
|
|
|
14
18
|
|
|
@@ -33,9 +37,3 @@
|
|
|
33
37
|
```
|
|
34
38
|
|
|
35
39
|
|
|
36
|
-
|
|
37
|
-
すなわち、Do メソッドは、結果用のクラスを生成して、そこにどのような処理となったかステータスを入れます。
|
|
38
|
-
|
|
39
|
-
|
|
40
|
-
|
|
41
|
-
データ変換に失敗することが普段から十分ありえる（織り込み済み）ならば、例外送出はせず結果用クラスに情報を入れて呼び出し側が判断、特別な対処をユーザーに要求したいほど滅多におきないことであれば例外を送出して処理を中断させる、と私ならばします。
|