1

我已经在这段代码上工作了一段时间，但我遇到了一个似乎无法调试的段错误。以下是相关代码：

typedef struct Halo* Halo;  
struct Halo  
{  
    float x, y, z;  
    float vx, vy, vz;  
    int n200a;  
    float m200a;  
    float r200a;  
    int n200c;  
    float m200c;  
    float r200c;  
    int n500a;  
    float m500a;  
    float r500a;  
    int n500c;  
    float m500c;  
    float r500c;  
};

全局变量：

光环 *halo_catalog;

失败的功能：

int loadHaloCatalog(char *filename)  
{  
    FILE *catalog_file;  
    long long halo_num;

    catalog_file = fopen(filename, "rb");
    if (catalog_file == NULL) {
        printf("Could not open halo catalog: %s\n", filename);
        return -1;
    }
    if (fread(&halo_num, sizeof(long long), 1, catalog_file) < 0) {  
        printf("Could not read number of halos\n");  
        return -1;  
    }  
    halo_catalog = (Halo *)calloc(halo_num, sizeof(struct Halo));  
    if (fread(halo_catalog, sizeof(struct Halo), halo_num, catalog_file) < 0) {  
        printf("Could not read that number of halos\n");  
        return -1;  
    }  
    printf("%f\n", halo_catalog[10000]->x);  
    printf("done\n");  
    fclose(catalog_file);  
    return (int)halo_num;  
}

它在 "printf("%f\n", halo_catalog[10000]->x);" 上失败 行，或对 fread 调用后分配的内存的任何其他访问。我知道我传入了一个有效文件，因为它可以正确读取 halo_num。它还从 fread 调用中收集有关 Halo 对象的正确信息，就像我调用 fread 并检查返回时它返回 halo_num 一样。

谢谢！

4

2 回答 2

3 
typedef struct Halo* Halo;

这是一个可怕的想法，可能是你的问题的原因。

你有一个全局变量

Halo *halo_catalog;

所以一个struct Halo**。但在

halo_catalog = (Halo *)calloc(halo_num, sizeof(struct Halo));  
if (fread(halo_catalog, sizeof(struct Halo), halo_num, catalog_file) < 0) {

你使用它就好像它是一个struct Halo*.

然后你索引

printf("%f\n", halo_catalog[10000]->x);

在距10000 * sizeof(struct Halo*)wherehalo_catalog点的字节距离处，并解释该位置的字节 - 这是某些floats 或ints 读入的值的一部分，而不是指针 - 并尝试访问x任何任意位置的组件是该误解的结果.

你应该

typedef struct Halo Halo;

并用于halo_catalog[10000].x解决该问题。

另一个问题是fread返回它已成功读取的项目数，如果小于请求的数字，它仍然可以是正数。

捕获 的返回值fread，并使用它来确定读取是否完全成功。此外，在尝试打印之前，请确保 10000 是有效索引halo_catalog[10000].x。

于 2013-04-08T20:22:36.290 回答
2

这真的可以编译吗？

问题是 halo_catalog 是指向数组的指针，而不是指针数组。你应该使用printf("%f\n", halo_catalog[10000].x);

我假设 10000 仅用于测试目的？因为我没有看到可以保证 halo_catalog 中有 10000 条记录。

于 2013-04-08T20:09:58.653 回答