我有这样的事情:
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 ++很陌生,因为我不得不学习它。
有什么想法吗?
答案 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
成员函数。