Ç -

Question

我已用C寫該功能的功能的內部動力分配/ rellocation（功能應當接收char*，分配必要的空間，並插入字符賦予一個指向字符的索引指針的後面）

void add_to_str(char *character, char ** string, int* index) 
{ 
    //we dont expect *string to be NULL !! no condition done 
    if (*index == 0) //if str is empty we will alloc AVGLEN(32) characters space 
     *string=malloc(AVGLEN*sizeof(char)); 

    if (*string == NULL) //malloc fails? 
    { 
     fprintf(stderr,errors[MALLOC]); 
     exit(99); 
    } 
    //string length exceeds 32 characters we will allocate more space 
    if (*index > (AVGLEN-1) || character== NULL)// or string buffering ended, we will free redundant space 
    { 
     *string=realloc(*string,sizeof(char)*((*index)+2));//+1 == '\0' & +1 bcs we index from 0 
     if (*string==NULL) //realloc fails? 
     { 
      fprintf(stderr,errors[REALLOC]); 
      exit(99); 
     } 
    } 
    *string[(*index)++]=*character; 
} 

當*index > 0，它給我就行了分段錯誤

*string[(*index)++]=*character; 

此功能（的一個變種後面char*只是malloc，然後指定字符string[i++]）完美運作。

Answer 1

你必須要小心這樣的說法：

*string[(*index)++]=*character; 

因爲數組索引比指針廢棄更高的優先級。因此，這與

*(string[(*index)++]) = *character; 

這不是你想要的。你想這樣的：

(*string)[(*index)++] = *character; 

的故障代碼，因爲在這種情況下，語句相當於**string，這仍然是有效的工程*index == 0，但是當index > 0，string會在一個無效的位置被取消引用：string+index。

Answer 2

需要注意的是做這樣的事情：

ptr = realloc(ptr, ...); 

是一個非常糟糕的模式。當realloc()失敗時，您的舊分配區域不再可供該程序訪問並泄漏。正確的模式是：

char* str_new = realloc(string, ...); 

if (str_new != NULL) 
    string = str_new; 
else 
    /* handle allocation error */ 

Answer 3

首先，功能fprintf需要一個格式字符串。

I.e.

fprintf(stderr,errors[MALLOC]); 

可能無效。

此功能的目的是什麼。似乎沒有敘述，