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

回答編集履歴

4

追記

2019/11/10 14:56

投稿

gentaro
gentaro

スコア8947

answer CHANGED
@@ -15,4 +15,82 @@
15
15
  訂正
16
16
  `}`をif文の後ろに持っていくと数が合いそう。
17
17
 
18
- 伝わりにくいかもしれんけど、とりあえず`{`と`}`の対応関係をチェックしてください。
18
+ 伝わりにくいかもしれんけど、とりあえず`{`と`}`の対応関係をチェックしてください。
19
+
20
+ ---
21
+ 追記
22
+
23
+ > 先程、Windows環境にファイルを移してVS codeで}の数を再確認しましたが、特に数が多すぎるといったことはありませんでした。
24
+ 結構ゴチャゴチャした感じのコードなのでわかりづらいですが…
25
+
26
+
27
+ そもそもきちんとインデントを付けられていないのでわからないんだろうと思います。
28
+ 整形してくれるIDEを利用するか、以下のようなサイトにソースコードを貼り付けて確認してみてください。
29
+
30
+ [https://www.tutorialspoint.com/online_java_formatter.htm](https://www.tutorialspoint.com/online_java_formatter.htm)
31
+
32
+ 質問文のコードを貼り付けて整形すると以下のようになります。
33
+
34
+ ```Java
35
+ public class BattleShip {
36
+ //潜水艦の座標を乱数で0-9の範囲で生成
37
+ int sx = (int)(Math.random() * 10);
38
+ int sy = (int)(Math.random() * 10);
39
+
40
+ public static void main(String[] args) throws IOException {
41
+ BufferedReader buf = new BufferedReader(new InputStreamReader(System.in)); //標準入力の準備
42
+ int count = 1; //砲撃回数カウント用の変数を作成
43
+
44
+ while (count <= 10) { //攻撃回数が10回を越えていない場合は繰り返す
45
+ //数値の入力
46
+ System.out.println("砲撃のx座標を入力してください。(0-9)"); //ユーザに入力を促す
47
+ int hx = Integer.parseInt(buf.readLine()); //標準入力からx座標を入力
48
+ while (!(0 <= hx <= 9)) { //0-9以外の値が入力された場合、正しい数が入力されるまで警告を表示して繰り返す
49
+ System.out.println("!エラー:0から9までの整数を入力してください!"); //ユーザに入力を促す
50
+ System.out.println("砲撃のx座標を入力してください。(0-9)"); //ユーザに入力を促す
51
+ int hx = Integer.parseInt(buf.readLine()); //標準入力からx座標を入力
52
+ }
53
+
54
+ System.out.println("砲撃のy座標を入力してください。(0-9)"); //ユーザに入力を促す
55
+ int hy = Integer.parseInt(buf.readLine()); //標準入力からy座標を入力
56
+ while (!(0 <= hy <= 9)) { //0-9以外の値が入力された場合、正しい数が入力されるまで警告を表示して繰り返す
57
+ System.out.println("!エラー:0から9までの整数を入力してください!"); //ユーザに入力を促す
58
+ System.out.println("砲撃のy座標を入力してください。(0-9)"); //ユーザに入力を促す
59
+ int hy = Integer.parseInt(buf.readLine()); //標準入力からy座標を入力
60
+ }
61
+ //数値入力終了、ゲーム開始
62
+ System.out.println("砲撃回数" + count); //砲撃回数の表示
63
+
64
+ if (sx == hx && sy == hy) { //命中判定
65
+ //(※砲撃が命中した場合の処理)
66
+ System.out.println("砲撃(" + hx + "," + hy + ")は命中しました!"); //命中座標の通知
67
+ System.out.println("相手の潜水艦を撃沈しました。 【YOU WIN!】"); //勝利メッセージ
68
+ System.out.println(); //表示を整えるために改行
69
+ count = 0; //ループ脱出のためにカウント変数を0に設定
70
+ } else { //(※砲撃が外れた場合の処理)
71
+ System.out.println("砲撃(" + hx + "," + hy + ")は外れました..."); //命中座標の通知
72
+ //ここからマンハッタン距離の算出
73
+ int xm = sx - hx; //潜水艦x座標と着弾x座標の誤差の算出
74
+ int ym = sy - hy; //潜水艦y座標と着弾y座標の誤差の算出
75
+ if (xm < 0) { //潜水艦x座標と着弾x座標の誤差が負の値になった場合の修正処理
76
+ xm = -xm;
77
+ }
78
+ if (ym < 0) { //潜水艦y座標と着弾y座標の誤差が負の値になった場合の修正処理
79
+ ym = -ym;
80
+ }
81
+ int man = xm + ym; //x座標、y座標それぞれの誤差の合計の算出
82
+ //マンハッタン距離の算出終了
83
+ System.out.println("相手潜水艦との誤差は" + man); //砲撃
84
+ count++; //カウント演算子を1増加
85
+ System.out.println(); //表示を整えるために改行
86
+ }
87
+ }
88
+ }
89
+
90
+ if (count > 10) { //カウント関数が10回を越えた(=10回以内に潜水艦を撃沈できずに失敗した)場合の処理
91
+ System.out.println("相手の潜水艦の撃沈に失敗しました。【GAME OVER!】"); //敗北メッセージ
92
+ }
93
+ }
94
+ ```
95
+
96
+ 最後のif文がmainメソッドの外側に書かれているのすぐわかるはずです。

3

typo修正

2019/11/10 14:56

投稿

gentaro
gentaro

スコア8947

answer CHANGED
@@ -13,6 +13,6 @@
13
13
  ~~`}`多いだけなんじゃ?~~
14
14
 
15
15
  訂正
16
- `}`をifの後ろに持っていくと数が合いそう。
16
+ `}`をifの後ろに持っていくと数が合いそう。
17
17
 
18
18
  伝わりにくいかもしれんけど、とりあえず`{`と`}`の対応関係をチェックしてください。

2

追記

2019/11/10 14:20

投稿

gentaro
gentaro

スコア8947

answer CHANGED
@@ -13,4 +13,6 @@
13
13
  ~~`}`多いだけなんじゃ?~~
14
14
 
15
15
  訂正
16
- `}`をif分の後ろに持っていくと数が合いそう。
16
+ `}`をif分の後ろに持っていくと数が合いそう。
17
+
18
+ 伝わりにくいかもしれんけど、とりあえず`{`と`}`の対応関係をチェックしてください。

1

訂正

2019/11/10 14:18

投稿

gentaro
gentaro

スコア8947

answer CHANGED
@@ -10,4 +10,7 @@
10
10
  }
11
11
  ```
12
12
 
13
- `}`多いだけなんじゃ?
13
+ ~~`}`多いだけなんじゃ?~~
14
+
15
+ 訂正
16
+ `}`をif分の後ろに持っていくと数が合いそう。