C言語でオセロに機能を追加したい

Question

前提・実現したいこと

C言語の初心者でC言語でオセロを作っています。
配置可能箇所を表示する機能を実装したいのですが、中々うまくいきません。
どうかお力添えいただけないでしょうか。

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

help機能の動作、呼び出しがうまくいきません。

該当のソースコード

#include <stdio.h>
#include <ctype.h>
//-----------------------------------マクロ定義
//盤面の大きさ
#define SIZE 8		// 8x8のサイズとする

//状態を定義
#define NONE 0		//空
#define BLACK 1		//黒
#define WHITE 2		//白
//-----------------------------------グローバル変数
//盤面
int board[ SIZE ][ SIZE ];	//board[ 縦 ][ 横 ]

//プレーヤー BLACK or WHITE
int player = BLACK;		//先手は黒

//コマを置く位置 x:横 y:縦 範囲は0～SIZE-1
int x = 0, y = 0;

//-----------------------------------関数のプロトタイプ宣言
void vanish( void );
void display( void );
void input( void );
int is_ok( void );
void put( void );
int check( int x, int y, int dx, int dy );
void flip( int x, int y, int dx, int dy);
int simple_check_end( void );
void help( int y,int x );
//-----------------------------------
//メイン関数
//
int main(int argc, char **argv)
{
	
	vanish();		//盤面の初期化
	
	player = BLACK;		//先手は黒
	while( 1 ){		//ゲームのループ（無限ループなので、このままでは終われない！）
	
					//ここで手番を表示
		
		display();		//盤面表示
		
		input();	//コマの位置を入力
		
		put();		//コマを置き，裏返す
		
		if( player == BLACK ){	//手番交替
			player = WHITE;	
		}
		else{
			player = BLACK;
		}
		if(simple_check_end()==1)
		{
			printf("\n");
			display();		//盤面表示
		  break;
		}
	}
	return 0;
	
}
//-----------------------------------
//初期化関数
//	機能：盤面に初期値を設定する
//	引数：なし
//	戻り値：なし
//
void vanish( void )
{
	int i,j ;
	 
	
	for( i = 0; i < SIZE; i++ ){		//縦のループ
		for( j = 0; j < SIZE; j++ ){	//横のループ
			board[ i ][ j ] = NONE;		//とりあえず全部空きにしておく	
			board[3][3] = WHITE;
			board[4][4] = WHITE;
			board[3][4] = BLACK;
			board[4][3] = BLACK;
		}
	}
	//それから中央にコマを配置する
	return;
}

//-----------------------------------
//盤面表示関数 display
//	機能：現在の盤面を表示する
//	引数：なし
//	戻り値：なし
//一文字の時にhelpを呼び出す
void display( void )
{
	int i ;
	int j ;
    printf(" ");
	printf("  A B C D E F G H  ");
	printf("\n");
	
	for( i = 0; i < SIZE; i++ ){	//縦のループ
		printf( "%2d", i );		//縦の位置を示す数字
		for( j=0; j < SIZE; j++ ){		//横のループ
			
			if( board[ i ][ j ] == NONE ){		//空なら
				printf( "・" );
			}else if( board[ i ][ j ] == BLACK ){	//黒なら
				printf( "●" );
			}else if( board[ i ][ j ] == WHITE ){	//白なら
				printf( "○" );
			}else {
				printf( "？" );		//今のところ、それ以外はない
			}
			
		}
		printf( "\n" );		//横の最後は改行しておく
	}
	return;
}
//-----------------------------------
//入力関数
//	機能：キーボードからコマを置く位置をx,yに入力する
//	引数：なし
//	戻り値：なし
//
void input( void )
{
//	int x, y;	//コマを置く位置
	char c;
	while( 1 ){		//正しく入力されるまで繰り返す

		printf( "横と縦を空白区切りで入力 >" );
		scanf( "%c %d", &c, &y );
		while(getchar()!='\n');
		x=tolower(c)-'a';
		//盤面内か確認
		
		if(c=="p"||y=="\0")
		{
			help(y,x);
			continue;
		}
		if(x<0 || x>=SIZE || y<0 || y>=SIZE )
		{
			printf("指定可能範囲外です。\n");
			continue;
		}
		//空いているか確認
		if(board[y][x]!=NONE)
		{
			printf("\n");
			printf("コマが存在します\n");
			continue;
		}
		//裏返すことができるか確認
		if(is_ok() == 0)
		{
			printf("\n");
			printf("コマを反転できません\n");
			continue;
		}
		return;		//正しく入力できたら戻る
	}
	//ここには来ないはず
}

//-----------------------------------
//コマを置き，裏返す関数 put
//	機能：全ての方向に，裏返せるならば，裏返す
//	引数：なし
//	戻り値：なし
//
void put( void )
{
	
	//まずコマを置く
	board[ y ][ x ] = player;

 
	//全方向について裏返せるか確認して、裏返す
	if( check( x, y, 1, -1 ) == 1 )	//右上
		flip( x, y, 1, -1 );
	if( check( x, y, 1, 0 ) == 1 )	//右
		flip( x, y, 1, 0 );
	if( check( x, y, 1, 1 ) == 1 )	//右下
		flip( x, y, 1, 1 );
	if( check( x, y, 0, -1 ) == 1 )	//上
		flip( x, y, 0, -1 );
	if( check( x, y, 0, 1 ) == 1 )	//下
		flip( x, y, 0, 1 );
	if( check( x, y, -1, -1 ) == 1 )	//左上
		flip( x, y, -1, -1 );
	if( check( x, y, -1, 0 ) == 1 )	//左
		flip( x, y, -1, 0  );
	if( check( x, y, -1, 1 ) == 1 )	//左下
		flip( x, y, -1, 1 );
	
	return;
}

//-----------------------------------
//チェック関数 check
//	機能：指定された方向について、裏返るコマがあるか確認する
//	引数：x,y:コマを置く位置，dx,dy:移動する方向への差分(±1)
//	戻り値：0は裏返らない，1は裏返る
//
int check( int x, int y, int dx, int dy )
{
	int flag = 0;	//相手のコマがあったかどうか 0:ない 1:ある
	while( 1 ){
		x += dx;	//x方向に移動してみる
		y += dy;	//y方向に移動してみる
		
		if( x < 0 || y < 0 || x > SIZE-1 || y > SIZE-1)
			return 0;		//盤面の外に出ていたら終了
		
		if( board[ y ][ x ] == NONE )
			return 0;		//空きマスだったら終了
		
		//相手のコマがあったらフラグを立てて、その先を探す
		if( board[ y ][ x ] != player ){	//このチェック方法は要注意！
			flag = 1;
			continue;	//続ける
		}
		if( board[ y ][ x ] == player ){		//ここまで来たら自分のコマのはず
			if( flag == 1 )
				return 1;		//もしフラグがたっていれば戻り値1で終了。いなければ終了
			else
				return 0;
		}
		//ここには来ないはず
	}
}

//-----------------------------------
//挟まれた相手のコマを裏返す関数 flip
//	機能：指定された方向について、実際に裏返す
//	引数：x,y:コマを置く位置，dx,dy:移動する方向への差分(±1)
//	戻り値：なし
//
void flip( int x, int y, int dx, int dy)
{
	while( 1 ){
		x += dx;
		y += dy;
		
		if( board[ y ][ x ] == player )
			break;		//自分のコマまできたら終了
		
		board[ y ][ x ] = player;		//そうでなければ相手のコマなので自分のコマに変える
	}
	return;
}

//-----------------------------------
//簡易版終了チェック関数 simple_check_end
//	機能：終了を調べる
//	引数：なし
//	戻り値：1:終了 0:終了でない
//
int simple_check_end( void )
{
	int i, j;
	int flag;
	
	flag = 1;	//とりあえず終了にしておく
	for( i = 0; i < SIZE; i++ ){		//縦のループ
		for( j = 0; j < SIZE; j++ ){	//横のループ
			if( board[ i ][ j ] == NONE ) flag = 0;		//空きがあった
		}
	}
	if( flag == 0 )
		return 0;		//空きがあれば続けるの戻り値
	else
		return 1;		//空きがなければ終了の戻り値
}
//-----------------------------------
//コマが置けるかどうか（裏返せるかどうか）調べる関数 is_ok
//	機能：いずれかの方向に置けるかを調べる
//	引数：なし
//	戻り値：0は置けない，1は置ける
//
int is_ok( void )
{
	int ok = 1; //裏返せる（置ける）と仮定
	if( check( x, y, 1, -1 ) == 1 )	//右上
		return ok; //置ける
	if( check( x, y, 1, 0 ) == 1 )	//右
		return ok; //置ける
	if( check( x, y, 1, 1 ) == 1 )	//右下
		return ok; //置ける
	if( check( x, y, 0, -1 ) == 1 )	//上
		return ok; //置ける
	if( check( x, y, 0, 1 ) == 1 )	//下
		return ok; //置ける
	if( check( x, y, -1, -1 ) == 1 )	//左上
		return ok; //置ける
	if( check( x, y, -1, 0 ) == 1 )	//左
		return ok; //置ける
	if( check( x, y, -1, 1 ) == 1 )	//左下
		return ok; //置ける
	// ここまで来たら
	ok = 0; //置けない
	return ok;
}
//-----------------------------------
//	配置確認関数　help
//	機能：配置可能個所の確認
//	引数：なし
//	戻り値：なし
void help( int y,int x )
{

	int i;
	int box[8][8]={
		{0,1,2,3,4,5,6,7},//0,n
		{0,1,2,3,4,5,6,7},//1,n
		{0,1,2,3,4,5,6,7},//2,n
		{0,1,2,3,4,5,6,7},//3,n
		{0,1,2,3,4,5,6,7},//4,n
		{0,1,2,3,4,5,6,7},//5,n
		{0,1,2,3,4,5,6,7},//6,n
		{0,1,2,3,4,5,6,7} //7,n
	};
	for(i=0; SIZE>box[y][x];i++){
		//盤面内か確認
		if(x<0 || x>=SIZE || y<0 || y>=SIZE )
		{
			i++;
		}
		//空いているか確認
		if(box[y][x]!=NONE)
		{
			i++;
		}
		//裏返すことができるか確認
		if(is_ok() == 0)
		{
			i++;
		}
		//配置可能である場合
		if(box[y][x]==NONE&&is_ok() == 1)
		{
			printf("%d\n",box[8][8]);
			i++;
		}
	}
return;
}

試したこと

・void help( int y,int x )が呼び出せているかの確認。
-呼び出せてはいるが、配置可能な箇所を表示せずに戻っている。
・入力関数にvoid help( int y,int x )呼び出し用のif(c=="p"||y=="\0")を追加。
-何を入力しても配置可能範囲外と表示されるようになってしまった。

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

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

Answer 1

C言語のコードを組みたいのであれば、デバッグ環境を整えましょう
WindowsであればVisualStudio、そうでなければEclipseなど、
これらの統合環境では、C言語のコードの任意の行で実行を停止させ、変数の値を参照したり、１行づつステップ実行させて変数の値を追いかけるなどができるようになります
そうすれば、あてずっぽでコードを組む、ということをしなくて済みます

Comments

[otogen]

回答ありがとうございます。
ツールに関して記す場所を見落としておりました、PC環境はwindowsで使用ツールに関しましてはGeanyを使っています。
が、やはりそちらの方が安心でしょうか。
導入してみて見直してみようと思います。

[y_waiwai]

ああ、単なる老婆心でのアドバイスです。
すでにソースデバッグできる環境をお持ちなら、使い慣れている環境のほうがいいでしょう

Answer 2

is_okがグローバル変数の x, y を参照しているのに対して、helpは引数の x, y を参照しているせいだと思います。この2つは別物です(ワーニング出てないでしょうか？)
x, yのような一時的な値をグローバル変数にするのはいつ値が変更されたのかがわかりにくくなりバグの元です。まずグローバル変数のx, yを削除しましょう。
is_okに引数 x, y を追加して、helpからis_ok(x, y)となるようにしてみてください。

Comments

[otogen]

回答ありがとうございます。
初期はただのオセロのプログラムだったのですが、数字を文字にする、今悩んでる機能を追加する。
としてる内に自分でもこんがらがってしまいそこまで目が回っていませんでした。
アドバイスを元に修正してみます。