回答編集履歴

2

冗長さを回避

2018/03/07 05:50

投稿

LouiS0616
LouiS0616

スコア35660

test CHANGED
@@ -9,32 +9,6 @@
9
9
  ---
10
10
 
11
11
  healメソッドを書き換えてしまえば良いのではないでしょうか。
12
-
13
- ```Java
14
-
15
- void heal(Hero h) {
16
-
17
- if(h.hp == 0) {
18
-
19
- System.out.println(h.name + "はもう死んでいる");
20
-
21
- return;
22
-
23
- }
24
-
25
-
26
-
27
- h.hp+=100;
28
-
29
- System.out.println(h.name+"は100回復した");
30
-
31
- }
32
-
33
- ```
34
-
35
-
36
-
37
- そう言えば、このメソッドは変ですね。引数はいらないんじゃないでしょうか。
38
12
 
39
13
  ```Java
40
14
 
@@ -60,4 +34,6 @@
60
34
 
61
35
 
62
36
 
37
+ ついでに引数も不要だと思ったので取り払いました。
38
+
63
- こうすればnameやhpをprivateフィールドに出来ます
39
+ nameやhpをprivateフィールドに出来るというメリットもあります。

1

追記

2018/03/07 05:50

投稿

LouiS0616
LouiS0616

スコア35660

test CHANGED
@@ -31,3 +31,33 @@
31
31
  }
32
32
 
33
33
  ```
34
+
35
+
36
+
37
+ そう言えば、このメソッドは変ですね。引数はいらないんじゃないでしょうか。
38
+
39
+ ```Java
40
+
41
+ void heal() {
42
+
43
+ if(this.hp == 0) {
44
+
45
+ System.out.println(this.name + "はもう死んでいる");
46
+
47
+ return;
48
+
49
+ }
50
+
51
+
52
+
53
+ this.hp+=100;
54
+
55
+ System.out.println(this.name+"は100回復した");
56
+
57
+ }
58
+
59
+ ```
60
+
61
+
62
+
63
+ こうすればnameやhpをprivateフィールドに出来ますね。