回答編集履歴
3
追加
answer
CHANGED
|
@@ -5,4 +5,9 @@
|
|
|
5
5
|
とメッセージに書いてある通りでしょう。
|
|
6
6
|
|
|
7
7
|
ご質問と関係ありませんが、case 内の行数が多過ぎで読みにくいです。
|
|
8
|
-
それだけの行数であれば、 case 毎にメソッドにした方が良いと思います。
|
|
8
|
+
それだけの行数であれば、 case 毎にメソッドにした方が良いと思います。
|
|
9
|
+
|
|
10
|
+
また、 Scanner のようなストリームを扱うクラスは、同一の入力からのインスタンスを複数作成するのは危険です。内部で先読みをしていた場合にデータが消える等の可能性があります。
|
|
11
|
+
|
|
12
|
+
選択の入力値が"範囲外か"のチェックに直接数値が書いてあり、しかも一部間違っているようです。
|
|
13
|
+
折角マップに番号が入っているのですから、そちらを使うべきではないでしょうか。
|
2
修正
answer
CHANGED
|
@@ -4,5 +4,5 @@
|
|
|
4
4
|
|
|
5
5
|
とメッセージに書いてある通りでしょう。
|
|
6
6
|
|
|
7
|
-
ご質問と関係
|
|
7
|
+
ご質問と関係ありませんが、case 内の行数が多過ぎで読みにくいです。
|
|
8
8
|
それだけの行数であれば、 case 毎にメソッドにした方が良いと思います。
|
1
追記
answer
CHANGED
|
@@ -2,4 +2,7 @@
|
|
|
2
2
|
|
|
3
3
|
"budgetTrackerDto" is null
|
|
4
4
|
|
|
5
|
-
とメッセージに書いてある通りでしょう。
|
|
5
|
+
とメッセージに書いてある通りでしょう。
|
|
6
|
+
|
|
7
|
+
ご質問と関係無いですが関係ありませんが、case 内の行数が多過ぎで読みにくいです。
|
|
8
|
+
それだけの行数であれば、 case 毎にメソッドにした方が良いと思います。
|