我有下面的代码,我正在尝试编写一个泛型函数,它接受2个迭代器和一个对象并检查是否发生任何事件并返回出现次数。
在我的简单课程
之下class person{
string name;
int age;
public:
person(string n, int a): name(n), age(a) {}
bool operator==(person &p);
};
bool person::operator==(person &p){
return (name == p.name && age == p.age);
}
以下是通用功能
template<typename Iter, typename Obj>
int count_obj(Iter iter1, Iter iter2, Obj &obj){
int count = 0;
for (; iter1 != iter2; iter1++){
if((*iter1) == obj)
count += 1;
}
return count;
}
我的主要:
int main(){
vector<person *> myp;
person a("ted", 21); person b("sun", 100); person c("ted", 21);
myp.push_back(&a);myp.push_back(&b);myp.push_back(&c);
cout<< "occurences for person objects " << count_obj(myp.begin(), myp.end(), a) << '\n';
}
完整错误
3b.cc: In function ‘int count_obj(Iter, Iter, Obj&) [with Iter = __gnu_cxx::__normal_iterator<person**, std::vector<person*> >, Obj = person]’:
3b.cc:61:79: instantiated from here
3b.cc:42:3: error: no match for ‘operator==’ in ‘iter1.__gnu_cxx::__normal_iterator<_Iterator, _Container>::operator* [with _Iterator = person**, _Container = std::vector<person*>, __gnu_cxx::__normal_iterator<_Iterator, _Container>::reference = person*&]() == obj’
make: *** [3b] Error 1
我似乎无法弄清楚我收到了这个错误。
答案 0 :(得分:5)
您有person *
的向量,并且您尝试将它们与person
进行比较。您需要将count_obj
中的代码行修改为:
if (*(*iter1) == obj)
或:
if ((*iter1) == &obj)
取决于您是否希望比较指针或对象。
[注意:您是否了解标准库中的std::count
功能?]
[注意(2):如另一个答案所述,你应该阅读“const correctness”。您应该将operator==
声明为const
,并且应该将const
引用作为参数。]
[注意(3):在容器中存储原始指针通常是个坏主意。例如,您是否意识到您实际上有内存泄漏?]