C++中<string, vector<double>>映射转<string, double[]>映射的问题排查

你的代码存在两个核心问题，直接导致了错误结果：

• 栈数组合法性问题：标准C++不允许用运行时变量（比如it.second.size()）作为栈数组的大小，你写的double temp_arr[size of it.second as a const];属于非标准语法（仅部分编译器支持VLA扩展，不可移植），本身就有编译隐患。
• 局部变量生命周期问题：temp_arr是循环内的栈局部变量，每次循环结束后这块栈内存会被系统回收。你把temp_arr的指针存入array_map，本质上存的都是野指针——指向已经被释放的内存区域。后续访问这些指针时，栈内存可能已经被其他数据覆盖，所以会出现所有数组内容相同且和原向量不一致的情况。

正确实现方式 #

方式一：动态分配数组（需手动管理内存） #

std::map<std::string, double*> array_map;
for (const auto& pair : vector_map) {
    const size_t vec_size = pair.second.size();
    // 用new动态分配堆内存，生命周期由手动控制
    double* temp_arr = new double[vec_size];
    std::copy(pair.second.begin(), pair.second.end(), temp_arr);
    array_map.insert({pair.first, temp_arr});
}

// 注意：使用完array_map后必须手动释放内存，避免内存泄漏
for (const auto& pair : array_map) {
    delete[] pair.second;
}

方式二：用智能指针自动管理内存（推荐） #

使用std::unique_ptr可以避免手动释放内存的麻烦，安全性更高：

#include <memory> // 必须包含此头文件

std::map<std::string, std::unique_ptr<double[]>> array_map;
for (const auto& pair : vector_map) {
    const size_t vec_size = pair.second.size();
    // 智能指针自动管理堆内存
    std::unique_ptr<double[]> temp_arr = std::make_unique<double[]>(vec_size);
    std::copy(pair.second.begin(), pair.second.end(), temp_arr.get());
    array_map.insert({pair.first, std::move(temp_arr)});
}

// 无需手动释放内存，unique_ptr会在元素被销毁时自动释放对应内存

额外说明 #

如果你的向量大小是编译期已知的固定值，可以考虑使用std::array替代动态数组，但从你的场景来看，向量大小应该是运行时动态变化的，因此上述两种动态分配的方式更合适。优先选择智能指针方案，能有效避免内存泄漏和野指针问题。

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