如何在C中使用带有双字符指针的realloc？

Question

我试图在双字符指针中将内存增加一行，但由于某种原因程序崩溃了，即使我没有收到任何警告。我用谷歌搜索过，人们一直在提供这种方式来重新分配，但它对我不起作用。你知道问题在哪里吗？它在最后一行。

提前致谢！

#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#define E 255

void Palindromas(char* Zodis, char eilute[E], int *kiekis, char** Ats);

int main()
{
    char Eilute [E];
    int i, kiekis = -1;
    char** Ats;
    char* Zodis;

    Zodis = malloc(E * sizeof(char));
    Ats = malloc(1 * sizeof(char*));

    // Failu aprasymas

    FILE *Duomenys = fopen("Duomenys.txt", "r");
    FILE *Rezultatai = fopen("Rezultatai.txt","w");

    //--------------------------------

    if (Ats == NULL)
    {
        printf("Atmintis nebuvo paskirta");
        exit(0);
    }

    if (Zodis == NULL)
    {
        printf("Atmintis nebuvo paskirta");
        exit(0);
    }

    if (Duomenys == NULL) printf("Nurodyto duomenu failo nera \n");

    while(fgets(Eilute, sizeof(Eilute), Duomenys))
    {
        Palindromas(Zodis, Eilute, &kiekis, Ats);
    }

    for (i = 0; i <= kiekis; i++)
    {
        fprintf(Rezultatai, "%s ", Ats[i]);
    }

    free(Ats);
    free(Zodis);
    fclose(Duomenys);
    fclose(Rezultatai);

    return 0;
}

void Palindromas(char* Zodis, char Eilute[E], int *kiekis, char** Ats)
{
    int i, l, j, index, ilgis, eil;

    for(i = 0; 1 == sscanf(Eilute + i, "%s%n", Zodis, &l); i = i + l)
    {

        *kiekis = *kiekis + 1;
        index = 1;
        ilgis = strlen(Zodis);

        Ats[*kiekis] = malloc(ilgis * sizeof(char));

        for (j = 0; j < ilgis; j++)
        {
            if (Zodis[j] != Zodis[ilgis - j - 1])
                index = 0;
        }

        if (index == 1) strcpy(Ats[*kiekis], Zodis);

        eil = *kiekis + 2;
        Ats = realloc(Ats, eil * sizeof(char*));
    }
}

Answer 1

我不得不用谷歌来理解你的代码，也许下次你可以帮我们翻译一下。

我有几点意见：

• if (Duomenys == NULL) 只是打印一条错误消息，但不会停止执行。它应该像之前的验证一样（如果返回错误代码更好）。
• Ats[*kiekis] = malloc(ilgis * sizeof(char)); 是错误的，应该是 Ats[*kiekis] = malloc((ilgis+1) * sizeof(char)); 因为它需要空结尾字符的空间。
• if (Zodis[j] != Zodis[ilgis - j - 1]) index = 0; 很好，但if (Zodis[j] != Zodis[ilgis - j - 1]) { index = 0; break; } 更好，因为这样循环将在第一个不同的字符处停止。
• if (index == 1) strcpy(Ats[*kiekis], Zodis); 必须是 if (index == 1) strcpy(Ats[*kiekis], Zodis);else --*kiekis 否则您的 Ats 数组也将不包含回文词。
• 别忘了释放Ats 数组中的每个元素，然后再使用for(i=0; i<=kiekis; i++) free(Ats[i]); 之类的东西释放数组本身

最后，回答您的问题，重新分配 Ats 数组即可：

Ats = realloc(Ats, (*kiekis+1) * sizeof(char*));

我会像这样重新组织你的功能：

void Palindromas(char* Zodis, char Eilute[E], int *kiekis, char** Ats)
{
    int i, l, j, index, ilgis;
    for(i = 0; 1 == sscanf(Eilute + i, "%s%n", Zodis, &l); i = i + l)
    {
        index = 1;
        ilgis = strlen(Zodis);
        for (j = 0; j < ilgis; j++)
        {
            if (Zodis[j] != Zodis[ilgis - j - 1])
            {
                index = 0;
                break;
            }
        }
        if (index == 1)
        {
            ++*kiekis;
            Ats = (char **)realloc(Ats, (*kiekis+1) * sizeof(char*));
            Ats[*kiekis] = dup(Zodis); // dup == malloc+strcopy
        } 
    }
}