c中结构返回值的逻辑错误

Question

我必须从 C 中的结构返回两个值：

我有以下数据结构：

struct Bar
{
        char *x;
        long y;
};

struct Bar funct();
struct Bar funct()
{
        struct Bar result;
        FILE *fp;
        long lSize;
        char *buffer;

        fp = fopen("list.txt", "rb");
        if (!fp) perror("list.txt"), exit(1);

        fseek(fp, 0L, SEEK_END);
        lSize = ftell(fp);
        rewind(fp);

        buffer = calloc(1, lSize+1);
        if(!buffer) fclose(fp), fputs("memory alloc fails", stderr), exit(1);

        if (1!=fread(buffer, lSize, 1, fp))
                fclose(fp), free(buffer), fputs("entire read fails", stderr), exit(1);

        fclose(fp);
        free(buffer);

        printf("%ld\n", lSize);

        result.x = (char *) buffer;
        result.y = lSize;
}

当我在 char 函数中实现相同的代码时，我用 char 数组而不是结构来获取缓冲区，而 lSize 为 744。

我的主要功能如下：

int main()
{
        char *buffer;
        printf("Reading file...\n");
        //buffer = readFile();
        struct Bar result;
        buffer = result.x;
        printf("%s\n", buffer);
        printf("%ld\n", result.y);
        //collect_character_distribution(buffer);


return 0;
}

但在结构中，我没有从 *buffer 中获取数组值，而是从 lSize 中获取 0。我需要返回 *buffer 和 lSize 值。出于这个原因，我必须使用结构。

我该如何解决这个问题？

谢谢，

Answer 1

你的代码有问题 -

 free(buffer);                      // don't free it earlier, you later use it 
 printf("%ld\n", lSize);
 result.x = (char *) buffer;       //casting is not needed

你 free buffer 然后让 result.x 指向它。 free 之后调用函数。

还有这个-

 if (1!=fread(buffer, lSize, 1, fp))
            fclose(fp), free(buffer), fputs("entire read fails", stderr), exit(1);

你应该避免这样写。它代表得相当不清楚。

你可以清楚地这样写-

 if(1!=fread(buffer, lSize, 1, fp)){
        fclose(fp);
        free(buffer);
        fputs("entire read fails", stderr);
        exit(1);
 }

相应地更改之前的if。

注意 - 你说你想从函数中return 一些东西，但你的代码似乎没有这样做。