scanf関数で文字数が正しくカウントされない

Question

前提

C言語で安全に文字入力を受け取るためのプログラムを書いていました。
scanf関数で改行まで最大255文字をcに入れ、入力された文字数をcharlengthに入れます。
EOFだったら異常終了扱いにします。
最初にEnterが押された、つまり文字が入力されなかったらその旨を伝えます。
最後に文字数を表示します。

実現したいこと

• 文字数を正しく表示する
• 文字が入力されなかったことを判定できるようにする

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

1つ目に、Enterを入力しても何も反応しません。

■■■■@penguin:~/■■■/■■■■■$ ./myconvert
アルファベットを入力 : (Enterを入力)
          (何も反応しなかったのでCtrl+D)
不正な入力が行われました。プログラムを終了します。
■■■■@penguin:~/■■■/■■■■■$

2つ目に、文字を入力したときに文字数が正しく表示されません。(下の場合3と出てほしかった)

■■■■@penguin:~/■■■/■■■■■$ ./myconvert
アルファベットを入力 : asd
896
■■■■@penguin:~/■■■/■■■■■$

該当のソースコード

/* myconvert.c */
#include <stdio.h>
#include <stdlib.h>

int main(void)
{
    char c[0X100];
    int charlength;

    printf("アルファベットを入力 : ");
    if (scanf("%255s[^\n]%n%*[\n]", c, &charlength) == EOF){
        printf("\n不正な入力が行われました。プログラムを終了します。\n");
        return EXIT_FAILURE;
    }
    if (charlength == 0){
        printf("入力が行われていません。\n");
    }
    printf("%d\n", charlength);

    return EXIT_SUCCESS;
}

補足情報（FW/ツールのバージョンなど）

名前	バージョン
ChromeOS	107.0.5304.110 (Official Build) (64-bit)
Linux(仮想環境)	Debian GNU/Linux 11 (bullseye)
gcc	10.2.1 20210110 (Debian 10.2.1-6)

Comments

[jimbe]

scanf でガンバルより 1 文字入力を回すほうが簡単な気がしますが。

Answer 1

scanfは書式指定と入力で順次マッチングを取って、マッチングが取れた部分に変換指定があれば変換して取り込む一方、マッチングしないことが確実になればそこで処理を諦めて以降の取得をせずにリターンするという動作をします。

"%255s[^\n]%n%*[\n]"がどういう入力にマッチするかというと、
%255sで255字までの文字列にマッチし
[で'['にマッチ...ところが、'['は先に文字列にマッチしてしまうので文字列に引き続いて[がマッチするのは255文字文字が続いてその次に[が来たときだけです。なので、この変換指定文字列は254文字以下の文字列が入力された段階(文字列に'\n'が続く状態)で以降はマッチしないことが確定し、%nに対する評価は行われません。
ということで、ローカル変数charlengthにはなんの作用も行われず、未初期化のローカル変数として不定値になります。
最後までマッチさせるとするなら"aaaa...(255文字)...a[^(改行)](改行)改行以外なにかの文字(改行)"と入力すればよいでしょう。"aaaa...(255文字)...a"が%255sにマッチし、[^(改行)]が"[^\n]"にマッチし、%*[\n]に改行がマッチして、改行以外のなにかの文字でマッチングが終了します。最後の改行はなにかの文字を標準入力に押し込むために必要。

また、"%255s"にいきなり改行を与えても、%sはデータ先頭の区切り文字(改行含む)は読み飛ばしますからscanfから抜けません。

多分、"%255[^\n]%n"とかしたかったのかなぁ、と想像はします。
scanfの沼にハマらずに、既に他の回答にありますがfgetsとsscanfあたりに逃げておいたほうが、同じことが楽にできたのではないかと思います。

ところで、この「質問」、よく読むと「何を知りたいか」書いてないんですけど。質問ではなくて不具合報告書ですか?

Comments

[退会済みユーザー]

scanf関数について詳しく説明していただきありがとうございます。
そうですね、%255と[^\n]の間にsを入れてしまったのは完全に見落としていました。また、scanf関数ではなくfgets、sscanf、strlen関数を使って処理しようと思っています。
此方が単に心の器が小さい動物であるだけなのかもしれませんが、回答が嫌みたらしい文で始まり、とても不快に感じ傷つきました。私は貴台の儕輩ではありません。言い方を間接的にしたり、ネガティブな内容は後に書いたりするだけでも文全体の印象が変わったと思います。

[thkana]

ご指摘により「後」にしてみました。

Answer 2

/* myconvert.c */
#include <stdio.h>
#include <stdlib.h>

int main(void)
{
    char c[256];
    int charlength = 0;  // ★

    printf("アルファベットを入力 : ");
    if (scanf("%255[^\n]%n", c, &charlength) != 1) {
        if (charlength == 0)
            puts("\n入力が行われていません。");
        else
            puts("\n不正な入力が行われました。プログラムを終了します。");
        return EXIT_FAILURE;
    }
    printf("%d\n", charlength);
    return EXIT_SUCCESS;
}

これで、Enter のみの場合と、asd という文字列の場合には対応できるはずですが、
256文字以上の文字列が入力された場合にはどういう対応にしたいのですか？

Comments

[退会済みユーザー]

255文字以降の入力は使わずに捨てます。
とりあえずscanfのフォーマット指定子で%*[\n]を使って読み飛ばそうとしました。
質問なんですが、最初のcharlengthの宣言で0で初期化することにはどういう意味があるのでしょうか。

[kazuma-s]

%255[^\n] は '\n' 以外の文字を 1～255文字取り込むという書式です。
Enter だけの入力の場合、1文字も読み込めず、この書式による処理は失敗します。
scanf の実行がそこで打ち切りになるので、次の "%n" は処理されず、charlength に値が設定されません。
if (charlength == 0) で charlength の値を得たいのなら初期化が必要です。

Answer 3

正しく表示されない、で投げ出してしまわないで、なにが起こってるのか確認しよう。
cにはなにが入ってるのか確認してみては。

Answer 4

カラの改行を検出するなら、fgetsｰsscanfのパターンの出番。
scanfの%sは非空白な文字が現れるまで戻ってこない。

    char s[280];
    printf("アルファベットを入力 : ");
    fgets( s, 280, stdin);
    if (sscanf(s, "%255s%n",  c, &charlength) == EOF){