从文件中填充 C 中嵌套结构的正确方法

Proper way to fill nested struct in C from file

提问人:user2030243 提问时间:6/9/2021 最后编辑:user2030243 更新时间:6/9/2021 访问量:147

问:

我从想要适合结构的文件中读取数据。此结构包含可变大小的数组。所以我使用 realloc 来即时填充它。不幸的是,我的程序失败了。Valgrind 报告了带有 realloc 的 pb:

==3170== Invalid write of size 8
==3170==    at 0x1093E6: charge_Un_BV (charge_un_bv.c:56)
==3170==    by 0x109193: main (charge_un_bv.c:87)
==3170==  Address 0x4e507a8 is 8 bytes before a block of size 8 alloc'd
==3170==    at 0x4A3F2CC: realloc (in /home/linuxbrew/.linuxbrew/Cellar/valgrind/3.17.0/libexec/valgrind/vgpreload_memcheck-amd64-linux.so)
==3170==    by 0x1093CD: charge_Un_BV (charge_un_bv.c:53)
==3170==    by 0x109193: main (charge_un_bv.c:87)

这是我的程序:

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

typedef struct st_BV {
    char nom_BV[40];
    int  id_bv;
    long  tabIndicePointsLength;
    long*  tabIndicePoints;
} BV;

BV* charge_Un_BV(char*); 
 
BV* charge_Un_BV(char* nomFileBV)
{
    BV* oneBV;
    oneBV = (BV*)malloc(sizeof(BV));
    if (oneBV == NULL) { printf("Erreur allocation mémoire\n"); exit(-6); }

    FILE* pt_fichier_BV = fopen(nomFileBV,"r");
    char ligne[100];
    long indice;
    float lon,lat;
    long count_ligne = 0;

    printf("start\n");  

    if (pt_fichier_BV == NULL)
    {
        printf("unable to open %s\n", nomFileBV);
        exit(-1);
    }
    else
    {
        printf("File opened\n");
        char* resultat = fgets(ligne,100,pt_fichier_BV);
        if (resultat == NULL) {printf("Error reading file\n"); exit(-2);}
        printf("First ligne skip: %s",ligne);
        
        
        oneBV->tabIndicePoints = (long*)malloc(sizeof(long));
        if (oneBV->tabIndicePoints == NULL) { printf("Error with alloc\n"); exit(-3); }

        int cr;
        
        do{
            cr = fscanf (pt_fichier_BV,"%ld %f %f",&indice, &lon, &lat);

            // reallocation according to the number of points read in the file
// BELOW LINE 53 :
            oneBV->tabIndicePoints = (long*)realloc(oneBV->tabIndicePoints,(count_ligne+1) * sizeof(long));
            if (oneBV->tabIndicePoints == NULL) { printf("Error realloc\n"); exit(-4); }
            
// BELOW LINE 56 :
            oneBV->tabIndicePoints[count_ligne-1] = indice;
            count_ligne++;
        }while(cr!=EOF);
        //total number of points
         oneBV->tabIndicePointsLength = count_ligne-1;
  //TEST
        printf("First indice (index): %ld\n",oneBV->tabIndicePoints[0]);
        printf("Fourth  indice (index): %ld\n",oneBV->tabIndicePoints[3]);
        printf("Total points: %ld\n",  oneBV->tabIndicePointsLength);

    }
    //close file
    int cr = fclose(pt_fichier_BV);
    if (cr != 0)
    {
        printf("Erro closing file\n");
        exit(-5);
    }   
    return oneBV;
}


// Programme TEST
int main()
{
    BV* lebv;
// BELOW LINE 87 :
    lebv = charge_Un_BV("../data/data.txt");
    
    free (lebv);

    return 0;
}

所以我想知道用结构填充动态数组的正确方法是什么。 问候

在第一个解决方案后编辑

我将这个问题添加到我的第一篇文章中。 valgrind 报告仍然给我关于内存泄漏的错误。我猜,这来自 realloc 之前的第一个 alloc 或最后一个 realloc。所以我有一块内存没有释放。这不会阻止程序工作,但我们可以改进它吗?我添加了“免费(lebv);”

==2858== 
début
Fichier ouvert
Première ligne: ind0 lon lat
Premier indice: 584119
Quatrième  indice: 584120
Nombre de points: 871
Quatrième  indice (2): 584120
==2858== 
==2858== HEAP SUMMARY:
==2858==     in use at exit: 6,976 bytes in 1 blocks
==2858==   total heap usage: 877 allocs, 876 frees, 3,050,688 bytes allocated
==2858== 
==2858== 6,976 bytes in 1 blocks are definitely lost in loss record 1 of 1
==2858==    at 0x4A3F2CC: realloc (in /home/linuxbrew/.linuxbrew/Cellar/valgrind/3.17.0/libexec/valgrind/vgpreload_memcheck-amd64-linux.so)
==2858==    by 0x1093ED: charge_Un_BV (charge_un_bv.c:53)
==2858==    by 0x1091B0: main (charge_un_bv.c:87)
==2858== 
==2858== LEAK SUMMARY:
==2858==    definitely lost: 6,976 bytes in 1 blocks
==2858==    indirectly lost: 0 bytes in 0 blocks
==2858==      possibly lost: 0 bytes in 0 blocks
==2858==    still reachable: 0 bytes in 0 blocks
==2858==         suppressed: 0 bytes in 0 blocks
==2858== 
==2858== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 0 from 0)
C 文件 结构 realloc alloc

评论

0赞 David C. Rankin 6/9/2021
在 C 中,不需要强制转换返回 ,这是不必要的。请参阅:我是否投射了 malloc 的结果?通常,它与指针一起使用,而不是与声明的类型一起使用。为什么? 不声明 3 个指针,它声明 1 个指针和 2 个字符变量。 说得很清楚。malloc'*'char* a, b, c;charchar *a, b, c;
0赞 user2030243 6/9/2021
感谢您的这些提示,我将从现在开始应用它们。但是,我发现 char* a;如果我们知道您的评论,则符号会更明确。
0赞 David C. Rankin 6/9/2021
只要你清楚效果,这才是最重要的。无论哪种方式都是完全可以的。

答:

2赞 idz 6/9/2021 #1

初始化为 0count_ligne

    long count_ligne = 0;

然后从中减去 1,然后递增并将其用作索引(访问索引 -1)

            oneBV->tabIndicePoints[count_ligne-1] = indice;
            count_ligne++;

您可能希望:

  • 不减去 1,
  • 增量后再使用。

评论

0赞 user2030243 6/9/2021
谢谢,这确实是问题所在,我想知道我怎么看不到它。但是,我还有另一个担忧,valgrind 给了我一个内存泄漏,这不会阻止程序正常运行。我将编辑我的帖子。
0赞 user2030243 6/9/2021
最后我找到了泄漏点。你必须做:free (lebv.tabIndicePoints);
0赞 idz 6/10/2021
很高兴你发现了问题。如果您在回答了第一个问题后有后续问题,您应该打开另一个问题。您将获得更快的响应,并使其他用户在遇到类似问题时更容易找到解决方案。