最大値・最小値が誤った数値で出てくる。

Question

前提・実現したいこと

c言語で任意の整数を標準入力から読み込み最大値、最小値、平均値を求めるという課題があり
r = scanf("%d",&x);
if (r < 0)
{
break;
}
↑のようなものをつかうほうほうでやりたいです。

該当のソースコード

#include <stdio.h>

int main(void) {
int x, m, r, u;

while (1) { //while

scanf("%d", &x);
m = x;
u = x;

scanf("%d", &x);
if (x > m) {
m = x;
}
if (x < u) {
u = x;

}
r = scanf("%d",&x);
if (r < 0)
{
break;
}

} //while

printf ("最大値は%dです\n", m);
printf ("最小値は%dです\n", u);

return 0;
}

試したこと

breakの位置を変える条件分岐をelse if やelseで試した。
標準入力のテキストデータの数値の順番を変えた

Comments

[azuapricot]

課題なら先生に質問しにいきましょうねー

[y_waiwai]

現状のコードはどうなるんでしょうか

[kenny-ackerman]

51
32
86
14
77
44
を読み込むと最大値が44最小値が44と出てしまいます

[cateye]

環境を書きましょう。
EOF(-1)を発生させる要因はOSによって違います。

Answer 1

変数rにscanfの結果を代入されていますが，scanfの返り値は

・代入された入力項目の個数
・失敗した場合EOF

となるので，printfを使ってrの値を見るとわかると思いますが，どんな値を代入してもrは負の値を取らないと思います．
scanfの値の代入先をxではなくrにしてはいかがでしょうか．

加えて，このプログラムの意図がいまいち分かっていないので蛇足かもしれませんが最初のm, nの初期化はwhile文から外してしまう方が良いかと．
任意のタイミングで負の値を代入することでループを脱するようなプログラムを考えているのならばwhile文の中にscanfは1つのほうがいい感じだと思いますよ．

Comments

[LouiS0616]

> どんな値を代入してもrは負の値を取らない

EOFは負のint型定数です。

Answer 2

なんで while ループ内に3カ所も scanf があるんでしょうね？
一つでいいはずですよ。

そのようにした上で、そのループ内で何をすればいいのか考えましょう。

※課題ということなのでヒントしか書きません