c言語 whileを使った文について教えて下さい。 初心者で初歩的な質問かもしれませんがよろしくお願いします。

Question

C言語について質問です。初心者なので、初歩的な質問ですがよろしくお願いいたします。
点数(point)を入力すると最高得点(max)と最低点(min)を表示するプログラムを作っています。
ifでもできるのですがwhileを使いたいです。
内容としてはwhile(1)でctrl＋cするまで終わらないプログラムで、入力した値とmax,minと比較して条件にあったら代入するといったプログラムを書いたつもりです。
しかし、結果は画像の右下の通りで、1回目はうまくいってるのですが、2回目から『前の点数→点数を入力→結果』といって順番で表示されてしまいます。一回めのように結果を表示し、改行で点数を入力といった表示にしたいのです。

#include <stdio.h>

int main(void)
{
    int  max = 0, min = 100;
    
    while(1)
    {
        int point;
        printf("点数を入力 >>");
        scanf("%d\n", &point);
        
        if(point >= 0)
        {
        if(point>max){max = point;}
        if(point<min){min = point;}
        }
        printf("最高点%d:最低点%d\n", max, min);
    }
    
    return(0);
}

Answer 1

この例に関してだけいうと、

        scanf("%d\n", &point);

を

        scanf("%d", &point);

に変えたら動きますよね。
"%d" は、空白や改行文字は読み飛ばしてくれるので。

こういう練習問題と解いている段階(かなりの初心者)でしたら、それでいいんでは？
他の方が助言しているとおりscanfなどの返り値チェックは本当は必要ですけども。

Comments

[EEE]

ありがとうございます
\nなくしたら動きました。

[otn]

誤って数字以外の文字を入力すると困った事態に。

[a_saitoh]

whileの使い方を学ぶ自主課題ですよね。課程の進度がI/Oエラーの正しい処理にはほどとおいので問題ないでしょう。
EOFを入れても困ったことになります。

Answer 2

scanf を含む入出力の動作には注意が必要です。
プログラムの動作を1ステップずつ追ってみましょう。

printf("点数を入力 >>");
scanf("%d\n", &point);
// 1. scanf で入力を待機
//    → ユーザが "20 -1\n" と入力
// 2. stdin のバッファに "20 -1\n" が蓄えられる
// 3. stdin のバッファから "20 " までを読み取る
//    ※scanf は空白を改行と同一視します
// 4. point に 20 が代入される
if(point >= 0) { /* ... ここの処理は狙い通り行われる */ }
printf("最高点%d:最低点%d\n", max, min);  /* ここも OK */

/* while ループの最初に戻る */

printf("点数を入力 >>");
scanf("%d\n", &point);
// 5. scanf で入力を待機
//    → ユーザが "60 -1\n" と入力
// 6. stdin のバッファに "60 -1\n" が追加され、"-1\n60 -1\n" となる
// 7. stdin のバッファから "-1\n" までを読み取る
// 8. point に -1 が代入される
if(point >= 0) { /* 通過しない！ */ }
printf("最高点%d:最低点%d\n", max, min);  /* if 文を飛ばすので前と同じ値が出力される */

/* while ループの最初に戻る */

printf("点数を入力 >>");
scanf("%d\n", &point);
// ⑦ stdin のバッファに "60 -1\n" があるので、scanf はそこから読み取る（入力を待機しない）
// ⑧ point に 60 が代入される
if(point >= 0) { /* ... 今回は通過する */ }
printf("最高点%d:最低点%d\n", max, min);  /* => 60:20 */
// 以下略

ポイントは、scanf は空白を改行と同一視するということと、stdin のバッファです。
そもそも入力に -1 を付けているのは「そうしないとうまくプロンプトが出ないから」みたいな理由だと思いますが、そういう「めんどくさいし、とりあえず狙い通り動いてるからいいや」という所にこそバグがあるものです。
まあ、今回の場合はそもそも狙い通り動いていなかったわけですけど……。

Comments

[EEE]

ありがとうございます。
とてもわかりやすく、助かります。
今後の勉強にもなりました。
また機会があればよろしくお願いします。

[EEE]

確かに-1はよくわからずつけていたので今後はちゃんと気をつけて、プログラムを作っていこうと思います。

Answer 3

#include <stdio.h>

int main(void)
{
    int  max = 0, min = 100;
    char line[100];

    while(1)
    {
        int point;
        printf("点数を入力 >>");
        fgets(line, sizeof line, stdin);
        sscanf(line, "%d\n", &point);

        if(point >= 0)
        {
        if(point>max){max = point;}
        if(point<min){min = point;}
        }
        printf("最高点%d:最低点%d\n", max, min);
    }

    return(0);
}

scanfは使い方が難しいので、fgets + sscanfを使いましょう。

Comments

[otn]

今回は単純なので良いですが、一般的にはscanf/sscanf/fscanfなどでは返り値をチェックする必要があります。

[EEE]

ありがとうございます。
一応完成しましたので、
今後の参考にさせていただきます。