オセロ作成に関する助言をお願いします

質問

• 初心者向けのC言語を使用したオセロ作成の助言をお願いします。
• 関数を使わない方法でのオセロ作成に困っています。
• 偶数か奇数かの判定で石の色を変える方法についても教えてください。

回答No.1

N-Ishikawa

そもそもおかしいところは1カ所ありますね。
bord[y][x]に1を設定した場合、その直後で必ず2にしています。
----------------------------------------------------------------
//ひっくり返す//
if(x>=0 && x<8 && y>=0 && y<8 && bord[y][x]==0){
if(bord[y][x] == 0){
bord[y][x]=1;
} ← ここに elseがないのが問題
if(bord[y][x]==1)
{
bord[y][x]=2;
}
}
----------------------------------------------------------------

デバッグで値の変化ぐらいは確認しましょう．．．．．

あと、オセロのルールに則れば、

１．必ずひっくり返せる場所にしかおけない
　　→おけるかどうかの判定が必要。
　　　入力エラーは入力直後に判定、間違っていても
　　　おける以上、相手に権利を移させない。
　　　ギブアップで-1,-1を入力すると手動で権利を移す

２．置いたらひっくり返す。
　　→ひっくり返せる方向は8方向
　　　私ならどの方向にひっくり返せるかのチェック関数を作成。
　　　1バイトフラグ（8bit）にて返す。（１．でも呼ぶ）
　　　ひっくり返す際はフラグに従い、設置位置隣接位置から目的の
　　　駒までひっくり返す。

ついでにいくつか気になった点を挙げておきます。

・プログラムですが、X,Y座標入力において
　以下の判定実行は scanf()の直後で実行すべきです。
　------------------------------------------
　//エラー処理//
　if(x<0 || x>=8 || y<0 || y>=8 )
　　　：
　------------------------------------------

　下記判定は上記判定の前段階で行っているのですが、
　同一if文内の 判定順序は C言語の処理系によっては
　後ろから．．．．なんてことが多々あります。
　bord[y][x]のように、配列にx,yを使用するのは必ず
　値チェックを先に行いましょう。
　（xに10000とか入力すると例外エラーの可能性が．．．．）
　------------------------------------------
　if(x>=0 && x<8 && y>=0 && y<8 && bord[y][x]==0){
　------------------------------------------
　そもそも、scanf()直後にエラー処理を行っていれば、
　「x>=0 && x<8 && y>=0 && y<8 &&」は不要ですよね。

・入力に失敗しただけで相手に権利が移ってしまうのは厳しくないですか？
　（そのように見えます。違っていたらすみません。）

・playerがカウントアップしていくのですが、うまく使えば終了条件に
　使用できますよね。
　今は駒を置くのに失敗してもカウントアップされてしまうようなので
　６４になっても必ず終了しているとは限りませんが、そこをうまく
　調整すると終了条件に使用できるようになるかと思います。
　（今は結局 CTRL+Cで終了させるように見えます．．．．．）

補足

こんなに詳細に回答してくれるかたがいるとは思いませんでした。
ありがとうございます。

>必ずひっくり返せる場所にしかおけない
>→おけるかどうかの判定が必要
これはクリアしました。

>入力エラーは入力直後に判定、間違っていても
>おける以上、相手に権利を移させない。
>ギブアップで-1,-1を入力すると手動で権利を移す
これもクリアしました。

＞1バイトフラグ（8bit）にて返す。（１．でも呼ぶ）
＞ひっくり返す際はフラグに従い、設置位置隣接位置から目的の
＞駒までひっくり返す。
お手上げです。

＞そもそもおかしいところは1カ所ありますね。
＞bord[y][x]に1を設定した場合、その直後で必ず2にしています。
＞--------------------------------------------------------------
＞//ひっくり返す//
＞if(x>=0 && x<8 && y>=0 && y<8 && bord[y][x]==0){
＞if(bord[y][x] == 0){
＞bord[y][x]=1;
＞} ← ここに elseがないのが問題
＞if(bord[y][x]==1)
＞{
＞bord[y][x]=2;
＞}
＞}
＞--------------------------------------------------------------
＞デバッグで値の変化ぐらいは確認しましょう．．．．．
この一番肝心なところがわかりません。。。
------------------------------------------------------------
//ひっくり返す
if(x>=0 && x<8 && y>=0 && y<8 && bord[y][x]==0){
if(bord[y][x] == 1){
bord[y][x]=2;
}else{
if(bord[y][x]==0)
bord[y][x]=1;
}
printf("\n");
printf("石が置かれました。\n");
player++;
}
------------------------------------------------------------
こんな感じになったんですけどやはりずっと同じ石しか置けません。

終了条件ですかそれはできそうですね。

--------------------------以下すべてのソース-------------------
#include<stdio.h>

int bord[8][8];
int x,y;
int i,j,l,a;
int player;

int main (void)
{
//ボードの初期化//
for(i=0;i<8;i++){
for(j=0;j<8;j++){
bord[i][j]=0;
}
}
bord[3][3]=1;
bord[4][4]=1;
bord[3][4]=2;
bord[4][3]=2;

//bordの表示//
player=0;
for(;;){ //何度も表示
printf("０１２３４５６７\n");
for(i=0;i<8;i++){
for(j=0;j<8;j++){
if(bord[i][j] == 0){
printf("＊");
}
else if(bord[i][j] == 1){
printf("●");
}
else{
printf("○");
}

}
printf("\n");
}

//黒か白か判定//
if(player%2==0){
bord[i][j]=1;
printf("黒の番です。\n");
printf("y座標を入力してください。(縦軸)");
scanf_s("%d", &y);
printf("x座標を入力してください。(横軸)");
scanf_s("%d", &x);
}
if(player%2==1){
bord[i][j]=0;
printf("白の番です。\n");
printf("y座標とx座標どちらも１００と入力すると相手のターンにします。\n");
printf("y座標を入力してください。(縦軸)");
scanf_s("%d", &y);
printf("x座標を入力してください。(横軸)");
scanf_s("%d", &x);

}

//ひっくりかえす//
//石を置く権利を相手に移す
if(x==100 && y==100){
printf("\n");
printf("石を置く権利を相手に渡します。\n");
player++;
continue;
}
//エラー処理（入力された数字が許容範囲外）
if(x<0 || x>=8 || y<0 || y>=8 )
{
printf("許容範囲外です。\n");
printf("もう一度入力してください。\n");
}
//エラー処理(同じところには置けない）
if(bord[y][x]!=0)
{
printf("\n");
printf("置かれてるよ！\n");
}
//ひっくり返す
if(x>=0 && x<8 && y>=0 && y<8 && bord[y][x]==0){
if(bord[y][x] == 1){
bord[y][x]=2;
}else{
if(bord[y][x]==0)
bord[y][x]=1;
}
printf("\n");
printf("石が置かれました。\n");
player++;
}

} //無限loop終了地点
return 0;
}
---------------------------------ソース終了---------------------