附加字符串会导致“realloc(): Invalid pointer”

Question

我有一个函数append_string，它将str2附加到str1：

void append_string(char* str1, char* str2) {
    int    new_length = strlen(str1)+strlen(str2);
    size_t new_size = sizeof(char)*(new_length);
    str1 = (char*) realloc(str1, new_size);
    strcat(str1, str2);
}

如函数所示，我正在尝试使用字符串的组合大小来增加大小。

每当我拨打append_str("", "adc"); 时，我都会收到"realloc(): Invalid pointer"

我做错了什么？

Answer 1

至少有这些问题：

正在尝试重新分配未分配的内容。

realloc(str1, new_size) 尝试重新分配 字符串文字 "" 导致“realloc(): Invalid pointer”。

尺寸缩小 1

新尺寸不考虑空字符。

// size_t new_size = sizeof(char)*(new_length);
size_t new_size = sizeof(char)*(new_length + 1);

代码丢失分配的指针

调用代码丢失了新指针str的值。

用于调整大小的弱类型

使用size_t。

---

相反，通过它的地址传递一个分配的指针或NULL。

void append_string(char** str, const char* appendage) {
  size_t new_length = strlen(*str) + strlen(appendage);
  size_t new_size = sizeof(char)*(new_length + 1);
  *str = realloc(*str, new_size);
  strcat(*str, appendage);
}

// Usage
char *s = malloc(1);
strcpy(s, "");
append_str(&s, "adc");
puts(s);

---

高级问题包括：

• 如果realloc()返回NULL怎么办？

• appendage重叠str如何处理？

• 不要使用strcat()。避免slow code。最好保留原始字符串长度并从那里复制。

    void append_string(char** str, const char* appendage) {
      size_t str_len = *str ? strlen(*str) : 0;
      size_t app_len = strlen(appendage);
      void *p = realloc(*str, str_len + app_len + 1);
      if (p == NULL) {    
        // Handle Error - various approaches
        free(*str);
        *str = NULL;
      } else {        
        strcpy(*str + str_len, appendage);
      }
    }
  

appendage与*str重叠时仍需处理大小写。

    void append_string_when_overlap_possible(char** str, const char* appendage) {
      size_t str_len = *str ? strlen(*str) : 0;
      size_t app_len = strlen(appendage);
      char *p = malloc(str_len + app_len + 1);
      if (p == NULL) {    
        // Handle Error - various approaches
        free(*str);
        *str = NULL;
      } else {        
        if (*str) {
          strcpy(p, *str);
        }
        strcpy(p + str_len, appendage);
        free(*str);
        *str = p;
      }
    }