0

このコードが機能しない理由について誰かに教えてもらえますか？

  • char *分割関数へのポインターを渡し、バッファーを分割します。
  • 渡された各引数（char *）のヒープにメモリを割り当てます。
  • 次に、この新しいバッファにサブストリングをstrcpyします。
  • メソッドから戻って変数のいずれかを出力しようとするまでは、すべて正常に機能します。
  • セグメンテーションの失敗
void split(char * buffer, int num, ...)
{
  char* string;
  char* tofree;
  string = strdup(trim(buffer));

  if (string != NULL) {
    tofree = string;

    va_list arguments; 

    //Initializing arguments to store all values after num 
    va_start ( arguments, num );           

    int i = 0;
    for (i = 0; i < num; i++ )        
    {
        //Item is the final store place of the split substring
        char * arg = va_arg ( arguments, char *);

        //Split the strings, delimiter is space
        char * splitBuffer = strsep(&string, " ");

        //Allocate the buffer memory to store the splitBuffer
        arg  =  malloc(sizeof(char*)*strlen(splitBuffer));

        strcpy(arg ,splitBuffer);
        printf("Buffer [%s] -- [%s]n", buffer, arg);
    }
    va_end ( arguments ); // Cleans up the list


    free(tofree);
  }
}




        char * a;
        char * b;
        char * c;
        split(buffer,3,a,b,c);
        printf("Print A = %s B = %s C = %s\n", a,b,c);
4

2 に答える 2

2

@tjamesonはこれを意味すると思います：

  void split(char * buffer, int num, ...)
  {
     char* string;
     char* tofree;
     string = strdup(trim(buffer));

     if (string != NULL)
     {
        tofree = string;

        va_list arguments; 

        //Initializing arguments to store all values after num 
        va_start ( arguments, num );           

        int i = 0;
        for (i = 0; i < num; i++ )        
        {
           //Item is the final store place of the split substring
           char ** arg = va_arg ( arguments, char **);

           //Split the strings, delimiter is space
           char * splitBuffer = strsep(&string, " ");

           //Allocate the buffer memory to store the splitBuffer
           *arg  =  malloc(sizeof(char*)*strlen(splitBuffer));

           strcpy(*arg ,splitBuffer);
           printf("Buffer [%s] -- [%s]\n", buffer, *arg);
        }
        va_end ( arguments ); // Cleans up the list

        free(tofree);
     }
  }


    char * a;
    char * b;
    char * c;
    split(buffer,3,&a,&b,&c);
    printf("Print A = %s B = %s C = %s\n", a,b,c);

正常に動作するはずです。

于 2013-03-11T15:33:26.027 に答える
1

Cでは、ポインタは値によって渡されます。関数へのポインターを渡し、その値（それが指すオブジェクトのアドレス）をその関数内で変更すると、C++参照のように元のポインターに伝播されません。

ここでは、 （、、または）が指すアドレスmallocを変更しますが、ローカルでのみ変更します。実際の、および（たとえば）は初期化されないままになります。あなたのコンパイラはおそらくそれについてあなたに警告するでしょう。argabcabcmain

これらのポインタを渡すときは、二重間接参照を使用します。

split(buffer,3, &a, &b, &c);

...そしてあなたのsplit関数の適切なコード、例えば：

char **arg = va_arg ( arguments, char ** );
*arg = malloc(...);
// etc.
于 2013-03-11T15:36:52.963 に答える