在for循环中使用malloc()分配结构体数组失效问题求助

核心错误原因 #

你在for循环里反复调用malloc()的方式完全错误：每次malloc都会分配一块全新的内存区域，之前分配的内存地址会被直接覆盖，导致之前存储的节点数据全部丢失，同时还会造成内存泄漏。这种方式根本无法正确累计所有符合条件的节点，自然会出现部分数据正确、部分错误的情况。

正确实现步骤 #

1. 先统计符合条件的节点数量
   先遍历一遍一维数组，数出所有大于80的数字的总个数，记为count。
2. 一次性分配结构体数组内存
   根据统计出的count，一次性调用malloc(sizeof(NODE) * count)分配足够的内存空间，避免重复分配带来的问题。
3. 再次遍历数组，填充结构体数组
   遍历过程中记录每个符合条件数字的X/Y坐标（注意坐标转换逻辑要和矩阵输出规则匹配），依次存入结构体数组。

修正后的代码示例 #

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

typedef struct {
    int x;
    int y;
    int value;
} NODE;

int main() {
    int arr[576];
    FILE *fp = fopen("data.txt", "r");
    if (!fp) {
        perror("Failed to open file");
        return 1;
    }

    // 读取576个数字到一维数组
    for (int i = 0; i < 576; i++) {
        fscanf(fp, "%d", &arr[i]);
    }
    fclose(fp);

    // 1. 统计符合条件的节点总数
    int count = 0;
    for (int i = 0; i < 576; i++) {
        if (arr[i] > 80) {
            count++;
        }
    }

    // 2. 一次性分配结构体数组内存
    NODE *nodes = (NODE*)malloc(sizeof(NODE) * count);
    if (!nodes) {
        perror("Failed to allocate memory");
        return 1;
    }

    // 3. 填充结构体数组（坐标按行优先转换：18行32列）
    int idx = 0;
    for (int i = 0; i < 576; i++) {
        if (arr[i] > 80) {
            nodes[idx].y = i / 32;  // Y对应行号
            nodes[idx].x = i % 32;  // X对应列号
            nodes[idx].value = arr[i];
            idx++;
        }
    }

    // 输出验证结构体数据
    printf("过滤后的节点数据：\n");
    for (int i = 0; i < count; i++) {
        printf("X: %d, Y: %d, Value: %d\n", nodes[i].x, nodes[i].y, nodes[i].value);
    }

    // 释放动态分配的内存
    free(nodes);
    return 0;
}

额外注意事项 #

• 坐标转换要和你的矩阵输出逻辑一致：如果矩阵是列优先存储（先存第一列18个，再第二列），需要把坐标转换改为x = i / 18，y = i % 18。
• 每次调用malloc后必须检查返回值是否为NULL，避免内存分配失败导致程序崩溃。
• 程序结束前一定要用free()释放动态内存，避免内存泄漏。

内容的提问来源于stack exchange，提问作者thenyik Jin