teratail header banner
teratail header banner
質問するログイン新規登録

回答編集履歴

3

追加

2021/12/06 18:13

投稿

jimbe
jimbe

スコア13355

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

2

修正

2021/12/06 18:13

投稿

jimbe
jimbe

スコア13355

answer CHANGED
@@ -4,5 +4,5 @@
4
4
 
5
5
  とメッセージに書いてある通りでしょう。
6
6
 
7
- ご質問と関係無いですが関係ありませんが、case 内の行数が多過ぎで読みにくいです。
7
+ ご質問と関係ありませんが、case 内の行数が多過ぎで読みにくいです。
8
8
  それだけの行数であれば、 case 毎にメソッドにした方が良いと思います。

1

追記

2021/12/06 17:52

投稿

jimbe
jimbe

スコア13355

answer CHANGED
@@ -2,4 +2,7 @@
2
2
 
3
3
  "budgetTrackerDto" is null
4
4
 
5
- とメッセージに書いてある通りでしょう。
5
+ とメッセージに書いてある通りでしょう。
6
+
7
+ ご質問と関係無いですが関係ありませんが、case 内の行数が多過ぎで読みにくいです。
8
+ それだけの行数であれば、 case 毎にメソッドにした方が良いと思います。