コンソールアプリケーションに関して - 通行不可の壁

[央真]

質問させて頂きます。

コンソールアプリケーションでＲＰＧ（っぽいもの）を作成しているのですが、
通行不可の壁の作り方がわかりません。
（元のソースは某サイト様のものを参考とし、自分なりに改良させて頂きました）。

マップは二次元配列を利用しています。

MoveBlock関数で、二次元配列のマップを展開し、
プレイヤー（主人公）の座標(px,py)が、
二次元配列に対応するようにと自分なりにコードを書いたのですが、
なかなか難しくて、とりあえずマップ座標(0,0)の場とプレイヤー座標が一致した場合に、
通行不可と表示しようとしたのですが、
どうしてもマップ座標(1,1)の場所に表示されてしまいます。

現状の悩みは上記のような感じで、最終的には通行不可の壁を作りたいと思っています。

どなたかご教授頂けたら幸いです。

コード:

#include "stdafx.h"
#include <stdio.h>
#include <conio.h>
#include <windows.h>
#include "Console.h"

#define KEY_UP    0x48
#define KEY_DOWN  0x50
#define KEY_RIGHT 0x4d
#define KEY_LEFT  0x4b

#define MAP_SIZE_Y 20
#define MAP_SIZE_X 20

int i = 0, j = 0;

int px = 3, py = 4;

int map[100][100] = {
	{1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1},	
	{1,0,0,1,0,0,0,1,1,1,0,0,0,0,0,0,0,1},	
	{1,0,0,0,0,0,0,1,1,1,0,0,0,0,0,0,0,1},	
	{1,0,1,0,1,0,0,0,0,0,0,0,0,0,0,0,0,1},	
	{1,0,0,0,0,0,0,1,1,1,0,0,0,0,0,0,0,1},		
	{1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1},
} ;

char *a = "@";//主人公
char *b = "#";//壁


void DrawMap()
{

	for ( j = 0; j < MAP_SIZE_Y; j++ )
	{
		for ( i = 0; i < MAP_SIZE_X; i++ )
		{
			if ( j == py && i == px ){
				printf(a);
				} else {
				switch ( map[j][i] ){
				default:printf(" "); break;
				case 1: printf(b); break;
				}
			}
		}
		printf("\n");
	}
}

void MoveBlock()
{
	for ( j = 0; j < MAP_SIZE_Y; j++ )
	{
		for ( i = 0; i < MAP_SIZE_X; i++ )
		{
			if( map[j][0] == py && map[0][i] == px ){
				if( j == 0 && i == 0 ){
				printf("通行不可");
				}
			}
		}
	}
}

void GetKey()
{
	switch ( getch() ){
		case KEY_UP:
			py--;
			break; //UP
		case KEY_DOWN:	
			py++; 
			break; //DOWN
		case KEY_LEFT:	
			px--; 
			break; //LEFT
		case KEY_RIGHT: 
			px++; 
			break;	//RIGHT
	}
}

void main()
{
	CConsole cnsl;

	cnsl.SetColor(H_GREEN, L_BLACK);

	while ( 1 )
	{
		system("cls");
		DrawMap();
		MoveBlock();
		GetKey();
	}
}

[box]

コード:

	{
		system("cls");
		DrawMap();
		MoveBlock();
		GetKey();
	}

しろうとである私が口を出していいかどうかわかりませんが、
MoveBlock() の後で GetKey() ですか？

GetKey() でどの方向へ動くかを決めた後で、MoveBlock() でその方向へ動けるかどうかを判断する、のではなくて、ですか？

[box]

なかなか難しくて、とりあえずマップ座標(0,0)の場とプレイヤー座標が一致した場合に、
通行不可と表示しようとしたのですが、
どうしてもマップ座標(1,1)の場所に表示されてしまいます。

コード:

			if( map[j][0] == py && map[0][i] == px ){
				if( j == 0 && i == 0 ){
				printf("通行不可");
				}
			}

(0, 0) 以外のマップ座標とプレイヤー座標が一致した場合に「通行不可」と表示するようにしたら、
どうなりますか？

[央真]

お返事ありがとうございます。

そうですね。これは僕の間違いです。
GetKey()の後にMoveBlock()を書き直しました。
プログラミングに対する考え方がまだよくわかっていないようです。
なので、ゲーム作りでＣの勉強をしていたんです。

コード:

 if( map[j][0] == py && map[0][i] == px ){
                if( j != 0 && i != 0 ){
                printf("通行不可");
                }
            } 

(0,0)以外とはつまりはこういう事なんでしょうか？
実行してみると、今度は座標(0,0)～(1,1)周辺にプレイヤー座標が来ると、
「通行不可」の文字が実行画面上に沢山出るようになってしまいました。

たぶん上記のコードが間違っているのでしょうが、自分にはどうすれば良いかわからず・・。
かれこれ数時間悪戦苦闘しています。
自分でコードを追加したものの、何か根本的な間違いをしているように思います。

壁「#」にプレイヤー「@」が接触したら、「通行不可」と表示させよう、というのが当面の課題です。

[softya（ソフト屋)]

やりたい事からすると、移動キーが押されたらと言う条件で通過不可か可能か表示する必要があります。
なので、GetKey()で移動したフラグを戻り値で返す　→　移動したならMoveBlock（）でマップと当たり判定する
としないと行けないでしょう。

[beatle]

コード:

            if( map[j][0] == py && map[0][i] == px ){
                if( j == 0 && i == 0 ){
                printf("通行不可");
                }
            }

思考を整理しましょう。以下 mapの「1」が壁で、「0」が通行可能な場所と仮定して話を進めます。
おそらく、次の1点だけ理解すればすんなりプログラムできるでしょう。
それは、プレイヤーが現在「壁」にいるのか「床」にいるのかの判断です。

ポイント: プレイヤー座標は(px,py)なので、map[py][px]でプレイヤーが「壁」と「床」どちらにいるか取得できる。

これが分かれば簡単で、プレイヤーが壁にいる場合に何か処理をするには、map[py][px] == 1のときに条件分岐すれば良いということになります。


以下、実際にプログラムの動作を追いかけつつ、何がおかしいかを考える道筋を示しました。
ただ、もしかしたら読むと混乱するので、隠してあります。長いですし。

► スポイラーを表示

恐らく、boxさんはこの5行がおかしいことに気付くかどうか見たかったのだと思います。
央真さんは「マップ座標(0,0)の場とプレイヤー座標が一致した場合に、通行不可と表示しようとした」のですよね。
まず、1行目のif文の動作を考えてみてください。

コード:

if( map[j][0] == py && map[0][i] == px )

map座標(0, 0)とプレイヤーの衝突を考えたいので、i == 0 && j == 0 のときを考えます。map[0][0]は1です。

プレイヤーが(0, 0)にいるときは (px, py) = (0, 0)ですから、このif文の条件式は成り立ちません。（if ( 1 == 0 && 1 == 0 ) となります。）
同様に考えると、プレイヤーが (0, y) もしくは (x, 0) にいる場合もこのif文の条件式は成り立ちません。（map[0][0] == pyかmap[0][0] == pxのどちらかの式が偽となります。）

今度はプレイヤーが (px, py) = (1, 1) にいるときを考えましょう。このときif文は if ( 1 == 1 && 1 == 1 )となって成り立ちます。このとき2行目のif文も、i == 0 && j == 0なので成り立ちます。
よってprintf("通行不可");が実行されます。

まとめると、i == 0 && j == 0 のときに px == 1 && py == 1 となると「通行不可」と表示されます。
なにかおかしいと思いませんか？

そもそも、map配列には1か0しか入っていませんから、

コード:

if( map[j][0] == py && map[0][i] == px )

これはどんなに頑張っても

コード:

if( (1 or 0) == py && (1 or 0) == px )

という条件式になります。
この条件式では、プレイヤーが左上の4マスの範囲にいる場合しかチェックできないのです。

[央真]

boxさん、 softyaさん、beatleさん、ありがとうございます。
それぞれの意見を参考にしコードを以下のように書き直した結果、
一応の完成をみました。
一応、というのはまだ曖昧な箇所があるからですが、満足な結果が得られたと思います。

コード:

int MoveBlock() //当たり判定を行う関数
{
	map[j][0] = py;
	map[0][i] = px;

	if( map[py][px] == 1 )
	{
		printf("通行不可");
		return 1;
	}
	else
	{
		return 0;
	}
}

void GetKey() //キー入力関数
{
	int flag = 0;

	switch ( getch() ){
		case KEY_UP:
			flag = MoveBlock(); //MoveBlock()の戻り値を変数flagに代入
			if( flag == 0 )     //変数flagに代入された戻り値が0のとき
				py--;           //プレイヤー座標を更新
			if( flag == 1 )		//変数flagに代入された戻り値が1のとき
				py++;			//プレイヤーは押し戻される
			break; //UP
		case KEY_DOWN:
			flag = MoveBlock(); //MoveBlock()の戻り値を変数flagに代入
			if( flag == 0 )     //変数flagに代入された戻り値が0のとき
				py++;			//プレイヤー座標を更新
			if( flag == 1 )		//変数flagに代入された戻り値が1のとき
				py--;			//プレイヤーは押し戻される
			break; //DOWN
		case KEY_LEFT:
			flag = MoveBlock(); //MoveBlock()の戻り値を変数flagに代入
			if( flag == 0 )     //変数flagに代入された戻り値が0のとき
				px--;			//プレイヤー座標を更新
			if( flag == 1 )		//変数flagに代入された戻り値が1のとき
				px++;			//プレイヤーは押し戻される
			break; //LEFT
		case KEY_RIGHT:
			flag = MoveBlock(); //MoveBlock()の戻り値を変数flagに代入
			if( flag == 0 )     //変数flagに代入された戻り値が0のとき
				px++;			//プレイヤー座標を更新
			if( flag == 1 )		//変数flagに代入された戻り値が1のとき
				px--;			//プレイヤーは押し戻される
			break;	//RIGHT
	}
}

また自分で試行錯誤してコードを書いていこうと思います。
皆様に感謝です。

[softya（ソフト屋)]

今の方法だと通過不可地形に刺さるので仮に移動させて当たり判定をして、マップで当たったら移動を取り消したほうが良いと思います。

あと

コード:

    map[j][0] = py;
    map[0][i] = px;

はマップを書き換えているんで非常にマズイです。