在C中连接大字符串以写入文件

Question

我是 C 的新手，所以我很难弄清楚这一点。我一次读取文件 500 个字节，计算文件中的行数，然后一旦达到 500 行，我将其写入另一个文件。该文件根本没有正确输出，我认为这与我在将其写入文件之前尝试连接一个长字符串有关。这是我现在拥有的代码的主循环：

   while (lineCount != 500)
   {
    if (lineCount == 500)
      break;
    readbytes = read(infd, buf, 500);
    for (str = buf; *str; ++str)
    {
      lineCount += *str == '\n';
      char *newPtr = realloc(forkBuf, (strlen(forkBuf) + strlen(str) + 2));
      forkBuf = newPtr;
      char* newFork = realloc(forkBuf, (strlen(forkBuf) + strlen(str) + 2));
      sprintf(forkBuf, "%s%s", newFork, str);
      if (lineCount == 500)
        break;
    }
    if (lineCount < 500)
    {
      if(readbytes < 500){
        write(outfd, forkBuf, (strlen(forkBuf) + 2));
        break;
      }
      continue;
    }
    write(outfd, forkBuf, (strlen(forkBuf) + 2));
  }

Answer 1

你有两行代码：

char* newFork = realloc(forkBuf, (strlen(forkBuf) + strlen(str) + 2));
sprintf(forkBuf, "%s%s", newFork, str);

这会导致未定义的行为；如果realloc 重新分配到一个新地址，forkBuf 将指向已释放的内存；即使realloc 工作到位，according to the standard：

原始指针 ptr [forkBuf 在此示例中] 已失效，对其的任何访问都是未定义的行为（即使重新分配已就地进行）。

我还要补充一点，如果（可能会发生）forkBuf 等于 newFork，则使用 sprintf 将字符串复制到自身也是未定义的行为。

此外，这里有一个for 循环，str 基本上被视为单个字符。我会重写整个循环，但快速破解可能是：

// One byte extra for the `\0`, and one for the new character
char *newPtr = realloc(forkBuf, (strlen(forkBuf) + 2));
if (newPtr == NULL)
    panic(); // Or whatever error handling you might want
forkBuf = newPtr;
size_t len = strlen(forkBuf);
forkBuf[len] = *str;
forkBuf[len + 1] = '\0';

请注意，循环中只需要一个realloc()，或者更好的是，循环外需要一个realloc(forkBuf, (strlen(forkBuf) + strlen(str) + 2));。