文字列を比較して任意の単語が含まれているかを検出するプログラム

Question

C言語で、文章と英単語を入力し，大文字／小文字を区別せず英単語が文章中に含まれていれば，その旨を表示するプログラムを作成しなさい、という課題に取り組んでいます。
strlen,toupper(tolower),strncomp関数を必ず使わなければいけません。

strtok関数を使って半角スペースで文字列を区切って表示させるところまではできたのですがコアダンプというエラーが出て動かなくなってしまいます。

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

egg apple
egg

ans0 = 2
Segmentation fault (コアダンプ)

該当のソースコード

#include <string.h>
#include <stdio.h>
#include<stdio.h>
#include<ctype.h>

int main(void)
{
    int  str_len, word_len, n, ans;
	char in_str[50] = {};
    char in_word[15];
    char large_str[50] = {};
    char large_word[15] = {};
	char *p, *w;

    str_len  = strlen(in_str);
    word_len = strlen(in_word);

    printf(">>");
    scanf("%[^\n]",in_str);
    printf(">>");
    scanf("%[^\n]",in_word);
    for(n = 0; n < str_len; n++) {
        large_str[n] = toupper(in_str[n]);
    }
    for(n = 0; n < word_len; n++) {
        large_word[n] = toupper(in_word[n]);
    }

    w = large_word;
	p = strtok(large_str," ");

	//1つ目のトークン表示
    ans = strncmp(p, w, word_len);
    if(ans == 0)
        printf("Hit!!\n");
	puts(p);

	//トークンがNULLになるまでループ
	while(p!=NULL){

		//2回目以降は第一引数はNULL
		p = strtok(NULL," ");
		if(p!=NULL){
            ans = strncmp(p, w, word_len);
                if(ans == 0)
                    printf("Hit!!\n");
			puts(p);
		}
	}
	return 0;
}

試したこと

//1つ目のトークン表示
    ans = strncmp(p, w, word_len);
    if(ans == 0)
        printf("Hit!!\n");
	puts(p);

ansへの代入の後にprintf関数を入れてansを表示させたところ、一回目のstrncmp関数で何が入ってきても0が代入されていました。
これでは比較できていないのでしょうか？

Answer 1

ansへの代入の後にprintf関数を入れてansを表示させたところ、一回目のstrncmp関数で何が入ってきても0が代入されていました。

ans だけでなく、str_len、word_len、n、in_str、in_word、large_str、
large_word、p、w を表示して何が入っているか見てください。

追記

#include <stdio.h>  // scanf, printf, puts
#include <string.h> // strlen, strncmp, strtok
#include <ctype.h>  // toupper

int main(void)
{
    char str[50], word[15];

    printf(">>"), scanf("%[^\n]", str);
    printf(">>"), scanf(" %[^\n]", word);
    for (int i = 0; str[i]; i++) str[i] = toupper(str[i]);
	int len = strlen(word);
    for (int i = 0; i < len; i++) word[i] = toupper(word[i]);
	char *p = strtok(str," ");
	while (p && strncmp(p, word, len)) p = strtok(NULL, " ");
    puts(p ? "その英単語は、英文に含まれています。"
	       : "その英単語は、英文に含まれていません。");
}

追記2
次のように書いても実行結果は同じです。

#include <stdio.h>  // scanf, printf, puts
#include <string.h> // strlen, strncmp, strtok
#include <ctype.h>  // toupper

int main(void)
{
    char str[50], word[15];

    printf(">>");
	scanf("%[^\n]", str);
    printf(">>");
	scanf("%s", word);
    for (int i = 0; str[i] != '\0'; i++)
		str[i] = toupper(str[i]);
    int len = strlen(word);
    for (int i = 0; i < len; i++)
		word[i] = toupper(word[i]);
    char *p = strtok(str," ");
    while (p != NULL && strncmp(p, word, len) != 0)
		p = strtok(NULL, " ");
	if (p != NULL)
		puts("その英単語は、英文に含まれています。");
	else
        puts("その英単語は、英文に含まれていません。");
}

!= '\0'、!= NULL、!= 0　はすべて省略できます。
これらは != の左の項が 0 ではないかどうかを判定しています。
for の第2式も while や if の式もその式の値が 0 でないかどうかで判定します。

Comments

[naki_a]

起床してからアドバイスに従い全部表示させたところ、ここ3日ほど自分がとんでもないことをしているのに気づきました。ありがとうございます。

[kazuma-s]

どのようにして解決したですか？
コードを質問に追記してください。

[naki_a]

追記いたしました、ご指摘ありがとうございます。

Answer 2

このように解決しました。

#include <string.h>
#include <stdio.h>
#include<stdio.h>
#include<ctype.h>

int main(void)
{
    int  str_len, word_len, n, ans;
	char in_str[50] = {};
    char in_word[15];
    char large_str[50] = {};
    char large_word[15] = {};
	char *p, *w;

    printf(">>");
    scanf("%[^\n]",in_str);
    printf(">>");
    scanf("%s",in_word);

    str_len  = strlen(in_str);
    word_len = strlen(in_word);
    printf("str_len = %d\nword_len = %d\n", str_len, word_len);

    for(n = 0; n < str_len; n++) {
        large_str[n] = toupper(in_str[n]);
    }
    printf("large_str = %s\n",large_str);
    for(n = 0; n < word_len; n++) {
        large_word[n] = toupper(in_word[n]);
    }
    printf("large_word = %s\n",large_word);

    w = large_word;
    puts(w);
	p = strtok(large_str," ");

    ans = strncmp(p, w, word_len);
    if(ans == 0)
        printf("その英単語は、英文に含まれています。\n");
    else{
        while(p!=NULL){
            p = strtok(NULL," ");
            if(p!=NULL){
                ans = strncmp(p, w, word_len);
                    if(ans == 0){
                        printf("その英単語は、英文に含まれています。\n");
                        break;
                    }
                puts(p);
            }
        }
    }
    if(ans != 0) {
        printf("その英単語は、英文に含まれていません。\n");
    }

	return 0;
}

Comments

[kazuma-s]

2つめの scanf の "%[^\n]" をどうするんだろうと思っていましたが、"%s" にしたんですね。
" %[^\n]" のように % の前にスペースを置くという手もあります。

ちょっと気になるのが strncmp を使っていることで、
これだと、str = "I am a teacher."、word = "tea" が
「含まれています」になってしまいます。

[kazuma-s]

#include <stdio.h> が 2つあることはまあいいとして、
= {}; のように空の初期化子は標準の C では許されていません。
また、in_word 以外の文字配列はすべて初期化して、これだけ初期化しないのはなぜでしょうか？

strncmp が 2回あって、同じ printf も 2回あるのはもっと簡単に書けます。
私の回答に追記します。

[naki_a]

#include<stdio.h>が2個あるのは多分コピペしたりしてるうちに重複してしまったんだと思います……。
in_wordをしてないのもおそらくこの配列は後でつけ加えたため初期化を忘れていたのだと思うので特に意味は無いです、= {}; がダメなのも初めて知りました。
追記拝見しました。私のプログラムの半分以下程度の長さで実装されていてここまでコンパクトになるんだと驚きました。
よろしければお答え頂けると嬉しいのですが、(p && strncmp(p, word, len)はどういう処理をしているんでしょうか？
strncmp関数は文字列が一致すれば0が返ってくるという挙動だと理解しているのですが、pとandをとっているのはどういう意味があるんでしょうか？？