C言語に関する質問でした

Question

前提・実現したいこと

C言語

発生している問題・エラーメッセージ

問題点１

問題点２

試したこと

Answer 1

• fgetsは改行\nも含めて読み込むため、キーワードとして入力する文字列末尾には、Enterキーによる改行が付いている。これを取り除かなければ、(比較対象の単語には改行は付いていないので)すべての比較が一致しない。
• fgetsの第2引数に、格納先の配列の要素数+1を指定してはいけない。配列の全要素に文字データが格納される状況で、自動的に付加されるヌル文字\0が配列の範囲を超えて書き込まれる。sizeofを利用して、配列の要素数を指定すべきだ。
• ファイルから読み込んだ文字列から単語を切り出す際、単語の長さが0であっても総単語数のカウンターを1増やしている。改行を含む文字列で総単語数が1増えてしまうのはこれが原因だ。カウンターの処理の前に、単語の長さを調べれば対処できる。
• fopenに失敗したら、ファイル処理を続けてはいけない。
• 配列名をそのまま使えばいい状況で、意味もなくポインター変数を使うのはやめよう。p_word, p_key, bufは、今回のコードではすべて不要だ。
• if文で単語を構成する文字かどうかをチェックするさい、isalnumを使ったチェックを二重に行なっている。一度目のチェックが成立しないなら、単語を構成しない文字であることは自明なので、チェックは一度でいい。つまり、if (alnum(...)) {...} else {...}の形で書ける。
• 文字列のクリアに配列の全要素を0クリアする必要はなく、先頭要素にヌル文字\0を設定するだけでいい。ヌル文字\0より後ろは文字列処理に使われないからだ。

#include <ctype.h>
#include <stdio.h>
#include <string.h>
#define SIZE 5000

int main(void)
{
    int target_count = 0;
    int all_count = 0;

    char keyword[32];
    printf("Input a word for search and count: ");
    fgets(keyword, sizeof(keyword), stdin);
    // 入力文字列の末尾に改行があれば、それを取り除く
    char *last = keyword + strlen(keyword) - 1;
    if (last >= keyword && *last == '\n') {
        *last = '\0';
    }
    printf("Input keyword is '%s'\n", keyword);

    FILE *fp;
    char text[SIZE];
    fp = fopen("test.txt", "r");
    if (fp == NULL) {
        printf("cannot open\n");
        return 1;
    }

    char current_word[32];
    int j = 0;
    while (fgets(text, sizeof(text), fp) != NULL) {
        for (int i = 0; text[i] != '\0'; i++) {
            if (isalnum(text[i])) {
                current_word[j] = text[i];
                j++;
            } else {
                current_word[j] = '\0';
                if (j == 0) {
                    continue;
                }
                all_count++;
                printf("current word is '%s'\n", current_word);
                if (strcasecmp(keyword, current_word) == 0) {
                    target_count++;
                }
                current_word[0] = '\0';
                j = 0;
            }
        }
    }
    fclose(fp);

    printf("全単語数: %d\n", all_count);
    printf("単語数: %d\n", target_count);
}

Comments

[Daregada]

かぶった。そしてだいたい同じ指摘をしている。

Answer 2

fgets で読み込むと '\n' を含むので次のようにして削除してください。

     printf("Input a word for search and count: ");
     fgets(p_key, 32+1, stdin);
+	char *p = strchr(p_key, '\n');
+	if (p != NULL) *p = '\0';
     printf("Input key = %s\n", p_key);

それから、fgets の第2引数で、32+1 とか size+1 のように 1足していますが、
これはいけません。実際に存在する配列の要素数にしてください。

fopen に失敗した場合でも、メッセージを出すだけで次に進み fgets を
実行するのは変です。return 1; などで、main を終了してください。

単語数が実際より増えるのは、行頭にスペースがあったり、単語間のスペースが
2個以上ある場合です。もう少しよく考えてみてください。

ポインタ p_key や p_word は、配列 key や word の先頭を指している
だけで一度も変更していません。
p_key や p_word を使用せず、key や word と書けばいいでしょう。
buf も不要です。text で十分です。

追記
解決済みなったので別解です。

#include <stdio.h>   // printf, fgets, fopen, fclose, fscanf, perror
#include <string.h>  // strchr, strcasecmp

int main(void)
{
    int cnt = 0, all = 0, *p;
    char key[100], word[100];

    FILE *fp = fopen("test.txt","r");
    if (fp == NULL) { perror("test.txt"); return 1; }

    printf("Input a word for search and count: ");
    if (!fgets(key, sizeof key, stdin)) return 2;
    if (p = strchr(key, '\n')) *p = '\0';
    printf("Input key = %s\n", key);

    while (1) {
		fscanf(fp, "%*[^a-zA-Z0-9]");  // 英数字以外を読み飛ばす
		if (fscanf(fp, "%99[a-zA-Z0-9]", word) != 1) break;  // 単語を取得
        all++;
        if (!strcasecmp(key, word)) cnt++;
	}
    fclose(fp);

    printf("全単語数：%d\n", all);
    printf("単語数：%d\n", cnt);
}