更新函数中的字符串数组

Question

我有一个将文件中的行复制到字符串数组中的工作示例。我想移动代码以将行复制到一个函数中，我只需将一个指向字符串数组的指针传递给该函数，行将存储在该数组中，以及一个指向文件的指针。但是，我试图将代码移动到一个函数中并不断出现段错误。我曾尝试使用 GDB 进行调试，似乎问题出在rows 的内存分配上。但我无法弄清楚问题所在。 realloc 似乎工作正常，因为我发现第 3 次迭代时行的大小增加了（使用 malloc_usable_size(*rows)），但随后出现了段错误。我在 Linux 上使用 gcc -Wall -Wextra -pedantic -std=c99 -g c_programs/read_file_function.c 进行编译。

工作示例

#define _GNU_SOURCE
#include <stdio.h>
#include <stdlib.h>
#include <string.h>

int main(int argc, char **argv)
{

  if (argc != 2)
  {
    fprintf(stderr, "Please supply a file path:\n%s <file path>\n", argv[0]);
    return EXIT_FAILURE;
  }
  FILE *fp = fopen(argv[1], "r");
  if (!fp)
  {
    perror("ERROR");
    return EXIT_FAILURE;
  }

  char **rows = (char **)malloc(sizeof(char *));
  char *lineBuf = NULL;
  size_t n = 0;
  size_t nLines = 0;
  ssize_t lineLength = 0;
  size_t i = 0;

  while ((lineLength = getline(&lineBuf, &n, fp)) != -1)
  {
    lineBuf[strcspn(lineBuf, "\n")] = 0;
    lineBuf[strcspn(lineBuf, "\r")] = 0;

    rows[i] = (char *)malloc(lineLength + 1);
    strcpy(rows[i], lineBuf);
    i++;
    nLines = i;
    rows = (char **)realloc(rows, (nLines + 1) * sizeof(char *));
  }

  printf("nLines: %lu\n", nLines);
  printf("row 1: %s\n", rows[0]);
  printf("row 2: %s\n", rows[1]);
  printf("row 2: %s\n", rows[10]);

  return 0;
}

非工作功能版本

#define _GNU_SOURCE
#include <stdio.h>
#include <stdlib.h>
#include <string.h>

size_t readFile(FILE **fp, char ***rows)
{
  char *lineBuf = NULL;
  size_t n = 0;
  size_t nLines = 0;
  ssize_t lineLength = 0;
  size_t i = 0;

  while ((lineLength = getline(&lineBuf, &n, *fp)) != -1)
  {
    lineBuf[strcspn(lineBuf, "\n")] = 0;
    lineBuf[strcspn(lineBuf, "\r")] = 0;

    *rows[i] = (char *)malloc(lineLength + 1);
    strcpy(*rows[i], lineBuf);
    i++;
    nLines = i;
    *rows = (char **)realloc(*rows, (nLines + 1) * sizeof(char *));
  }
  return nLines;
}

int main(int argc, char **argv)
{

  if (argc != 2)
  {
    fprintf(stderr, "Please supply a file path:\n%s <file path>\n", argv[0]);
    return EXIT_FAILURE;
  }
  FILE *fp = fopen(argv[1], "r");
  if (!fp)
  {
    perror("ERROR");
    return EXIT_FAILURE;
  }

  char **rows = (char **)malloc(sizeof(char *));
  size_t nLines = readFile(&fp, &rows);

  printf("nLines: %lu", nLines);
  printf("row 1: %s", rows[0]);
  printf("row 2: %s", rows[1]);

  return 0;
}

Answer 1

*rows[i] 正在执行*(rows[i]) - 访问行数组中的ith 元素，然后取消引用它。您想做(*rows)[i] - 取消引用rows，然后访问ith 元素。

我建议：

readFile(..., char ***rows0) {
    char **rows = NULL; // temporary internal variable

    ...
       // use rows normally
       rows = stuff();
    ...
   
    // when finished, assign once
    *rows0 = rows;
    return nLines;
}

但不要成为3-star programmer。充其量，使用一个结构，-> 很容易使用。喜欢：

struct string {
    char *str;
};

struct lines {
    struct string *strs;
    size_t cnt;
};


// @return 0 on success, otherwise error
int readFile(...., struct lines *p) {
    // initialization
    p->cnt = 0;
    p->strs = NULL;

    ...
       void *pnt = realloc(p->strs, (p->cnt + 1) * ....);
       if (!pnt) { /* handle error */ return -1; }
       p->strs = pnt;
       p->strs[p->cnt]->str = malloc(lineLenght + 1);
       if (!p->strs[p->cnt]->str) { /* handle error */ return -2; }
       strcpy(p->strs[p->cnt]->str, lineBuf);
       p->cnt++;
    ...

    return 0; /* success */
}
    

int main(int argc, char **argv) {
   struct lines p = {0};
   if (readFile(..., &p)) {
       /* handle error */
   }
   printf("nLines: %zu\n", p.cnt);

不要预先分配内存。使用NULL 初始化内存并使用内存调用realloc 之前。 realloc(NULL 与 malloc() 相同。

检查分配错误。