segmentation fault

フォーラム(掲示板)ルール
フォーラム(掲示板)ルールはこちら  ※コードを貼り付ける場合は [code][/code] で囲って下さい。詳しくはこちら
grayxx

segmentation fault

#1

投稿記事 by grayxx » 13年前

以下のプログラムを作ったのですが降順か昇順かを決めたところで、segmentation faultと出て止まってしまいます。
どうしたら良いでしょうか
#include<stdio.h>
#include<string.h>
#include<time.h>

#define N 5

char *A[N];
char B[N][128];



void Sort(char *S[], int n)
{
int a, b, c, o;
char *temp;

printf("ascending order is 0.\ndescending order is 1\n");
scanf("%d", &o);
switch(o){
case 0:

for(a=0;a<n;a++){
for(b=0;b<n;b++){
if((strcmp(S, S[a]))>0){
temp=S[a];
S[a]=S;
S=temp;
}
}
}
}
switch(o){
case 1:

for(a=0;a<n;a++){
for(b=0;b<n;b++){
if((strcmp(S, S[a]))<0){
temp=S[a];
S[a]=S;
S=temp;
}
}
}
}
}


int main(void){
int i;
FILE *fp;
char filename[256];
printf("please enter file \n");
gets(filename);
strcat(filename,".txt");
fp=fopen(filename, "r");

if(fp==NULL){
perror("faile no open\n");
return -1;
}
else{
fscanf(fp,"%s",&A[N]);

}

printf("\n\n sort start\n\n");

Sort(A, N);

printf("sort end\n");
for(i=0;i<N;i++) {
printf("%s ", A);
}
printf("\n");
}

よろしくお願いします

box
記事: 2002
登録日時: 15年前

Re: segmentation fault

#2

投稿記事 by box » 13年前

Sort関数におけるswitch文の使い方がユニーク(もっと簡潔に書けるはず)なのは
いったん横に置くとして…。
そもそも
grayxx さんが書きました:

コード:

#define N 5

char *A[N];
           fscanf(fp,"%s",&A[N]);
このコードでファイルの中身を読み取れていると解釈すること自体に
誤りがあるように思います。
Nは5という固定値で、かつ、A[]という配列の定義範囲外の領域です。

main関数のfscanf()で配列の定義範囲外の領域に書き込んだというバグが、
Sort関数における昇順・降順の選択のところで顕在化している、ということでありましょう。
バグのないプログラムはない。
プログラムは思ったとおりには動かない。書いたとおりに動く。

アバター
みけCAT
記事: 6734
登録日時: 15年前
住所: 千葉県
連絡を取る:

Re: segmentation fault

#3

投稿記事 by みけCAT » 13年前

配列Aに入っているポインタが(グローバル変数なので)NULLのままstrcmpに渡されているからではないでしょうか?
fscanfを「正しく」使って文字列(=ポインタとしてはゴミ)を代入したところで、アクセス違反は出ると思いますが。
複雑な問題?マシンの性能を上げてOpenMPで殴ればいい!(死亡フラグ)

閉鎖

“C言語何でも質問掲示板” へ戻る