无效使用非静态成员函数

时间:2015-03-26 19:06:22

标签: c++ function const member non-static

我有这样的事情:

class Bar
      {
      public:
        pair<string,string> one;
        std::vector<string> cars;
        Bar(string one, string two, string car);
      };

class Car
      {
      public:
        string rz;
        Bar* owner;
        Car(string car, Bar* p);
      };

class Foo
       {
         public:
          Foo  ( void );
          ~Foo  ( void );
          int Count ( const string & one, const string &  two) const;
          int comparator (const Bar & first, const Bar & second) const;            
            std::vector<Bar> bars;
       };

int Foo::comparator(const Bar & first, const Bar & second) const{
  return first.name < second.name;
}

int Foo::Count  ( const string & one, const string & two ) const{
  int result=0;
  Bar mybar = Bar( one, two, "" );
  std::vector<Bar>::iterator ToFind = lower_bound(bars.begin(), bars.end(), mybar, comparator);
  if (ToFind != bars.end() && ToFind->one == mybar.one ){
    result = ...
  }
  return result;
}

方法Foo::Count应使用std::lower_bound()根据两个字符串对在vector<Bar>中查找元素。 现在这部分不起作用。致lower_bound()我正在提供方法comparator()。我认为没关系,但g ++说:

c.cpp: In member function ‘int Foo::Count(const string&, const string&) const’:
c.cpp:42:94: error: invalid use of non-static member function
std::vector<Bar>::iterator ToFind = lower_bound(bars.begin(), bars.end(), mybar, comparator);

方法Count()必须保持const ...

我对C ++很陌生,因为我不得不学习它。

有什么想法吗?

3 个答案:

答案 0 :(得分:13)

最简单的解决方法是使比较器功能保持静态:

static int comparator (const Bar & first, const Bar & second);
^^^^^^

Count中调用时,其名称为Foo::comparator

现在的方式,成为非静态成员函数是没有意义的,因为它不使用Foo的任何成员变量。

另一种选择是使其成为非成员函数,特别是如果除了Foo之外其他代码可能使用此比较器是有意义的。

答案 1 :(得分:7)

您必须使Foo::comparator静态或将其包装在std::mem_fun类对象中。这是因为lower_bounds要求比较器是一个具有调用操作符的对象类,如函数指针或函子对象。此外,如果您使用的是C ++ 11或更高版本,您也可以按dwcanillas建议并使用lambda函数。 C ++ 11也有std::bind

示例:

// Binding:
std::lower_bounds(first, last, value, std::bind(&Foo::comparitor, this, _1, _2));
// Lambda:
std::lower_bounds(first, last, value, [](const Bar & first, const Bar & second) { return ...; });

答案 2 :(得分:2)

你应该传递一个this指针来告诉函数哪个对象可以工作,因为它依赖于它而不是static成员函数。