无法使realloc（）工作

Question

FILE *file;
file = fopen(argv[1], "r");
char *match = argv[2];
if (file == NULL) {
    printf("File does not exist\n");
    return EXIT_FAILURE;
}
int numWords = 0, memLimit = 20;
char** words = (char**) calloc(memLimit, sizeof(char));
printf("Allocated initial array of 20 character pointers.\n");
char string[20];

while (fscanf(file, "%[a-zA-Z]%*[^a-zA-Z]", string) != EOF) {
    words[numWords] = malloc(strlen(string) + 1 * sizeof(char));
    strcpy(words[numWords], string);
    printf("Words: %s\n", words[numWords]);
    numWords++;     /*keep track of indexes, to realloc*/ 
    if (numWords == memLimit) {
        memLimit = 2 * memLimit;
        words = (char**) realloc(words, memLimit * sizeof(char*));   /*Fails here*/
        printf("Reallocated array of %d character pointers.\n", memLimit);
    }

}

代码应打开并读取包含标点符号，空格等单词的文件并存储在字符串中，但是经过20次尝试，它会引发错误，而且我似乎无法在这里使用realloc（）来工作，我期望是问题。 动态分配给该数组20个char指针，当达到限制时，该数组应按两倍重新分配。 我该如何解决？

Answer 1

两个笔记。 首先，永远不要转换calloc / malloc / realloc的返回值。 请参阅此以获取更多信息。

其次，正如其他人在评论中指出的那样，第一个calloc语句使用sizeof(char)而不是应该使用sizeof(char*) 。

Answer 2

words是指向指针的指针。 这个想法是分配一个指针数组。
以下是错误的，因为它分配了memLimit字符而不是memLimit指针。
这是主要问题

char** words = (char**) calloc(memLimit, sizeof(char)); // bad

因此，请使用一个简单的习惯用法：分配任何words指向的memLimit组。 编写，阅读和维护更加容易。

char** words = calloc(memLimit, sizeof *words);

---

避开while (scanf() != EOF)孔。 回想一下，各种结果可能来自scanf()系列。 它返回成功扫描的字段或EOF的计数。 通常，这是至少3个选项中的1个。 因此，不要测试您不想要的结果，请测试您想要的结果。

// while (fscanf(file, "%[a-zA-Z]%*[^a-zA-Z]", string) != EOF) {
while (fscanf(file, "%[a-zA-Z]%*[^a-zA-Z]", string) == 1) {

上面的示例可能并非每次都返回0，但是下面的示例很容易返回。

int d;
while (fscanf(file, "%d", &d) == 1) {

@Enzo Ferber正确建议使用"%s" 。 进一步建议遵循上述习惯用法，并将输入宽度限制为小于缓冲区大小的1。

char string[20];
while (fscanf(file, "%19s", string) == 1) {

---

建议检查分配结果的习惯。

    // better to use `size_t` rather than `int `for array sizes.
    size_t newLimit = 2u * memLimit;
    char** newptr = realloc(words, newLimit * sizeof *newptr);
    if (newptr == NULL) {
       puts("Out-of-memory");
       // Code still can use old `words` pointer of size `memLimit * sizeof *words`
       return -1;
    }
    memLimit = newLimit;
    words = newptr;
  }

Answer 3

失误

• 不要转换malloc / calloc返回。 不需要它。
• 您的第一个sizeof错误。 它应该是sizeof(char*)
• 该scanf()格式字符串。 %s做得很好。

码

以下代码对我有用（每行打印一个单词）：

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

int main(int argc, char *argv[])
{
    FILE *file;
    file = fopen(argv[1], "r");
    char *match = argv[2];
    if (file == NULL) {
        printf("File does not exist\n");
        return EXIT_FAILURE;
    }
    int numWords = 0, memLimit = 20;
    char **words = calloc(memLimit, sizeof(char*));
    printf("Allocated initial array of 20 character pointers.\n");
    char string[20];

    while (fscanf(file, "%s", string) != EOF) {
        words[numWords] =
            malloc(strlen(string) + 1 * sizeof(char));
        strcpy(words[numWords], string);
        printf("Words: %s\n", words[numWords]);
        numWords++; /*keep track of indexes, to realloc */
        if (numWords == memLimit) {
            memLimit = 2 * memLimit;
            words = realloc(words, memLimit * sizeof(char *));  
            printf
                ("Reallocated array of %d character pointers.\n",
                 memLimit);
        }

    }
}

用./realloc realloc.c调用

希望能帮助到你。

Answer 4

您的第一个分配是问题。 您分配20个字符，并将其视为20个字符指针。 您超出了分配的缓冲区并破坏了内存。

第二个分配失败，因为堆已损坏。