如何正确分配和解除分配向量图

Question

我试图将数据保存在无序的向量映射中，使用 uint32_t 地址作为键。向量需要存储在堆中，所以 即使函数超出范围，我仍然可以访问数据。我是这样分配的：

using vectorBuffer = std::vector<uint8_t>;
using I2cVectorMap = std::unordered_map<address_t, vectorBuffer>;
using I2cVectorEntry = std::pair<address_t, vectorBuffer>;

if(auto i2cIter = i2cVectorMap.find(address); i2cIter == i2cVectorMap.end()) {
    vectorBuffer * receiveVector = new vectorBuffer(maxReplyLen);
    i2cVectorMap.insert(I2cVectorEntry(address, *receiveVector));
}
else {
    // Already in map. readjust size
    vectorBuffer * existingVector = &i2cIter->second;
    existingVector->resize(maxReplyLen);
    existingVector->shrink_to_fit();
}

像这样的释放（根据各种来源的建议，将向量与空向量交换）：

// getting the address from another object.
address_t deviceAddress = i2cCookie->getAddress();
if(auto i2cIter = i2cVectorMap.find(deviceAddress); i2cIter != i2cVectorMap.end()) {
    vectorBuffer().swap(i2cIter->second);
}

这是一个正确的实现吗？我会将地址传递给映射条目，或者传递给启动 I2C 传输的驱动程序函数，或者传递给另一个处理数据的对象。我想确保我没有内存泄漏。

提前非常感谢！

Answer 1

不，您不应该使用new。首先需要直接news 的情况很少见。

您无需手动创建任何具有动态存储持续时间（“在堆上”）的对象。标准库容器直接存储对象，而不是对象的引用或指针。他们将为这些对象分配并自己构造它们。您只需向容器成员函数提供构造函数参数，容器（复制/移动）从中构造存储的元素。

您可以将对象声明为自动变量（即“在堆栈上”）：

vectorBuffer receiveVector(maxReplyLen);
i2cVectorMap.insert(I2cVectorEntry(address, receiveVector));

或为中间向量实例使用临时变量而不是命名变量（这也将使用移动而不是复制操作，因此性能更好）：

i2cVectorMap.insert(I2cVectorEntry(address, vectorBuffer(maxReplyLen)));

如果您使用emplace 而不是insert，您也可以删除冗余对类型：

i2cVectorMap.emplace(address, vectorBuffer(maxReplyLen));

---

您不需要指针来调用映射中对象的方法：

vectorBuffer& existingVector = i2cIter->second;
existingVector.resize(maxReplyLen);
existingVector.shrink_to_fit();

---

要从地图中删除一个元素，您可以使用它的.erase 成员函数：

i2cVectorMap.erase(deviceAddress);

你不需要做任何其他事情。这也会破坏和释放向量。

当您不再需要地图及其包含的矢量时，您也无需执行任何操作。地图的析构函数会在地图定义的范围离开时自动清理所有内容。