回答編集履歴

3

追加

2021/12/06 18:13

投稿

jimbe
jimbe

スコア13168

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

2

修正

2021/12/06 18:13

投稿

jimbe
jimbe

スコア13168

test CHANGED
@@ -10,6 +10,6 @@
10
10
 
11
11
 
12
12
 
13
- ご質問と関係無いですが関係ありませんが、case 内の行数が多過ぎで読みにくいです。
13
+ ご質問と関係ありませんが、case 内の行数が多過ぎで読みにくいです。
14
14
 
15
15
  それだけの行数であれば、 case 毎にメソッドにした方が良いと思います。

1

追記

2021/12/06 17:52

投稿

jimbe
jimbe

スコア13168

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