右のキーを押した際に配列で移動可能か判断して、一マス右に移動できるようにしたい。

Question

環境

• Windows10
• visual studio 2019
• DXライブラリ

行いたいこと
例えばプログラムに書いてある座標(nx，ny)=(2，2)に位置するキャラが右に動く際にnx=nx+1にnx=2を代入させて、idou[nx + 1][ny + 0] は
idou[3][2] となり、配列idou[5][5]よりidou[3][2] の座標の位置は0なので移動可能となり、キャラが描画できるというプログラムを作りたいです。
以上の説明はあくあで一例です。出来れば、以上をループのようにして、どの座標からでも右のキーを押すたびに配列idou[nx + 1][ny + 0]から右に移動可能か判断して右に一マス進むようにしたいです。

これは、自分なりに以下のようにプログラムを作りました。ですがうまく機能していません。
出来ればプログラムの原型を崩さずに右キーを押すたびに右に一マス動けるようにしたいです。

		if (Key[KEY_INPUT_RIGHT] == 1)
		{
			
			
			if (idou[nx + 1][ny + 0] == idou[3][0]) {// 移動可能にして、その座標に移動する
				nx = nx + 1;
				nx == 3;
				idou[nx + 1][ny + 0] == idou[3][2];

				playerphoto = gh[7];
			}

そして、以下は全体のプログラムです。

#include "DxLib.h"



int Key[256]; // キーが押されているフレーム数を格納する

// キーの入力状態を更新する
int gpUpdateKey() {
	char tmpKey[256]; // 現在のキーの入力状態を格納する
	GetHitKeyStateAll(tmpKey); // 全てのキーの入力状態を得る
	for (int i = 0; i < 256; i++) {
		if (tmpKey[i] != 0) { // i番のキーコードに対応するキーが押されていたら
			Key[i]++;     // 加算
		}
		else {              // 押されていなければ
			Key[i] = 0;   // 0にする
		}
	}
	return 0;
}

// プログラムは WinMain から始まります
int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance, LPSTR lpCmdLine, int nCmdShow)
{
	SetGraphMode(1600, 680, 32); // ウィンドウの大きさを指定
	ChangeWindowMode(TRUE);
	
	int playerphoto;
	int nx=2, ny =2 ; // 最初のキャラのX座をdxとY座標をdyである。配列の真ん中の0を最初の座標としたため(2,2)となる

	if (DxLib_Init() == -1)        // ＤＸライブラリ初期化処理
	{
		return -1;            // エラーが起きたら直ちに終了
	}



	//1. 3x3マスの2次元配列
	int idou[5][5] = {
					{1,1,1,1,1 },
					{1,0,0,0,1 },
					{1,0,0,0,1 },
					{1,0,0,0,1 },
					{1,1,1,1,1 },
	};

	
	//DrawGraph(dx, dy, playerphoto, FALSE);





	//グラフィックハンドル格納用配列
	int gh[12];
	LoadDivGraph("charall.png", 12, 3, 4, 49, 66, gh);//画像読み込み
	

	SetDrawScreen(DX_SCREEN_BACK);
	while (ScreenFlip() == 0 && ProcessMessage() == 0 && gpUpdateKey() == 0) {
		DrawGraph(nx, ny, gh[5], FALSE);// プレイヤーの画像を描画
	
		// カーソルキーの右が押されている

		if (Key[KEY_INPUT_RIGHT] == 1)
		{
			
			
			if (Key[KEY_INPUT_RIGHT] == 1)
		{
			
			
			if (idou[nx + 1][ny + 0] == idou[3][0]) {// 移動可能にして、その座標に移動する
				nx = nx + 1;
				nx == 3;
				idou[nx + 1][ny + 0] == idou[3][2];

				playerphoto = gh[7];
			}
			else if (nx == 1 && idou[nx + 1] == idou[2]) {
				// 移動不可能
				playerphoto = gh[9];
			}


			if (Key[KEY_INPUT_LEFT] == 1)
			{
				nx = nx - 1;
				playerphoto = gh[2];
			}

			if (Key[KEY_INPUT_UP] == 1)
			{
				ny = ny + 1;
				playerphoto = gh[4];
			}
			if (Key[KEY_INPUT_DOWN] == 1)
			{
				ny = ny - 1;
				playerphoto = gh[3];
			}

			if (nx == 0 && ny == 0)
			{

				playerphoto = gh[7];
			}

			ScreenFlip();
			// 画面をクリア
			ClearDrawScreen();

			DrawGraph(nx, ny, playerphoto, FALSE);

		}
	}

	DxLib_End();                // ＤＸライブラリ使用の終了処理

	return 0;                // ソフトの終了 

}

編集が遅くなりました。すいません。jimbeさんのプログラムをそのまま入れてみました。ですが、思うように動きませんでした。もしかしたら組み込む場所が違ったのかもしれません。

#include "DxLib.h"



int Key[256]; // キーが押されているフレーム数を格納する

// キーの入力状態を更新する
int gpUpdateKey() {
	char tmpKey[256]; // 現在のキーの入力状態を格納する
	GetHitKeyStateAll(tmpKey); // 全てのキーの入力状態を得る
	for (int i = 0; i < 256; i++) {
		if (tmpKey[i] != 0) { // i番のキーコードに対応するキーが押されていたら
			Key[i]++;     // 加算
		}
		else {              // 押されていなければ
			Key[i] = 0;   // 0にする
		}
	}
	return 0;
}

// プログラムは WinMain から始まります
int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance, LPSTR lpCmdLine, int nCmdShow)
{
	SetGraphMode(1600, 680, 32); // ウィンドウの大きさを指定
	ChangeWindowMode(TRUE);
	
	int playerphoto;
	int nx=2, ny =2 ; // 最初のキャラのX座をdxとY座標をdyである。配列の真ん中の0を最初の座標としたため(2,2)となる

	if (DxLib_Init() == -1)        // ＤＸライブラリ初期化処理
	{
		return -1;            // エラーが起きたら直ちに終了
	}



	//1. 3x3マスの2次元配列
	int idou[5][5] = {
					{1,1,1,1,1 },
					{1,0,0,0,1 },
					{1,0,0,0,1 },
					{1,0,0,0,1 },
					{1,1,1,1,1 },
	};

	
	//DrawGraph(dx, dy, playerphoto, FALSE);





	//グラフィックハンドル格納用配列
	int gh[12];
	LoadDivGraph("charall.png", 12, 3, 4, 49, 66, gh);//画像読み込み
	

	SetDrawScreen(DX_SCREEN_BACK);
	while (ScreenFlip() == 0 && ProcessMessage() == 0 && gpUpdateKey() == 0) {
		DrawGraph(nx, ny, gh[5], FALSE);// プレイヤーの画像を描画
	
		// カーソルキーの右が押されている

		if (Key[KEY_INPUT_RIGHT] == 1)
		{
			
			//①if (idou[ny + 1][nx + 1]) {nx==0,}//紙を読む nx==0でidou[nx + 1]がidou[1]の時移動可能にして、その座標に移動する、nx==1でidou[nx + 1]がidou[2]の時移動不可能と
			//①のidou[ny + 1][nx + 1]を利用してif文で書きたいが、idou[ny + 1][nx + 1]を単体でidou[ny + 1]として使えるか？
			if (idou[nx + 1][ny + 0] == 0) { //移動しようとする先が空いていれば
   //移動可能
				nx = nx + 1; //移動
				playerphoto = gh[7];
			}
			else {
				// 移動不可能
				playerphoto = gh[9];
			}

			if (Key[KEY_INPUT_LEFT] == 1)
			{
				nx = nx - 1;
				playerphoto = gh[2];
			}

			if (Key[KEY_INPUT_UP] == 1)
			{
				ny = ny + 1;
				playerphoto = gh[4];
			}
			if (Key[KEY_INPUT_DOWN] == 1)
			{
				ny = ny - 1;
				playerphoto = gh[3];
			}

			if (nx == 0 && ny == 0)
			{

				playerphoto = gh[7];
			}

			ScreenFlip();
			// 画面をクリア
			ClearDrawScreen();

			DrawGraph(nx, ny, playerphoto, FALSE);

		}
	}

	DxLib_End();                // ＤＸライブラリ使用の終了処理

	return 0;                // ソフトの終了 

}

編集
入門書を見るなりしてjimbeさんから頂いたコードを正しく使おうとやっているのですが
どうもうまくいきません。
条件文としては正しいと思うのですが
右のキーが押される＆右のマスが0という条件で
写真[7]を表示するプログラムとして

if (Key[KEY_INPUT_RIGHT] == 1)
{
if (idou[nx + 1][ny + 0] == 0)
{ //移動しようとする先が空いていれば
   //移動可能
nx = nx + 1; //移動
playerphoto = gh[7];
}

else {
// 移動不可能
playerphoto = gh[9];

}

}
と書いたんですが、うまくいきません。

jimbeさんのプログラム正しかったです。キャラの移動を50倍してみると正しく機能していることがわかりました。

Answer 1

#include "DxLib.h"

int Key[256];  // キーが押されているフレーム数を格納する配列

int gpUpdateKey() // キーの入力状態を更新する
{
	char tmpKey[256];  // 現在のキーの入力状態を格納する配列
	GetHitKeyStateAll(tmpKey);  // 全てのキーの入力状態を得る
	for (int i = 0; i < 256; i++) {
		if (tmpKey[i] != 0) {  // i番のキーが押されていたら
			Key[i]++;  // 加算
		}
		else {  // 押されていなければ
			Key[i] = 0; // 0にする
		}
	}
	return 0;
}

int WINAPI WinMain(HINSTANCE hi, HINSTANCE hp, LPSTR cl, int cs)
{
	SetGraphMode(1600, 680, 32);        // ウィンドウの大きさを指定
	ChangeWindowMode(TRUE);             // 全画面ではなくウインドウを使用
	if (DxLib_Init() == -1) return -1;  // ＤＸライブラリ初期化処理
	SetDrawScreen(DX_SCREEN_BACK);      // 裏画面を使用する設定

	int idou[5][5] = {
		{ 1, 1, 1, 1, 1 },
		{ 1, 0, 0, 0, 1 },
		{ 1, 0, 0, 0, 1 },
		{ 1, 0, 0, 0, 1 },
		{ 1, 1, 1, 1, 1 },
	};
	int nx = 2, ny = 2;  // プレイヤーの初期位置
	int move = 0;  // 移動状態

	int gh[12];  //グラフィックハンドル格納用配列
			// 5:正面、7:右向き、2:左向き、4:上向き、3:下向き、9:移動不可
	LoadDivGraph("charall.png", 12, 3, 4, 49, 66, gh);  //画像読み込み
	int playerphoto = gh[5];

	while (ProcessMessage() == 0) {
		gpUpdateKey();  // キーの入力状態を取得
		if (Key[KEY_INPUT_RIGHT] == 1) {  // カーソルキーの右が押されている
			move = 1;
			if (idou[nx + 1][ny] == 0) {  //移動先が空いていれば
				nx = nx + 1; playerphoto = gh[7];  // 右向き
			}
			else {
				playerphoto = gh[9];  // 移動不可能
			}
		}
		if (Key[KEY_INPUT_LEFT] == 1) {
			move = 1;
			if (idou[nx - 1][ny] == 0) {  //移動先が空いていれば
				nx = nx - 1; playerphoto = gh[2];  // 左向き
			}
			else {
				playerphoto = gh[9];  // 移動不可能
			}
		}
		if (Key[KEY_INPUT_UP] == 1) {
			move = 1;
			if (idou[nx][ny - 1] == 0) {  //移動先が空いていれば
				ny = ny - 1; playerphoto = gh[4];  // 上向き
			}
			else {
				playerphoto = gh[9];  // 移動不可能
			}
		}
		if (Key[KEY_INPUT_DOWN] == 1) {
			move = 1;
			if (idou[nx][ny + 1] == 0) {  //移動先が空いていれば
				ny = ny + 1; playerphoto = gh[3];  // 下向き
			}
			else {
				playerphoto = gh[9];  // 移動不可能
			}
		}

		ClearDrawScreen();  // 裏画面をクリア
		if (move > 0 && ++move == 20) {
			move = 0; playerphoto = gh[5];  // 移動終了
		}
		int x = nx * 60 + 100, y = ny * 80 + 100;  // 表示位置に変換
		DrawGraph(x, y, playerphoto, FALSE);  // プレイヤーを裏画面に描画
		ScreenFlip();  // 裏画面を表画面に反映
	}

	DxLib_End();  // ＤＸライブラリ使用の終了処理
	return 0;  // ソフトの終了 
}

プレイヤーを描画する前に壁を描画してもよいでしょう。

		for (int i = 0; i < 5; i++) {
			for (int j = 0; j < 5; j++) {
				if (idou[i][j] == 1) {
					int x = j * 60 + 100, y = i * 80 + 100;
					DrawGraph(x, y, gh[11], FALSE);  // 壁を裏画面に描画
				}
			}
		}
		int x = nx * 60 + 100, y = ny * 80 + 100;  // 表示位置に変換
		DrawGraph(x, y, playerphoto, FALSE);  // プレイヤーを裏画面に描画

Comments

[carnage0216]

どうもありがとうございます。
https://teratail.com/questions/207686?modal=q-compにも答えて頂けると!

Answer 2

私だったらこんな感じに分けますね

int dx = 0, dy = 0;
int nextphoto; // intなのかは知らんがplayerphotoと同じ型

// キー判定部分
if (Key[KEY_INPUT_RIGHT] == 1)
{
    // カーソルキーの右が押されている
    dx = 1;
    nextphoto = gh[7];
} else if(Key[KEY_INPUT_LEFT] == 1)
{
    // カーソルキーの左が押されている
    dx = -1;
}
// 省略

// 実際の判定および移動部分
if (
    (dx != 0 || dy != 0) &&
    nx + dx >= 0 && nx + dx < 5 &&
    ny + dy >= 0 && ny + dy < 5 &&
    idou[nx+dx][ny+dy] == 0
   )
{
    nx += dx;
    ny += dy;
    playerphoto = nextphoto;
}

Comments

[carnage0216]

いつもどうもありがとうございます。

Answer 3

ご説明が無茶苦茶です.
右移動部分だけですが

  if(idou[nx + 1][ny + 0] == 0) { //移動しようとする先が空いていれば
    //移動可能
    nx = nx + 1; //移動
    playerphoto = gh[7];
  }
  else {
    // 移動不可能
    playerphoto = gh[9];
  }

これだけです.

Comments

[episteme]

× idou[nx + 1][ny + 0] == 0
〇 idou[(nx+1) + 1][ny + 1] == 0　// 目的地(x+1,y)にそれぞれ+1のゲタ

ですねー

[fana]

・「Java」になってます．コードのところ．
・+1のゲタは要らないような？
・idou[こっちがxで][こっちがy]なことに関してはＯＫなのか否か謎

[episteme]

- 移動可能領域の左上隅を(1,1)とするならゲタは要らんすね。
- Cでは(メモリ・レイアウト上は) idou[y][x] ですしねー

[jimbe]

>「Java」になってます
間違えました. ご指摘ありがとうございます.

> 目的地(x+1,y)にそれぞれ+1のゲタ
何故でしょうか.

[episteme]

再度: -移動可能領域の左上隅を(1,1)とするならゲタは要らんすね。

[carnage0216]

お二方、解答ありがとうございます。
jimbeさんのプログラムを使ったものを載せさせて頂きます。

[carnage0216]

どうもありがとうございました。

Answer 4

配列要素を見て移動の可否を決めることに関しては既に存在する他の方の回答で十分に思いますが，
２次元配列のindexに関して不安に思うので指摘(?)しておきます．

例えば

int A[2][3] = {
  {  0,  2,  0 },
  { 10,  0,  0 }
};

という配列があるとき，値が2の要素はA[0][1]で，値が10の要素がA[1][0]です．
２次元的な並びで示せば

A[0][0], A[0][1], A[0][2]
A[1][0], A[1][1], A[1][2]

という感じです．
indexは
A[下に行くほどこっちが増える][右に行くほどこっちが増える]
というルール．

あなたのコードの【配列idou[][]に用いるindex】と【座標値(nx,ny)を考えている座標系(軸の向き)】との対応関係が
あなたの意図した関係になっているか？に関しても確認されると良いのではないかと思います．