动态增加 C 字符串的大小

Question

我目前正在创建一个程序来捕获用户的按键并将它们存储在一个字符串中。我希望存储按键的字符串是动态的，但我遇到了一个问题。 我当前的代码如下所示：

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

typedef struct Foo {
  const char* str;
  int size;
} Foo;

int main(void)
{
  int i;
  Foo foo;
  foo.str = NULL;
  foo.size = 0;

  for (;;) {
    for (i = 8; i <= 190; i++) {
      if (GetAsyncKeyState(i) == -32767) { // if key is pressed
        foo.str = (char*)realloc(foo.str, (foo.size + 1) * sizeof(char)); // Access violation reading location xxx
        sprintf(foo.str, "%s%c", foo.str, (char)i);
        foo.size++;
      }
    }
  }

  return 0;
}

任何帮助将不胜感激，因为我没有任何想法了。 :(
我是否也应该动态分配 Foo 对象？

Answer 1

1. const char *str - 你声明指向const char 的指针。您不能在调用 UB 时写入引用的对象
2. 您使用sprintf 只是为了添加char。这毫无意义。
3. 结构中不需要指针。

您需要设置编译器选项来编译**为 C 语言”而不是 C++

我会做一些不同的方式：

typedef struct Foo {
    size_t size;
    char str[1];
} Foo;

Foo *addCharToFoo(Foo *f, char ch);
{
    if(f)
    {
        f = realloc(f, sizeof(*f) + f -> size);
    }
    else
    {
        f = realloc(f, sizeof(*f) + 1);
        if(f) f-> size = 0
    }
    if(f) //check if realloc did not fail
    {
        f -> str[f -> size++] = ch;
        f -> str[f -> size] = 0;
    }
    return f;
}

在main

int main(void)
{
    int i;
    Foo *foo = NULL, *tmp;

    for (;;) 
    {
        for (i = 8; i <= 190; i++) 
        {
            if (GetAsyncKeyState(i) == -32767) { // if key is pressed
            if((tmp = addCharToFoo(f, i))
            {
                foo = tmp;
            }
            else
            /* do something - realloc failed*/
            }
        }
    }

    return 0;
}

Answer 2

sprintf(foo.str, "%s%c", foo.str, (char)i); 格式错误：第一个参数不能是 const char *。您应该会看到编译器错误消息。

修复此问题后（使str 成为char *），则行为未定义，因为%s 读取的源内存与目标重叠。

相反，您需要使用其他方法来附加不涉及重叠读写的字符（例如，使用[ ] 运算符来写入字符并且不要忘记空终止）。

Answer 3

首先，为了处理好事情，你需要定义

typedef struct Foo {
    char* str;
    int size
} Foo;

否则，Foo 正确地进行变异真的很烦人 - 您通过在 realloc 调用之后以任何方式修改 foo->str 来调用未定义的行为。

段错误实际上是由sprintf(foo.str, "%s%c", foo.str, (char)i);引起的，而不是对realloc的调用。 foo.str 通常不会以空值结尾。

事实上，您调用sprintf 就是在重复工作。 realloc 已经复制了之前f.str 中的所有字符，所以你所要做的就是通过添加单个字符

f.str[size] = (char) i;

编辑以回复评论：

如果我们想一起追加到字符串（或者更确切地说，两个 Foo），我们可以这样做：

void appendFoos(Foo* const first, const Foo* const second) {
    first->str = realloc(first->str, (first->size + second->size) * (sizeof(char)));
    memcpy(first->str + first->size, second->str, second->size);
    first->size += second->size;
}

appendFoos 函数通过在 first 上附加 second 来修改它。

在整个代码中，我们将Foos 保留为非空终止。但是，要转换为字符串，您必须在读取所有其他字符后添加最后一个空字符。