キャラクターの現在位置の値が入りません。

Question

###前提・実現したいこと
将来的にゲームを作りたいと思っています。
そのため、まずC言語でキーボード入力でキャラクターの座標を動かすプログラムを作成しようと思いました。
###発生している問題・エラーメッセージ
数字の0,1,2,それ以外で上下左右の方向に移動するように割り当てたのですが、どの数字をキーボードで入力してもxy座標が(75,-25)になってしまい、なおかつ変わりません。

ここで行き詰ってしまったので、ぜひ皆さんの知恵を貸してください!

###ソースコード

#define _CRT_SECURE_NO_WARNINGS
#include <stdio.h>
#include <stdlib.h>
#include <time.h>
#define _USE_MATH_DEFINES
#include <math.h>

typedef struct car {
	int x;	//現在位置x
	int y;	//現在位置y
	double max_x;	//xの最大値
	double max_y;	//yの最大値
	int L = 50;
}car;

int move(car car, int a, int area);

int main() {
	FILE *file;
	file = fopen("car.csv", "w");
	car car;

	int num_step;	//ステップ数
	int j;
	int a, sd;
	int area;

	int reward;

	int num_a;	//行動
	int num_s;	//状態


	car.max_x = 250;
	car.max_y = 250;
	num_a = 4;

	//エリアの数
	num_s = 25;

	//初期設定
	car.x = 25;
	car.y = 25;
	area = car.x / 50 + car.y / 50 * 5;

		for (j = 0; j < 10; j++) {
			fprintf(file, "j=%d\n", j);

			scanf("%d", &a);
			sd = move(car, a, area);

			//ここで障害物の設定
			//障害物があるareaに入るとrewardに-10が入り試行を終了する
			//Goalに入るとrewardに+10が入り試行を終了
			//その他はreward=0なのでそのまま続行
			if (area = 24) {
				reward = 10;
			}
			else if (area = 5 || 8 || 13 || 22) {
				reward = -10;
			}
			else {
				reward = 0;
			}
			//もしxとyが環境の範囲からでたらreward=-1を入れる
			if (car.x < 0 || car.max_x < car.x || car.y < 0 || car.max_y < car.y) {
				reward = -1;
			}

			if (reward < 0) {
				//失敗
				car.x = 0;
				car.y = 0;
				area = car.x / 50 + car.y / 50 * 5;
				break;
			}
			else if (reward < 0) {

				//成功
				car.x = 0;
				car.y = 0;
				area = car.x / 50 + car.y / 50 * 5;
				break;
			}
			else {
				//続行
				area = sd;
			}fprintf(file, "%d,%d,%d,%d,%d\n", car.x, car.y, a, sd, area);
	}
	fclose(file);
	return 0;
}

int move(car car, int a,int area) {
	if (a = 0) {
		car.y = car.y + 50;
	}
	if (a = 1) {
		car.x = car.x + 50;
	}
	if (a = 2) {
		car.y = car.y - 50;
	}
	else {
		car.x = car.x - 50;
	}

	printf("%d,%d\n", car.x, car.y);
	int sd;
	//sdはアクションが起こった後のareaを表す
	sd = car.x / 50 + car.y / 50 * 5;

	return area;
}

###補足情報(言語/FW/ツール等のバージョンなど)

Answer 1

一つの原因は、関数 int move(car car, int a,int area) 内の比較式が誤っていることが原因と思います。

int move(car car, int a,int area) {
    if (a = 0) {
        car.y = car.y + 50;
    }
    if (a = 1) {
        car.x = car.x + 50;
    }
    if (a = 2) {
        car.y = car.y - 50;
    }
    else {
        car.x = car.x - 50;
    }

おそらく使用したい演算子は代入演算子の = ではなく、比較演算子は == ではないですか？

ex.)
if (a == 0) {

---

この場合、比較式ではなく記述された代入式の値が０かどうかで偽か真かが決まります。

if (a = 0) の式 a = 0 の結果は 0 ですので、条件式は常に成立しません。
if (a = 1) の式 a = 1 の結果は 1 ですので、条件式は常に成立します。
if (a = 2) の式 a = 2 の結果は 2 ですので、条件式は常に成立します。

つまり move は a の値にかかわらず、呼び出されるたびに

car.x = car.x + 50;
car.y = car.y - 50;

が実行されてしまっています

---

その後、追記

他も見てみると、if 文の比較式の記述が、かなり誤っているみたいです。

        if (area = 24) {    ←　常に真になる　　　

　　　　　↓
if (area == 24) {

        else if (area = 5 || 8 || 13 || 22) { 　 ←　常に真になる

　　　　　↓
else if ((area == 5) || (area == 8) || (area ==13) || (area ==22)) {
自分なら switch(area) を使いますけど…

Comments

[mikamikan]

回答ありがとうございます!
=の使い方、思いっきり間違っていたので勉強になりました。
本当にありがとうございました!

Answer 2

=についてはT.Kannoさんのご指摘通りです。初心者は間違えやすいので気をつけましょう。

もう一つ重要な問題点があります。
int move(car car, int a, int area)
構造体の変数が「値渡し」になっています。これではmove関数内でcar変数を変更しても、呼び出し元には反映されません。

この場合、C言語では「ポインタ渡し」をする必要があり、このように書きます。
int move(car *car, int a, int area)

呼び出し側は
sd = move(&car, a, area);
このように変数名に&を付けることでポインタが渡されるようになります。

move関数内でcar変数の値を取得したり設定したりする際は、
car->y = car->y + 50;
このように.ではなく->でメンバ変数をつなげます。

それと、構造体名とtypedefによる別名、そして変数名をどれも同じ名前にしているので判りづらくなっています。区別が付くような名前を付けた方がいいでしょう。

追加でもう一点。
move関数のreturnがareaとなっていますが、sdの間違いでは？

Comments

[mikamikan]

回答ありがとうございます!
値が反映されない謎は値渡しをしていたからだったんですね。
とても参考になりました。
ありがとうございました!