Question

我的代码看起来基本上是这样的：

std::map<int, int> map1, map2;
BOOST_FOREACH(int i, map1)
{
    // do steps 1-5 here...
}
BOOST_FOREACH(int i, map2)
{
    // do steps 1-5 (identical to above) here...
}

有没有办法连接映射以消除第二个循环中的重复代码？或者一种方法来扩展BOOST_FOREACH以一次迭代两个不同的地图？显然我不想增加程序的时间复杂度（否则我可以创建一个新的map并插入map1和map2）。我有一种感觉，我在这里遗漏了一些不成熟的东西。

Answer 1

您可以定义一个函数：

typedef std::map<int, int> IntMap;

void doStuffWithInt(IntMap::value_type &i)
{
  // steps 1 to 5
}

BOOST_FOREACH(IntMap::value_type &i, map1)
  doStuffWithInt(i);
BOOST_FOREACH(IntMap::value_type &i, map2)
  doStuffWithInt(i);

虽然在这种情况下使用std::for_each可能更简单：

for_each(map1.begin(), map1.end(), doStuffWithInt);
for_each(map2.begin(), map2.end(), doStuffWithInt);

Answer 2

这里的想法是编写一种特殊类型的迭代器来虚拟地合并两个容器，就BOOST_FOREACH而言。请注意，我不是从现有的两个容器中创建一个新容器。我只是从第一个容器的end（）跳转到第二个容器的begin（）迭代器。我没有尝试编写实际的merged_iterator类，但是虽然编写起来可能有点长，但这在技术上并不困难。我真的很惊讶没有找到像谷歌这样的东西。不过我看起来并不长！

template<typename Container>
boost::iterator_range<
  merged_iterator<Container::iterator>
  >
concat_containers( Container& c1, Container& c2 )
{
  typedef merged_iterator<typename Container::iterator> MergedIterator;
  typedef boost::iterator_range<MergedIterator> IteratorRange;
  return IteratorRange(
    MergeIterator( c1.begin(), c1.end(), c2.begin(), c2.end() ),
    MergeIterator( c2.end(), c1.end(), c2.begin(), c2.end() ) );
}

// Now use a bit of magic to define merged_iterator<...>
// And you'll be able to write

BOOST_FOREACH( std::pair<int, int> i, concat_containers( map1, map2 ) )
{
// Do whatever you want here
}

Answer 3

除了我推荐的1800的解决方案之外，还有各种hacky解决方案：

for (int stage = 0; stage < 2; stage++) {
    BOOST_FOREACH(int i, stage == 0 ? map1 : map2) {
        ...
    }
}

typedef std::map<int, int> intmap;
std::vector<intmap *> v;
v.push_back(&map1);
v.push_back(&map2);
BOOST_FOREACH(intmap *m, v) {
    BOOST_FOREACH(int i, *m) {
        ...
    }
}

注意：当我看到同事写下这样的代码时，有时候我会被一种不可抗拒的冲动所克服。使用风险自负。

Answer 4

最简单的方法是：

std::map<int, int> map1, map2;
int key, value;
BOOST_FOREACH(boost::tie(key, value), boost::join(map1, map2))
{
    // do steps 1-5 here...
}

不要担心，由于括号，这些逗号不会混淆预处理器。

Answer 5

在我的头脑中，我会尝试

std::map<int, int> map1, map2;
std::map<int, int>& maps = { map1, map2 }
BOOST_FOREACH(std::map<int, int> map, maps)
  BOOST_FOREACH(int i, map)
  {
      // do steps 1-5 here...
  }

Answer 6

解释here。

你可以这样做：

std::map<int,int> m;
typedef std::pair<int,int> pair_t;
BOOST_FOREACH(pair_t p, m)

如何使用BOOST_FOREACH和两个std :: maps？

6 个答案: