zsh: segmentation fault が出たのですが原因がわかりません

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

typedef struct ufo {
char time[50];
char city[50];
char shape[50];
double lat;
double lon;
}UFO;

void printUfo(UFO ufo) {
printf("%s,%s,%s,%lf,%lf\n", ufo.time, ufo.city, ufo.shape, ufo.lat, ufo.lon);
}

int main() {
FILE *fp;
char line[256];
UFO ufo, list[80333];
int num=0;

if ((fp = fopen("ufo_sighting_data.csv", "r")) == NULL) {
    printf("file open error!\n");
    exit(EXIT_FAILURE);
}

while (fgets(line, 256, fp) != NULL) {
    strcpy(ufo.time, strtok(line, ",\n"));
    strcpy(ufo.city, strtok(NULL, ",\n"));
    strcpy(ufo.shape,  strtok(NULL, ",\n"));
    ufo.lat = atof(strtok(NULL, ",\n"));
    ufo.lon = atof(strtok(NULL, ",\n"));
        
    list[num] = ufo;
        
    num++;
}
fclose(fp);

for (int i=0; i<num; i++) {
    printUfo(list[i]);
}

return 0;

}

コード

tatsu99

2021/01/16 08:30

インデントがずれてますね。 ```ここに言語を入力コード ``` が表示されていれば ```① ② ``` ①の個所にCと入力し ②の個所にあなたのソースを貼り付けます。つまり、以下のようにします。 ```C あなたのソースをはりつけたもの ``` そうすると、きれいに、インデントがそろいます。もし、 ```ここに言語を入力コード ``` がなければ <code>をクリックしてください。 ```ここに言語を入力コード ``` が表示されます。

tatsu99

2021/01/16 08:33

ufo_sighting_data.csvの内容を提示してください。（行数が多いなら、10程度でも構いません）又、それは、およそ何行ありますか。

wwwwwa.k.awww

2021/01/16 08:41

9/9/1995 22:50,silverdale,,47.6447222,-122.6936111 9/9/1996 05:30,savannah,disk,32.0833333,-81.1 9/9/1996 08:30,san francisco,oval,37.775,-122.4183333 9/9/1997 16:00,las vegas,formation,36.175,-115.1363889 9/9/1997 21:30,roseburg,light,43.2166667,-123.3405556 9/9/1998 01:25,tomah,cylinder,43.9786111,-90.5038889 9/9/1999 00:40,renton,light,47.4830556,-122.2158333 9/9/1999 05:00,los altos,circle,37.3852778,-122.1130556 9/9/1999 12:35,shasta/sand flat,fireball,40.650984,-122.363897 9/9/1999 19:00,marathon,disk,24.7133333,-81.0905556 9/9/1999 20:45,charlottetown (canada),disk,46.233333,-63.133333 9/9/1999 22:00,bothell,other,47.7625,-122.2041667 9/9/1999 22:00,mount shasta,,41.31,-122.3094444 9/9/1999 22:00,santa cruz,other,36.9741667,-122.0297222 9/9/1999 22:00,shasta/sand flat,disk,40.650984,-122.363897 80332行あります。

行動規範の内容に同意します

回答3件

ベストアンサー

原因は２つあります。
1つ目は、list[80333]が大きすぎるためです。これの解決方法は、kazuma-sさんが提示しています。
もう１つは、データの１行目のように
9/9/1995 22:50,silverdale,,47.6447222,-122.6936111
のような,,のデータがあることが原因です。
,,の中にはデータがないので、strtokはNULLを返します。
従って、strcpy又はatofでセグメンテーションフォールトになります。
対策としては
1案：,,のデータを修正する。（必ずデータを埋める。）
2案：,,があることを前提にソースを修正する。

とりあえず、２案を採用するとして、
,,のデータの場合、文字列なら"",数値なら0.0をセットするようにしました。
確認は、提示されたデータで行いました。80333行のデータでは行っていません。

下記が修正後のソースです。

C
1#include <stdio.h>
2#include <string.h>
3#include <stdlib.h>
4typedef struct ufo {
5    char time[50];
6    char city[50];
7    char shape[50];
8    double lat;
9    double lon;
10} UFO;
11UFO list[80333];
12void printUfo(UFO ufo)
13{
14    printf("%s,%s,%s,%lf,%lf\n", ufo.time, ufo.city, ufo.shape, ufo.lat,
15           ufo.lon);
16}
17
18int main()
19{
20    FILE *fp;
21    char line[256];
22    UFO ufo;
23    int num = 0;
24	char *p;
25    if ((fp = fopen("ufo_sighting_data.csv", "r")) == NULL) {
26        printf("file open error!\n");
27        exit(EXIT_FAILURE);
28    }
29
30    while (fgets(line, 256, fp) != NULL) {
31		p = strtok(line, ",\n");
32        if (p==NULL){
33        	strcpy(ufo.time, "");
34		}else{
35        	strcpy(ufo.time, p);
36        }
37        p = strtok(NULL, ",\n");
38        if (p==NULL){
39	        strcpy(ufo.city, "");
40		}else{
41        	strcpy(ufo.city, p);
42        }
43        p = strtok(NULL, ",\n");
44        if (p==NULL){
45        	strcpy(ufo.shape,"");
46		}else{
47        	strcpy(ufo.shape, p);
48        }
49        p = strtok(NULL, ",\n");
50        if (p==NULL){
51        	ufo.lat = 0.0;
52		}else{
53        	ufo.lat = atof(p);
54        }
55        p = strtok(NULL, ",\n");
56        if (p==NULL){
57	        ufo.lon = 0.0;
58		}else{
59        	ufo.lon = atof(p);
60		}
61        list[num] = ufo;
62
63        num++;
64    }
65    fclose(fp);
66
67    for (int i = 0; i < num; i++) {
68        printUfo(list[i]);
69    }
70
71    return 0;
72}
73

投稿2021/01/16 09:53

編集2021/01/16 09:55

tatsu99

総合スコア5438

wwwwwa.k.awww

2021/01/16 11:13

ありがとうございます。解決しました！

行動規範の内容に同意します

sizeof(UFO) は 168 なので、sizeof(list) は 13495933 です。
12メガバイト以上になります。
Linux を使っているのなら、ulimit -a または ulimit -s でスタックサイズを
調べてみてください。8192kbytes になっていませんか？

static UFO list[80333]; のように静的にメモリを確保してみてください。

投稿2021/01/16 09:08

kazuma-s

総合スコア8224

wwwwwa.k.awww

2021/01/16 11:15

ありがとうございます。参考になりました。

行動規範の内容に同意します

UFO ufo, list[80333];

ローカル変数は一般的にスタックに取られますんで、あんまし大きいサイズの変数を定義するとスタックを食いつぶしてエラーとなります。
グローバル変数にしてみては。

投稿2021/01/16 08:19

y_waiwai

総合スコア87774

wwwwwa.k.awww

2021/01/16 08:30

試してみましたが、同じエラーが出ました。

y_waiwai

2021/01/16 08:34

ならあとは、strtokのところでNULLが帰ってませんか。 C言語のコードを書くなら、デバッグできる環境を整えましょう。 Eclipseや、WindowsならVisualStudioなど。コードの任意の場所で実行を止め、変数のナカミを見ることができます。そこから１行づつ実行して、コードの流れを見れるようになりますそうすれば、アテズッポでコードを書かなくて済むようになります。

行動規範の内容に同意します

あなたの回答