たすけて！

質問

こんにちは（こんばんわ？）。
わたしは某大学の学生です。
現在課題のためにプログラムを制作中なのですが、
どうしてもエラーが消えなくて困っています。
提出期限がすでに切れているので、大至急お答え願います！たすけてー！

エラー内容：セグメンテーション違反
　　　　　（実行時のみ、コンパイルエラーはなし）
エラー箇所（多分）：strcmpの使い方？

ソース（一部）：
（前略）
int Lookup(char *p[],char buffer[])
{
int i;

for(i=0;i< parray_size;i++)
if(!strcmp(p[i],buffer))
break;

if(i== parray_size) return 0;
else return 1;
}

回答No.2

tailkuppa · ベストアンサー

1. #include <string.h> は入っていますか？
　今回は問題ないけど、引数が間違っていたりするときに参考になるので、入れておいた方が無難ですよ。

2. parray_size はちゃんと p のサイズになっていますか？
　というか、どこで宣言していますか？
　これがp の配列数より大きいと質問のようなエラーになりそうです。

3. この関数(Lookup)を呼んでいる箇所のソースが欲しいです。
　そっちで間違っている可能性のほうがありそう。

4. とりあえず、下記のような感じで動くことを確認しました。

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

int parray_size = 3;

int
main(void) {

　static char *p[3] = { "ほげ", "ふが", "もげ" };
　char buffer[10];

　printf ("%d\n", Lookup(p, "ない"));
　printf ("%d\n", Lookup(p, "ほげ"));

}

int Lookup(char *p[],char buffer[])
{
　int i;

　for (i=0;i< parray_size;i++) {
　　if (!strcmp(p[i],buffer)) {
　　　break;
　　}
　}

　if (i== parray_size) {
　　return 0;
　}
　else {
　　return 1;
　}
}

お礼

tailkuppaさん回答ありがとう御座いました！
問題のほうはめでたく解決しました！
サンプルのほう、とっても役に立ちました。
ではでは～。

補足

１．string.hは入ってます。

２．parray_sizeはプリプロセッサで、
　　#define parray_size 10000
　　の様に、定義しています。

３．これが呼び出し部分です。
　　/*（前略）*/　　
void main(void)
{
　　char *p[parray_size];
　　char buffer[parray_size];
　　/*（中略）*/　　
　　printf("What's a word to lookup? ");

　　scanf("%s",buffer);

　　if(Lookup(p,buffer))
　　　　printf("%s is exist.\n\n", buffer);
　　else printf("%s is not exist.\n\n", buffer);
}