使用struct成员创建映射会抛出bad_alloc

Question

所以我有一个结构

struct float3
{
    float x, y, z;
};

我正在尝试创建一个函数来获取x，y，z值并将它们映射到键0,1,2各自的维度。我编写了下面的代码，但它抛出了一个错误的alloc_exception。看起来我的内存不足了。

KdTree::float2map(std::vector<float3>& data)
    {    
       std::vector<std::map<int, float> > m_pnts;
       int cnt = 0;
       for(int i = 0; i = data.size(); i++)
          {
             std::map<int, float> tmp;
             tmp.insert(std::make_pair(0, data[i].x));
             tmp.insert(std::make_pair(1, data[i].y));
             tmp.insert(std::make_pair(2, data[i].z));

             m_pnts.push_back(tmp);
             std::cout << m_pnts.size() << std::endl;
            }

       }

       return m_pnts;
    }

我仍然是C ++的新手，所以我确信还有很多其他方法可以做到这一点或优化这种方法。问题是我必须做一个33,914,095个浮点数，我想不出另一种方法来实现这一点。任何帮助将不胜感激。

Answer 1

看看这行代码：

for(int i = 0; i = data.size(); i++)

你在i声明的条件下将data.size()设置为for。这将导致越界访问，因为data.size()将成为data向量的索引，其元素从0索引到data.size() - 1。

那可能不是你的意图。在for循环中使用适当的条件。它应该是：

for(int i = 0; i < data.size(); i++)

或者更好的是，使用range-based for loop有助于避免此类错误：

for(const auto& ele: data)
{
    std::map<int, float> tmp;
    tmp.insert(std::make_pair(0, ele.x));
    tmp.insert(std::make_pair(1, ele.y));
    tmp.insert(std::make_pair(2, ele.z));
    m_pnts.push_back(tmp);
    std::cout << m_pnts.size() << std::endl;
}