• • 已关闭**。此问题需要超过focused。当前不接受答案。
• • 想要改进此问题吗？**更新此问题，使其仅关注editing this post的一个问题。

7天前关闭。
Improve this question
在C语言中最痛苦的工作之一就是找出最好和最有效的处理字符串的方法。我已经搜索和尝试了将近一天了。不管怎么说，基本上，我试图创建一个动态字符串构建器函数，所以我得到了以下结果：

#include <stdarg.h>
#include <stdio.h>
#include <stdlib.h>

void *alloc(void **ptr, size_t len) {
  free(*ptr);
  void *mem = malloc(len);
  if (mem == NULL) {
    fprintf(stderr, "err: alloc failed");
    exit(EXIT_FAILURE);
  }
  return mem;
}

char *str(char **src, const char *fmt, ...) {
  if (fmt == NULL) {
    return NULL;
  }
  va_list args;
  va_start(args, fmt);
  size_t len = vsnprintf(NULL, 0, fmt, args);
  if (len < 0) {
    return NULL;
  }
  *src = alloc((void **) src, sizeof(char) * (len + 1));
  vsnprintf(*src, len + 1, fmt, args);
  va_end(args);
  return *src;
}

以下是我目前的使用方法：

int main(int argc, char *argv[]) {
  char *s = str(&s, "FOUND!");
  puts(s);
  puts(str(&s, "NOT %s", s));
  puts(s);
  return 0;
}

当它确实起作用时，我在想这样的事情：

• 可能的内存泄漏。
• 以及我在动态内存分配方面做错的任何事情。
• 以及如何用str(&pointer, "value")初始化char *（传递未初始化的&pointer是好还是坏）。

代码是否有任何错误以及可能的修复/改进？

编辑

也许我一开始就不该这么做XD
根据答案/建议修复：

#include <stdarg.h>
#include <stdio.h>
#include <stdlib.h>

void *alloc(void *ptr, size_t len) {
  void *mem = realloc(ptr, len);
  if (mem == NULL) {
    free(ptr);
    perror(NULL);
  }
  return mem;
}

char *str(char *src, const char *fmt, ...) {
  if (fmt == NULL) {
    return NULL;
  }
  va_list args1, args2;
  va_copy(args2, args1);
  va_start(args1, fmt);
  int len = vsnprintf(NULL, 0, fmt, args1);
  va_end(args1);
  if (len < 0) {
    return NULL;
  }
  char *val = alloc(src, len + 1);
  va_start(args2, fmt);
  int res = vsnprintf(val, len + 1, fmt, args2);
  va_end(args2);
  free(src);
  if (res < 0) {
    free(val);
    return NULL;
  }
  return val;
}

int main(int argc, char *argv[]) {
  char *s = str(NULL, "FOUND!");
  puts(s);
  free(s);
  s = str(s, "NOT %s", s);
  puts(s);
  free(s);
  return 0;
}

变更内容（以及注解（Ns），供将来参考）：

• • • N**：传递未初始化的&pointer是错误的，实际上是伪装的错误。
• • • N**：使用realloc代替malloc，后者处理NULL初始值并释放先前分配的内存。
• 将**参数更改为*。**背后的最初想法是能够直接修改函数中*src指针的值（在原始代码中以某种方式工作）。
• 初始声明更改为char *s = str(NULL, "value")，更新值需要重新分配s = str(s, "updated: %s", s)。
• 对于vsnprintf的返回值，将size_t len更改为int。size_t是我最初用来匹配malloc的size_t参数的值。
• • • N**：第二次vsnprintf调用可能在第一次调用后返回不同的结果，因此使用va_copy复制第一个va_list用于第二次调用。
• free旧字符串，然后返回实际值。