как найти объект в std :: vector - PullRequest
2 голосов
/ 22 июня 2011

У меня есть объект, как показано ниже

 class MyClass{
   int a;
   double b;
 };

и

 vector<MyClass> vMyClass;

тогда я хочу поиск в vMyClass попробуйте использовать функцию алгоритма:

 if( std::find(vMyClass.begin(), vMyClass.end(), aiField) == vMyClass.end())

Я тоже определил operator==

 int operator ==(Field &Left,Field &Right)
 {
    return memcmp(&Left,&Right,sizeof(Field));
 }

или

int operator ==(Field &Right)
{
    return memcmp(this,&Right,sizeof(Field));
}

но я получаю сообщение об ошибке.

1>c:\Program Files (x86)\Microsoft Visual Studio 9.0\VC\include\algorithm(40) : error C2678: binary '==' : no operator found which takes a left-hand operand of type 'MyClass' (or there is no acceptable conversion)
1>        C:\Program Files\Microsoft SDKs\Windows\v6.0A\include\guiddef.h(192): could be 'int operator ==(const GUID &,const GUID &)' [found using argument-dependent lookup]
1>        c:\users\shmuelian\documents\visual studio 2008\projects\src\examples\DLgenerator\DLParser.h(85): or       'int operator ==(MyClass &,MyClass &)' [found using argument-dependent lookup]
1>        c:\users\shmuelian\documents\visual studio 2008\projects\src\examples\DLgenerator\DLParser.h(70): or       'int MyClass::operator ==(MyClass &)'
1>        while trying to match the argument list '(MyClass, const MyClass)'
1>        c:\Program Files (x86)\Microsoft Visual Studio 9.0\VC\include\algorithm(74) : see reference to function template instantiation '_InIt std::_Find<std::_Vector_iterator<_Ty,_Alloc>,_Ty>(_InIt,_InIt,const _Ty &)' being compiled
1>        with
1>        [
1>            _InIt=std::_Vector_iterator<MyClass,std::allocator<MyClass>>,
1>            _Ty=MyClass,
1>            _Alloc=std::allocator<MyClass>
1>        ]
1>        c:\users\shmuelian\documents\visual studio 2008\projects\src\examples\DLgenerator\DLParser.h(108) : see reference to function template instantiation '_InIt std::find<std::_Vector_iterator<_Ty,_Alloc>,MyClass>(_InIt,_InIt,const _Ty &)' being compiled
1>        with
1>        [
1>            _InIt=std::_Vector_iterator<MyClass,std::allocator<MyClass>>,
1>            _Ty=MyClass,
1>            _Alloc=std::allocator<MyClass>
1>        ]
1>CommandArgParser.cpp
1>Generating Code...

Ответы [ 4 ]

14 голосов
/ 22 июня 2011

Они должны быть постоянными ссылками. И memcmp очень, очень плохо - просто проведите сравнение для каждого члена.

7 голосов
/ 22 июня 2011

Вы забыли квалификатор const в вашем операторе ==, плюс он должен вернуть bool, а не int. Просто используйте это вместо:

bool operator ==(const Field &Left, const Field &Right)
{
    return (Left.a == Right.a) && (Left.b == Right.b);
}
1 голос
/ 22 июня 2011

Реализуйте оператор внутри самого класса:

class MyClass
{    
public:
    int a;    
    double b;  

    bool operator ==(const MyClass &Right) const
    {      
        return (a == Right.a) && (b == Right.b);  // 'this' is left
    } 
}; 
1 голос
/ 22 июня 2011

У вас есть вектор MyClass, но оператор сравнения для Field. Вы должны написать
bool operator == (const MyClass &, const MyClass &).

Этот оператор должен (как указано другими) не использовать memcmp, а выполнять сравнение по элементам.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...