空白があったらもう一度入力させたい

Question

入力の最初や途中に空白文字があったらもう一度入力させるプログラムを作りたいのですが、文字が入っていても繰り返されてしまいます。なぜでしょうか？

#include <stdio.h>

int main() {
  char name[10];
  char Name_Len;
  char flag = 0;
  char cnt = 0;
  char *check;

  while (flag == cnt) {
    printf("名前入力して>>");
    fgets(name, 10, stdin);

    // 配列の最後を\0にする処理
    Name_Len = strlen(name) - 1;
    name[Name_Len] = '\0';

    check = strchr(name, ' ');

    if ((strlen(name) <= 10) && !check) {
      printf("%s:ok\n", name);
      cnt++;
    } else {
      printf("%s:no\n", name);
    }
  }

  return 0;
}

Answer 1

fgets(name, 10, stdin); で入力したら、
(strlen(name) <= 10) は常に真です。書いても意味がありません。

次のように書けば、flag も cnt も不要です。

#include <stdio.h>   // printf, fgets
#include <string.h>  // strchr, strlen

int main(void)
{
	char name[256];

	while (1) {
		printf("名前入力して>>");
		if (!fgets(name, sizeof name, stdin)) break;
		char *p = strchr(name, '\n');
		if (p) *p = '\0';
		if (strlen(name) <= 10 && !strchr(name, ' ') && name[0]) {
			printf("%s:ok\n", name);
			break;
		}
		printf("%s:no\n", name);
	}
	return 0;
}

Comments

[gyoruo]

そんなスマートなやり方もあるんですね！
ご回答ありがとうございます。

Answer 2

以下のコードでできました！
if ((strlen(name) <= 10) && !strchr(name, ' ') && !(name[0] == '\0'))のように変更しました。

#include <stdio.h>

int main() {
  char name[10];
  char flag = 0;
  char cnt = 0;
  char i = 0;

  while (flag == cnt) {
    printf("名前入力して>>");
    fgets(name, 10, stdin);

    // 配列の最後を\0にする処理
    name[strlen(name) - 1] = '\0';

    if ((strlen(name) <= 10) && !strchr(name, ' ') && !(name[0] == '\0')) {
      for (i = 0; i < strlen(name);i++) {
        printf("[%c]\n", name[i]);
      }
      printf("%s:ok\n", name);
      cnt++;
    } else {
      printf("%s:no\n", name);
    }
  }

  return 0;
}

Comments

[thkana]

"12345678 " (8の次が空白)という入力に対してはどのような結果が得られることを期待しますか?

[gyoruo]

アドバイスありがとうございます！
試しましたが、確かにnoの方にいってしまいますね・・・
直してみます！

[gyoruo]

最後に空白が入ったとしてもそれまでの値は実行させたいので直してみます！

[kazuma-s]

"12345678 " (8の次に空白が 3つ) という入力は ok ですか、no ですか？
ok の場合、空白も name に残したままでいいんですか？
10文字以下という制限は空白を除いてですか、それとも後続の空白も含めてですか？

[gyoruo]

空白が1つどの位置にあってもnoにしようとしています。
確かに、nameに残したままだと余計なメモリ食ってしまうので、その処理も必要ですね。。。

[thkana]

> 最後に空白が入ったとしてもそれまでの値は実行させたいので直してみます！
これは okにしたい、ということですか?
では、"123 "(3の次が空白)もokになるのですか?
> 空白が1つどの位置にあってもnoにしようとしています。
と矛盾するような気がしますが。

[gyoruo]

> 最後に空白が入ったとしてもそれまでの値は実行させたいので直してみます！
いや、すみません、空白が1つでもどこかの場所に入った時点でnoにしたいです。

[thkana]

わたしのてもとでは、
"12345678 " (8の次が空白)という入力に対して"ok"になったので先のコメントだったわけですが。"no"になりましたか? どちらが期待する動作ですか。

[kazuma-s]

文字数制限の仕様もはっきりさせてください。

[gyoruo]

混乱させて申し訳ございません。
文字数制限は特になく([256]バイト以下)半角スペースが1箇所でも入ればnoにしたいといった処理を実現したかったです。
次からはより具体的に記載します。

Answer 3

flag、cntが初期化されずに使用されています
まずはこれをどーにかしよう

そして、strchr関数の戻り値はcharではありません。

Comments

[gyoruo]

アドバイスありがとうございます。

[gyoruo]

ポインタでしたね、、、、