2Dゲームにおける左向きに攻撃する処理の方法

Question

概要
現在、右向きに進んでいく2Dのスクロール系のゲームを作成しております。
ゲームの主人公(コード内におけるheroという名前です)が左向きに攻撃する処理は描けているのですが、右向きに攻撃をする処理をうまく描くことができず、質問をさせていただきました。
以下は対象のコードです。
コード

void DrawBall(int key) {
	// 発射処理
    // 右向き 
	if (IsBKeyTrigger(key) == TRUE && g_stagedata.hero.turn == FALSE) {
		// 空きを探す
		int i;
		for (i = 0; i < MAX_BALL; i++) {
			if (g_stagedata.balls[i].living == FALSE) break;
		}
		if (i < MAX_BALL) {
			g_stagedata.balls[i].living = TRUE;
			g_stagedata.balls[i].x = g_stagedata.hero.x + IMG_CHIPSIZE;
			g_stagedata.balls[i].y = g_stagedata.hero.y;
		}
        // 左向き
	}else if (IsBKeyTrigger(key) == TRUE && g_stagedata.hero.turn == TRUE) {
		// 空きを探す
		int i;
		for (i = 0; i < MAX_BALL; i++) {
			if (g_stagedata.balls[i].living == FALSE) break;
		}
		if (i < MAX_BALL) {
			g_stagedata.balls[i].living = TRUE;
			g_stagedata.balls[i].x = g_stagedata.hero.x - IMG_CHIPSIZE;
			g_stagedata.balls[i].y = g_stagedata.hero.y;
		}
	}
	// ボールの描画と移動
	float mv = 350.0f * g_frametime; // 移動量計算
		for (int i = 0; i < MAX_BALL; i++) {
			if (g_stagedata.balls[i].living == FALSE) continue;
			g_stagedata.balls[i].x += mv;

			// マップ当たり判定
			AtariInfo atari = CheckBlock(g_stagedata.balls[i].x, g_stagedata.balls[i].y,
				g_stagedata.balls[i].x);
			if (atari.DR == TRUE || atari.UR == TRUE) g_stagedata.balls[i].living = FALSE;
			// 画面外に出ても消滅
			if (g_stagedata.balls[i].x > g_stagedata.scrollx + 1000) {
				g_stagedata.balls[i].living = FALSE;
			}

			DrawGraph((int)(g_stagedata.balls[i].x - g_stagedata.scrollx),
				(int)g_stagedata.balls[i].y,
				g_imghandles.ball, TRUE);
		}
	 if (g_stagedata.hero.turn == TRUE) {
		for (int i = 0; i < MAX_BALL; i++) {
			if (g_stagedata.balls[i].living == FALSE) continue;
			g_stagedata.balls[i].x -= mv;

			// マップ当たり判定
			AtariInfo atari = CheckBlock(g_stagedata.balls[i].x, g_stagedata.balls[i].y,
				g_stagedata.balls[i].x);
			if (atari.DR == TRUE || atari.UR == TRUE) g_stagedata.balls[i].living = FALSE;
			// 画面外に出ても消滅
			if (g_stagedata.balls[i].x > g_stagedata.scrollx - 1000) {
				g_stagedata.balls[i].living = FALSE;
			}

			DrawGraph((int)(g_stagedata.balls[i].x - g_stagedata.scrollx),
				(int)g_stagedata.balls[i].y,
				g_imghandles.ball, TRUE);
		}
	 }
}

// キートリガー処理
BOOL IsBKeyTrigger(int key) {
	if (key & PAD_INPUT_B) {
		if (g_stagedata.g_bkey_prev == FALSE) {
			g_stagedata.g_bkey_prev = TRUE;
			return TRUE;
		}
	}
	else {
		g_stagedata.g_bkey_prev = FALSE;
	}
	return FALSE;
}

発生している問題
右向き(g_stagedata.hero.turn==FALSE)に攻撃（ボール）を飛ばすことはできているのですが、左(g_stagedata.hero.turn==TRUE)を向いたときに左にボールを飛ばすことができません。また、右向きでボールを飛ばした後に、左を向いた場合、右向きに飛ばしたボールが消えてしまう現象も発生しております。

解決するために考えている方法
現在、自分の考えている解決方法として蛇足的ではありますが、void DrawBall()関数内の処理を左向きと右向きで完全に分けて記述する方法で、解決できるのではないかと考えております。
もし、上記のコードの処理において、左向きに攻撃する（ボールを飛ばす）かつ、一度右向きに飛ばしたボールを消させない方法がわかる方がいらっしゃいましたら、アドバイスをいただけると幸いです。

補足
主人公＝hero
攻撃＝ball(ボール)
向き＝turn（FALSE：右向き、TRUE＝左向き）

Answer 1

右と左で計算式の一部値の符号しか変わらないので、それを返す関数を一つ作るだけで、左右別の条件を作る必要が無くなります。
また、ボールの移動方向は「発射時の hero の turn 」のみで決まるのは大前提ですが、turn は mv の符号にしか影響しませんので、最初から mv を balls に作っておいて発射時に mv / -mv を入れてしまえば、移動時に向きを見る必要も無くなります。
(ついでに mv の計算も発射時に毎回する必要は無さそうに思います。→ 発射ボタンによっては早い/遅いボールを撃てる?)
(さらについでに、 "DrawBall" という名前の関数で "移動" から "発射処理" まで行うのは変な気がします。)

※コンパイルも何も出来ないので切り貼りしたイメージコードです。コンパイルエラーその他が発生する可能性があります。

//turn==TRUE(左向き) なら -v, そうで無いなら v を返す
int GetTurnedValue(BOOL turn, int v) {
  return turn == TRUE ? -v : v;
}

void DrawBall(int key) {
  // 発射処理
  if (IsBKeyTrigger(key) == TRUE) {
    for (int i = 0; i < MAX_BALL; i++) {
      if (g_stagedata.balls[i].living == FALSE) { // 空き
        g_stagedata.balls[i].living = TRUE;
        g_stagedata.balls[i].x = g_stagedata.hero.x + GetTurnedValue(g_stagedata.hero.turn, IMG_CHIPSIZE);
        g_stagedata.balls[i].y = g_stagedata.hero.y;

        float mv = 350.0f * g_frametime; // 移動量計算
        g_stagedata.balls[i].mv = GetTurnedValue(g_stagedata.hero.turn, mv); //構造体? に mv メンバ追加
        break;
      }
    }
  }

  // ボールの移動・描画
  for (int i = 0; i < MAX_BALL; i++) {
    if (g_stagedata.balls[i].living == FALSE) continue;

    g_stagedata.balls[i].x += g_stagedata.balls[i].mv;

    // 画面外に出たら消滅
    if (g_stagedata.balls[i].x > g_stagedata.scrollx + 1000 ||
      g_stagedata.balls[i].x < g_stagedata.scrollx - 1000) {
      g_stagedata.balls[i].living = FALSE;
      continue;
    } else {
      // マップ当たり判定
      AtariInfo atari = CheckBlock(g_stagedata.balls[i].x, g_stagedata.balls[i].y, g_stagedata.balls[i].x);
      if (atari.DR == TRUE || atari.UR == TRUE) {
        g_stagedata.balls[i].living = FALSE;
        continue;
      }
    }

    DrawGraph((int)(g_stagedata.balls[i].x - g_stagedata.scrollx), (int)g_stagedata.balls[i].y, g_imghandles.ball, TRUE);
  }
}

// キートリガー処理
BOOL IsBKeyTrigger(int key) {
  if (key & PAD_INPUT_B) {
    if (g_stagedata.g_bkey_prev == FALSE) {
      g_stagedata.g_bkey_prev = TRUE;
      return TRUE;
    }
  } else {
    g_stagedata.g_bkey_prev = FALSE;
  }
  return FALSE;
}

Comments

[maguro2020]

ご回答いただきありがとうございます。
確かに、DrawBall関数内で"移動"と"発射処理"を同時に行うのは違和感がありました。
jimbeさんのコードも参考にしながら、自分のコードを改良したいと思います。
改めてご回答してくださり、ありがとうございました。

Answer 2

まず、下記のif文の範囲が意図とあっていないと思います。

	// ボールの描画と移動
	float mv = 350.0f * g_frametime; // 移動量計算
		for (int i = 0; i < MAX_BALL; i++) {
 		 		:
 		}
	 if (g_stagedata.hero.turn == TRUE) { //←ここ
		for (int i = 0; i < MAX_BALL; i++) {
 		 		:
 		}
	}

そして、プレイヤーの左右の向きで、ボールの動きを制御しようとしていますが、プレイヤーの向きとボールの動きが無関係だと思うので、ロジック的におかしいです。g_stagedata.ballsの構造体に向きのメンバ変数(turn)を持たせてそれでボールの動きを制御すべきです。

ifでの分岐では全体に右向きと左向きで似たようなコードが出現するので、ボールのメンバ変数(turn)でコードをまとめるとよいかと思います。

Comments

[maguro2020]

ご回答いただきありがとうございます。
現在、ボールの向きと主人公の向きが同時に動いてしまう（ボールを投げた後に向きを変えた場合、ボールの向きも変わる現象です）、別の問題が発生していたため、とても助かります。もう少し自分でもボールの発射方法について、変更を加えてみたいと思います。

Answer 3

自己解決しました。
ありがとうございました。

Comments

[fiveHundred]

具体的な解決手順が書かれていないので、低評価します。

[jimbe]

[4-2. どのように解決したのかを伝えましょう]
https://teratail.com/help/question-tips#questionTips42

>あなたがどのように解決したかを書き残しておくことで、あなたの質問は貴重なコンテンツ資産となります。
>いつか他の誰かがあなたと同じ問題に陥った時、そのコンテンツにたどり着いたらすぐに解決できるように情報を残しておいてあげてください