入力した数値の最大値、２番目に大きい数値、３番目に大きい数値

Question

１０個の整数を入力して、１番大きい数値、２番目に大きい数値、３番目に大きい数値を表示したいのですがうまくいきません。
３番目の数値が０になってしまいます。
改善の仕方を教えていただければ幸いです。

該当のソースコード

#include <stdio.h>

int main(void)
{
	int number[10];
	int no_1 = 0;
	int no_2 = 0;
	int no_3 = 0;
	int i;

	printf("10個の整数を入力\n");
	for (i = 0; i<10; i++) {
		scanf("%d", &number[i]);

		if (no_1 < number[i]) {
			no_2 = no_1;
			no_1 = number[i];
		}
		else if (no_2 < number[i]) {
			no_3 = no_2;
			no_2 = number[i];
		}
	}

	printf("1番目に大きい数字は%dです。\n", no_1);
	printf("2番目に大きい数字は%dです。\n", no_2);
	printf("3番目に大きい数字は%dです。\n", no_3);
	return 0;
}

Answer 1

あえてモダンなC++で書いてみると

#include <iostream>
#include <algorithm>

int main()
{
  int v[10];
  int top3[3];
  
  for(int& it : v)
    std::cin >> it;

  using std::begin;
  using std::end;
  std::partial_sort_copy( begin(v), end(v), 
                          begin(top3), end(top3),
                          [](int a, int b){return a > b;} );
  int top_n = 0;
  for(int& it : top3)
    std::cout << ++top_n << "番目に大きい数字は" << it << "です。" << std::endl;
}

Answer 2

#include <stdio.h>

int main(void) {
    int number;
    int no_1 = 0;
    int no_2 = 0;
    int no_3 = 0;

    printf(" 0 以上の整数を入力 (0 で終了)\n");
    for (int i = 0; i < 10; i++) {
        scanf("%d", &number);
        if (number <=0 ) {
          break;
        }

        if (no_1 < number) {
            no_3 = no_2;
            no_2 = no_1;
            no_1 = number;
        }
        else if (no_2 < number) {
            no_3 = no_2;
            no_2 = number;
        }
        else if (no_3 < number) {

            no_3 = number;
        }
    }

    printf("1番目に大きい数字は%dです。\n", no_1);
    printf("2番目に大きい数字は%dです。\n", no_2);
    printf("3番目に大きい数字は%dです。\n", no_3);
    return 0;
}

num_1, num_2, num_3 を num[] 配列で扱うようにすれば、もっとコードは短くできそうに思います。

4 つの配列をつくる、num[0] に入力した値をいれたら、それを昇順にソートする。
そうすれば常ににいままでいれた数字の上位 3 つが常に num[1], .. num[3] に在るようになります。

Answer 3

こんにちは、
まず10個のデータを大きい順にソートして　for文で大きい順に3つ　出力すればいいと思います。
(入力するデータがすべて異なるデータであることを前提としています。すみません)
(3という数字も記号定数にすべきかもしれません)

#include <stdio.h>
#include <stdlib.h>

#define BUFSIZE 256
#define MAX_NUMBER 10

int main(void)
{
	char buf[BUFSIZE];
	int num[MAX_NUMBER];
	int answer[3];
	int i,j;
	int tmp;

	for(i = 0; i < MAX_NUMBER; i++){
		fgets(buf,sizeof(buf),stdin);
		num[i] = atoi(buf);
	}
	
	for(i = 0; i < MAX_NUMBER; i++){
		for(j = i + 1; j < MAX_NUMBER; j++){
			if(num[i] < num[j]){
				tmp = num[i];
				num[i] = num[j];
				num[j] = tmp;
			}
		}
	}
	
	for(i = 0; i < 3; i++){
		printf("%d番目に大きな数字は%dです。\n",i + 1,num[i]);
	}
	
	return 0;
}