ecCodes（GRIB读取库）内存未释放问题：是代码错误还是预期行为？

兄弟，你遇到的内存持续增长问题完全是代码里的手动内存管理错误导致的，和ecCodes库本身没关系。咱们来拆解一下问题：

问题根源 #

你在处理GRIB文件的while循环里，每次调用codes_handle_new_from_file()都会创建一个新的codes_handle句柄，但你把codes_handle_delete(h)放在了while循环外面——这就意味着只有最后一次循环生成的句柄被释放了，前面所有迭代创建的句柄都一直占着内存没被回收！四个1.5GB的文件，加上句柄关联的内存开销，最终内存涨到6.5GB就完全说得通了。

修正后的代码 #

把codes_handle_delete(h)移到while循环内部，每次处理完一个GRIB字段就立刻释放对应的句柄，同时补上必要的错误检查（你的原代码里缺失了不少错误处理逻辑）：

#include <string>
#include <vector>
#include <iostream>
#include "eccodes.h"

int main() {
    std::string filenames[] = {"../data/era5_model.grib", "../data/era5_model2.grib", "../data/era5_model3.grib", "../data/era5_model4.grib"};
    std::vector<long> vec = {};

    for (auto & filename : filenames) {
        FILE* f = fopen(filename.c_str(), "r");
        if (!f) { // 检查文件打开是否成功
            std::cerr << "Failed to open file: " << filename << std::endl;
            continue;
        }

        int err = 0;
        codes_handle* h;
        while ((h = codes_handle_new_from_file(nullptr, f, PRODUCT_GRIB, &err)) != nullptr) {
            long k1 = 0;
            err = codes_get_long(h, "level", &k1);
            if (err != CODES_SUCCESS) { // 检查获取key是否成功
                std::cerr << "Failed to get 'level' key: " << codes_get_error_message(err) << std::endl;
                codes_handle_delete(h); // 出错也要释放句柄，避免泄漏
                continue;
            }
            vec.push_back(k1);
            codes_handle_delete(h); // 处理完立刻释放句柄！
        }

        if (err != CODES_SUCCESS && err != CODES_END_OF_FILE) { // 区分正常结束和异常错误
            std::cerr << "Error reading file " << filename << ": " << codes_get_error_message(err) << std::endl;
        }

        fclose(f);
    }

    if (vec.size() > 52) { // 避免越界访问的安全检查
        std::cout << vec[52];
    } else {
        std::cerr << "Vector index 52 is out of bounds!" << std::endl;
    }

    return 0;
}

额外优化建议 #

如果你用的是C++11及以上版本，推荐用RAII（资源获取即初始化）封装codes_handle，彻底避免手动管理内存的失误。比如写个简单的智能指针封装：

#include <memory>

struct CodesHandleDeleter {
    void operator()(codes_handle* h) const {
        if (h) codes_handle_delete(h);
    }
};

using UniqueCodesHandle = std::unique_ptr<codes_handle, CodesHandleDeleter>;

之后在循环里用这个智能指针，完全不用手动调用释放函数：

UniqueCodesHandle h(codes_handle_new_from_file(nullptr, f, PRODUCT_GRIB, &err));
while (h) {
    // 处理逻辑...
    h.reset(codes_handle_new_from_file(nullptr, f, PRODUCT_GRIB, &err));
}

验证效果 #

修正后，处理每个文件时，内存会在处理完所有GRIB字段后回到初始水平（大概1.5GB左右），不会再持续累积。至于你提到的Rust FFI调用里也存在同样问题，大概率也是相同原因——你没有在每次创建句柄后都调用对应的释放函数。

内容的提问来源于stack exchange，提问作者J.Lewandowski