0

私は以下のコードを持っています.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

このエラーが発生していることがわからないようです。

4

1 に答える 1

5

のベクトルがありperson *、それらを と比較しようとしていますperson。count_objのコード行を次のいずれかに変更する必要があります。

if (*(*iter1) == obj)

また：

if ((*iter1) == &obj)

ポインターとオブジェクトのどちらを比較するかによって異なります。

[注：std::count標準ライブラリの関数を知っていますか？]

[注 (2): 別の回答で述べたように、おそらく「定数の正しさ」を読む必要があります。operator==asを宣言し、参照を引数としてconst取る必要があります。const]

[注 (3): 生のポインターをコンテナーに格納することは、多くの場合、悪い考えです。たとえば、実際にメモリ リークが発生していることに気付いていますか? ]

于 2011-12-26T17:23:18.637 に答える