回答編集履歴
2
追記
test
CHANGED
@@ -14,7 +14,7 @@
|
|
14
14
|
|
15
15
|
|
16
16
|
|
17
|
-
3,個人的な好みとしては`cutResult`
|
17
|
+
3,個人的な好みとしては`cutResult`メソッドは作らずに`Math.min`を使って最大値の設定をします。
|
18
18
|
|
19
19
|
|
20
20
|
|
@@ -56,7 +56,7 @@
|
|
56
56
|
|
57
57
|
public static void main(String[] args){
|
58
58
|
|
59
|
-
|
59
|
+
// 以下の変数 sample1, result1 も同様に適切な名前に変更すると後で見直した時に後悔しないです。
|
60
60
|
|
61
61
|
Pot sample1 = new Pot(150);
|
62
62
|
|
1
追記
test
CHANGED
@@ -2,23 +2,23 @@
|
|
2
2
|
|
3
3
|
|
4
4
|
|
5
|
-
1,クラス名はSampleではなく、具体的なモノを表す方が後で(3ヶ月後
|
5
|
+
1,クラス名は`Sample`ではなく、具体的なモノを表す方が後で(3ヶ月後にコード読んだ時)に後悔しずらいかと。
|
6
6
|
|
7
|
-
今回の場合は給湯器なのでクラス名はSampleではなくPotクラス。
|
7
|
+
今回の場合は給湯器なのでクラス名は`Sample`ではなく`Pot`クラス。
|
8
8
|
|
9
9
|
|
10
10
|
|
11
11
|
2,メソッド名や引数も同様です。
|
12
12
|
|
13
|
-
setResultではなくsetTemperature、変数名もnumberではなくtemperature。
|
13
|
+
`setResult`ではなく`setTemperature`、変数名も`number`ではなく`temperature`。
|
14
14
|
|
15
15
|
|
16
16
|
|
17
|
-
3,個人的な好みとしてはcutResult関数は作らずにMath.minを使って最大値の設定をします。
|
17
|
+
3,個人的な好みとしては`cutResult`関数は作らずに`Math.min`を使って最大値の設定をします。
|
18
18
|
|
19
19
|
|
20
20
|
|
21
|
-
サンプルコードです、ご参考まで
|
21
|
+
サンプルコードです、ご参考まで→[Wandbox](https://wandbox.org/permlink/S0FBVEyWEVb2VPuf)
|
22
22
|
|
23
23
|
|
24
24
|
|