c - strtok と strcpy エラー

Question

文字strtok列を分割していました。

[更新]以下の新しいバージョンのコメントと回答を使用しましたが、ATM が機能しませんでした

int Crawl :: splitUrl(char ***tmp, int max_length, char *url)
{
 int idx=0;
 char * p;
 int i;

 p = strtok (url,"/");
 while (p != NULL && idx < max_length)
  {

   for (i=0;i<maxUrlSize-1 && p[i] != '\0';i++)
    (*tmp)[idx][i] = p[i];
   for ( ; i< maxUrlSize-1;i++)
    (*tmp)[idx][i] = '\0';

   printf("tmp[idx[%d]] %s\n",idx,(*tmp)[idx]);

   idx++;
   p = strtok (NULL, "/");
  }

 return idx;
};

はprintf("tmp[idx] ...正しく印刷されます。

しかし、メソッドを実行した後のメインでは：

  split_url = new char * [ maxUrlSplits ];
  for (int k=0;k<maxUrlSplits;k++)
   split_url[k] = new char [maxUrlSize];

  arr_size = crawl->splitUrl(&split_url,maxUrlSplits,url);

配列split_urlは空です。

コンパイラと gdb は問題ありません。

誰かがアイデアを持っていますか？

Answer 1

ポインターが指している場所を変更していないので、関数に char * を渡すだけで済みます。そう

int Crawl :: splitUrl(char *tmp, int max_length, char *url)

Answer 2

for (i=0;i<maxUrlSize || p[i] != '\0';i++)
    tmp[idx][i] = p[i];

この for ループは正しくありません。いずれかの条件が真である限り、バイトを長くコピーします。両方が真の場合にのみコピーする必要があると思います。

Answer 3

はい、これは正しいです。C のすべてのパラメーターは、値による呼び出しとして機能します。変更 (配列を埋める) が必要な場合は、ポインターを使用する必要があります。

       int Crawl :: splitUrl(char ***tmp, int max_length, char *url)
        {
         int idx=0;
         char * p;
         int i;

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

           for (i=0;i<maxUrlSize || p[i] != '\0';i++)
            (*tmp)[idx][i] = p[i];
           for ( ; i< maxUrlSize-1;i++)
            (*tmp)[idx][i] = '\0';

           printf("tmp[idx] %s\n",(*tmp)[idx]);

           idx++;
           p = strtok (NULL, "/");
          }


         return idx;
        };

...
      arr_size = crawl->splitUrl(&split_url,maxUrlSplits,url);