回答編集履歴
5
修正
answer
CHANGED
|
@@ -54,12 +54,12 @@
|
|
|
54
54
|
//getNumDifference
|
|
55
55
|
//パラメタ２-パラメタ１の差を返却する
|
|
56
56
|
//パラメタのどちらかがnullの場合は0を返却する
|
|
57
|
-
public int getNumDifference(Integer
|
|
57
|
+
public int getNumDifference(Integer subtrahend,Integer minuend){
|
|
58
58
|
int difference;
|
|
59
|
-
if(
|
|
59
|
+
if(subtrahend== null || minuend== null){
|
|
60
60
|
return 0;
|
|
61
61
|
}
|
|
62
|
-
difference =
|
|
62
|
+
difference = minuend - subtrahend;
|
|
63
63
|
return difference;
|
|
64
64
|
}
|
|
65
65
|
```
|
4
修正
answer
CHANGED
|
@@ -47,7 +47,7 @@
|
|
|
47
47
|
}
|
|
48
48
|
```
|
|
49
49
|
|
|
50
|
-
あとは、これは投稿用に適当につけた名前だと
|
|
50
|
+
あとは、これは投稿用に適当につけた名前だと思いますので本質的な指摘ではないのですが、可読性の観点からはもう少し変数名や関数名ににこだわると良いのではないでしょうか。
|
|
51
51
|
|
|
52
52
|
```Java
|
|
53
53
|
|
3
修正
answer
CHANGED
|
@@ -47,4 +47,19 @@
|
|
|
47
47
|
}
|
|
48
48
|
```
|
|
49
49
|
|
|
50
|
+
あとは、これは投稿用に適当につけた名前だということだと思うのですが、可読性の観点からはもう少し変数名や関数名ににこだわると良いのではないでしょうか。
|
|
51
|
+
|
|
52
|
+
```Java
|
|
53
|
+
|
|
54
|
+
//getNumDifference
|
|
55
|
+
//パラメタ２-パラメタ１の差を返却する
|
|
56
|
+
//パラメタのどちらかがnullの場合は0を返却する
|
|
57
|
+
public int getNumDifference(Integer minuend,Integer subtrahend){
|
|
58
|
+
int difference;
|
|
59
|
+
if(minuend == null || subtrahend == null){
|
|
60
|
+
return 0;
|
|
61
|
+
}
|
|
62
|
+
difference = subtrahend - minuend;
|
|
50
|
-
|
|
63
|
+
return difference;
|
|
64
|
+
}
|
|
65
|
+
```
|
2
修正
answer
CHANGED
|
@@ -1,4 +1,4 @@
|
|
|
1
|
-
koko_uさんが既にご指摘されていますが、
|
|
1
|
+
koko_uさんが既にご指摘されていますが、パラメタのnull判定をしてどちらかがnullだった場合の
|
|
2
2
|
```Java
|
|
3
3
|
return num3;
|
|
4
4
|
```
|
1
修正
answer
CHANGED
|
@@ -45,4 +45,6 @@
|
|
|
45
45
|
num3 = num2-num1;
|
|
46
46
|
return num3 ;
|
|
47
47
|
}
|
|
48
|
-
```
|
|
48
|
+
```
|
|
49
|
+
|
|
50
|
+
そのほかは特に普通かなと思います。
|