5

我想添加两个地图以及以下行为。

如果键存在-> 将两个键值相加。

如果键不存在 -> 插入对映射。

我看过一些标准库算法。即转换，但似乎没有做我想要的。

取自此链接

template < class InputIterator, class OutputIterator, class UnaryOperator >
  OutputIterator transform ( InputIterator first1, InputIterator last1,
                             OutputIterator result, UnaryOperator op )
{
  while (first1 != last1)
    *result++ = op(*first1++);  // or: *result++=binary_op(*first1++,*first2++);
  return result;
}

我的想法是，在使用时我的第二张地图中只有一个迭代器和适当的仿函数

 *result++=binary_op(*first1++,*first2++);

因此，我将无法遍历我的第二张地图以找到关键值。

一个想法是让我自己的算法稍微改变一下。 

template < class InputIterator, class ContainerType, class BinaryOperator >
  void myTransform ( InputIterator first1, InputIterator last1,
                               ContainerType cont2,  
                               BinaryOperator binary_op )
{
  while (first1 != last1)
    binary_op(first1++, cont2); //cont2 passed by reference 
}

然后我就可以使用：

cont2.find()在我的函子中搜索整个地图。

这将是我在想什么的更完整的例子，但我似乎得到了一个我无法解决的编译错误（我有点猜测 BinaryOperator 类型......见下文）？

#include <map>
#include <string>
#include <iostream>

template < class InputIterator, class ContainerType, class BinaryOperator >
void myTransform ( InputIterator first1, InputIterator last1,
                 ContainerType &cont2, 
                 BinaryOperator binary_op )
{
  while (first1 != last1)
    binary_op(first1++, cont2); //cont2 passed by reference
}

template<class IteratorType, class ContainerType>
struct AddMapValues:
  std::binary_function<IteratorType, ContainerType, void>
{
  void operator()(IteratorType itr, ContainerType& cont)
  {
    if( cont.find(itr->first) != cont.end() ) cont[itr->first] = cont.find(itr->first).second + itr->second;
    else cont.insert( (*itr) );
  }
};


int main()
{
  typedef std::map<std::string, double> stringDoubleMap;
  typedef std::map<std::string, double>::iterator stringDoubleMapItr;
  typedef void (*ptrfnt)(stringDoubleMapItr, stringDoubleMap& );

  stringDoubleMap map1;
  stringDoubleMap map2;

  map1.insert( stringDoubleMap::value_type("Test1",1.0) );
  map1.insert( stringDoubleMap::value_type("Test2",2.0) );
  map1.insert( stringDoubleMap::value_type("Test3",3.0) );

  map2.insert( stringDoubleMap::value_type("Test1",1.0) );
  map2.insert( stringDoubleMap::value_type("Test2",2.0) );
  map2.insert( stringDoubleMap::value_type("Test3",3.0) );

  myTransform( map1.begin(), map1.end(),
               map2, 
               AddMapValues< stringDoubleMapItr, stringDoubleMap >() );

  return 0;

}

这是我的编译器错误：

testingMapTransforms.cxx: In function ‘int main()’:

testingMapTransforms.cxx:52:85: error: no matching function for call to     ‘myTransform(std::map<std::basic_string<char>, double>::iterator, std::map<std::basic_string<char>, double>::iterator, stringDoubleMap&, std::map<std::basic_string<char>, double>::iterator, AddMapValues<std::_Rb_tree_iterator<std::pair<const std::basic_string<char>, double> >, std::map<std::basic_string<char>, double> >)’

testingMapTransforms.cxx:52:85: note: candidate is:

testingMapTransforms.cxx:12:20: note: template<class InputIterator, class ContainerType, class   OutputIterator, class BinaryOperator> OutputIterator myTransform(InputIterator, InputIterator,  ContainerType, OutputIterator, BinaryOperator)

似乎有另一个迭代器来自某个地方并且容器类型没有正确读取？

有任何想法吗？

我在用

gcc - GNU 项目 C 和 C++ 编译器

和

Ubuntu/Linaro 4.6.3-1ubuntu5

谢谢

笔记：

我在答案中更新了上面代码的工作版本。如果您认为我应该更改问题代码，请告诉我。不确定最佳实践

4

4 回答 4

12 
I want to add two maps together with the following behavior:
If key exists add two key values together.
If key does not exist. Insert pair to map.

我认为一个简单的 for 循环会做你想要的：

for(auto it = map2.begin(); it != map2.end(); ++it) map1[it->first] += it->second;

如果键存在，则该值将添加到现有键中。如果键不存在，operator[]将插入它并且它的值将被默认初始化（0.0 for double）。

我不认为在这里使用适用于任何容器的通用函数是明智的。say vector 和 map 的 insert() 和 operator[] 的语义差别太大了。

于 2012-06-13T11:21:10.167 回答
2

您的代码不可读。它容易出错。

手动完成，但一次只做一件事。不要混合递增指针、取消引用它们、将元素添加到地图和从另一个地图查询都在同一行中。结果一团糟。

于 2012-06-13T11:14:44.197 回答
2

我不认为你可以做到这一点transform。你有两个容器，你需要两个迭代器对，当一个元素只在其中一个中时，你需要分别推进它们。两个序列版本transform同步推进它们。

用手做应该不会太难。可能类似于以下内容：

typedef std::map<std::string, std::double> Map

Map
addValues( Map const& m1, Map const& m2 )
{
    Map results;
    Map::const_iterator i1 = m1.begin();
    Map::const_iterator i2 = m2.begin();
    while ( i1 != m1.end() && i2 != m2.end() ) {
        if ( i1->first < i2->first ) {
            results.insert( results.end(), *i1 );
            ++ i1;
        } else if ( i2->first < i1->first ) {
            results.insert( results.end(), *i2 );
            ++ i2;
        } else {
            results.insert( results.end(),
                            Map::value_type( i1->first, i1->second + i2->second ) );
            ++ i1;
            ++ i2;
        }
    }
    results.insert( i1, m1.end() );
    results.insert( i2, m2.end() );
    return results;
}

（在将其作为模板之前，我会先让它像这样工作。）

于 2012-06-13T11:25:08.400 回答
0

这是上面的修订版。问题代码中有很多错误，但这似乎有效并且是相同的想法：

template < class InputIterator, class ContainerType, class BinaryOperator >
void myTransform ( InputIterator first1, InputIterator last1, ContainerType &cont2, BinaryOperator binary_op )
{
  while (first1 != last1)
    (*binary_op)(first1++, cont2); //cont2 passed by reference
}

template<class IteratorType, class ContainerType>
struct AddMapValues:
  std::binary_function<IteratorType, ContainerType, void>
{
  static void Add(IteratorType itr, ContainerType& cont)
  {
    cont[itr->first] += itr->second;
  }
};

int main()
{

  typedef std::map<std::string, double> stringDoubleMap;
  typedef std::map<std::string, double>::iterator stringDoubleMapItr;
  typedef void (*ptrfnt)(stringDoubleMapItr, stringDoubleMap& );

  stringDoubleMap map1;
  stringDoubleMap map2;

  map1.insert( stringDoubleMap::value_type("Test1",1.0) );
  map1.insert( stringDoubleMap::value_type("Test2",2.0) );
  map1.insert( stringDoubleMap::value_type("Test4",3.0) );

  map2.insert( stringDoubleMap::value_type("Test1",1.0) );
  map2.insert( stringDoubleMap::value_type("Test2",10.0) );
  map2.insert( stringDoubleMap::value_type("Test3",3.0) );

  myTransform( map1.begin(), map1.end(), map2, AddMapValues<stringDoubleMapItr, stringDoubleMap >::Add );

  for(stringDoubleMapItr itr = map2.begin(); itr != map2.end(); ++itr ){
    std::cout << "INFO: Key: " << itr->first << " | Value: " << itr->second  << std::endl;
  }

  return 0;

}
于 2012-06-13T13:29:51.343 回答