C++循环读取DCM文件转cv::Mat存入二维vector仅保留最后值如何解决

你遇到的问题和for循环局部变量保存到外部vector无关，C++的vector值语义本身就会拷贝局部变量内容到外部容器，问题出在你的img_processing函数内部实现存在内存复用/浅拷贝问题，导致每次返回的vector实际都对应同一块被反复覆盖的内存，最后只有最后一次读取的内容生效。

问题排查与修复方案 #

1. 先修复基础语法错误
   你当前的img_processing函数返回值缺少模板参数，属于语法错误，首先修正为：

// 明确返回值类型为存储16位像素的vector
vector<Uint16> img_processing(string path) {
    // 内部实现
}

2. 核心问题：确保img_processing内部不存在内存复用/浅拷贝
   90%以上的同类问题都是因为内部实现用了全局/静态缓存、或者cv::Mat转vector时没有做深拷贝：

• 不要在函数内用static修饰存储像素的vector变量，静态变量只会初始化一次，每次调用都会覆盖同一块内存，最终所有返回值都对应最后一次修改的内容
• cv::Mat转vector时必须做全量值拷贝，正确写法示例：

// 假设你已经把DCM转成了cv::Mat类型的dcm_mat，类型为CV_16U
vector<Uint16> A(dcm_mat.begin<Uint16>(), dcm_mat.end<Uint16>());
return A;

不要直接把Mat的data指针强转赋值给vector，否则vector只会持有指针，Mat内存被后续读取覆盖时，vector里的值也会跟着变。
3. 验证读取DCM的逻辑
如果你用DCMTK等库读取DCM文件，确认每次读取新文件时，都是为当前文件单独申请的像素缓存，没有复用同一个全局缓存区。
4. 可选优化写法
如果担心push_back的扩容问题，可以提前给外部vector预留空间再赋值：

int main(){
    vector<string> files; // 已经填充了所有DCM文件路径
    vector<vector<Uint16>> array(files.size()); // 提前初始化对应大小
    for (int i = 0; i < files.size(); i++){
        array[i] = img_processing(files[i]);
    }
    // 后续处理array即可
}

内容的提问来源于stack exchange，提问作者許煜承 Wistar