3

我对std::multimap容器有些头疼,我想知道实现目标的正确方法是什么。基本上,这是我的SSCCE

#include <iostream>
#include <map>

int main ()
{
    typedef std::multimap<int, int> CollectionType;
    typedef std::pair<CollectionType::iterator, CollectionType::iterator> RangeType;

    CollectionType multiMap;

    multiMap.insert(std::make_pair(1, 1));
    multiMap.insert(std::make_pair(1, 2));
    multiMap.insert(std::make_pair(2, 3));
    multiMap.insert(std::make_pair(2, 4));
    multiMap.insert(std::make_pair(2, 5));
    multiMap.insert(std::make_pair(3, 1));

    RangeType range = multiMap.equal_range(2);
    for (CollectionType::iterator iterator = range.first; iterator != range.second; ++iterator)
    {
        if (iterator->second == 4)
        {
            //multiMap.erase(iterator);//ISSUE 1
        }
        else
        {
            //iterator->first = -1;//ISSUE 2
        }
    }

    return 0;
}

正如您在上面看到的,我需要multimap为给定键选择一个范围,然后:

  1. 我需要从范围中删除某些行
  2. 我需要从范围内更改其他行的键

关于 1,由于“对已擦除元素的引用和迭代器无效”,我该如何删除这些元素?我应该在某个容器中推送特定的迭代器并在循环结束后对其进行迭代吗?我看到了这个答案,但它似乎有点hackish/丑陋/容易出错/等等......

关于2,由于“愚蠢”的方法(显然)不起作用,那么实现我需要的好方法是什么?一旦我解决了问题 1,我可以删除元素并在它们的位置插入新元素,但是如果假设我错误地插入了一个与我刚刚删除的键相同的项目,那可能不会弄乱迭代吗? ..

int second = iterator->second;
localEdges.smarter_erase(iterator);
localEdges.insert(std::make_pair(-1, second));
4

1 回答 1

4

erase返回一个迭代器,因此将您的迭代器重置为该迭代器,它将继续有效。

只需重新插入新密钥,擦除原始密钥

http://ideone.com/0Pr6Qc

#include <iostream>
#include <map>

void printMultimap(const std::multimap<int, int>& multiMap)
{
    std::cout << "MultiMap:\n";
    for (const auto& pair : multiMap)
    {
        std::cout << pair.first << ":" << pair.second << "\n";
    }
}

int main()
{
    std::multimap<int, int> multiMap;
    multiMap.insert(std::make_pair(1, 1));
    multiMap.insert(std::make_pair(1, 2));
    multiMap.insert(std::make_pair(2, 3));
    multiMap.insert(std::make_pair(2, 4));
    multiMap.insert(std::make_pair(2, 5));
    multiMap.insert(std::make_pair(3, 1));

    printMultimap(multiMap);

    auto range = multiMap.equal_range(2);
    for (auto iterator = range.first; iterator != range.second;)
    {
        if (iterator->second != 4)
        {
            multiMap.insert(std::make_pair(-1, iterator->second));
        }
        iterator = multiMap.erase(iterator);
    }

    printMultimap(multiMap);

    return 0;
}
于 2013-03-12T02:25:28.393 回答