Question

我有一个简单的地图程序。它需要一个班级作为关键。该班有多个成员。我认为我的比较功能是正确的。我遵循严格的弱顺序。问题是，它允许输入重复的密钥。

以下是我的代码。

#include <iostream>
#include <string.h>
#include <map>

class mapkey
{
public:
    std::string mInterface;
    std::string mDestination;
    int         mPrefixLen;
    std::string mNextHop;
    int         mMetric;

    mapkey() {}
   ~mapkey() {}
    mapkey(std::string a, std::string b, int c, std::string d, int e)
    {
      mInterface = a;
      mDestination = b;
      mPrefixLen = c;
      mNextHop = d;
      mMetric = e;
    }
};

struct mapcomp
{
  bool operator() (const mapkey left, const mapkey right);
};

bool mapcomp::operator() (const mapkey left, const mapkey right)
{
  if(strcmp(left.mInterface.c_str(), right.mInterface.c_str()) < 0)
    return true;
  if(strcmp(left.mInterface.c_str(), right.mInterface.c_str()) > 0)
    return false;

  if(strcmp(left.mDestination.c_str(), right.mDestination.c_str()) < 0)
    return true;
  if(strcmp(left.mDestination.c_str(), right.mDestination.c_str()) > 0)
    return false;

  if(strcmp(left.mNextHop.c_str(), right.mNextHop.c_str()) < 0)
    return true;
  if(strcmp(left.mNextHop.c_str(), right.mNextHop.c_str()) > 0)
    return false;

  if(left.mPrefixLen < right.mPrefixLen)
    return true;
  if(left.mPrefixLen > right.mPrefixLen)
    return false;

  if(left.mMetric < right.mMetric)
    return true;
  if(left.mMetric > right.mMetric)
    return false;
}

typedef std::map<mapkey, std::string, mapcomp> script_map;
script_map mm;

void print_map()
{
   script_map::const_iterator iter;
   for (iter = mm.begin(); iter != mm.end(); iter++)
   {
     std::cout << "value is - " << iter->second << std::endl;
   }
}

int main()
{
   mapkey test1("eth1", "50.60.70.80", 1, "90.10.20.30", 1);
   mm[test1] = "first";

   mapkey test2("eth1", "50.60.70.40", 1, "90.10.20.30", 1);
   mm[test2] = "second";

   mapkey test3("eth1", "50.60.70.20", 1, "90.10.20.30", 1);
   mm[test3] = "third";

   mapkey test4("eth1", "50.60.70.80", 1, "90.10.20.30", 1);
   mm[test4] = "fourth";

   print_map();

   return 0;
}

以上程序，第一和第四个键是相同的。当我打印地图时，输出如下

g ++ --std = c ++ 11 map.cpp

./ a.out的

值是 - 第三

值是 - 秒

值是 - 第四

值是 - 第一个

我缺少什么？第四个条目应该没有添加。

Answer 1

原因：您的比较功能已损坏。

解决方案：使用惯用的C ++编写一个新的。

struct mapcomp
{
  bool operator() (mapkey const& l, mapkey const& r) {
      return 
           std::tie(l.mInterface, l.mDestination, l.mPrefixLen, l.mNextHop, l.mMetric) 
           <
           std::tie(r.mInterface, r.mDestination, r.mPrefixLen, r.mNextHop, r.mMetric)
      ;
  }
};

我通过mapkey const&代替mapkey以避免复制。
我使用元组比较和std::tie从你的成员中制作元组。

您还应该从mapkey中删除所有构造函数和析构函数。鉴于您可以通过通用初始化初始化成员，它们没有用处。

我还会考虑将结构更改为您的类的operator<（也许operator==）的重载。 map接受它就足够了，而不通过任何其他比较器。

Answer 2

从这里开始：

if(strcmp(left.mInterface.c_str(), right.mInterface.c_str()) < 0)
    return true;
if(strcmp(left.mInterface.c_str(), right.mInterface.c_str()) > 0)
    return false;

strcmp()返回零，只有零，当字符串相等时。

然后查看第一行：在所有其他成员都有机会进行比较之前返回true。

Answer 3

您的mapcomp方法在所有情况下都不返回值。您需要在函数末尾返回false，以表示项目相等。

我建议使用内置字符串＆lt;运算符并重写这样的mapcomp方法：

bool mapcomp::operator() (const mapkey& left, const mapkey& right)
{
  if(left.mInterface < right.mInterface) return true;
  if(left.mInterface > right.mInterface) return false;

  if(left.mDestination < right.mDestination) return true;
  if(left.mDestination > right.mDestination) return false;

  if(left.mNextHop < right.mNextHop) return true;
  if(left.mNextHop > right.mNextHop) return false;

  if(left.mPrefixLen < right.mPrefixLen) return true;
  if(left.mPrefixLen > right.mPrefixLen) return false;

  if(left.mMetric < right.mMetric) return true;
  if(left.mMetric > right.mMetric) return false;

  return false; // items are equal  
}

Answer 4

对于C ++，然后才能使用

bool mapcomp::operator() (const mapkey left, const mapkey right)
{
    if (left.mInterface != right.mInterface)        return left.mInterface < right.mInterface;
    if (left.mDestination != right.mDestination)    return left.mDestination < right.mDestination;
    if (left.mNextHop != right.mNextHop)            return left.mNextHop < right.mNextHop;
    if (left.mPrefixLen != right.mPrefixLen)        return left.mPrefixLen < right.mPrefixLen;
    if (left.mMetric != right.mMetric)              return left.mMetric < right.mMetric;
    return false; //equal != less
}

将类作为键的映射允许重复键

4 个答案: