回答編集履歴

add_book() を改善

2021/06/11 08:58

投稿

rubato6809

スコア1382

answer CHANGED Viewed

@@ -200,26 +200,25 @@
     return 0;        // ok
 }
-TPBook * add_book(char *name, char *number, TPBook *head)
+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));
+    TPBook * tp = malloc(sizeof(TPBook));
     // メンバを初期化する
     tp->next = NULL;
     strcpy(tp->name, name);
     strcpy(tp->number, number);
-    if (p == NULL)      // 一回目は tp 自身が先頭になる
+    if (head == NULL)    // 一回目は tp 自身が先頭になる
         return tp;
+    TPBook *p = head;
+    while (p->next)
+        p = p->next;
+    // この時点で p はリストの最後を指している
     // 今作った構造体 tp をリストの最後に追加する。
     p->next = tp;       // 新しい構造体をリストの最後に
     return head;        // 二回目以降は head そのまま

完全版を追加

2021/06/11 08:58

投稿

rubato6809

スコア1382

answer CHANGED Viewed

@@ -102,4 +102,143 @@
     }
     return 0;
 }
+```
+---
+サービス。私の手元で動作確認した完全版。
+```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

answer CHANGED Viewed

@@ -58,6 +58,9 @@
 P.S.
 それにしてもヘンな判定をしてますね。リスト操作をしようという段階になって、isdigit(), isupper(), islower() といった標準ライブラリ関数を知らないの？
 それに、いちいちローカルな配列にコピーしなくても、そのまま判定できますよ。
+for (int i = 1; i <= len; i++) {    これはバグ
+for (int i = 1; i <  len; i++) {  　これが正しい
 ```C
 #include <ctype.h>
 int check_name(char *name)
@@ -72,7 +75,7 @@
     if (!isupper(name[0]))       // 先頭は英大文字であるべし
         return 1;
-    for (int i = 1; i <= len; i++) {
+    for (int i = 1; i < len; i++) {   // ！！条件を修正した！！
         if (!islower(name[i]))   // 残りは英小文字であるべし
             return 1;
     }

main() にも #ifdef ORG を

2021/06/10 12:50

投稿

rubato6809

スコア1382

answer CHANGED Viewed

@@ -47,8 +47,12 @@
 ```C
 int main(void)
 {
+#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
 ```
 P.S.

main() の修正を追加

2021/06/10 12:45

投稿

rubato6809

スコア1382

answer CHANGED Viewed

@@ -43,6 +43,14 @@
 }
 ```
+質問者のコードは head = NULL; でスタートしており、それだとORGコードが動作するが、私のコードではSegmentation Fault を起こし動作しない。そこで例えば次のようにダミーエントリを作っておく手がある。当座をしのいで動作確認はできるはず。
+```C
+int main(void)
+{
+	struct TelephoneBook dummy = { "Dummy", "000-0000-0000", NULL };
+	struct TelephoneBook *head = &dummy;    // ダミーエントリをポイントして開始
+```
 P.S.
 それにしてもヘンな判定をしてますね。リスト操作をしようという段階になって、isdigit(), isupper(), islower() といった標準ライブラリ関数を知らないの？
 それに、いちいちローカルな配列にコピーしなくても、そのまま判定できますよ。

修正した関数を追加

2021/06/10 12:38

投稿

rubato6809

スコア1382

answer CHANGED Viewed

@@ -44,4 +44,47 @@
 ```
 P.S.
-それにしてもヘンな判定をしてますね。リスト操作をしようという段階になって、isdigit(), isupper(), islower() といった標準ライブラリ関数を知らないの？
+それにしてもヘンな判定をしてますね。リスト操作をしようという段階になって、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;
+}
+```