strtokとポインタの扱い方がわかりません．

Question

前提・実現したいこと

.csvファイルをJSON化するプログラムを制作しています．csvファイルの読み取りまでは上手くいったのですが，strtokとポインタの使い方がいまいち掴めず，データがboxに格納されているのか判断できません．
.csvファイルを1行ごと読み込み，strtokを使用して分解，その分解したデータを格納するタイミングで問題が起きているようです．

当該のソースコード

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

#define N 512 // 1行の最大文字数(バイト数)
#define X 30

int main(void) {
    FILE *fp; // FILE型構造体
    char fname[] = "test2.csv";
    char str[N];
    char *box[X];
    char *p;

    fp = fopen(fname, "r"); // ファイルを開く。失敗するとNULLを返す。
    if(fp == NULL) {
        printf("%s file not open!\n", fname);
        return -1;
    }


    fgets(str, N, fp);

    p = strtok(str, ",");

    puts(p);

    box[0] = p;

    while(p!=NULL){
        p = strtok(NULL,",");

        if(p!=NULL){
            int i = 1;
            puts(p);
            box[i] = p;
            //printf("%s\n", box[i]);
            i++;
        }
    }

    printf("%s\n", str);




    fclose(fp); // ファイルを閉じる

    for(int j=0; j<=X-1; j++){
        printf("%c", *box[j]);
    }

    printf("\n");*/

    return 0;
}

出力されるコード

./a.out
Settings
Toilet_Duration
Toilet_Duration
0:01
0:01
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0

0

Settings
Segmentation fault: 11

編集後の該当のソースコード

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

#define N 512 // 1行の最大文字数(バイト数)
#define X 512

int main(void) {
    FILE *fp; // FILE型構造体
    char fname[] = "test2.csv";
    char str[N];
    char *box[X];
    char *p;
    int i = 0;

    fp = fopen(fname, "r"); // ファイルを開く。失敗するとNULLを返す。
    if(fp == NULL) {
        printf("%s file not open!\n", fname);
        return -1;
    }


    fgets(str, N, fp);

    p = strtok(str, ",");

    puts(p);

    box[i] = p;
    i++;

    while(p!=NULL){
        p = strtok(NULL,",");

        if(p!=NULL){

            puts(p);
            box[i] = p;
            printf("%s\n", box[i]);
            i++;
        }
    }

    printf("%s\n", str);




    fclose(fp); // ファイルを閉じる

    printf("%c\n", *box[1]);

    printf("\n");

    return 0;
}

編集後の出力されるコード

hermes$ gcc line.c 
hermes$ ./a.out
Settings
Toilet_Duration
Toilet_Duration
0:01
0:01
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0
0

0

Settings
T

元の.csvデータ

Settings,Toilet_Duration,0:01,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0

試したこと

どの地点でSegmentation faultが起きているのか知る為にprintf関数を使用して文字列を出力させた．

Answer 1

i がずっと 1 です。

for(int j=0; j<=X-1; j++){

X - 1 まで入っていないのでそこまで列挙してはいけません。

Comments

[hermes]

コードを訂正しました．まだ問題があるので質問の方を編集します．

[Zuishin]

一問一答です。別の問題なら仕切り直してください。完成までは面倒みられません。

[Zuishin]

それと、問題はなるべく自分で解決してください。

[hermes]

失礼致しました．以後注意します．

[Zuishin]

不当な低評価を戻しなさい。

[hermes]

まだ低評価の機能が解放されていなくて，僕は押してないのですが．勘違いであったら申し訳ありません．

[Zuishin]

低評価したならどこが間違っているのか書いてみなさい。ちょっと考えたらあなたの方が間違っていることがわかるでしょう。

[Zuishin]

削除

[haruniku]

また強要してますね笑

[Zuishin]

低評価の理由を書きなさい。

[ai_2013_dev]

私も、この回答がマイナス評価される理由を知りたい

Answer 2

コードを良く見てください。明らかにおかしい場所が一点あります。

        if(p!=NULL){
            int i = 1;
            puts(p);
            box[i] = p;
            i++;
        }

i = 1 としているので、この中では常に box[1] = p となりますね。

そして、最後の printf で参照しているのは j=0, 1, 2,.... となるので、 j = 2 の時点で初期化されていない box[2] を参照しようとしてエラーとなります。

printfデバッグについて

printf("%s\n", str);

のように改行文字を追加しましょう。でないと高確率でコンソールに出力される前にプログラムが終了してしまいまうので、「その箇所を通ってない」と勘違いしやすいです。

Comments

[hermes]

ありがとうございます．訂正したものを元の質問の方に再掲載します．まだ問題があるようです．

[mather]

質問を編集するときはプレビューも確認しながら更新しましょう。

[mather]

更新後のコードでは何が問題なのでしょうか？
質問に記載してください。

[mather]

ついでに、質問を更新するときは上書きではなく更に下に追加するほうが良いですよ。
最初の状態で回答した内容が更新された質問に対して的はずれなものになってしまいます。

[Zuishin]

次の質問なので、解決済みにして新しく質問すべきと思います。

[mather]

それもそうですね。

[Zuishin]

コードも全く変わっているし、この調子で人の話を聞かない人に完成まで追加質問されても付き合いきれません。mather さんの回答をベストアンサーに選び、新しく質問してください。

[hermes]

申し訳ありません．次回以降気をつけます．

[Zuishin]

今回からです。

[Zuishin]

低評価しやがった。

[Zuishin]

削除

[hermes]

先ほど削除してしまった質問を再掲載しました．
新しく書いたコードでは最後の

printf("%c\n", *box[1]);

の部分で表示されるのが
"T"
1文字となってしまいます．私としては
"Toilet_Duration"
と出力して欲しいです．

何かアドバイスなどありますでしょうか？どうぞよろしくお願い致します．

[Zuishin]

個人情報が含まれていたので削除

[mather]

Zuishinさんの言う通り、できれば別の質問にしたほうが良いですね。
%sと違って、%c ってchar型の値を表示するんです。

[Zuishin]

こんなのに餌をやらなくていい。

[hermes]

ご指摘ありがとうございます．

[mather]

Zuishinさん、そんなに感情的にならなくてもいいと思いますよ。嫌になったら見ないようにしたほうが良いです。
hermesさんも回答をいただけたことにまずは感謝して、聞きたいことが変わったなら別の質問を立てて聞くべき(Zuishinさんのおっしゃる通り）ですし、何でもかんでも教えてもらえると思わずに、まずはリファレンスをしっかり確認して自分で考える癖を付けましょう。

[Zuishin]

嫌なら見ない方がいいですよ。

[hermes]

承知致しました．私も”聞けばわかる”と安易に考えてしまっていました．反省しております．今後は自分から動いて（リファレンスなどを探って）成長していけたらと思います．ご指摘本当に感謝致します．ありがとうございます．

[hermes]

Zuishinさん
本当に申し訳ありませんでした．聞く前にまず調べる事を今後は徹底していきたいと考えております．どうぞよろしくお願い致します．

[Zuishin]

不当な低評価を戻しなさい。

[hermes]

今日teratailを始めたばかりで低評価の機能がまだ解放されておりません．私は押していないと思うのですが，，，勘違いでしたら申し訳ございません．

[Zuishin]

削除

[Zuishin]

削除

[mather]

お二人とも、ここは「回答へのコメント」をする場だと思いますので、一旦この場での会話は止めていただけませんか。よろしくお願いします。

[Zuishin]

終わってから言わないでくださいな。また始まるでしょ。

[Zuishin]

ここは回答のコメントをするところなので、火に油を注ぎたいなら、他へどうぞ。

[Zuishin]

低評価 2 の回答と同意文なので低評価しました。

Answer 3

char* box[30];
だと、box[0] からbox[29]までです。それ超えると範囲外になりますね

---

んで、Cでコードを組むなら、VisualStudioやEclipseなど、デバッグできる環境を整えましょう
ソースコードの任意の行で実行を止め、変数のナカミを見ることができます。
また、１行づつ実行したりできますんで、当てずっぽでコードを組まなくて済むようになります

Comments

[hermes]

ありがとうございます．デバッグ環境も整えていきたいと思います．