2番目に小さい値とその添字の出力

Question

###実現したいこと
2番目に小さい値とその添え字を出力したい。
※全部異なる値が入力されるとします。
###問題点
いまのプログラムだと下記の実行結果になります。
10から降順にまた昇順にならべるとうまくいきません。

10個の正整数を入力します
1番目 10
min1=10 min2=10
2番目 9
min1=9 min2=10
3番目 8
min1=8 min2=10
4番目 7
min1=7 min2=10
5番目 6
min1=6 min2=10
6番目 5
min1=5 min2=10
7番目 4
min1=4 min2=10
8番目 3
min1=3 min2=10
9番目 2
min1=2 min2=10
10番目 1
min1=1 min2=10
2番目に小さい数は0番目に入力された10です

ランダムだとうまくいきます。

10個の正整数を入力します
1番目 4
min1=4 min2=4
2番目 7
min1=4 min2=4
3番目 2
min1=2 min2=4
4番目 9
5番目 0
min1=0 min2=4
6番目 5
min1=0 min2=4
7番目 1
ans=7
min1=0 min2=1
8番目 3
min1=0 min2=1
9番目 8
min1=0 min2=1
10番目 10
min1=0 min2=1
2番目に小さい数は7番目に入力された1です

###コード

#include<stdio.h>
int main(void)
{
    int i,data,maxans;
    int min1=0; //1番小さな値
    int min2=0; //2番目に小さな値
    int ans=0;
    //添え字を保存しておく
   
    printf("10個の正整数を入力します\n");
    for(i=0;i<10;i++){
        printf("%d番目 ",i+1);
        scanf("%d",&data);
        if(i==0){
            min1=data;
            min2=min1;
        }else if(min2>data){
            if(min1<data){
            min2=data;
            ans=i+1;
            printf("ans=%d\n",ans);
            }
            if(min1>data){
            min1=data;
            }
            //ans=i+1;
            //printf("ans=%d\n",ans);
        }
         printf("min1=%d min2=%d\n",min1,min2);
    } 

    printf("2番目に小さい数は%d番目に入力された%dです\n",ans,min2);
    return 0;
}

Comments

[pepperleaf]

最初から、10個なんてせずに、3個くらいから、確認したら、どうでしょう。
そして、プログラムの動きを一行づつ追う。
多分、簡単に分かると思う。。。
まず、一回目のループで、 min1 と min2が同じ値になってますが、良いでしょうか?

[kazuma-s]

10個全部同じ値を入力したときは、どうするつもりですか？

[langhtorn]

全部異なる値が入力されることになっています。

[SHOMI]

ならばそれを前提条件として質問内容に追記してください。
コメント欄はデフォルト非表示なので回答者に読んでもらえるとは限りません。

Answer 1

min1 と min2 に初期値として 0 を入れておくと、data より小さくなるので
適切な判断ができなくなります。

また、次のような処理をしてみてはいかがですか？

• data が min1 より小さければ、min2 = min1, min1 = data。
• そうでなくて、data が min2 より小さければ、min2 = data。

追記
質問のコードで「うまくいかない」わけを知りたいのですよね。
10, 9, 8,... と入れるとどうなるか考えてみれば分かるはずです。
各変数の値の変化を見てください。

10 を入力すると、data = 10, min1 = 10, min2 = 0。
9 を入力すると、data = 9。i = 1, min2 < data (0 < 9) なので何もしない。
変ですよね。
9が一番小さく、10 が二番目に小さいから、
min1 = 9、min2 = 10 になっていなくてはいけないのに。

こんな風に自分の書いたコードを確かめてみようと思わないのですか？

私なら次のようなコードを書きます。

#include <stdio.h>

int main(void)
{
    int min1 = 0x7fffffff; // 1番小さな値 (minimum number)
    int min2 = 0x7fffffff; // 2番目に小さな値
    int ent1 = 0; // min1 が何番目の入力か (entry number)
    int ent2 = 0; // min2 が何番目の入力か
	int data;     // 入力データ

    puts("10個の正整数を入力します");
    for (int i = 1; i <= 10; i++) {
        printf("%d番目 ", i);
        scanf("%d", &data);
		if (data < min1)
			min2 = min1, ent2 = ent1, min1 = data, ent1 = i;
        else if (data < min2)
			min2 = data, ent2 = i;
    } 
    printf("2番目に小さい数は%d番目に入力された%dです\n", ent2, min2);
}

コンマ演算子を多用するなど、模範的なコードではありません。
内容を理解して、自分のコードを書いてみてください。

Comments

[momon-ga]

あと、ansも複数用意しないとindexを表示できないです。

[pepperleaf]

確かにコンマ多用で、お薦めとは言えませんが、よさそうですね。