回答編集履歴

2

補足を追記しました

2017/01/06 04:54

投稿

grovion
grovion

スコア145

test CHANGED
@@ -161,3 +161,17 @@
161
161
  * `new Member()`を使うものは`MyException`を投げる可能性があるので、`try`節に移動した
162
162
 
163
163
 
164
+
165
+ # 補足
166
+
167
+
168
+
169
+ 「論理的に問題があると思われるif文」とは `if(id < 100 && id > 999)`のことです。
170
+
171
+
172
+
173
+ 「100よりも小さく、999より大きい数」多分ないです。
174
+
175
+ 多分したいことは「100よりも小さい、または、999より大きい数」だと思うので、
176
+
177
+ `if(id < 100 || id > 999)`です。ですが、else文で問題がないと思われるので削除しました。

1

ソースコードの変更点を明記しました

2017/01/06 04:54

投稿

grovion
grovion

スコア145

test CHANGED
@@ -1,6 +1,8 @@
1
1
  # ソースコード
2
2
 
3
3
 
4
+
5
+ 変更箇所にはコメントいれました。
4
6
 
5
7
 
6
8
 
@@ -146,6 +148,16 @@
146
148
 
147
149
 
148
150
 
151
+ # 変更点
149
152
 
150
153
 
154
+
155
+ * `MyException`クラスの作成
156
+
157
+ * `Member`のコンストラクタに`throws`を追加し`MyException`を投げることを宣言
158
+
151
- 変更箇所はコメントいました。
159
+ * 論理的問題があると思わるif文を削除
160
+
161
+ * `new Member()`を使うものは`MyException`を投げる可能性があるので、`try`節に移動した
162
+
163
+