for文とif文の複合

Question

Q.　5人の点数(0点～100点)をランダムに表示させた後、高い順に並べ替える。

例）
1番目 = ×点
2番目 = ×点
3番目 = ×点　　　A
4番目 = ×点
5番目 = ×点

↓

3番目 = ×点　
2番目 = ×点
5番目 = ×点　　　B
4番目 = ×点
1番目 = ×点

A
↓
B
をコンソール画面で表示させたいです。

下記コードですと”A”しか表示されないです。
足りないコードを教えて下さい。

#include <time.h>

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


  int test[5];

  for (int i = 0; i < 5; i++) 
  {
    test[i] = rand() % (100 + 1);
    printf("%d番目 = %d点\n", i + 1, test[i]);

    for (int j = i + 1; j < 5; j++) 
    {
      if (test[i] < test[j])
      {
	　　　int date = test[i]; 
	　　　test[i] = test[j]; 
	　　　test[j] = date; 
      }
    }
  }

  // キー入力待ち
  rewind(stdin);
  _getch();

  return 0;

}

Comments

[y_waiwai]

提示のコードではどういう不具合があるんでしょうか

[f.Shou]

不具合はないです。
AとBを表示させたいのにAしか表示されないのです。

Answer 1

まずは「1番目～5番目」のデータを生成して、それから並べ替えるほうがいいのではないかと思います。

乱数を生成しながら並べ替えようとしているために、何をやっているのかわからないことになりますし、test[0]しか値を書き込んでいないのにtest[1]以降にアクセスするという、やってはいけないことをやっているコードとなっています。

Comments

[f.Shou]

わかりましたありがとうございます。

Answer 2

以下の処理になるように作りなおしてください。

1. ランダムの値でtest配列を初期化
2. testの値を表示
3. testをソート
4. testの値を表示

今の作りは上記をいっぺんにやろうとしています。
.

Comments

[f.Shou]

それのコード化ができなくて止まっています、、、

[ttyp03]

落ち着いて順序良くやればできるでしょ。
それくらいの力量はありそうなコードですが。
もっとも自力で書いていればの話ですが。
今のコードを分解して再構成すればいいのですよ。

[maisumakun]

「5つの値の生成と表示が済んでから並べ替えを行う」というように、順を追ってやるようにしてみましょう。

「表示しながら並べ替えを行う」ようなこともやれなくはないですが、バラバラにやるのと比べて考えることが増えます。

[f.Shou]

そうなのですね、、、わかりましたありがとうございます。