質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

89.13%

C言語 Segmentationfault11の解決策が不明

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 317

hermes

score 13

前提・実現したいこと

.csvファイルをJSON化するプログラムです.現段階ではcsvファイルを読み込み,配列に格納する所まで制作しました.

発生している問題・エラーメッセージ

2行目のcsvデータを表示し終えた後にSegmentationFault11が発生.3行目の読み込みが出来ない.
もう一つ気がかりなのが,csvファイルの最後の列である[name]の"]"だけが表示されない事.

./a.out
[Settings] [Toilet_Duration] [0:01]
[date] [sleep_state.start1] [sleep_state.end1] [sleep_state.dur1] [sleep_state.start2] [sleep_state.end2] [sleep_state.dur2] [toilet.start1] [toilet.end1] [toilet.start2] [toilet.end2] [toilet.start3] [toilet.end3] [toilet.start4] [toilet.end4] [toilet.start5] [toilet.end5] [toilet.start6] [toilet.end6] [toilet.start7] [toilet.end7] [toilet.start8] [toilet.end8] [bath.start] [bath.end] [bath.dur] [remarks] [name
Segmentation fault: 11

該当のソースコード

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

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

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

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

    /*****************
     * 1行目
     */

    //ファイルを1行ごとに呼び出し
    fgets(str, N, fp);
    //", " で分解
    p = strtok(str, ",");

    //puts(p);
    //box[]に値を収納
    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);

    //printしてるのここ
    printf("[%s] [%s] [%s]\n", box[0], box[1], box[2]);

    /*******************
     * 2行目
     */

    //ファイルを1行ごとに呼び出し
        while (fgets(str, N, fp)!=NULL) {
            i = 0;
            //", " で分解
            p = strtok(str, ",");

            //puts(p);
            //box[]に値を収納
            box[i] = p;
            i++;

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

                if (p != NULL) {

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

            //printしてるのここ
            for (int j = 0; j < Colum; j++) {
                printf("[%s] ", box[j]);
            }
            printf("\n");
        }


    /******************************************
     * 最後の〆
     */
    fclose(fp); // ファイルを閉じる

    printf("\n");

    return 0;
}

試したこと

Segmentation Faultに陥る状況の例などを参照.
自身のファイルにそれぞれの条件が当てはまらないか比較.
While文を除去し2行目だけ実行.結果は変わらず.

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

checkベストアンサー

+1

"test2.csv" の中身は、どうなってますか?
特に 2行目。 512 byteを越えてませんか?

#define N 512 を 1024 とかにしても変わりませんか?

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/07/11 22:53

    おっしゃる通りでした.NとXを2048にしたところ問題なくプログラムが動きました.本当にありがとうございます.

    キャンセル

+1

while (fgets(str, N, fp)!=NULL) {
との絡みで、データファイルの「2行目」が512バイトを超えているとそういうこともあるかと思うけど。

あと、現時点では関係なさそうだけど、
for (int j = 0; j < Colum; j++) {
って大丈夫かしら?
for (int j = 0; j < i; j++) {
の方が安全では。
(さらに、box[]の範囲外アクセスがないようにガードもすべきだとも思う。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/07/11 23:00

    ありがとうございます!本当に助かります.

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 89.13%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる