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

Question

#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;

}

コード

Comments

[tatsu99]

インデントがずれてますね。
```ここに言語を入力
コード
```
が表示されていれば

```①
②
```

①の個所にCと入力し
②の個所にあなたのソースを貼り付けます。

つまり、以下のようにします。

```C
あなたのソースをはりつけたもの
```

そうすると、きれいに、インデントがそろいます。

もし、
```ここに言語を入力
コード
```
がなければ
<code>をクリックしてください。
```ここに言語を入力
コード
```
が表示されます。

[tatsu99]

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

[wwwwwa.k.awww]

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行あります。

Answer 1

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

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

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

#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;
UFO list[80333];
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;
    int num = 0;
	char *p;
    if ((fp = fopen("ufo_sighting_data.csv", "r")) == NULL) {
        printf("file open error!\n");
        exit(EXIT_FAILURE);
    }

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

        num++;
    }
    fclose(fp);

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

    return 0;
}

Comments

[wwwwwa.k.awww]

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

Answer 2

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

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

Comments

[wwwwwa.k.awww]

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

Answer 3

UFO ufo, list[80333];

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

Comments

[wwwwwa.k.awww]

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

[y_waiwai]

ならあとは、strtokのところでNULLが帰ってませんか。

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