Question

我在将文件读入C中的字符串数组时遇到问题。这是我的代码：

  char mylittleBuffer[BUFFER_SIZE];     //BUFFER_SIZE is 4096
  char *a;
  char **c;

  fprintf(stderr, "LOL\n");
  for(int i = 0; (a = fgets(mylittleBuffer, sizeof(mylittleBuffer), myInput)) != NULL; i++)
    {
      fprintf(stderr, "a: %s\n", a);
      c[i] = malloc(sizeof(a));
      if (c[i] == NULL)
        printf("c[i] is NULL");
      c[i] = strdup(a);
      //      fprintf(stderr, "mylittleBuffer: %s\n", mylittleBuffer);                                                                                                                                                   
      fprintf(stderr, "c[i] %s\n", c[i]);
    }

看起来只读取文件的一行，因为程序将打印出a一次。其他线路怎么了？我没有收到任何错误消息......

这里有什么问题？
如何解决此问题？

Answer 1

您尚未初始化c指向任何内容（您需要为其分配空间），因此当您使用c[i]时，您正在使用未定义的内存位置，从而调用未定义的行为。它不会崩溃是一个奇迹。您将需要为字符指针数组分配空间。

enum { INITIAL_SIZE = 2 };            // Set to a larger number when not debugging
char mylittleBuffer[BUFFER_SIZE];     //BUFFER_SIZE is 4096
char *a;
size_t c_size = INITIAL_SIZE;
char **c = malloc(c_size * sizeof(*c));

if (c == NULL)
{
    fprintf(stderr, "out of memory (for c)\n");
    return;
}

fprintf(stderr, "LOL\n");
for (int i = 0; (a = fgets(mylittleBuffer, sizeof(mylittleBuffer), myInput)) != NULL; i++)
{
    fprintf(stderr, "a: %s\n", a);
    if (i >= c_size)
    {
         // Reallocate c to get more space
         size_t new_size = c_size * 2;
         void *new_space = realloc(c, new_size * sizeof(*c));
         if (new_space == 0)
         {
             // Release the already allocated c[i]
             // Release c
             fprintf(stderr, "Out of memory (for more c)\n");
             return;
         }
         c_size = new_size;
         c = new_space;
    }
    // c[i] = malloc(sizeof(a));  // Leak - you use strdup() too
    c[i] = strdup(a);
    if (c[i] == NULL)
    {
        fprintf(stderr, "c[i] is NULL\n");
        // Release the already allocated c[i] strings
        // Release c
        return;
    }
    //      fprintf(stderr, "mylittleBuffer: %s\n", mylittleBuffer);
    fprintf(stderr, "c[%d] <<%s>>\n", i, c[i]);  // <<>> show where the string ends
}

我主要保留了你的代码。如果是我的，a将不存在，mylittleBuffer将只是buffer，这就是我在循环体中使用的a。我可能会为c使用更长的名字，虽然我通常使用比其他人更短的名字。

注意如果调用失败，使用realloc()的代码如何避免丢失指向先前分配的内存的指针。如果直接分配给c，您将失去对先前内存分配的唯一引用 - 可能是主要泄漏。

鉴于清理代码需要两次，我会编写一个函数来执行此操作：

static void release_memory(size_t num, char **strings)
{
    for (size_t i = 0; i < num; i++)
        free(strings[i]);
    free(strings);
}

它将被称为：

release_memory(i, c);

将文件读入字符串数组C

1 个答案: