こんにちは(こんばんわ?)。
わたしは某大学の学生です。
現在課題のためにプログラムを制作中なのですが、
どうしてもエラーが消えなくて困っています。
提出期限がすでに切れているので、大至急お答え願います!たすけてー!
エラー内容:セグメンテーション違反
(実行時のみ、コンパイルエラーはなし)
エラー箇所(多分):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ベストアンサー
- 回答日時:
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;
}
}
この回答への補足
1.string.hは入ってます。
2.parray_sizeはプリプロセッサで、
#define parray_size 10000
の様に、定義しています。
3.これが呼び出し部分です。
/*(前略)*/
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);
}
tailkuppaさん回答ありがとう御座いました!
問題のほうはめでたく解決しました!
サンプルのほう、とっても役に立ちました。
ではでは~。
No.5
- 回答日時:
下の者ですが、大ボケをかましてしまいました。
>2の対策ですが、strncmpという関数がありますので、ご検討下さい。
これですが、buffer の大きさをstrlenで求めて(これを例えば、buf_sizeとします)、
for(i=0;i< parray_size;i++)
を
for(i=0;i< parray_size-buf_size;i++)
にすればOKですね…
後、parray_sizeも同様にstrlenで求めた方が安全ですね^^;
速度が重要でなければ、strlenで計算する方をおすすめします。
strlenも\0で終了している必要がありますのでご注意を
No.4
- 回答日時:
セグメンテーション違反 は確保した範囲外のメモリにアクセスした場合におこります。
このプログラムで、考えられるのは
1. parray_size が p 用に確保されたサイズより大きく、ループ内で、p[i] が範囲外をさしている
2. strcmp は、いずれかが、'\0' まで探索をします。
つまり、このプログラムで、pもしくは、bufferの末尾に'\0'が含まれていなければ、探索は範囲外に及ぶ
3. p、buffer のいずれかに、NULL が渡されている。
の可能性です。
1の対策は、他の部分を見ないことには、、、
2の対策ですが、strncmpという関数がありますので、ご検討下さい。
3の対策は、関数の最初にポインタのチェックを加えることです。
まず、どの箇所で、落ちているのかを確認してください。
fprintf(stderr, ".."); // .. には何か好きなメッセージ
で、ポイントポイントで、メッセージを出力し、何処まで表示されたかを確認するのが簡単で、効果的な方法です。
以上、簡単ですが。
参考URL:http://www.linux.or.jp/JM/html/LDP_man-pages/man …
No.3
- 回答日時:
質問のソースを見た限りでは何も言えないし、tailkuppaさまのプログラムならエラーがないと思われるので、ちょっと余談を。
char *p[]
という引数の定義ですが、このように書くように推奨されてはいるものの、意味があいまいになりかねないので
char **p
を私はオススメします。
どちらも同じ表現ですが、人間から解釈すると前者は配列のポインタを渡し、後者はポインタのポインタを渡すと解釈できます。
しかし、実際に内部では、ポインタのポインタが渡されているのです。
焦っているのに余談なんて失礼しました。
配列とポインタは似通っていて、ポインタを要求されるところでは配列を渡すことができる場合が多いのですが、多次元になるとそれが通用しない場合があります。
二次元配列、ポインタの配列、配列のポインタ、ポインタのポインタ、これらは混同しやすいので微妙に使い分けることが重要です。
ポインタマニアの、はぽるんでした。
はぽるんさん回答ありがとう御座いました!
問題のほうはめでたく解決しました!
tailkuppaさんのサンプルを元に試行錯誤。
その結果、今まで*p[]の初期化に
for(i=0;i < parray_size;i++)
p[i]=NULL;
のようにヌルポインタを使っていたものを、
for(i=0;i < parray_size;i++)
p[i]="\0";
の様にしたらスッパリ解決しました。
………なんででしょ?(笑)
はぽるんさんはポインタに詳しいんですよね?
結局strcmpはあんまり関係なかったようです…
んー…Cは奥が深いっす。
それではこのへんで~。
お探しのQ&Aが見つからない時は、教えて!gooで質問しましょう!
関連するカテゴリからQ&Aを探す
おすすめ情報
- ・漫画をレンタルでお得に読める!
- ・人生のプチ美学を教えてください!!
- ・10秒目をつむったら…
- ・あなたの習慣について教えてください!!
- ・牛、豚、鶏、どれか一つ食べられなくなるとしたら?
- ・【大喜利】【投稿~9/18】 おとぎ話『桃太郎』の知られざるエピソード
- ・街中で見かけて「グッときた人」の思い出
- ・「一気に最後まで読んだ」本、教えて下さい!
- ・幼稚園時代「何組」でしたか?
- ・激凹みから立ち直る方法
- ・1つだけ過去を変えられるとしたら?
- ・【あるあるbot連動企画】あるあるbotに投稿したけど採用されなかったあるある募集
- ・【あるあるbot連動企画】フォロワー20万人のアカウントであなたのあるあるを披露してみませんか?
- ・映画のエンドロール観る派?観ない派?
- ・海外旅行から帰ってきたら、まず何を食べる?
- ・誕生日にもらった意外なもの
- ・天使と悪魔選手権
- ・ちょっと先の未来クイズ第2問
- ・【大喜利】【投稿~9/7】 ロボットの住む世界で流行ってる罰ゲームとは?
- ・推しミネラルウォーターはありますか?
- ・都道府県穴埋めゲーム
- ・この人頭いいなと思ったエピソード
- ・準・究極の選択
デイリーランキングこのカテゴリの人気デイリーQ&Aランキング
-
Run-Time Check Failure #3とい...
-
init関数の意味
-
コンストラクタでnewを失敗した...
-
トリプルポインタが必須!とな...
-
基本アルゴリズムの『返す』の...
-
free()への引数について
-
C言語のポインタに直接アドレス...
-
ポインタについてアドバイスお...
-
C言語でのconstを返す関数
-
参照の参照、struct record **s...
-
_TCHAR、LPCWSTRについて
-
ポインタって?
-
C言語 よく使うのは?
-
IStream / VBA
-
ExcelVBAでのkernel32(64bit)
-
C言語ソースコードの解析
-
ファイルポインタのヘッダーフ...
-
c言語で任意のファイルから読み...
-
単方向リストの解釈
-
クイックソートの比較交換回数...
マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング
-
C言語のポインタに直接アドレス...
-
init関数の意味
-
セグメントエラー
-
fopne で失敗する原因
-
戻り値で構造体を返すことは可...
-
Run-Time Check Failure #3とい...
-
ExcelVBAでのkernel32(64bit)
-
LPSTR型の初期化について
-
C言語でのconstを返す関数
-
ハンドルはポインタか
-
【C言語】戻り値が構造体の関数
-
nullポインタを逆参照とは?
-
基本アルゴリズムの『返す』の...
-
参照型で受け取った引数をポイ...
-
#define NULL ((void *)0) の弊害
-
アプリを32bitから64bit移行
-
ハンドル、アドレス、ポインタ...
-
デバイスハンドルとは?
-
C++で関数ポインタから関数名を...
-
Cで作成したDLL関数をVBから呼...
おすすめ情報