使用malloc动态扩展浮点数组遇内存分配异常问题求助

嘿，我来帮你捋捋这个动态数组分配的问题～你遇到的核心问题是每次循环都只分配了单个float的空间，没有对数组进行扩容——直接用malloc的话，每次都会创建新的内存块，覆盖之前的指针，不仅会丢失之前的数据，还会造成内存泄漏。

问题根源 #

你当前的代码逻辑应该是每次循环里给pole_cien直接分配sizeof(float)的空间，这相当于每次都重新创建一个只能存1个浮点数的数组，完全没有保留之前的元素，也没有扩容数组的长度。

正确的实现思路 #

要实现“每次添加新浮点数就扩容数组”，你需要用realloc函数来完成内存的动态扩容——它会在原有内存的基础上，重新分配更大的空间，并且自动保留原有数据（如果内存足够的话）。具体步骤如下：

• 初始时将数组指针设为NULL，元素计数初始化为0，方便后续扩容逻辑。
• 每次读取到新的浮点数字符串后，用realloc申请当前元素数+1个float的空间。
• 用临时指针接收realloc的返回值，避免分配失败时丢失原有内存指针。
• 将转换后的浮点数存入数组的新位置，然后递增元素计数。
• 处理内存分配失败的情况，避免内存泄漏。

修正后的代码示例 #

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

float *funkcia_e(FILE **subor, int *pocet_cien) {
    float *pole_cien = NULL;
    char nazov[256]; // 定义足够大的缓冲区存储每行内容
    *pocet_cien = 0; // 初始元素数设为0

    while (fgets(nazov, sizeof(nazov), *subor) != NULL) {
        // 移除fgets读取到的换行符，避免转换浮点数出错
        nazov[strcspn(nazov, "\n")] = '\0';

        // 扩容数组：申请能容纳当前元素数+1个浮点数的空间
        float *temp_ptr = realloc(pole_cien, (*pocet_cien + 1) * sizeof(float));
        if (temp_ptr == NULL) {
            // 分配失败，释放已分配的内存避免泄漏
            free(pole_cien);
            fprintf(stderr, "内存分配失败！\n");
            return NULL;
        }
        pole_cien = temp_ptr;

        // 将字符串转换为浮点数，存入数组的新位置
        pole_cien[*pocet_cien] = atof(nazov);
        (*pocet_cien)++;
    }

    return pole_cien;
}

关键细节说明 #

• 用realloc而非malloc：realloc是动态扩容数组的标准方式，它会处理原有内存的复制和释放，避免手动管理多个内存块的麻烦。
• 临时指针接收返回值：如果realloc分配失败会返回NULL，直接赋值给pole_cien会导致之前的内存指针丢失，造成内存泄漏，所以先用临时指针接收，确认成功后再赋值。
• 处理换行符：fgets会把换行符\n也读入缓冲区，用strcspn可以快速定位并替换为字符串结束符，避免atof转换时出错。
• 内存泄漏防护：如果分配失败，一定要先释放之前已经分配的内存，再返回错误。
• 调用后记得释放内存：在调用funkcia_e的代码中，使用完返回的数组后，一定要用free(pole_cien)释放内存，避免长期运行导致内存泄漏。

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