malloc/realloc分配数组的内存释放方法及代码优化咨询

正确的释放方式：方式一 #

malloc、realloc分配的是整块连续的堆内存，对应的释放操作只需要调用一次free(ptr)即可，完全不需要逐个元素释放。

方式二的问题：

• ptr+i指向的是数组内部的元素地址，并非malloc/realloc返回的原始分配指针，对这类地址调用free属于未定义行为，会直接破坏堆结构，引发程序崩溃或内存错误。
• 即便循环释放元素后再释放ptr，也会造成重复释放——整块内存已经被标记为可回收，二次释放会触发严重的内存异常。

另外，C标准明确规定free(nullptr)是安全操作，所以甚至可以省略if (ptr != nullptr)的判断，直接写free(ptr);。

第二段代码的优势 #

你判断得完全正确：第二段代码比第一段更健壮。
第一段代码中，如果realloc失败返回nullptr，会直接覆盖原有的ptr，导致之前已经分配的内存地址丢失，造成内存泄漏；而第二段用new_ptr接收realloc结果，失败时ptr仍保留之前有效的内存地址，后续可以正常释放已分配的内存。

进一步优化方向 #

1. 修复随机数生成的逻辑问题 #

FillMem函数中每次调用srand(time(0))会重置随机数种子，如果循环执行速度快（1秒内完成多次调用），time(0)返回的时间戳相同，会导致生成的随机数序列完全重复。应该把srand(time(0))移到main函数开头，只初始化一次：

int main() {
    srand(time(0)); // 仅初始化一次随机数种子
    // ... 其他代码
}

2. 用C++标准容器替代手动内存管理 #

在C++中，std::vector是动态数组的最优选择，它会自动处理内存分配、扩容和释放，完全避免手动调用malloc/realloc/free的风险：

#include <vector>

int main() {
    const size_t size_iter = 10000000;
    int n = 30;
    std::vector<int> vec;
    vec.reserve(size_iter); // 预分配初始空间，减少扩容次数

    for (int i = 1; i <= n; ++i) {
        size_t size = size_iter * i;
        vec.resize(size);
        FillMem(vec.data(), size);
        
        std::cout << '[' << i << ']' << " address: " << static_cast<void*>(vec.data()) 
                  << ", size: " << size << " *********************" << std::endl;
    }
    // 无需手动释放内存，vector销毁时自动回收
}

3. 规范类型转换与变量类型 #

• 替换C风格强制转换，用static_cast<int*>替代(int*)，更符合C++的类型安全要求。
• 将size_iter、size的类型改为size_t（内存大小/数组索引的标准类型），避免int类型溢出的风险：

const size_t size_iter = 10000000;
size_t size = size_iter;

4. 统一输出风格 #

不要混合std::cout和printf，保持代码风格一致，比如用std::cout输出地址：

std::cout << '[' << i << ']' << " address: " << static_cast<void*>(ptr) 
          << ", size: " << size << " *********************" << std::endl;

5. 完善错误处理逻辑 #

当内存分配失败时，第二段代码应该释放已经分配的内存，避免泄漏：

if (new_ptr == nullptr) {
    printf("memory allocation error\n");
    free(ptr); // 释放之前已分配的内存
    break;
}

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