stdin

からの入力の読み取りstdinからのユーザー入力をプレーンなC。問題は、エラーに対して堅牢で、ユーザーを特定の入力に制限し、複雑さの点で問題がない、健全な実装が必要なことです。関数get_strings()は次のように入力文字を読み取ります。改行（\n）、EOFがなく、すべての文字がisalpha()テスト。ただし、スペースを確保したい。

レビュー中に（私が思うに）特別な注意を払うに値するいくつかのポイント：

-削除したい基本的に、ユーザーが意味のある入力なしでEnterキーを押したかどうかをテストする外側のwhileループ。

-ferror（）を使用する必要もありますか？fgetcは、問題が発生したときにすでにEOFを返しますが、ユーザーに問題が発生したことを通知するのではなく、ストリームからの読み取りを停止します。

-毎回発生するわけではありませんが、発生します。\ nはstdinストリームに残り、次に意味のある入力を取得したいときfgetc（）はスキップされます。ここでは問題ではありませんが、1文字の「はい/いいえ」の質問をしている場合は問題になります。この問題を取り除くには、stdinに残っている以前のすべてのものをクリアする構造を使用する必要があります。これについては2番目のコードブロックを参照してください。 。

それで、私はこれを完全に間違って処理していますか？受け入れるためのより良い方法はありますか？それは私には本当に不格好に見え、不格好は常に悪いです。

/** @brief Contains the dictionary */ static char **strings = NULL; /** @brief Helps with containing the dicionary */ static char *string; /* Reads input char by char with fgetc() */ static char *get_strings() { char *string = NULL; char ch; size_t len = 0; while (string == NULL && ch != EOF) { while (EOF != (ch = fgetc(in_stream)) && ch != "\n") { if (ch != " " && isalpha((int)ch) == 0) { fprintf(stderr, "Only [a-z] is a valid input. | \t" "| Input another or end with CTRL+D: "); continue; } string = (char*) realloc(string, len+2); if (string == NULL) { bail_out(EXIT_FAILURE, "realloc(3) failed"); } string[len++] = toupper(ch); if (len >= MAX_DATA) { bail_out(EXIT_FAILURE, "Input too long\n"); } } if (ferror(in_stream)) { bail_out(EXIT_FAILURE, "Error while reading from stream"); } } if(string) { string[len] = "\0"; } else { printf("\nFinished dictionary...\n"); } printf("Added string: %s | Input another or end with CTRL+D: ", string); return string; } /* Saves the returned strings from get_strings() in a linked list */ static void read_dict() { int index; for (index = 0; (string = get_strings()); ++index) { if (string[0] == "\0") continue; strings = (char**) realloc(strings, (index+1)*sizeof(*strings)); if (strings == NULL) { bail_out(EXIT_FAILURE, "realloc(3) failed"); } strings[index] = string; } /* Take a note of how many entries we have yet. */ dict_size = index; } 

より単純なケースの2番目のCodeBlock：

while(1) { char tmp; printf("Please enter your guess [a-z]: "); guess = fgetc(stdin); /* Jump back to start of loop */ if (guess == "\n") { continue; } /* HERE IS THE CLEAR FOR STDIN This part really just eats all remaining \ns from the user, so that later inputs can start uninterrupted. Can I get rid of it in some better way? */ while((tmp = getchar()) != "\n" && tmp != EOF); if(!isalpha(guess)) { fprintf(stderr, "Enter a valid letter [a-z]!\n"); continue; } } 

コメント

• クイック注：< stdbool.h >のブール値を使用するか、1と0ではなく定義します。’より明確な意味
• @Zorgatone私はあなたに半分同意します。常にstdbool.hを使用しますが、’自分のブール値をロールしようとしないでください。