C言語 ゲーム

このプログラムに関数を一つ増やしたいのですが、うまくできません。どこを、どうすればいいでしょう？
迷路の内容は未完成です。

#include <stdio.h>
#include <conio.h>
#include <stdlib.h>
#include <string.h>

int KEY(int *Kn,int *Y,int *X);


int main(void)
{
//数値格納
int kn; //入力キー
int sty,stx; //座標
int fy,fx; //マップ範囲
int jm[25][40]; //移動可不判定
char mapc[3]={0};//複写されたマップ構成情報

//カウント
int y,x; //判定生成、座標生成・描写用


//マップチップ
char mc[2][3]={
/*mc[0] 移動可能*/"　",
/*mc[1] 移動不可*/"■"};

//マップ構成 (最大値 25行x40列)
char map[][81]={
/*00*/"■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■",
/*01*/"■　■　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　■",
/*02*/"■　■■■■　■　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　■",
/*03*/"■　■■　　　■　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　■",
/*04*/"■　　■　■■■　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　■",
/*05*/"■■　■　■　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　■",
/*06*/"■　　■　■　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　■",
/*07*/"■　■■　■　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　■",
/*08*/"■　　　　■　■　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　■",
/*09*/"■　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　■",
/*10*/"■　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　■",
/*11*/"■　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　■",
/*12*/"■　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　■",
/*13*/"■　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　■",
/*14*/"■　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　■",
/*15*/"■　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　■",
/*16*/"■　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　■",
/*17*/"■　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　■",
/*18*/"■　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　■",
/*19*/"■　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　　■",
/*20*/"■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■"};

//マップ範囲算出
fy=sizeof(map)/81; //行
fx=strlen(map[0])/2; //列




//キャラクター初期座標指定
sty=8,stx=8;


/* 判定生成 */
for(y=0;y<fy;y++){
for(x=0;x<fx;x++){
strncpy(mapc ,&map[y][(x)*2] ,2 );
if(strncmp(mc[1],mapc,2)==0){jm[y][x]=1;} //移動不可判定
else if(strncmp(mc[0],mapc,2)==0){jm[y][x]=0;} //移動可能判定
}
}

/* 画面出力 */

while(1){
system("cls"); //画面消去



/* 座標生成、描写 */
for(y=0;y<fy;y++){
for(x=0;x<fx;x++){
strncpy(mapc ,&map[y][(x)*2] ,2);
if(jm[y][x]==0){
if(y==sty&&x==stx)
{printf("%s","○");} //キャラ表示
else if(strncmp(mc[0],mapc,2)==0)
{printf("%s",mapc );} //移動可能表示
}
else if (strncmp(mc[1],mapc,2)==0)
{printf("%s",mapc );} //移動不可表示
}
if(fx<40){printf("\n");} //fxが40未満の場合、改行
}

if(sty==1&&stx==1){
printf("ゴールしました\n");
}


/* 入力キー、移動座標出力 */
KEY(&kn,&sty,&stx);

/* 壁、マップ外への侵入防止 */
for(y=0;y<sty+1;y++){
for(x=0;x<fx;x++){
if((y==sty&&x==stx&&jm[y][x]==1)||(fy<=sty||fx<=stx)){
if(kn==0x4b){stx++;}
else if(kn==0x4d){stx--;}
else if(kn==0x48){sty++;}
else if(kn==0x50){sty--;}
break;
}
}
}


/* 終了操作 */
if(kn==0x1b){printf("▼終了します。\n") ;break;}
}

}

/* 入力キー、移動座標出力 */
int KEY(int *Kn,int *Y,int *X){
while(1){
*Kn=getch( ); //1:通常キー
if (*Kn==0){
*Kn=getch( ); //2:特殊キー
if (*Kn==0x4b){(*X)--;}// ←
else if (*Kn==0x4d){(*X)++;}// →
else if (*Kn==0x48){(*Y)--;}// ↑
else if (*Kn==0x50){(*Y)++;}// ↓
else if (*Kn==0x80){ }//特殊キー追加場所
else {continue;}
break;
}
else if (*Kn==0x1b){}// Esc
//else if (*Kn== ){} 通常キー追加場所
else {continue;}
break;
}
return 0;
}

No.1 ベストアンサー 回答者： kmee 回答日時： 2013/12/28 09:38

目的もなく、ただ「関数を一つ増やしたい」と言われても困ります。

目的を決まらなければ、どんな関数にしたらよいかがわかりません。
どんな関数にしたらよいかがわからなければ、プログラミングのしようがありません。


以下、余談にはなりますが。
> char map[][81]={
■が2バイトだと決めつけていますが、UTF-8だと3バイトです。
プログラム中この2バイトであると決めつけた「2」という数値が出てきます。
これは「よくないプログラミング手法」です。

//マップチップ
char mc[2][3]={
と用意しているのですから、マップデータを表示と同じにする必要はありません。
char map[][81]={
/*00*/"********************",
/*01*/"* * * * *",
...
などとすれば、2を掛けたり割ったりする必要がなくなります。
strncpy,strcmp関数もほとんど不要です。



あるいは
int jm[25][40]; //移動可不判定
にマップ情報をいれるのですから、最初から
int jm[][41]={
{1,1,1,1,1,1,1,1,1,1,....,1,2},
{1,0,1,0,0,1,1,1,1,1,0,....,1,2},
...
{2}
}
等とjmに直接マップデータを記述して、 char map[][81] をまったく使用しない、という方法もあります。
// 列の終端、行の終端に 2 を使用しました。
// sizeofによる計算や、strlen関数の代りに、終端までの数を数えるようにします。
// 現行のmapでも、終端行の目印(空文字列とか)を入れて、そこまでの行数を数えるようにする方が、sizeofを使うものに比べて汎用性がよいです。


/* 壁、マップ外への侵入防止 */
for(y=0;y<sty+1;y++){
for(x=0;x<fx;x++){
if((y==sty&&x==stx&&jm[y][x]==1)||(fy<=sty||fx<=stx)){

条件をよく見てください。
この条件が成立するのは、次のどちらかが成立している場合です。
1. (fy<=sty||fx<=stx)
2. (y==sty&&x==stx&&jm[y][x]==1)
1.には、変数 x,y が使われていないので、ループの影響は受けません。
2. では、 少なくとも、y==sty&&x==stx が成立している必要があります。これ以外のx,yでは不成立です。
　つまり、 このforループでは、y==sty&&x==stxの1回以外は無駄なループです。

sty,stxが範囲外だと、jm[sty][stx]==1 で動作に不具合が出ます。
ですが、
(sty,stxが範囲外である) || (jm[sty][stx]==1)
とすると、 || 演算子の特性( 左が真なら、右は計算しない) によって、 jm[sty][stx]==1 を評価するのは sty,stxが が範囲内の場合のみ、となります。

もっとも、私なら次のように作ります
newX=stx ;
newY=sty ;
/* 入力キー、移動座標出力 */
KEY(&kn,&newY,&newX);

/* 壁、マップ外への侵入防止 */
if ( newX,newYがマップの範囲内 && jm[newY][newX]==0 ) {
stx=newX;
sty=newY;
}

キーコードで判定して移動したものを、またキーコードで判定して元に戻す、なんて、面倒ではなかったですか?
「移動」に必要なのは、次の座標と、そこへ移動できるかどうかの判断だけで、キーコードは不要です。

この回答へのお礼

無茶なこと質問した自分にこんなにも回答ありがとうございます。すごくためになりました。
自分でももう一度考えてみたいと思います。

お礼日時：2013/12/28 12:08