这是向容器添加元素的正确方法吗？

Question

自从我编写C++以来已经有一段时间了,所以我想确认一下我是否正确地做了这个:

vector<Mat>
VideoHash::dct3d(vector<Mat> cube)
{
    vector<Mat> dctPlanes;
    for (int k = 0; k < TEMPORAL_DIM; ++k)
    {
        Mat spatial;
        Mat freq;
        cube[k].convertTo(spatial, CV_64F);
        dct(spatial, freq);
        dctPlanes.push_back(freq);
    }

    // Do other stuff here
}

我有一个矩阵向量作为函数的输入.对于向量中的每个矩阵,我转换为不同的数据类型(double),对转换结果执行DCT,然后将DCT的结果存储在另一个向量中.

这看起来是对的吗？我对for循环中创建的矩阵有疑问.循环退出后,它们将超出范围.分配给堆栈中那些矩阵的内存会发生什么变化？循环后是否仍然可以访问该内存(例如,在"在此处执行其他操作"部分).

Answer 1

当您将矩阵推回到向量中时,它会按值复制到由向量分配的内存中.只要存在dctPlanes,其中的矩阵仍然存在.当然,你的临时(空间,频率)将在每个循环迭代中超出范围.

编辑:为了解决优化问题,您可以考虑这样的事情:

VideoHash::dct3d(vector<Mat> & cube)
{
    Mat spatial;
    vector<Mat> dctPlanes;
    dctPlanes.resize(TEMPORAL_DIM);
    for (int k = 0; k < TEMPORAL_DIM; ++k)
    {
        cube[k].convertTo(spatial, CV_64F);
        dct(spatial, dctPlanes[k]);
    }

    // Do other stuff here
}

它并不完美,因为理想情况下,你需要避免在调整向量大小时调用所有构造函数的成本 - 并且有很多方法 - 但与原始代码相比,这将获得不错的收益.