書いたマルバツゲームの判定方法の間違いが分かりません。

Question

### 前提 NxNマスのマルバツゲームを作っています。プレイヤー同士で対戦します。実感としては90％ぐらいで来たと思います。 ### 実現したいこと詰まっている点は2つです。 1. 判定方法のところのコードが間違っているのですが、それがなぜなのか全く分かりません。 2. 勝負がついた時に、どのようにしてループを抜け出すのか（一応自分が終わって欲しいと思うところにbreakとは書いているのですが、下のパターン1、2の様に判定方法が間違っているせいなのかループから抜け出してくれません。） ### 発生している問題・エラーメッセージ ``` パターン1 OX_ XXO OXO x座標を0〜2から入力してください： //この様になっても終わらない。続いてしまう。さらに続けると OXO XXO OXO 引き分けです //となってしまう。＿＿＿＿＿＿＿＿＿＿＿＿＿＿＿＿＿＿＿＿パターン2.0 OXO XO_ OX_ //この様になっても終わらないパターン2.1 OXO XOX OXO Oの勝ちです！ //パターン2.0をこの状態まで続けるとなぜか終わる。 ``` ### 該当のソースコード ```C言語 #include int main() { int n; int while_count = 0;//全体をカウントする。これの偶奇でプレイヤー1、2を判断する。 printf("マスの数を入力してください："); scanf("%d", &n); //ここからボードを作ってく int x = n, y = n; char board[y][x]; int i, j; for (i = 0; i < n ; i++){ printf(" "); for(j = 0; j < n ; j++){ printf("_ "); board[i][j] = '_'; } } printf(" "); /* whileの中でやること 1.◯を入力する(◯スタート。偶数回目であれば◯、奇数回目であればXを入力。1回目からスタート？それとも0回目？調べる) 2.◯の場所が（A）範囲内で（B）空白なのかどうか確認 3.◯を記入して、ボードを表示 4.勝敗を判断 5.ターンを進める 6.1に戻る */ while(1){ //1.入力する＋2.入力チェック do{ printf("x座標を0〜%dから入力してください：", n - 1); scanf("%d", &x); printf("y座標を0〜%dから入力してください：", n - 1); scanf("%d", &y); if(x < 0 || x > n - 1 || y < 0 || y > n - 1){ printf("入力された値が不正です。 "); }else if(board[y][x] != '_'){ printf("そのマスは埋まってます。 "); } }while((x < 0 || x > n - 1 || y < 0 || y > n - 1) || (board[y][x] != '_')); //3.◯xを記入して、 if(while_count % 2 == 0){ board[y][x] = 'O'; }else{ board[y][x] = 'X'; } //ボードを表示 for (i = 0; i < n ; i++){ printf(" "); for(j = 0; j < n ; j++){ printf("%c", board[i][j]); } } printf(" "); //4勝敗の判断。縦横斜めが同じかどうかをループで判断引き分け（全マス埋まるも考える）。 char kigou; //if文を使って偶数回目であればkigou='O',奇数回目であればkigou='X'を代入 if(while_count % 2 == 0){ kigou = 'O'; }else{ kigou = 'X'; } int count1 = 0; for(y = 0; y < n; y++){//横方向の行をn行分チェック。countで数えてその回数とnが同じであれば勝ちになるはず。 count1 = 0; for(x = 0; x < n; x++){ if(board[y][x] == kigou){ printf("count1 %d ", count1); count1++; } } if(count1 == n){ break; } } if(count1 == n){ printf("%cの勝ちです！ ", kigou); break; } int count2 = 0; for(x = 0; x < n; x++){//縦方向の行をn列分チェック。countで数えてその回数とnが同じであれば勝ちになるはず。 count2 = 0; for(y = 0; y < n; y++){ if(board[y][x] == kigou){ printf("count2 %d ", count2); count2++; } } if(count2 == n){ break; } } if(count2 == n){ printf("%cの勝ちです！ ", kigou); break; } int count3 = 0; int a; for(a = 0; a < n; a++){//左上から右下に向かってチェックする。countで数えてその回数とnが同じであれば勝ちになるはず。 if(board[a][a] == kigou){ printf("count3 %d ", count3); count3++; } } if(count3 == n){ printf("%cの勝ちです！ ", kigou); break; } int count4 = 0; for (a = 0; a < n; a++){//右上から左下に向かってチェックする。countで数えてその回数とnが同じであれば勝ちになるはず。 if(board[n - 1 - a][a] == kigou){ printf("count4 %d ", count4); count4++; } } if(count4 == n){ printf("%cの勝ちです！ ", kigou); break; } if(while_count == n*n - 1){// printf("引き分けです"); break; } //5.ターンを進める printf(" %d周目終わり ", while_count); while_count++; printf(" %d周目始まる ", while_count); } printf(" "); return 0; } ``` ### 試したこと判定方法に問題があると考えてので、76行目から125行目を変えたりして試しました。ただ自分が判定方法の何が問題なのか理解できていなかったため、何も分からなかったのが本音です。判定方法の考え方としては、例えば76行目から86行目が横方向への判定方法なのですが、y＝0（0行目）の時、二つ目のforでxを0からn-1(3x3のゲームであれば、nは2)まで動かし、if文の中でkigouが何度boardと同じなのかを数え、countの数とnが同じになれば”勝ち”となる様に考えました。しかし実際には勝っていても終わらなかったり、引き分けと判定してしまうのです。 ### 補足情報（FW/ツールのバージョンなど） Macbook Air M1 2020 Ver 12.6 Xcode　を使っています。ここにより詳細な情報を記載してください。

Accepted Answer

例えば以下の判定部分を見てみましょう。最初の for には「横方向の行をn行分チェック」と書かれています。本当に n 行分チェックが実行されているでしょうか? count の変化と if による判定は妥当でしょうか? プログラムが想定通りに動かない時、一番面倒なのが作った本人の思い込みです。考えた通りに動くように書いたはず・つもりですので、大量の「わ」の中に1つだけ「ね」があっても分からないみたいに(まぁ"ね"があると分かっているだけで難易度は相当下がりますが)、細かい所の違いが見えてきません。一旦頭をリセットして、他人の目・頭で知らないコードを解析するように少しずつ動作を調べていくことをやってみるのも必要になりまず。 ```c int count = 0; for(y = 0; y < n; y++){//横方向の行をn行分チェック。countで数えてその回数とnが同じであれば勝ちになるはず。 for(x = 0; x < n; x++){ if(board[y][x] == kigou){//あとでkigouに変える count++; } } if(count == n){ printf("%cの勝ちです！ ", kigou); } break; } ``` ついでみたいになりますが ```c count = 0; for (a = 0; a < n; a++){//右上から左下に向かってチェックする。countで数えてその回数とnが同じであれば勝ちになるはず。 if(board[n - 1 - a][n - 1 - a] == kigou); } ``` 右上から左下になっていますか? 数えてますか? --- 大体整ったようですので、さらに纏めてみた感じのを提示させて頂きます。共通として表示部分だけを関数化していますが、他の部分は質問のコードに似せる感じにしています。 (Windows/Cygwin/Eclipse で確認しています) ```c #include //2箇所で表示するため、共通処理として関数化 void print_board(int n, char board[n][n]) { for(int y = 0; y < n; y ++) { for(int x = 0; x < n; x++) { printf("%c", board[y][x]); } printf(" "); } printf(" "); } int main() { int n; printf("マスの数を入力してください："); scanf("%d", &n); //ボードを作る char board[n][n]; for(int i = 0; i < n; i++) { for(int j = 0; j < n; j++) { board[i][j] = '_'; } } print_board(n, board); /* ~~while~~forの中でやること 1.◯を入力する(◯スタート。偶数回目であれば◯、奇数回目であればXを入力。1回目からスタート？それとも0回目？調べる) 2.◯の場所が（A）範囲内で（B）空白なのかどうか確認 3.◯を記入して、ボードを表示 4.勝敗を判断 5.ターンを進める 6.1に戻る ← ループの中でやることでは無いのでこれは不要, 逆にループをいつ終わるのかを書いておく必要がある */ char player = ' '; //元 kigou int match_over = 0; //決着が着く(どちらかが勝つ)と 1 for(int turn = 0; turn < n * n && match_over == 0; turn++) { //turn は元 while_count. ターンが過ぎるか決着が着いたら終わる. [5.ターンを進める](turn++) を含む printf(" %d周目始まる ", turn); //このターンのプレイヤー if(turn % 2 == 0) { player = 'O'; } else { player = 'X'; } //[1.入力する]＋[2.入力チェック] int x, y; while(1) { printf("x座標を0〜%dから入力してください：", n - 1); scanf("%d", &x); printf("y座標を0〜%dから入力してください：", n - 1); scanf("%d", &y); if(x < 0 || x > n - 1 || y < 0 || y > n - 1) { printf("入力された値が不正です。 "); } else if(board[y][x] != '_') { printf("そのマスは埋まってます。 "); } else { printf(" "); break; //エラーでは無い＝正常な入力なら入力ループを抜ける } } //[3.◯xを記入して、ボードを表示] board[y][x] = player; print_board(n, board); //[4.勝敗の判断] 縦横斜めが同じかどうかをループで判断。各カウントを一度に行う。 int count[n + n + 2]; //[0～n-1]:水平, [n～n+n-1]:垂直, [n+n～n+n+1]:斜め //クリア for(int i = 0; i < n + n + 2; i++) { count[i] = 0; } //カウント for(int y = 0; y < n; y++) { for(int x = 0; x < n; x++) { if(board[y][x] != player) { continue; //条件に合わない場合に早々にスキップすることで、コードのインデントを深くしない(下の方で'}'が並ぶと分かり難くなる) } count[y]++; //水平 count[n + x]++; //垂直 if(x == y) { //斜め(＼) count[n + n]++; } if(n - 1 - x == y) { //斜め(／) count[n + n + 1]++; } } } //チェック for(int i = 0; i < n + n + 2; i++) { if(count[i] == n) { match_over = 1; //決着が着いた. ここで turn のループを break しなくても、 turn のループ条件に引っ掛かって終わってくれる. break; } } printf(" %d周目終わり ", turn); } if(match_over == 1) { printf("%cの勝ちです！ ", player); } else { printf("引き分けです "); } return 0; } ```

前提

実現したいこと

発生している問題・エラーメッセージ

該当のソースコード

試したこと

補足情報（FW/ツールのバージョンなど）

関連した質問