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

回答編集履歴

1

問題点を具体的に指摘

2018/09/20 04:27

投稿

dice142
dice142

スコア5158

answer CHANGED
@@ -14,4 +14,82 @@
14
14
  3と4のパターンもCPUが勝った(プレイヤーが負けた)として考えれば同様です。
15
15
 
16
16
  つまり、前回の勝者(lastWinner)を更新している箇所が間違っています。
17
- 前回の勝者の更新は、その値を使い終わった後にするべきです。
17
+ 前回の勝者の更新は、その値を使い終わった後にするべきです。
18
+
19
+ ---
20
+
21
+ 【追記】
22
+ 必要部分を抜粋したコードを追いながら説明します。
23
+ (提示されたコードの空行を消すなど若干手を加えましたが、処理自体は変えてません)
24
+
25
+ ```java
26
+ /* 前提条件として「前回はCPUが勝った」として追います。 */
27
+
28
+ // 勝ち負け判定
29
+ result = janken.judge(); /* 今回はプレイヤーが勝ったとします。 */
30
+
31
+ if(result == S結果.あいこ) {
32
+ System.out.println("あいこでしょ!");
33
+ } else if (result == S結果.勝ち) { /* ここの分岐に来ます */
34
+ System.out.println("playerの勝ち");
35
+ playerPt ++; /* プレイヤーに1点加算されます。 */
36
+ } else if (result == S結果.負け) {
37
+ System.out.println("cpuの勝ち");
38
+ cpuPt ++;
39
+ } else {
40
+ break;
41
+ }
42
+
43
+ /* この分岐に入ります。 */
44
+ if(result == S結果.勝ち) {
45
+ /* 前回の勝者をプレイヤーに更新します。 */
46
+ /* ※この時点で前回の勝者はCPUだったはずなのに、
47
+  プレイヤーに更新されてしまいます。 */
48
+ lastWinner = 1;
49
+ }
50
+
51
+ /* 直前の更新でlastWinnerが1になってしまい、今回はプレイヤーが勝ったので
52
+ この分岐に入ってしまいます。 */
53
+ if(lastWinner == 1 && result == S結果.勝ち) {
54
+ playerPt--;
55
+ lastWinner = 0;
56
+ }
57
+ ```
58
+
59
+ コード内にコメントで書きましたが改めて。
60
+ ** 前回はCPUが勝ち、今回はプレイヤーが勝ったとして話を進めます。 **
61
+
62
+ まず、プレイヤーが勝ったのでplayerPtが1増えます。
63
+ ```java
64
+ } else if (result == S結果.勝ち) {
65
+ System.out.println("playerの勝ち");
66
+ playerPt ++;
67
+ }
68
+ ```
69
+
70
+ 【問題点】次に、プレイヤーが勝ったのでlastWinnerを1(プレイヤー)にします。
71
+ ** 前回の勝者はCPUだったのに、ここで前回の勝者はプレイヤーと書き換えられます。 **
72
+ ```java
73
+ if(result == S結果.勝ち) {
74
+ lastWinner = 1;
75
+ }
76
+ ```
77
+
78
+ その結果、連勝と判断されて、本来は入らない連勝の分岐に入り、
79
+ playerPtが1減ります。
80
+ ```java
81
+ if(lastWinner == 1 && result == S結果.勝ち) {
82
+ playerPt--;
83
+ lastWinner = 0;
84
+ }
85
+ ```
86
+
87
+ このような流れで、連勝していないのに連勝と判断され、ポイントが+1→-1で0になります。
88
+
89
+ ---
90
+
91
+ プログラムは書いたとおりに実行されます。
92
+ お書きになったプログラムを実行して結果を見る他に、
93
+ 実際にコードを見ながら「今ここが実行され、次はここに来て…」と指で指して、
94
+ 混乱しないように変数の値を紙か何かにメモしながら値の更新を確認し、
95
+ プログラムの流れを追うことができるようにしていきましょう。