全角10文字以内を入力し、それ以上だと再入力させるプログラムを作りたい

Question

以下のプログラムで全角10文字までを入力可能としているのに、「あ」と20文字入れても代入されてしまいます。
MAXDATAで11などの制限をかけるとそれ以上入力した場合にコアダンプで処理が落ちてしまうため、多く値をとることで改行文字かの判定をifで行っています。なぜ10文字以上でも代入されてしまうのでしょうか？

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

#define MAXDATA 256

int main(void) {
    char test[MAXDATA];
    char flag = 0;

    while (flag == 0) {
      puts("何か文字を入力して、エンターキーを押してください");
      printf(">>");
      // 半角全角10文字以内を想定
      fgets(test,MAXDATA,stdin);
      if (test[11] <= '\n') {
        printf("10文字以下で、%sと入力されました",test);
        flag = 1;
      } else {
        // 範囲を超えて入力された場合のオーバーラン対策
        if (strchr(test, '\n') == NULL) {
          while (fgetc(stdin) != '\n');
        }
      }
    }

  return 0;
}

Comments

[dodox86]

> if (test[11] <= '\n') {
もしかしてこのコードは、「test[0]～test[11]」の範囲に改行コード('\n')が入っているかを判定しよう（したい）としていませんか。もしそうだとすれば、意味が全然違います。

[Daregada]

使用しているエンコーディング(UTF-8とかShift_JISとか)を追記してください。

[gyoruo]

お返事ありがとうございます。
そのように考えていましたが、よくよく考えてみれば、文字コードによって全角は[0][1][2]を占領したりするのでできないと判明しました。
ちなみに、文字コードはUTF-8です。

Answer 1

amiyaさんのアドバイス通り、文字判定のif文を変更したら意図した通りに動きました。

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

#define MAXDATA 256

int main(void) {
    char test[MAXDATA];
    char flag = 0;

    while (flag == 0) {
      puts("何か文字を入力して、エンターキーを押してください");
      printf(">>");
      // 半角全角10文字以内を想定
      fgets(test,MAXDATA,stdin);
      if (strchr(test, '\n') - test <= 31) {
        printf("%d\n", strchr(test, '\n') - test);
        printf("10文字以下で、%sと入力されました",test);
        flag = 1;
      }
      else {
        // 範囲を超えて入力された場合のオーバーラン対策
        if (strchr(test, '\n') == NULL) {
          while (fgetc(stdin) != '\n');
        }
      }
    }

  return 0;
}

Comments

[kazuma-s]

質問に追記するのではなく、新規の回答として投稿したんですね。
それは、まあいいでしょう。

全角文字をすべて 3バイトだと仮定して、「<= 31」で判定しています。

コード内のコメントに「// 半角全角10文字以内を想定」とありますが、
半角文字を 12文字入力しても
「10文字以下で、abcdefghijklと入力されました」
になりますよ。

[gyoruo]

ご返信ありがとうございます！
全角で10文字超えなければ大丈夫なので半角は何文字でもOKです！

Answer 2

OS やコンパイラは何ですか？

次のコードは期待通りに動きますか？

#include <stdio.h>   // printf, fgets, getchar
#include <stdlib.h>  // mbstowcs
#include <string.h>  // strchr
#include <locale.h>  // setlocale

#define MAXDATA 256

int main(void)
{
	char test[MAXDATA];

	setlocale(LC_CTYPE, "");  // マルチバイト文字を使用するため

	while (1) {
		printf("何か文字を入力して、エンターキーを押してください\n>>");
		if (!fgets(test, MAXDATA, stdin)) break;
		char *p = strchr(test, '\n');   // 改行文字を探す
		if (p) *p = '\0';  // 改行文字があれば、削除
		else {             // 改行文字が無ければ、
			int c;         //     改行文字まで読み飛ばす
			while ((c = getchar()) != EOF && c != '\n') ;
		}
		int n = mbstowcs(NULL, test, 0);  // 文字数を数える
		if (n >= 0 && n <= 10) {
			printf("10文字以下で、%sと入力されました\n", test);
			break;
		}
	}
	return 0;
}

Comments

[gyoruo]

ご回答ありがとうございます。
OSはCentOSで、コンパイラはatomについているものを用いています。
上記のプログラムでは全角10文字以内でも10文字以下でも何度も入力させられました。

[kazuma-s]

私は、Ubuntu で、環境変数 LANG=ja_JP.UTF-8 で動いています。

Answer 3

文字列が何文字かはstrlen関数で求めることができます。
その返り値と文字数制限を比較してください。

ただし、正確にはこれで分かるのはバイト数であり、1文字が2バイト以上の文字コードを使った場合には、
文字数とは限りません。

fgetsは改行まで含めてしまうので、その分を減らすか、始めから複数行を前提で、「改行まで」の文字数を検知したいのであれば、
strchr(test,'\n')-test 等として求めてください(これもバイト数です)

今の状態だと、入力内容次第で問題が起きそうなので…こんな感じ？

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

#define MAXDATA 256

int main(void) {
    char test[MAXDATA];
    char flag = 0;

    while (flag == 0) {
      puts("何か文字を入力して、エンターキーを押してください");
      printf(">>");
      // 半角全角10文字以内を想定
      fgets(test,MAXDATA,stdin);
      
      char *p = strchr(test, '\n');   // 改行位置を取得
      if (p != NULL){                 // 改行があった場合
          *p = '\0';       　　　　　　// 改行位置で文字列を終了させる
      } else {                        //ついでにMAXDATAオーバーの処理
      // 範囲を超えて入力された場合のオーバーラン対策
        while (fgetc(stdin) != '\n');
      }

      int len = strlen(test);         // 改めて長さを取得
      if (len <= 30) {                // 一文字3Byte固定なら30？ 
        printf("%d\n", len);
        printf("10文字以下で、%sと入力されました",test);
        flag = 1;
      }
    }

  return 0;
}

scanfとかsscanfを使った方が手っ取り早い気がする。

Comments

[gyoruo]

お返事とアドバイスありがとうございます。
半角と全角でバイト数が異なるので、そもそもこの判定の仕方が無理でした。。。

[gyoruo]

アドバイス通りにしてみたところ、意図した通りに動きました！
ありがとうございます。

[kazuma-s]

意図したとおりに動いたというコードを、質問に追記してもらえませんか。
質問は編集できます。

[gyoruo]

ご指摘ありがとうございます。
追記しておきました。