回答編集履歴

add_book() を改善

2021/06/11 08:58

投稿

rubato6809

スコア1382

test CHANGED Viewed

@@ -402,46 +402,44 @@
-TPBook * add_book(char *name, char *number, TPBook *head)
+TPBook * add_book(char *name, char *number, TPBook *head) // 修正版
-{
+{
+    // 追加する構造体メモリを獲得
+    TPBook * tp = malloc(sizeof(TPBook));
+    // メンバを初期化する
+    tp->next = NULL;
+    strcpy(tp->name, name);
+    strcpy(tp->number, number);
+    if (head == NULL)    // 一回目は tp 自身が先頭になる
+        return tp;
     TPBook *p = head;
-    while (p && p->next) {
+    while (p->next)
         p = p->next;
-    }
     // この時点で p はリストの最後を指している
-    // 追加する構造体メモリを獲得
-    TPBook *tp = malloc(sizeof(TPBook));
-    // メンバを初期化する
-    tp->next = NULL;
-    strcpy(tp->name, name);
-    strcpy(tp->number, number);
-    if (p == NULL)      // 一回目は tp 自身が先頭になる
-        return tp;
     // 今作った構造体 tp をリストの最後に追加する。
     p->next = tp;       // 新しい構造体をリストの最後に

完全版を追加

2021/06/11 08:58

投稿

rubato6809

スコア1382

test CHANGED Viewed

@@ -207,3 +207,281 @@
 }
 ```
+---
+サービス。私の手元で動作確認した完全版。
+```C
+#include <stdio.h>
+#include <string.h>
+#include <stdlib.h>
+#include <ctype.h>
+typedef struct TelephoneBook {
+    char name[16];
+    char number[16];
+    struct TelephoneBook *next;
+} TPBook;
+TPBook * add_book(char *name, char *num, TPBook *head);
+void print_book(TPBook *p);
+void free_book(TPBook *p);
+int check_name(char *name);
+int check_number(char *number);
+int main(void)
+{
+    TPBook *head = NULL;
+    char name_number[33];
+    char name[16];   // 入力した名前を格納する
+    char number[16]; // 入力した電話番号を格納する
+    int check_na;    // 仕様通りに入力されてなければ1，\qが入力されていれば2が代入される
+    int check_nu;    // 仕様通りに入力されてなければ1が代入される
+    while (1) {
+        printf("input>");
+        fgets(name_number, sizeof(name_number), stdin);
+        sscanf(name_number, "%s %s", name, number);
+        check_na = check_name(name);
+        if (check_na == 2) {
+            print_book(head);
+            free_book(head);
+            return EXIT_SUCCESS;
+        }
+        if (check_na == 1) {
+            printf("error");
+            return EXIT_FAILURE;
+        }
+        check_nu = check_number(number);
+        if (check_nu == 1) {
+            printf("error");
+            return EXIT_FAILURE;
+        }
+        if (check_na == 0 && check_nu == 0) {
+            printf("Ok\n");
+        }
+        head = add_book(name, number, head);
+        print_book(head);     // show result
+    }
+}
+int check_name(char *name)
+{
+    char *chr1 = "\q";
+    char *chr2 = "\Q";
+    if (strcmp(name, chr1) == 0 || strcmp(name, chr2) == 0)
+        return 2;
+    int len = strlen(name);
+    if (!isupper(name[0]))       // 先頭は英大文字であるべし
+        return 1;
+    for (int i = 1; i < len; i++) {
+        if (!islower(name[i]))   // 残りは英小文字であるべし
+            return 1;
+    }
+    return 0;                    // ok
+}
+int check_number(char *number)
+{
+    /* format: 090-1234-5678 */
+    for (int i = 0; i <= 2; i++) {  // 0, 1, 2 が
+        if (!isdigit(number[i]))    // 数字でないなら
+            return 1;
+    }
+    for (int i = 4; i <= 7; i++) {  // 4, 5, 6, 7 が
+        if (!isdigit(number[i]))    // 数字でないなら
+            return 1;
+    }
+    for (int i = 9; i <= 12; i++) { // 9, 10, 11, 12 が
+        if (!isdigit(number[i]))    // 数字でないなら
+            return 1;
+    }
+    if (number[3] != '-' || number[8] != '-') {
+        return 1;
+    }
+    return 0;        // ok
+}
+TPBook * add_book(char *name, char *number, TPBook *head)
+{
+    TPBook *p = head;
+    while (p && p->next) {
+        p = p->next;
+    }
+    // この時点で p はリストの最後を指している
+    // 追加する構造体メモリを獲得
+    TPBook *tp = malloc(sizeof(TPBook));
+    // メンバを初期化する
+    tp->next = NULL;
+    strcpy(tp->name, name);
+    strcpy(tp->number, number);
+    if (p == NULL)      // 一回目は tp 自身が先頭になる
+        return tp;
+    // 今作った構造体 tp をリストの最後に追加する。
+    p->next = tp;       // 新しい構造体をリストの最後に
+    return head;        // 二回目以降は head そのまま
+}
+void print_book(TPBook *p)
+{
+    while (p) {
+        printf("%s : %s\n", p->name, p->number);
+        p = p->next;
+    }
+}
+void free_book(TPBook *p)
+{
+    while (p) { /* 次ポインタがNULLまで処理 */
+        TPBook *p2 = p->next;
+        free(p);
+        p = p2;
+    }
+}
+```

for 文のバグを修正

2021/06/10 13:31

投稿

rubato6809

スコア1382

test CHANGED Viewed

@@ -118,6 +118,12 @@
 それに、いちいちローカルな配列にコピーしなくても、そのまま判定できますよ。
+for (int i = 1; i <= len; i++) {    これはバグ
+for (int i = 1; i <  len; i++) {  　これが正しい
 ```C
 #include <ctype.h>
@@ -146,7 +152,7 @@
-    for (int i = 1; i <= len; i++) {
+    for (int i = 1; i < len; i++) {   // ！！条件を修正した！！
         if (!islower(name[i]))   // 残りは英小文字であるべし

main() にも #ifdef ORG を

2021/06/10 12:50

投稿

rubato6809

スコア1382

test CHANGED Viewed

@@ -96,9 +96,17 @@
 {
+#ifdef ORG
+    struct TelephoneBook *head = NULL;
+#else
-	struct TelephoneBook dummy = { "Dummy", "000-0000-0000", NULL };
+    struct TelephoneBook dummy = { "Dummy", "000-0000-0000", NULL };
-	struct TelephoneBook *head = &dummy;    // ダミーエントリをポイントして開始
+    struct TelephoneBook *head = &dummy;    // ダミーエントリをポイントして開始
+#endif
 ```

main() の修正を追加

2021/06/10 12:45

投稿

rubato6809

スコア1382

test CHANGED Viewed

@@ -83,6 +83,22 @@
     return head;
 }
+```
+質問者のコードは head = NULL; でスタートしており、それだとORGコードが動作するが、私のコードではSegmentation Fault を起こし動作しない。そこで例えば次のようにダミーエントリを作っておく手がある。当座をしのいで動作確認はできるはず。
+```C
+int main(void)
+{
+	struct TelephoneBook dummy = { "Dummy", "000-0000-0000", NULL };
+	struct TelephoneBook *head = &dummy;    // ダミーエントリをポイントして開始
 ```

修正した関数を追加

2021/06/10 12:38

投稿

rubato6809

スコア1382

test CHANGED Viewed

@@ -91,3 +91,89 @@
 P.S.
 それにしてもヘンな判定をしてますね。リスト操作をしようという段階になって、isdigit(), isupper(), islower() といった標準ライブラリ関数を知らないの？
+それに、いちいちローカルな配列にコピーしなくても、そのまま判定できますよ。
+```C
+#include <ctype.h>
+int check_name(char *name)
+{
+    char *chr1 = "\q";
+    char *chr2 = "\Q";
+    if (strcmp(name, chr1) == 0 || strcmp(name, chr2) == 0)
+        return 2;
+    int len = strlen(name);
+    if (!isupper(name[0]))       // 先頭は英大文字であるべし
+        return 1;
+    for (int i = 1; i <= len; i++) {
+        if (!islower(name[i]))   // 残りは英小文字であるべし
+            return 1;
+    }
+    return 0;                    // OK
+}
+int check_number(char *number)
+{
+    /* number[] format: 090-1234-5678 */
+    for (int i = 0; i <= 2; i++) {  // 0, 1, 2 が
+        if (!isdigit(number[i]))    // 数字でないなら
+            return 1;
+    }
+    for (int i = 4; i <= 7; i++) {  // 4, 5, 6, 7 が
+        if (!isdigit(number[i]))    // 数字でないなら
+            return 1;
+    }
+    for (int i = 9; i <= 12; i++) { // 9, 10, 11, 12 が
+        if (!isdigit(number[i]))    // 数字でないなら
+            return 1;
+    }
+    if (number[3] != '-' || number[8] != '-') {
+        return 1;
+    }
+    return 0;
+}
+```