C言語で区切り文字で区切る

Question

C言語でcallocを使いテキストファイルから読み込んだ奴をbuffer に入れてそれを区切り文字で区切ってbucket
の中に入れたいです。例えば　buffer の中身が　aaa bbbb cccc, dddd;　とかであればそれを下のコードに書いてあるspliterで区切ってその区切りもじをbucketの中にどんどん入れたいです。　この場合bucket の中が４個になるはずで。

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

正しいbucketの数を表示させたいのに　segmentfaultになってしまいます。
どの様に下の　関数　tokenize_stringを変更すれば良いですか

segmentation faultと出てきます

該当のソースコード

ソースコード　使った関数やストラクチャーです。
buffer = (char *) calloc(10000, sizeof(char));
if (buffer == NULL){
printf(" can not get memory\n");
return 0;
}

size = readfile(filename,buffer) ; // ファイルサイズです
size = size -1;

// tokernize ---------------------------------------------------

bucket = (word_t *) calloc(1500, sizeof(word_t));

bucket-> len = 0;
bucket-> word = (char*)calloc(1500,sizeof(char));

count = tokenize_string(buffer, size, bucket);

printf("%d",count); // delete later

free(buffer);
free(bucket->word);
free(bucket);

//---------------------------------------------------------------

int tokenize_string(char* buffer, size_t size, word_t* bucket){

size_t len = 0;
int count = 0;
char* p = NULL;

char* spliter = " .,;()\n";

p = strtok(buffer, spliter);

while(p!= NULL){
    
    len = strlen(p);

    if(len != 0){


    strcpy(bucket->word, p);
    bucket->len = len;
    bucket++;
    count++;
    }
    p = strtok(NULL,spliter);
}
return count;

}
// struct--------------------------------------------

typedef struct word_t
{
int len;

char* word;

} word_t;

試したこと

ここに問題に対して試したことを記載してください。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Comments

[y_waiwai]

このままではコードが読みづらいので、質問を編集し、<code>ボタンを押し、出てくる’’’の枠の中にコードを貼り付けてください

[yumetodo]

本筋からは外れますが、英文法的にcan notとは書きません。can'tかcannotです

Answer 1

ソースコードは一部を貼るのではなくてすべて記載してください。またcodeを貼る記法を使って読みやすくしてください。main関数どこいったし。

で現状で分かる範囲ですが、bucketに使っている構造体word_tのメンバー変数wordのメモリー確保に問題があります。

callocによってword_tの配列を動的確保しているはずですが、wordのメモリー確保がたった一度しかありません。

---

それはそうと実はそもそもwordのメモリー確保は必要ないのです。strtok関数は文字列を解釈する時に同時に入力文字列にNULL文字を書き込むことで文字列分割を達成します。つまり、strtok関数の戻り値を単に保持すればいいわけです。

ただしもととなる入力文字列の先頭へのポインタは保持しておきbuketの寿命が尽きる時にfreeする必要があります。今回の例だとreadfileの第二引数に渡しているbufferですね。つまり

typedef struct {
    void* ptr;//あとでfreeするためだけのもの、これにポインタ演算はしない
    world_t* data;
} bucket_t;

のような構造体で管理すればよかったわけです。

こうすることでstrcpyする必要がないのでそこでbuffer overrunする心配がないし(なので通常はstrcpyは使うべきではなく、memcpyを利用してくださいね)、コピーが減るので高速化できるし、コード量も減るし、いい事ずくめです。もっと楽をしましょう。