realloc，用于 C 中的字符串数组

Question

当你想将可变大小的单词添加到字符串数组时，是否有正确的方法来使用 realloc？我遇到了分段错误。 请告诉我有什么问题

// This function puts every word found in a text file, to a String array, **words
char **concordance(char *textfilename, int *nwords){
    FILE * fp;
    char *fileName = strdup(textfilename);
    fp = fopen(fileName, "r");
    if(fp == NULL) {
        perror("fopen");
        exit(1);
    }
    char **words = malloc(sizeof(char));
    // char **words = NULL

    char line[BUFSIZ];
    while(fgets(line, sizeof(line), fp) != NULL){
        char *word = strdup(line);
        word = strtok(word, " ");
        do{
            words = realloc(words, (*nwords+1) * sizeof(char(*)));
            words[*nwords] = word;
        } while((word = strtok(NULL, " ")) != NULL);
    }
    return words;
}


int main(int argc, const char * argv[]) {
    int *nwords = malloc(sizeof(int));
    nwords = 0;
    concordance("test.txt", nwords);
}

Answer 1

您似乎以错误的方式将nwords 初始化为0。由于您已将其声明为指针，因此您无法直接访问它。相反，您应该使用取消引用运算符*。

在main 函数中进行以下更改

*nwords = 0; 而不是nwords = 0;

nwords = 0 将nwords 指向的位置修改为地址为0 的位置，您无权访问且无法分配。

警告：

1. 最好不要在同一个指针上执行realloc，如果realloc失败会使指向位置NULL，导致之前存在的数据丢失。相反，正如@David 建议的那样，您可以使用一个临时变量到realloc 内存，然后检查它是否不是NULL，然后将其内容分配给words 指针。

    //your code
    char *tmp = realloc(words, /* new size*/);
    if(tmp != NULL)
        words = tmp;
    // your code

1. 在使用realloc 时，您通常使用它来分配数据块，而不是单个位置。

Answer 2

当您初始化nwords 的值时，您正在覆盖它的指针地址，而不是它的值。

此外，正如评论者所说，char **words = malloc(sizeof(char)); 行不正确。但是您总是重新分配变量words，因此代码仍然可以按预期工作。为了让它超级安全，你应该把它改成char **words = malloc(sizeof(char*));

我使用*nwords = 0; 行，现在它按预期工作。

#define BUFSIZ 1000
#include<stdio.h>
// This function puts every word found in a text file, to a String array, **words
char **concordance(char *textfilename, int *nwords){
  FILE * fp;
  char *fileName = strdup(textfilename);
  fp = fopen(fileName, "r");
  if(fp == NULL) {
    perror("fopen");
    exit(1);
  }
  char **words = malloc(sizeof(char));
  // char **words = NULL

  char line[BUFSIZ];
  while(fgets(line, sizeof(line), fp) != NULL){
    char *word = strdup(line);
    word = strtok(word, " ");
    printf("word='%s'\n",word);
    do{
      *nwords=*nwords+1;
      printf("nwords=%d\n",*nwords);
      words = realloc(words, (*nwords+1) * sizeof(char(*)));
      words[*nwords] = word;
    } while((word = strtok(NULL, " ")) != NULL);
  }
  return words;
}


int main(int argc, const char * argv[]) {
  int *nwords = malloc(sizeof(int));
  *nwords = 0;
  concordance("test.txt", nwords);
}