編集履歴

回答編集履歴

qiita

2019/01/14 11:20

投稿

スコア5850

test CHANGED Viewed

@@ -212,7 +212,13 @@
 after_loop:
+    {
+        char* const lf = strchr(s, '\n');
-    strchr(s, '\n')[0] = '\0';
+        if(NULL != lf) lf[0] = '\0';
+    }
 }
@@ -398,10 +404,18 @@
 }
 ```
-[https://wandbox.org/permlink/b7Ev0PdEj3iu7JJP](https://wandbox.org/permlink/b7Ev0PdEj3iu7JJP)
+[https://wandbox.org/permlink/NuxCyJqbMdnrsPCc](https://wandbox.org/permlink/NuxCyJqbMdnrsPCc)
+追記:
+Qiitaに記事書きました、上記コードに誤りがあった場合はQiitaのほうのみ修正することにします。
+[「苦しんで覚えるc言語」のとあるコードをもっと安全に、より良く](https://qiita.com/yumetodo/items/bb2885f8c99e02e3d104)

全面改訂

2019/01/14 11:20

投稿

yumetodo

スコア5850

test CHANGED Viewed

@@ -11,3 +11,397 @@
 あと`age`と`sex`が`int`型なのが個人的にはどうかと思うのでenumとかunsignedな何かに書き換えたいt頃です。
+---
+暇ができたので全面書き直ししました。他にもいろいろ思うところがあったので。
+```c
+#include <stdio.h>
+#include <stdlib.h>
+#include <string.h>
+#include <limits.h>
+#include <errno.h>
+#include <stdbool.h>
+#include <ctype.h>
+#include <float.h>
+#include <assert.h>
+#ifndef static_assert
+#   define NO_STDC_STATIC_ASSERT
+#   define static_assert(...)
+#endif
+#if defined(_MSC_VER) || defined(__cplusplus)
+#   define restrict
+#endif
+/**
+ * @brief 文字列が文字を持っているか調べます。
+ * @param str 対象文字列へのポインタ
+ * @return false: nullptrか空白文字のみの文字列 true:それ以外
+ */
+static inline bool str_has_char(const char *str)
+{
+    if (NULL == str) return false;
+    bool ret = false;
+    for (; !ret && *str != '\0'; str++) ret = (*str != ' ');
+    return ret;
+}
+/**
+ * @brief 文字列が文字を持っているか調べます。
+ * @param io 書き換えるbool型変数へのポインタ、呼び出し後はポインタが指す変数にnew_valueが代入される
+ * @param new_value 新しい値
+ * @return ioが指すbool変数がもともと持っていた値
+ */
+static inline bool exchange_bool(bool* restrict const io, const bool new_value)
+{
+    const bool tmp = *io;
+    *io = new_value;
+    return tmp;
+}
+/**
+ * @brief fgetsで失敗したときにストリームをクリアしてループする関数
+ * @param s ストリームから読み取った文字列を格納するための領域へのポインタ
+ * @param buf_size ストリームから読み取った文字列を格納するための領域の大きさ
+ * @param stream FILE構造体へのポインタかstdin
+ * @param message_on_error エラー時に表示してループする
+ * @return 成功時は0, new line at the end of fileのときは-1
+ */
+static inline int fgets_wrap(char* restrict const s, size_t buf_size, FILE* restrict const stream, const char* restrict message_on_error)
+{
+    size_t i = 0;
+    for (bool first_flg = true; i < 100 && NULL == fgets(s, buf_size, stream); ++i) {
+        if (feof(stdin)) return -1;
+        if (!exchange_bool(&first_flg, false)) puts((message_on_error) ? message_on_error : "再入力してください");
+    }
+    if (100u == i) exit(1);//無限ループ防止
+    if (feof(stdin)) return 0;
+    //改行文字が入力を受けた配列にない場合、入力ストリームにごみがある
+    const size_t len = strlen(s);
+    //短すぎる入力
+    if (0 == len || (1 == len && '\n' == s[0])) return 1;
+    //長過ぎる入力
+    if ('\n' != s[len - 1]) {
+        //入力ストリームを掃除
+        while (fgetc(stream) != '\n');
+        return 2;
+    }
+    return 0;
+}
+/**
+ * @brief 標準入力から文字列の入力を受ける
+ * @param s ストリームから読み取った文字列を格納するための領域へのポインタ
+ * @param buf_size ストリームから読み取った文字列を格納するための領域の大きさ
+ * @param message 入力を受ける前にputsに渡す文字列。表示しない場合はnullptrか空白文字のみで構成された文字列へのポインタを渡す
+ * @param message_on_error エラー時に表示してループする
+ */
+static inline void input_str(char* restrict const s, size_t buf_size, const char* message, const char* restrict message_on_error)
+{
+    if (str_has_char(message)) puts(message);
+    size_t i = 0;
+    for (; i < 100u; ++i) {
+        //長過ぎる入力以降の無限ループ防止にerrnoをクリアする
+        errno = 0;
+        switch (fgets_wrap(s, buf_size, stdin, message_on_error)) {
+        case -1: return;//EOF
+        case 1://短すぎる入力
+        case 2://長過ぎる入力
+            continue;
+        default: goto after_loop;
+        }
+    }
+    if (100u == i) exit(1);//無限ループ防止
+after_loop:
+    strchr(s, '\n')[0] = '\0';
+}
+/**
+ * @brief 標準入力から入力を受け、unsigned int型に変換する
+ * @details fgetsしてstrtodしている。max, minの条件に合わないかエラー時はループ
+ * @details errnoの値を書き換える
+ * @param message 入力を受ける前にputsに渡す文字列。表示しない場合はnullptrか空白文字のみで構成された文字列へのポインタを渡す
+ * @param message_on_error エラー時に表示してループする
+ * @param max 入力値を制限する。最大値を指定
+ * @param min 入力値を制限する。最小値を指定
+ * @return 入力した数字、EOFのときは0
+ */
+static inline unsigned int input_uint(const char* message, const char* restrict message_on_error, const unsigned int max, const unsigned int min)
+{
+    if (str_has_char(message)) puts(message);
+    char s[30];
+    static_assert(sizeof(unsigned int) < 8, "err");
+    unsigned long t = 0;
+    size_t i = 0;
+    for (char* endptr = s; ((0 == t && endptr == s) || 0 != errno || t < min || max < t) && i < 100u; ++i) {
+        //長過ぎる入力以降の無限ループ防止にerrnoをクリアする
+        errno = 0;
+        switch (fgets_wrap(s, sizeof(s), stdin, message_on_error)) {
+        case -1: return 0;//EOF
+        case 1://短すぎる入力
+        case 2://長過ぎる入力
+            endptr = s;//ループ制御フラグとして流用
+            continue;
+        default: break;
+        }
+        t = strtoul(s, &endptr, 10);
+    }
+    if (100 == i) exit(1);//無限ループ防止
+    return ((unsigned int)(t));
+}
+#ifdef NO_STDC_STATIC_ASSERT
+#   undef static_assert
+#   undef NO_STDC_STATIC_ASSERT
+#endif
+#ifndef _countof
+#define _countof(arr) (sizeof(arr) / sizeof(*arr))
+#endif
+typedef struct
+{
+    char name[256];
+    unsigned int age;
+    unsigned int sex;
+}People;
+void InputPeople(People *data)
+{
+    do {
+        input_str(data->name, _countof(data->name), "名前を入力してください", "不正な長さの入力です、もう一度入力してください");
+        printf("%sでよろしいですか？？（「合ってます」：「1」を入力、「間違いました」：「1以外」を入力）\n", data->name);
+    } while (!feof(stdin) && 1 != input_uint(NULL, NULL, UINT_MAX, 0));
+    if (feof(stdin)) exit(1);
+    do {
+        data->age = input_uint("年齢を入力してください", NULL, UINT_MAX, 0);
+        printf("%dでよろしいですか？？（「合ってます」：「1」を入力、「間違いました」：「1以外」を入力）\n", data->age);
+    } while (!feof(stdin) && 1 != input_uint(NULL, NULL, UINT_MAX, 0));
+    if (feof(stdin)) exit(1);
+    do {
+        data->sex = input_uint("性別を入力してください（「男性」：「1」を入力、「女性」：「2」を入力）", NULL, 2, 1);
+        printf("%sでよろしいですか？？（「合ってます」：「1」を入力、「間違いました」：「1以外」を入力）\n", (1 == data->sex) ? "男性" : "女性");
+    } while (!feof(stdin) && 1 != input_uint(NULL, NULL, UINT_MAX, 0));
+    if (feof(stdin)) exit(1);
+    putchar('\n');
+}
+void ShowPeople(const People* data)
+{
+    printf(
+        "名前：%s\n"
+        "年齢：%d\n"
+        "性別：%s\n",
+        data->name, data->age, (data->sex == 1) ? "男性" : "女性"
+    );
+    putchar('\n');
+}
+int main(void)
+{
+    People data[3];
+    for (size_t i = 0; i < 3; i++) {
+        InputPeople(&data[i]);
+    }
+    for (size_t i = 0; i < 3; i++) {
+        ShowPeople(&data[i]);
+    }
+    return 0;
+}
+```
+[https://wandbox.org/permlink/b7Ev0PdEj3iu7JJP](https://wandbox.org/permlink/b7Ev0PdEj3iu7JJP)