[初心者]c言語でオセロを作成中の問題

Question

前提・実現したいこと

c言語でオセロを作成中の初心者です。ひととおり全体的な枠組みは完成したのですが、うまく挙動しません。
オセロで相手のコマをひっくり返すところがうまくいってないとは思うのですが、どこを改善すればよいのかわからないので教えてほしいです。

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

put_checkという関数の部分がおかしいのだと思いますが、自分ではどこをかえたらいいのかわからないので教えてほしいです。よろしくお願いします。
所々いらないことを書いている部分もあるとは思うのですが、とりあえずひととり動くものを作りたいです。

↓実行結果　自分のコマ（このコードではｗ固定）を置く位置を入力したあと、きちんと置かれず、謎の数字が一番上に追加されます。

 12345678
1........
2........
3........
4...wb...
5...bw...
6........
7........
8........
0
あなたのターンです。
縦方向3
横方向5
 1234119678
1........
2........
3........
4...wb...
5...bw...
6........
7........
8........

該当のソースコード

#include<stdio.h>
#include<stdlib.h>
#include<time.h>
#define WHITE 1
#define BLACK 2
char board[9][9];
int turn;//自分のターン
int sentaku;
int i_put,j_put;
int bamen_sentaku=1;
//char o,a;//oは敵,aは自分
void shokika(){
    board[0][0]=' ';
    for(int a=1;a<9;a++){  
        board[0][a]=a;
        board[a][0]=a;
    }
    for(int i=1;i<9;i++){
        for(int j=1;j<9;j++){
            board[i][j]='.';
        }
   
    }
    board[4][4]=board[5][5]='w';//一旦wは自分,bが敵だとする
    board[4][5]=board[5][4]='b';
}

void display(){

    for(int i=0;i<9;i++){
        for(int j=0;j<9;j++){
            if(j==0&&i==0){
                printf("%c",board[i][j]);
            }else if(j==0||i==0){//盤面の数字の部分はint型
                printf("%d",board[i][j]);
            }else{
                printf("%c",board[i][j]);
            }
            }
        printf("\n");
    }

}
void shouhai(){
    int w_number=0;
    int b_number=0;
    int sum=0;
    for(int i=1;i<9;i++){
        for(int j=1;j<9;j++){
            if(board[i][j]=='w'){
                ++w_number;
            }else if(board[i][j]=='b'){
                ++b_number;
            }
        }
    }
    if(sum==64){
        printf("白%d枚、黒%d枚\n",w_number,b_number);
        if(w_number==b_number){
            printf("引き分けです\n");
        }else if(w_number<b_number){
            printf("あなたの勝利です\n");
        }else{
            printf("あなたの敗北です\n");
        }
    }
}

int num_kaeseru(int x,int y,int iro){//そこに打つといくつ返せるか
    if(board[y][x]!='.'){
        return -1;
    }
    char my_iro;
    char aite_iro;
    if(iro==1){//自分の色がｗなら
        my_iro ='w';
        aite_iro ='b';
    }else{
        aite_iro = 'w';
        my_iro = 'b';
    }
    int total=0;
    for(int dy=-1;dy<=1;dy++){
        for(int dx=-1;dx<=1;dx++){
            int sx,sy;
            int k=0;//めくる枚数
            sx = x;
            sy = y;
            while(1){
                sx += dx;
                sy += dy;
                if(sx<=0 || sy<=0 || sx>8 || sy>8){//盤面を超えたら
                    break;
                }
                if(board[sy][sx]=='.'){
                    break;
                }
                if(board[sy][sx]==aite_iro){
                    k += 1;
                }
                if(board[sy][sx]==my_iro){
                    total += k;
                    break;
                }
            }
        }
    }
    return total;
}

void put_check(int x,int y,int iro){//iroは自分の色
char my_iro;
char aite_iro;
if(iro==1){
    my_iro ='w';
    aite_iro ='b';
}else{
    my_iro ='b';
    aite_iro ='w';
}
    for(int dy=-1;dy<=1;dy++){
        for(int dx=-1;dx<=1;dx++){
            int sx,sy;
            int k=0;//めくる枚数
            sx = x;
            sy = y;
            while(1){
                sx += dx;
                sy += dy;
                if(sx<=0 || sy<=0 || sx>8 || sy>8){//盤面を超えたら
                    break;
                }
                if(board[sy][sx]=='.'){
                    break;
                }
                if(board[sy][sx]==aite_iro){
                    k += 1;
                }
                if(board[sy][sx]==my_iro){
                    for(int i=0;i<k;i++){//めくる作業
                        sx -= dx;
                        sy -= sy;
                        board[sy][sx]=my_iro;
                    }
                    break;
                }
            }
        }
    }
}

void put(){
    while(1){
        printf("あなたのターンです。\n");
        printf("縦方向"); scanf("%d",&j_put);
        printf("横方向"); scanf("%d",&i_put);
        if(num_kaeseru(i_put,j_put,WHITE)>0){
            break;
        }
    }
    put_check(i_put,j_put,WHITE);
}

int uteru_masu(int iro){
    for(int y=1;y<=8;y++){
        for(int x=1;x<=8;x++){
            if (num_kaeseru(x,y,iro)>0){
                return 1;//1=打てるマスあり
            }
        }
    }
    return 2;//打てるマスなし
}

void computer_1(){
    int c_x,c_y;//コンピュータが置く位置
    srand((unsigned int)time(NULL));
    while(1){
        c_x = rand()%8+1;
        c_y = rand()%8+1;
        if(num_kaeseru(c_x,c_y,BLACK)>0){
            break;
        }
    }
    put_check(c_x,c_y,2);
}
void menu(){
    while(1){
    if(bamen_sentaku==1){
    printf("/////////////////オセロ/////////////////\n");
    //printf("ゲームモードを選択してください//なお２Pゲームしか実装されていない模様\n");
    //printf("1Pゲーム　---1\n2Pゲーム　---2\n --->"); scanf("%d",&sentaku);
    shokika();
    display();
    printf("%d\n",num_kaeseru(5,1,1));
    bamen_sentaku=2;
    }else if(bamen_sentaku==2){
        put();
        display();
        bamen_sentaku=3;
    }else if(bamen_sentaku==3){
        computer_1();
        display();
        bamen_sentaku=4;
    }else if(bamen_sentaku==4){
        if(uteru_masu(WHITE)==2 && uteru_masu(BLACK)==2){
            printf("試合終了です");
            bamen_sentaku=5;
        }else if(uteru_masu(WHITE)==2 && uteru_masu(BLACK)==1){
            printf("あなたは打てないのでパスです\n");
            bamen_sentaku=3;
        }else if(uteru_masu(WHITE)==1 && uteru_masu(BLACK)==2){
            printf("コンピュータが打てるコマがないのでパス。\n");
        }else{
            bamen_sentaku=2;
        }
    }else if(bamen_sentaku==5){
        shouhai();
        break;
    }
    }
}

int main(){
    menu();
    return 0;
}

試したこと

このコードを実行したところ、コマを置く位置を指定したときに、カードがうまくいかず、おかしな挙動をします。pythonゲームアルゴリズム入門という書籍を参考しつつ、c言語で作ってみようとしているのですが、うまく行かないので、改善点を教えてほしいです。

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

ここにより詳細な情報を記載してください。

Comments

[thkana]

漠然と「うまく挙動しません」「おかしな挙動をします」というのではなく、
・こういう動作を期待して
・こういうプログラムを組んだ(この部分はありますが)ら
・期待と違う、こういうことが起こった
という「観察できる事実」を具体的に書いてください。そうでないと、回答者はあなたのプログラムを全部チェックしなければならなくなります。

[tatsu99]

computer_1();
は
computer_LV1();
の間違いでしょうか。提示されたソースをコンパイルすると、
computer_1でエラーになります。

[jimbe]

全然ご質問に関係無く「どうでもいい」と言われそうですが、変数名や関数名に英語とローマ字日本語を混在させないで頂きたいです。

[ka-kosya19212]

tatsu99様
コードを修正いたしました。

[ka-kosya19212]

jimbe様
基本的にはpythonのゲームプログラミングの本の関数の名前をそのままつかっています。
混在しないほうがいいのは、そのとおりだと思うのであとで改善したいと思います。

[ka-kosya19212]

thkana様
実行結果を載せました。

[jimbe]

> 本の関数の名前をそのままつかっています
駄文に返信ありがとうございます、了解です。
python では有りなのでしょうけれど、なんともまぁ。

Answer 1

見た感じ、ここじゃないでしょうか。

          for(int i=0;i<k;i++){//めくる作業
            sx -= dx;
            sy -= sy; //!!!!!
            board[sy][sx]=my_iro;
          }

---

弄ってみました。
置ける場所等を何度も探すのを避けるため、points 配列を用意しました。
Delta 構造体と Point 構造体が同じですが、まぁ雰囲気ということで。

#include <stdio.h>
#include <stdlib.h>
#include <memory.h>
#include <time.h>
#include <stdarg.h>

#define WHITE 'w'
#define BLACK 'b'
#define SPACE '.'

#define TRUE 1
#define FALSE 0

#define SIZE 8

#ifndef NULL
#define NULL ((void)0)
#endif

char board[SIZE][SIZE];

//初期化
void init(){
  srand((unsigned int)time(NULL));

  memset(board, SPACE, sizeof(board));
  board[SIZE/2-1][SIZE/2-1]=board[SIZE/2][SIZE/2]=WHITE;//一旦wは自分,bが敵だとする
  board[SIZE/2-1][SIZE/2]=board[SIZE/2][SIZE/2-1]=BLACK;
}

//盤面表示
void print(){
  printf(" ");
  for(int x=0; x<SIZE; x++){
    printf("%d", x+1);
  }
  printf("\n");
  for(int y=0; y<SIZE; y++){
    printf("%d", y+1);
    for(int x=0; x<SIZE; x++){
      printf("%c", board[y][x]);
    }
    printf("\n");
  }
}

//結果
void judgement(){
  int w_count = 0;
  int b_count = 0;
  for(int y=0; y<SIZE; y++){
    for(int x=0; x<SIZE; x++){
      if(board[y][x] == WHITE){
        ++w_count;
      } else if(board[y][x] == BLACK){
        ++b_count;
      }
    }
  }

  printf("白%d枚、黒%d枚\n",w_count, b_count);
  if(w_count == b_count){
    printf("引き分けです\n");
  }else if(w_count > b_count){
    printf("あなたの勝利です\n");
  }else{
    printf("あなたの敗北です\n");
  }
}

//8方向
typedef struct Delta { int x, y; } Delta;
Delta deltas[] = {
    {-1,-1}, {0,-1}, {1,-1},
    {-1, 0},         {1, 0},
    {-1, 1}, {0, 1}, {1, 1}
};

//座標保持用バッファ
typedef struct Point { int x, y; } Point;
Point points[SIZE*SIZE];

//x,y に打った場合にひっくり返せる数を返す.
//keepPoints に TRUE を指定すると, ひっくり返せる各座標を points に格納する.
int acquiredPoints(int x, int y, int color, int keepPoints){
  int total = 0;
  if(board[y][x] != SPACE) return total;

  for(int i=0; i<sizeof(deltas)/sizeof(Delta); i++) {
    Delta *d = &deltas[i];

    for(int sx=x+d->x, sy=y+d->y, count=0;
        sx>=0 && sy>=0 && sx<SIZE && sy<SIZE && board[sy][sx]!=SPACE;
        sx+=d->x, sy+=d->y, count++){
      if(board[sy][sx] == color) { //挟める
        total += count;
        break;
      }
      if(keepPoints) {
        points[total + count].x = sx;
        points[total + count].y = sy;
      }
    }
  }
  return total;
}

//x,y と points の各座標に color を設定する.
void acquired(int x, int y, int total, int color) {
  board[y][x] = color;
  for(int i=0; i<total; i++) {
    board[points[i].y][points[i].x] = color;
  }
}

//打てる各座標を points に格納しその件数を返す.
int availablePoints(int color) {
  int count = 0;
  for(int y=0; y<SIZE; y++){
    for(int x=0; x<SIZE; x++){
      if(acquiredPoints(x, y, color, FALSE) > 0) {
        points[count].x = x;
        points[count].y = y;
        count ++;
      }
    }
  }
  return count;
}

int input(int min, int max, char *fmt, ...) {
  int v;
  va_list arg_ptr;
  do {
    va_start(arg_ptr, fmt);
    vprintf(fmt, arg_ptr);
    va_end(arg_ptr);

    scanf("%d",&v);
  } while(v < min || max < v);
  return v;
}

void turnPlayer(int color){
  int count = availablePoints(color);
  if(count == 0) {
    printf("あなたは打てないのでパスです\n");
    return;
  }

  int x, y, total;
  printf("あなた('%c')のターンです。\n", color);
  while(1) {
    x = input(1, SIZE, "横方向(x:1-%d)", SIZE);
    y = input(1, SIZE, "縦方向(y:1-%d)", SIZE);
    if(x>=1 && y>=1 && x<=SIZE && y<=SIZE && (total = acquiredPoints(x-1, y-1, color, TRUE)) > 0){
      acquired(x-1, y-1, total, color);
      print();
      return;
    }
    printf("そこには打てません。\n");
  }
}

void turnComputer(int color) {
  int count = availablePoints(color);
  if(count == 0) {
    printf("コンピュータが打てるコマがないのでパス。\n");
    return;
  }

  Point point = points[rand() % count]; //コピー
  int total = acquiredPoints(point.x, point.y, color, TRUE);
  acquired(point.x, point.y, total, color);
  printf("コンピュータ('%c')は x%d,y%d に打ちました。\n", color, point.x+1, point.y+1);
  print();
}

int main(){
  printf("/////////////////オセロ/////////////////\n");
  init();
  print();

  while(1) {
    turnPlayer(WHITE);
    turnComputer(BLACK);

    if(!availablePoints(WHITE) && !availablePoints(BLACK)){
      break;
    }
  }

  printf("試合終了です\n");
  judgement();
  return 0;
}

Comments

[ka-kosya19212]

コードを書き換えたら正しく動作するようになりました。返信ありがとうございました。

[jimbe]

念の為、teratail はバグを取ってもらう所ではありませんので、ご留意ください。

Answer 2

void put_check(int x,int y,int iro){//iroは自分の色
    char my_iro;
    char aite_iro;
    
    if(iro==1){
        my_iro ='w';
        aite_iro ='b';
    }else{
        my_iro ='b';
        aite_iro ='w';
    }
    
    board[y][x]=my_iro; //----追加---- (x,y) に my_iro を置く
    
    for(int dy=-1;dy<=1;dy++){
        for(int dx=-1;dx<=1;dx++){
            int sx,sy;
            int k=0;//めくる枚数
            sx = x;
            sy = y;
            while(1){
                sx += dx;
                sy += dy;
                if(sx<=0 || sy<=0 || sx>8 || sy>8){//盤面を超えたら
                    break;
                }
                if(board[sy][sx]=='.'){
                    break;
                }
                if(board[sy][sx]==aite_iro){
                    k += 1;
                }
                if(board[sy][sx]==my_iro){
                    for(int i=0;i<k;i++){//めくる作業
                        sx -= dx;
                        sy -= dy; //----修正--- sy -= sy; になっちゃってますよ
                        board[sy][sx]=my_iro;
                    }
                    break;
                }
            }
        }
    }
}

ですね。

2点あって、めくる前に、そもそもboard[y][x]の自分の石を置かないといけません。

board[y][x]=my_iro;

それから、

sy -= dy;

ですね。ベクトル (dx,dy) は、置いた石から検査する方向を示しています。
自分の石がみつかったら、バックして元の石までひっくり返す、ということを
していますね。おそらく -sy は打ち間違いだと思いますが。

初心者とは思えません。プログラムは綺麗だと思いました。

Comments

[ka-kosya19212]

そのとおりに書き換えたら正しく動作するようになりました！わざわざ丁寧にありがとうございました。