strtok関数

[虎ichi]

学校の課題のプログラムを書く過程で、strtok関数を使って","区切りで配列に格納したいと考えてます。
今、例として"ABC,EFG,HIJ,KLM"を","区切りで配列に格納して出力するプログラムを書いて実行したのですが、
ABC
EFG
HIJ
KLM
となるはずのところが、セグメンテーション違反が出てしまいました。
どこがおかしいのでしょうか？
ご指摘お願いします。

コード:

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

int main(void)
{

  char str[] = "ABC,EFG,HIJ,KLM";
  char *tp;
  int i, count = 1;
  char str2[32][32];


  tp = strtok(str, ",");
  strcpy(str2[0], tp);

  while(tp != NULL)
    {
 
      tp = strtok(NULL, ",");
      strcpy(str2[count++], tp);
    }

  for(i = 0; i < 4; i++)
    {

        printf("%s\n", str2[i]);

    }

  return 0;

}

[a5ua]

コード:

while(tp != NULL)
{
	tp = strtok(NULL, ",");		// ここで、tpがNULLになると
	strcpy(str2[count++], tp);	// strcpyにNULLを渡すことになり、エラーとなる
}

[non]

前のスレッドを、ほったらかしにするのは、マナーが悪いですね。

[虎ichi]

ほったらかしにしてすみません。
ご指摘ありがとうございます。

コード:

 tp = strtok(NULL, ",");
      strcpy(str2[count++], tp);

この部分をどのようにかえればいいのでしょうか？
strtok関数でいろんなサイトを見て
1回目は tp = strtok(str, ",");
2回目以降は tp = strtok(NULL, ",");

と書いてたのでそのまま引用しましたが、ただ単に出力するのとコピーするのは違うということなんでしょうか？

[non]

NULLとは数値でいうと0ですから、0番地からの内容をコピーするのでマズイです。

[虎ichi]

コード:

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

int main(void)
{

  char str[] = "ABC,EFG,HIJ,KLM";
  char *tp = ;
  int i, count = 0;
  char str2[32][32];


  tp = str;

  while(tp != NULL)
    {
 
      tp = strtok(tp, ",");
      strcpy(str2[count++], tp);
    }

  for(i = 0; i < 4; i++)
    {

        printf("%s", str2[i]);

    }

  return 0;

}

こういうことですか？

うーん・・・わかるようでわかりません＞＜

[みけCAT]

tp = strtok(tp, ",");
と
strcpy(str2[count++], tp);
の間にtpがNULLでないかの判定を入れないといけないと思います。

追記
今更ながら修正。あとに→間に

[初級者]

ふつうは、こういう風に書くんでしょうね、きっと。

コード:

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

int main(void)
{
    char str[] = "ABC,EFG,HIJ,KLM";
    char str2[32][32];
    char *tp;
    int i, count = 0;

    tp = strtok(str, ",");
    while (tp) {
        strcpy(str2[count++], tp);
        tp = strtok(NULL, ",");
    }

    for (i = 0; i < count; i++) {
        printf("%s\n", str2[i]);
    }
    return 0;
}

[虎ichi]

ありがとうございます！
while文の中身が逆でした。やっと意味がわかりました！