Function pointer : 'function does not evaluate to a function which takes 2 arguments'

Hey guys ! I have an error that I can't get rid of. It's 'function does not evaluate to a function which takes 2 arguments'. I have tried modifying cmp_func to (*cmp_func) and (&cmp_func) but neither was the solution.

Here's the (shortened) code.

Code:

class CoursMgr
{
public:


private:
	typedef bool (CoursMgr::*CMP_FUNC)(int, int);

	// Gros HACK pas beau..
	bool FONCTION_NULLE(int, int) { }

};

Code:

if(cmp_func == FONCTION_NULLE)
	if(ComparePPE(trav_it->NoteObtenue, a + b) && ComparePGE(trav_it->NoteObtenue, a - b))
		ls_resultat.back().Travaux.push_back(*trav_it);
else 
	if(cmp_func(trav_it->NoteObtenue, donnee))
		ls_resultat.back().Travaux.push_back(*trav_it);

What's weird is that cmp_func can only be a pointer to ComparePPE, ComparePGE and another function, yet the code in blue compiles but not the code in red.
Any help will be much appreciated.

Where is cmp_func declaration and assignment?

Oh I'm sorry, I think I cut too much code :P

Code:

if(type & C_TRAVAIL_NOTE)
{
	CMP_FUNC cmp_func = TrouveCmpFunc(str);

	if(cmp_func != 0)
	{
		if(cmp_func == FONCTION_NULLE)
			if(ComparePPE(trav_it->NoteObtenue, a + b) && ComparePGE(trav_it->NoteObtenue, a - b))
				ls_resultat.back().Travaux.push_back(*trav_it);
		else 
			if(cmp_func(trav_it->NoteObtenue, donnee))
				ls_resultat.back().Travaux.push_back(*trav_it);
	}
}

// ...

CoursMgr::CMP_FUNC CoursMgr::TrouveCmpFunc(const std::string& critere)
{
	CMP_FUNC func = 0;

	// On regarde si on a ">=x"
	boost::regex e_pge("^>=\\b[0-9]{1,2}\\b$");
	if(boost::regex_match(critere, e_pge))
		func = ComparePGE;

	// On regarde si on a "<=x"
	boost::regex e_ppe("^<=\\b[0-9]{1,2}\\b$");
	if(boost::regex_match(critere, e_ppe))
		func = ComparePPE;

	// On regarde si on a "x"
	boost::regex e_exact("^\\b[0-9]{1,2}\\b$");
	if(boost::regex_match(critere, e_exact))
		func = CompareExact;

	// On regarde si on a "x+-y"
	boost::regex e_pm("^\\+-\\b[0-9]{1,2}\\b$");
	if(boost::regex_match(critere, e_pm))
		func = FONCTION_NULLE;

	return func;
}

About the FONCTION_NULLE thing: It's a hack because if TrouveCmpFunction() returns 0, it means that no match was found. However, since the last case of the TrouveCmpFunction() is a special case(i.e. it contains 2 numbers instead of only 1) I cannot assign a function to func, I have to treat this case apart but I cannot return -1 because CMP_FUNC cannot be assigned the value -1 so I made an empty function called FONCTION_NULLE().

member fuctions are actually receive an additional hidden parameter - pointer to this...
I don't know how to call them by pointer...
What you can do - make Comparison functions - static members to remove this hidden parameter, and declare the pointer to function as

typedef bool (*CMP_FUNC)(int, int);
Then you actualli have a pointer to function receiving 2 parameters and it is working

Thank you my friend. Moving my comparison functions and typedef outside of the class (as well as minor adjustments) has solved my problem. Thanks again.

This has nothing to do with the main topic of this thread but I didn't want to make another thread for such a small issue.

Okay so I use stringstreams every now and then but in one function, I use it twice.. so I just create a stringstream object and work with it. However, while debugging, I noticed that the dummy test would never produce the expected result.. thus I have decided to go step by step with my debugger and found out that the x variable is changed to the correct value while the 'y' variable is *not* changed at all. I have verified and it seems that 'propre' contains the proper result. I have verified my code many times and concluded that it was the fault of the stringstream.. and it was. I tried to use its flush() and ignore() functions to reset it to a working state, but apparently it didn't work either as the 'y' variable still hadn't changed. However, when I created a second stringstream object and used it for 'y', the variable changed accordingly and I got the expected result. Could you guys enlighten me and tell me how I can modify this really simple piece of code so that it works (i.e. modify 'y') without using a second stringstream object ?

Code:

void CoursMgr::ElimineCaracteresPM(int& x, int& y, const std::string& critere)
{
	std::string::const_iterator it;
	for(it = critere.begin(); it != critere.end(); it++)
		if(*it == '+')
			break;

	std::string propre;
	propre.append(critere.begin(), it);

	std::stringstream ss;
	ss << propre;
	ss >> x;

	propre.clear();
	propre.append(it + 2, critere.end());

	ss << propre;
	ss >> y;
}

Many thanks.

use ss.clear() before entering new data

I'll try that ! I don't know why I haven't -.-

Thanks.

To clear the value inside the stringstream, use ss.str(""), or in this case ss.str(propre) to reset it to the new value you want to use. Adding ss.clear() is probably wise in case an eofbit or failbit was set with the previous string, but that won't clear the contents of the stream.

Originally Posted by Desolation

Thank you my friend. Moving my comparison functions and typedef outside of the class (as well as minor adjustments) has solved my problem. Thanks again.

you might also want to look into boost.function or boost.bind for all kinds of member-function binding fun!

see my sig for boost link