Re: std::list question
"Jack" <jl@knight.com> wrote in message
news:%23Yut$nE$HHA.4568@TK2MSFTNGP02.phx.gbl
struct _myval
{
long idx;
std::string src_val;
std::string dst_val;
} myval;
bool IsEquals(const _myval& val1, const std::string& str2)
{
return val1.dst_val == str2;
}
...
list<_myval> g_MyvalList;
...
std::list<_myval>::iterator it = g_MyvalList.begin();
it = find_if (it, g_MyvalList.end(), IsEquals);
==========================
Error:
Error 7 error C2198: 'bool (__cdecl *)(const _myval &,const
std::string &)'
too few arguments for call c:\program files\microsoft visual studio
8\vc\include\algorithm 87
Have you noticed that you haven't at any point specified the actual
string you were supposedly looking for? Does it not surprise you?
Make it:
struct Comparator {
Comparator(const string& s) : compare_to(s) {}
bool operator()(const _myval& v) const {
return v.dst_val == compare_to;
}
private:
string compare_to;
};
std::list<_myval>::iterator it =
find_if(g_MyvalList.begin(), g_MyvalList.end(), Comparator("xyz"));
--
With best wishes,
Igor Tandetnik
With sufficient thrust, pigs fly just fine. However, this is not
necessarily a good idea. It is hard to be sure where they are going to
land, and it could be dangerous sitting under them as they fly
overhead. -- RFC 1925