標準入力された値の最大値・平均値・最小値を表示するC言語プログラム

Question

前提・実現したいこと

下記に記載している機能を持つC言語プログラム作成したいと思っています。

---

標準入力から1個以上の整数値データが与えられる。 これら全ての整数値の最大値、平均値、最小値をそれぞれ1行としてこの順に書き出す。 この3個の数値は、それぞれ左詰に1行として書き出す。 特に、平均値は、小数点以下2桁まで書き出す。

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

最大値・平均値・最小値の個々の機能実装はうまくいっているのですが、一つのソフトとしてまとめることが出来ず困っています。

該当のソースコード

C言語

#include <stdio.h>

int main(int argc, char*argv[]){
    int d,i,n,M,N;
 
    scanf("%d", &n);
    for(i= 1; scanf("%d", &d)==1; i++){
        if ( d >= n ) M= d;  
        if ( d <= n ) N= d;  
        n+= d;
    }
    printf("%d\n", M);
    printf("%.2f\n", (double)n/i);
    printf("%d\n", N);
       
    return 0;
}

試したこと

上記のコードで実行しましたが、思った通りには動きませんでした。

補足情報

因みにこれは大学の自由課題です。ご教授いただければ幸いです。

Comments

[pepperleaf]

全く動かないのでしょうか? それとも期待する値が出力されないのでしょうか?

[Taka787]

期待する値が出力されないです。

[Taka787]

おかげさまで出来ました。ありがとうございました。

Answer 1

すでに解決済みですが一応。

ループ内の「今読んだ値がそれまでの最大値より大きな数だったら、最大値を更新」というロジックは「それまでの最大値」がない状態、ループの1回目に対応できません。
初期値としてこういう設定をするのが一つの手段。

• 入力されるかもしれない最小の値より小さい値→最大値、
• 入力されるかもしれない最大の値より大きな値→最小値

1個目のデータだけ別扱いするのがもう一つの手段。疑似コードで示すとだいたいこんな感じ。
（今回はデータゼロ個の場合は扱わなくていいので、これでOK)

if( 整数値をdを読み取り　が失敗)｛エラー停止｝
count=1;
total=minimum=maximum=d;

while(整数値をdに読み取り、が成功)｛
 if(minimum>d)minimum=d;
 if(maximum<d)maximum=d;
 total+=d;
 count++; //for文で実装しても可
｝
average=total/(double)count;

Comments

[Taka787]

分かりやすい擬似コードありがとうございます。参考になりました。

Answer 2

Cなど不要。そう、シェル芸ならね！

$ cat numbers
6768
4458
203
5313
2712
2204
2889
1993
9488
1222
$ cat numbers|awk '!a||$1<a{a=$1}$1>b{b=$1}{c+=$1}END{printf"%d\n%.2f\n%d\n",b,c/NR,a}'
9488
3725.00
203

Comments

[Taka787]

ありがとうございました。できました。

Answer 3

講義ノートや講義でつかっている教科書で解決できるはず。
あるいは、図書館でたのテキストで勉強したり、googke で C 入門のページで勉強してみてください。
あるいは同級生などと一緒にかんがえてみるとよいです。

(このレベルの処理を QA サイトで質問してヒントやコード例をもらって解決しているようでは、この先 苦労することになる気がします)

... 思った通りには動きません ...

それを解決していくのが課題をこなす事の意味だと思います。
print 文を埋込んで調べる、 debugger の使い方を修得する, 類似の問題を探してそれを元に考える など...

Comments

[Taka787]

ご指摘ありがとうございます。自分のためになる勉強方法を実践していきます。
又、ご指導お願いします。

Answer 4

演習課題とのことなので、まず、気づいた事。

• M, N の初期設定値がありません。

• 終了条件が scan()==1 となっていますが、大丈夫ですか?

こちらで確認したら、ctrl-D で終了しました。

まずはこの辺で。

Comments

[Taka787]

適切なアドバイス有難うございました。

Answer 5

for(i= 1; scanf("%d", &d)==1; i++){
        if ( d >= n ) M= d;
        if ( d <= n ) N= d;

を

M = n;
N = n;
for(i= 1; scanf("%d", &d)==1; i++){
        if ( d > M) M= d;
        if ( d < N) N= d;

に変更してみてはどうでしょうか

Comments

[Taka787]

ご教授有難うございました。

Answer 6

比較がおかしい
なぜ合計値と比較しているのか？

Comments

[Taka787]

指摘ありがとうございます。修正してみましたら出来ました。

Answer 7

最終的にどうやって解決したのですか？