射的ゲーム 得点をランキング画面に反映させたい

Question

前提

DXライブラリを使って射撃ゲームを作成しています。
C++初心者です。

実現したいこと

スコアをランキング画面に反映させたい

発生している問題・エラーメッセージ

配列の中の全部の要素にスコアが反映されてしまいます。

該当のソースコード（一部抜粋）

void Ranking_Draw() {
	int rankX = 50, rankY = 150;　  // 〇位の座標
	int scoreX = 70, scoreY = 150;  // スコアの座標
	// スコアランキング　１位から５位まで表示
	static int rank[5] = {};
	

	if (rank[0] < score) {
		rank[0] = score;
	}
	else if (rank[1] < score) {
		rank[1] = score;
	}
	else if (rank[2] < score) {
		rank[2] = score;
	}
	else if (rank[3] < score) {
		rank[3] = score;
	}
	else if (rank[4] < score) {
		rank[4] = score;
	}

	// バブルソート（降順）
	int i, j, buf;

	for (i = 0; i < 5; i++)
	{
		for (j = 0; j < 4; j++)
		{
			if (rank[j] < rank[j + 1])
			{
				buf = rank[j];
				rank[j] = rank[j + 1];
				rank[j + 1] = buf;
			}
		}
	}

	// ランキング表示
	int x = 1;
	SetFontSize(10);
	DrawString(200, 110, "ランキング画面です。", GetColor(253, 222, 165));
	for (int i = 0; i < 5; i++) {
		DrawFormatString(rankX, rankY, GetColor(253, 222, 165), "%d", x);
		DrawFormatString(scoreX, scoreY, GetColor(255, 255, 255), "%d", rank[i]);
		rankY += 30;
		scoreY += 30;
		x += 1;
	}

	DrawString(0, 450, "Bキーを押すとメニュー画面に戻ります。", GetColor(255, 255, 255));


}

Answer 1

Ranking_Draw() なる関数名から推測するに，この関数はランキングの描画が必要なときに毎度呼ばれるのではないでしょうか．

で，この関数には　データ更新処理　と　描画処理　が混在している様子ですから，この関数を呼ぶたびにデータ更新処理も走ってしまうことになります．

配列の中の全部の要素にスコアが反映されてしまいます

というのは，そのような理由によって発生しているものと推測します．
この関数が繰り返し呼ばれた際にデータがどのように変わっていくことになるのか？　を考えてみると良いでしょう．

---

「描画のための関数」であれば描画のみを行うべきであり，データの更新処理までをも行うべきではありません．
データ更新処理　と　描画処理　は個別の関数に分け，それぞれを必要なときにのみ呼ぶようにすると良いのではないでしょうか．

Comments

[kinakomoti-_-]

コメントありがとうございます。試みてみます

[kinakomoti-_-]

データ更新処理と描画処理に分けてみたんですが、直りませんでした

[fana]

実際に何をどうしたらどうなったのか，こちらには不明ですから，「頑張って原因究明作業とかをしてください」としか言えませんが…

> 必要なときにのみ呼ぶようにする

ということがしっかりとできているのですか？

---

質問文に提示されたコードの 8 行目からの if ～ else if ～ ... について言えば，
配列 rank[] の要素群のうち，ここを１度だけ実行した際に値が変化し得る要素というのは１個だけ（：複数個が変化することはない）ですよね．
そしたら，rank[] の全要素の値が変化するためには，少なくともここの処理が５回は走る必要がある，ということ．
なので，ここの処理を１回しか実施しない（＝必要なときのみ実施する）ならば，「全要素に反映される」という現象は起こらないハズ．

（この if～else if～... のところの実装が「ランキングデータの更新として意味的に正しいかどうか」というのは別の問題なので置いとくとして）

[fana]

念のため明言しておくけど，

> 「頑張って原因究明作業とかをしてください」

これはガチなアドバイスですぞ．
平たく言えば「デバッグしてください」とかになる．
なにはともあれ，「実際に何がどう動いた結果として観測した状況になっているのか？」というところをあなた自身が掴まないと，「何をどうすれば良いのか？」といった方針等を考えることもできないよ．

[kinakomoti-_-]

わかりました。ありがとうございます！頑張って原因究明作業します

[kinakomoti-_-]

原因究明作業をした結果、ここの処理が終了するまで実施されていました。
一回だけ実施したいって時はフラグで制御した方がいいですかね？

[fana]

どういう方法が良いのか？　に関しては，あなたのプログラムの作り等々に依るでしょうから，そういう制御が適しているのか否かも他者には不明です．

が，「スコアのランキング」という言葉から想像するに，更新すべきタイミングというのは「ある１回のゲームプレイの最終スコア値が定まった時点」なのではないかと推測しますので，そのようなタイミング（ゲームクリアしたときやゲームオーバーになったとき？）に実施すれば良いだけの話なのかもしれません．