C言語：scanfで10桁の整数のみを受付ける方法

Question

初心者の学生です。どうぞ宜しくお願いいたします。

電話番号を入力するプログラムの課題を解いています。
10桁の数字以外（ハイフン等）が入力された場合にエラーを返す処理を書いたつもりですが、上手く動きません。
バッファーのような気がしているのですが、どうクリアしたら良いかわからず質問させていただきました。

お知恵をお借りできれば幸いです。お時間ありがとうございます。

出力画面

※変数の値を確認のため表示しており、見づらくて申し訳ありません

1回目は意図した動きをしています。

1回目にハイフン入りの入力をすると、再入力時に正しく入力しても正常に処理してくれません。

該当のソースコード

#include <stdio.h>
#include <string.h>
#include <ctype.h>

void clearKeyboard(void)
{
    while (getchar() != '\n'); // バッファークリア
}

int main()
{
    char phoneNum[11];
    int needInput = 1;
    char *p = phoneNum;
    int flg = 0;

    printf("10桁の整数を入力してください: ");
    while(needInput == 1)
    {
        scanf("%[^\n]", phoneNum);
        clearKeyboard();
        while(*p)
        {
            if(!isdigit(*p++)) // phoneNumが 整数じゃなかったら フラグ付与
            {
                flg = 1;
            }
        }
            if(flg || strlen(phoneNum) != 10) // phoneNumが整数じゃない or phoneNumが10桁じゃなかったら実行
            {
                needInput = 1;
                printf("10桁の整数のみを入力してください: (needInput:%d, flg:%d, phoneNum:%s)", needInput, flg, phoneNum);
            }
            else
            {
                needInput = 0;
                printf("10桁の整数が入力されました (needInput:%d, flg:%d, phoneNum:%s)", needInput, flg, phoneNum);
            }
}
    return 0;
}

試したこと

• scanf の %10s を %12s にしたりした（結果変わりませんでした）
• clearKeyboard 関数をエラーメッセージの後に入れた（再入力時に改行されてしまいました）

補足情報

このサイトでコンパイルしました。
https://www.onlinegdb.com/online_c_compiler

Answer 1

scanfを使わずに、fgetsで１行の文字列として取り込み、文字操作により数字と桁数のチェックを行いましょう。

Comments

[aluminium]

fgetsという方法があるのですね！
学校の課題でscanfが必須でして... 今後fgetsを使えるように勉強します。
ありがとうございました！

Answer 2

pの値が、最初にchar *p = phoneNum;で初期化された後、
while(*p)のループの中でp++され、その後、放置されています。
その為、二週目以降のwhile(*p)のループでは、前回ポインタを進めた場所からの開始となります。
結果、おそらくまともな値にはならず、整数ではないと判断され、flg = 1;が実行されます。
flg = 1;の時点で、if(flg || strlen(phoneNum) != 10)は、真となり、
needInput = 1;が実行されてループは継続されます。

while(*p)の前にp = phoneNum;として、ポインタの位置を戻してやると良いかと思います。

---

それはそれとして、数字だけを受け取りたいのであれば、
scanf("%[^\n]", buf);で一行すべて受け取った後に、
sscanf(buf,"%[0-9]",phoneNum);とかして、先頭からの連続した数字だけを読み取り、
if(strlen(phoneNum) != 10) とかで桁数だけをチェックでOKな気もします。
(一つも合致しなかったときの為にsscanfの戻り値のチェックも必要かな)

Comments

[aluminium]

scanfでの方法をご教示いただき かつ 最初にご回答くださりありがとうございます！
"%[0-9]" で意図した処理になり、無事、課題を提出することができました。
ありがとうございました！

Answer 3

タイトルに「scanf で」とありますが、scanf を使わない方法を示します。

#include <stdio.h>  // fgets, printf
#include <string.h> // memcpy
#include <ctype.h>  // isdigit

int main(void)
{
    char phoneNum[11] = "";
	char buf[256];

    printf("10桁の整数を入力してください: ");
	while (fgets(buf, sizeof buf, stdin)) {
		int i = 0;
		while (i < 10 && isdigit(buf[i])) i++;
		if (i == 10 && buf[i] == '\n') {
			memcpy(phoneNum, buf, 10);
			printf("10桁の整数が入力されました (phoneNum:%s)\n", phoneNum);
			break;
		}
        printf("10桁の整数のみを入力してください: ");
	}
}

ひらがなや漢字などを入力する可能性がある場合は、
isdigit(buf[i]) を isdigit((unsigned char)buf[i]) に変更してください。

Comments

[aluminium]

isdigitを配列で使えるようにするには、ご教示いただいた様にインクリメントすればいいんですね！
ありがとうございます。今後使ってみたいと思います！