回答編集履歴
5
修正
test
CHANGED
@@ -110,17 +110,17 @@
|
|
110
110
|
|
111
111
|
//パラメタのどちらかがnullの場合は0を返却する
|
112
112
|
|
113
|
-
public int getNumDifference(Integer
|
113
|
+
public int getNumDifference(Integer subtrahend,Integer minuend){
|
114
114
|
|
115
115
|
int difference;
|
116
116
|
|
117
|
-
if(
|
117
|
+
if(subtrahend== null || minuend== null){
|
118
118
|
|
119
119
|
return 0;
|
120
120
|
|
121
121
|
}
|
122
122
|
|
123
|
-
difference =
|
123
|
+
difference = minuend - subtrahend;
|
124
124
|
|
125
125
|
return difference;
|
126
126
|
|
4
修正
test
CHANGED
@@ -96,7 +96,7 @@
|
|
96
96
|
|
97
97
|
|
98
98
|
|
99
|
-
あとは、これは投稿用に適当につけた名前だとい
|
99
|
+
あとは、これは投稿用に適当につけた名前だと思いますので本質的な指摘ではないのですが、可読性の観点からはもう少し変数名や関数名ににこだわると良いのではないでしょうか。
|
100
100
|
|
101
101
|
|
102
102
|
|
3
修正
test
CHANGED
@@ -96,4 +96,36 @@
|
|
96
96
|
|
97
97
|
|
98
98
|
|
99
|
+
あとは、これは投稿用に適当につけた名前だということだと思うのですが、可読性の観点からはもう少し変数名や関数名ににこだわると良いのではないでしょうか。
|
100
|
+
|
101
|
+
|
102
|
+
|
103
|
+
```Java
|
104
|
+
|
105
|
+
|
106
|
+
|
107
|
+
//getNumDifference
|
108
|
+
|
109
|
+
//パラメタ2-パラメタ1の差を返却する
|
110
|
+
|
111
|
+
//パラメタのどちらかがnullの場合は0を返却する
|
112
|
+
|
113
|
+
public int getNumDifference(Integer minuend,Integer subtrahend){
|
114
|
+
|
115
|
+
int difference;
|
116
|
+
|
117
|
+
if(minuend == null || subtrahend == null){
|
118
|
+
|
119
|
+
return 0;
|
120
|
+
|
121
|
+
}
|
122
|
+
|
123
|
+
difference = subtrahend - minuend;
|
124
|
+
|
99
|
-
|
125
|
+
return difference;
|
126
|
+
|
127
|
+
}
|
128
|
+
|
129
|
+
```
|
130
|
+
|
131
|
+
|
2
修正
test
CHANGED
@@ -1,4 +1,4 @@
|
|
1
|
-
koko_uさんが既にご指摘されていますが、
|
1
|
+
koko_uさんが既にご指摘されていますが、パラメタのnull判定をしてどちらかがnullだった場合の
|
2
2
|
|
3
3
|
```Java
|
4
4
|
|
1
修正
test
CHANGED
@@ -93,3 +93,7 @@
|
|
93
93
|
}
|
94
94
|
|
95
95
|
```
|
96
|
+
|
97
|
+
|
98
|
+
|
99
|
+
そのほかは特に普通かなと思います。
|