二次元配列の書き込みについて

Question

はじめに

C/C++の外部ライブラリなしでゲームを作っているものです。(YouTubeの動画を参考に(パズ◯ラ))
初心者でどこがどのように間違っているのかわからないので質問させていただきます・・・

困っているところ

変数への書き込みについて困っています。
こちらが書いているコードです↓

#include <cstdio>
#include <cstdlib>
#include <conio.h>
#include <iostream>
#include <ctime>
using namespace std;

#define BOARD_WIDTH 6
#define BOARD_HEIGHT 5

enum {
	DROP_NONE,	
	DROP_FIRE,	// 火
	DROP_WATER,	// 水
	DROP_WOOD,	// 木
	DROP_LIGHT,	// 光
	DROP_DARK,	// 闇
	DROP_MAX
};

enum {
	DROP_STATUS_DEFAULT,
	DROP_STATUS_HOLD,
	DROP_STATUS_MAX
};

typedef struct {
	int x, y;
}VEC2;
char bropAA[DROP_MAX][DROP_STATUS_MAX][2 + 1] = {
	{"  ","  "},	// DROP_NONE,
	{"▽","▼"},	// DROP_FIRE,	// 火
	{"△","▲"},	// DROP_WATER,	// 水
	{"◇","◆"},	// DROP_WOOD,	// 木
	{"◯","●"},	// DROP_LIGHT,	// 光
	{"☆","★"}	// DROP_DARK,	// 闇
};

int board[BOARD_HEIGHT][BOARD_WIDTH];

VEC2 cursorPos = {};

bool holding = false;

int main() {
	srand((unsigned int)time(NULL));

	for (int y = 0; y < BOARD_HEIGHT; y++) 
		for (int x = 0; x < BOARD_WIDTH; x++) 
			board[y][x] = 1 + rand() % (DROP_MAX - 1);
	
	while (true) {
		system("cls");
		for (int y = 0; y < BOARD_HEIGHT; y++) {
			for (int x = 0; x < BOARD_WIDTH; x++) {
				int status = DROP_STATUS_DEFAULT;
				if(holding && (x == cursorPos.x) && (y == cursorPos.y))
					status = DROP_STATUS_HOLD;
				cout << bropAA[board[y][x]][status];
			}

			if (y == cursorPos.y)
				cout << "←";
			cout << endl;
		}

		for (int x = 0; x < BOARD_WIDTH; x++)
			if (x == cursorPos.x)
				cout << "↑";
			else
				cout << "  ";

		VEC2 lastCursorPos = cursorPos;
		switch (_getch()) {
		case 'w':cursorPos.y--;	break;
		case 's':cursorPos.y++;	break;
		case 'a':cursorPos.x--;	break;
		case 'd':cursorPos.x++;	break;
		default:
			holding = !holding;
			break;
		}

		if (holding)
			int temp = board[cursorPos.y][cursorPos.x];
			board[cursorPos.y][cursorPos.x] = board[lastCursorPos.y][lastCursorPos.x];
			board[lastCursorPos.y][lastCursorPos.x] = temp;
	}
}

このコードの中で困っているのは最後の方の

if (holding)
			int temp = board[cursorPos.y][cursorPos.x];
			board[cursorPos.y][cursorPos.x] = board[lastCursorPos.y][lastCursorPos.x];
			board[lastCursorPos.y][lastCursorPos.x] = temp;

の部分です。
int temp にboard[省略][省略]を書き込もうとしてもtempが定義されていませんとエラーが出ます。
参考にしている動画ではコンパイルが通っています。

環境

OS：Windows 10 Home 64bit
コンパイラ：Visual Studio 2022 Preview
使用言語：C/C++

さいごに

こんなことで質問してしまってすみません・・・まだ初心者なもので・・・
一応コードは理解してるつもりなのですが・・・

回答してくださる心優しい方を待ってます・・・

追記

エラーメッセージの詳細を忘れていました・・・
エラーメッセージは以下です。

重大度レベル コード 説明 プロジェクト ファイル 行 抑制状態

エラー (アクティブ) E0020 識別子 "temp" が定義されていません Puzzle and Dragons C:\Users\user\Documents\Programming\C - C++\Puzzle and Dragons\Puzzle and Dragons\source.cpp 87

Answer 1

(YouTubeの動画を参考に(パズ◯ラ))

それよりも先に基礎からやりましょう。
今やっていることは、料理で例えると、「レシピ通りには作らずに、オリジナリティを出そうとして、余計なものを入れまくっている状態」です。

そりゃ、まともなものは作れませんし、仮に出来たとしても、たまたまうまくいっただけです。

それと、「エラーメッセージぐらいは読みましょう」。
エラーメッセージにヒントが隠されています。

int temp にboard[省略][省略]を書き込もうとしてもtempが定義されていませんとエラーが出ます。

(最初の質問)

だと仮定します。

まず、エラーメッセージに行番号が書かれているはずなのでそこを見ましょう。

メッセージから考えると、

        if (holding)
            int temp = board[cursorPos.y][cursorPos.x];
            board[cursorPos.y][cursorPos.x] = board[lastCursorPos.y][lastCursorPos.x];
            board[lastCursorPos.y][lastCursorPos.x] = temp;

のようですね。

……えーっと、とりあえず「C言語 if文」とかで検索してみてください。
(修正: なぜかfor文になってた…)

C/C++でのfor文とかは、"{}" でくくったものが一つのブロックと見なされます。

"{}"を省略すると一行が対象になるのです。

他の方の回答のコメント部にあるリンク先には、

そもそも｛｝は複文を書くための手段なのであって，それが無い状態が「省略形」なのではない．

と書かれていますね。(発想が逆だったらしい)

(打消し文での)「一行」が紛らわしいとご指摘があったので、ここも修正します。
一行というよりは1ステートメントってところでしょうか。

つまり、"{}"を省略せずに書くと、

        if (holding){
            int temp = board[cursorPos.y][cursorPos.x];
        }
            board[cursorPos.y][cursorPos.x] = board[lastCursorPos.y][lastCursorPos.x];
            board[lastCursorPos.y][lastCursorPos.x] = temp;

という感じでしょうか。

C/C++では Pythonみたいにインデントでブロックを表現する…わけじゃないので、"{}"で一つの対象範囲とします。

次に、コードを読みましょう。

コードを読むコツは『一行レベルで、何をしているかを考えながら読む』です。

今回のコードは長いので問題の部分だけを読みます。

        // holding が trueなら(?)
        if (holding){
            // temp に カーソルがある場所のボードの位置の値を設定
            int temp = board[cursorPos.y][cursorPos.x];
        }
        // 前回のカーソルの場所にする?
        board[cursorPos.y][cursorPos.x] = board[lastCursorPos.y][lastCursorPos.x];
        // 前回のカーソルの場所をtempの値にする
        board[lastCursorPos.y][lastCursorPos.x] = temp;

これを「疑似コード」にする。

1. holding が trueなら(?)
    1.1. temp に カーソルがある場所のボードの位置の値を設定
2. 前回のカーソルの場所にする?
3. 前回のカーソルの場所をtempの値に変更

temp のスコープを考えてください。

(1.1)しか対象範囲にありません。
そうなると、(3)でtempの値を使おうとしていますが、「おいおい、そんなもん、ねーぞ」と言われますよね。

つまり基礎が出来ていない事が原因。

私なら、(2), (3) も (1)の対象範囲として"{}" の中に書くか、
別途、変数を用意して、if文の外に置いて使う。

---

[追記1]

失礼ですが、「参考に」というのが誤解を生んでしまいました、参考というよりかは、実際の作り方を見てこの場合はこうやる、この関数を使うと何々ができる、というのを学ぶのがが目的でした。

基礎はできています。

あ、そっちですか。それなら上での発言は言い過ぎましたね。
初心者~中級者ぐらいの人はそういう勉強方法もありっちゃアリですが、やっぱり良くないと思います。

関数とかの使い方とかは公式リファレンスを参考にすべきです。
とはいえ、C言語やC++の公式リファレンスってのは無いようなので、有志が作ったものになりますが。
たとえば cppjpとかがありますね。

for文の後ろに{}をつけるのは常識だと思っていました。

常識っていうか、これがデフォです。一行だけの場合、ブロックを明示しなくてもわかるので、省略がされがちなだけです。

ですが、その動画ではfor文に{}を付けずにPythonのようにインデントでやっていました。

インデントっていうか、それっぽく見えただけでは?

その動画は見ていませんが、おそらく、コンパイルはしていないのでは? (動画内で)
その手の動画を見るのなら、ざっくりと見るか、コンパイル(ビルド)しているのも含まれている前提ですね。私なら。

if文も、命令文が一つならインデントでできますが今回の場合、命令文が１つ以上なのに{}を付けなかったのが原因ですよね。

つまり、BeatStarさんの{}を省略して書くと・・・の場合、tempの定義はできますがあとの２つは
スコープ外になってエラーになるということですね。

YES. 一応、

int a = 1;

int tmp = 2;
if( a == 1 ) tmp = 10;

int b = a * tmp;

とかみたいな感じでブロックの外に変数宣言を出して……であれば可能です。

Comments

[gift-0618]

失礼ですが、「参考に」というのが誤解を生んでしまいました、参考というよりかは、実際の作り方を見てこの場合はこうやる、この関数を使うと何々ができる、というのを学ぶのがが目的でした。
基礎はできています。
for文の後ろに{}をつけるのは常識だと思っていました。
ですが、その動画ではfor文に{}を付けずにPythonのようにインデントでやっていました。
if文も、命令文が一つならインデントでできますが今回の場合、命令文が１つ以上なのに{}を付けなかったのが原因ですよね。
つまり、BeatStarさんの{}を省略して書くと・・・の場合、tempの定義はできますがあとの２つは
スコープ外になってエラーになるということですね。

[gift-0618]

あともう一つ、誤字ってましたね・・・すみません。
「int temp にboard[省略][省略]を書き込もうとしてもtempが定義されていませんとエラーが出ます。」
ではなく、boardにtempを書き込むでした（エラーが出るのは）

[BeatStar]

本文に書きますね。

[fana]

（互いに話が通じているようなので余計かとは思いますが）
細かい文言が気になります．

> "{}"を省略すると一行が対象になるのです。

（これは他の方の回答のコメント欄にリンクが示されていますが）括弧を書かない状態が「省略系」なのではありません．
また，「一行」と言うと語弊があるのではないでしょうか（C/C++はマルチステートメント可なのですし）．

> インデントでできますが

インデントには何の効力もありません．

[fana]

あと，至極どーでもいいことなのですが，

> とりあえず「C言語 for文」とかで検索してみてください

これ，何で for だったのでしょう？
（問題のコードは if なのに． ifだと検索結果がイマイチとかそういう理由？）

[BeatStar]

@ fanaさん
確かにそうですね。(ざっと読んだだけだったみたいです…)
修正します。

Answer 2

-if (holding)
+if (holding) {
            int temp = board[cursorPos.y][cursorPos.x];
            board[cursorPos.y][cursorPos.x] = board[lastCursorPos.y][lastCursorPos.x];
            board[lastCursorPos.y][lastCursorPos.x] = temp;
+        }

Comments

[gift-0618]

ありがとうございます！

if文には波括弧が必要なのでしょうか？

[thkana]

たまたま他の質問があったので。参考になるかと思います。
https://teratail.com/questions/360649

[gift-0618]

thkanaさん、ありがとうございます！