Discussion :

[b4u]

Bonsoir,
avec le morceau de code suivant:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
	std::map<std::string, void (ServerConfig::*)(const char *)>::const_iterator item = _item_map.find(attr->Name());
	if (item != _item_map.end())
		(item->second)("toto");

j'obtiens l'erreur (je compile avec Visual Studio):
error C2064: term does not evaluate to a function taking 1 arguments
sur l'appel (item->second)("toto");. Quelqu'un peut resoudre ca?
Merci d'avance

[Laurent Gomila]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

(item->*second)("toto");

Par ailleurs, tu devrais faire un typedef pour ta map...

[b4u]

salut,
j'obtiens cette fois-ci:
error C2065: 'second' : undeclared identifier

[Volune]

Ta fonction semble nécessiter un objet de classe ServerConfig pour être appelée, et je n'en vois aucun dans ton code.

[b4u]

Désolé, je n'ai pas bien précisé certains éléments.
J'ai une classe ServerConfig, qui contient un membre privé _item_map déclaré comme ca:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

std::map<std::string, void (ServerConfig::*)(const char *)> _item_map;

Voici le contructeur de la classe:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
ServerConfig::ServerConfig()
{
	_item_map["port"] = &ServerConfig::set_port;
	_item_map["log_path"] = &ServerConfig::set_log_path;
	_item_map["n_max_connection"] = &ServerConfig::set_n_max_connection;
 
	_port = 80;
	_log_path = "toto.log";
	_n_max_connection = 4200;
 
}

(à ce propos, pourquoi _item_map["port"] = &set_port; ou _item_map["port"] = set_port; ne compilent pas?)

Maintenant la méthode privée qui exploite la map:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
void ServerConfig::parse_xml_element(TiXmlElement *elmt)
{
	if (elmt != 0)
		if ((std::string(elmt->Value())) == std::string("item"))
			for (TiXmlAttribute *attr = elmt->FirstAttribute(); attr; attr = attr->Next())
			{
				(_item_map.find(attr->Name())->second)("toto");
			}
}

l'implémentation est écrite autrement que dans mon premier post, mais j'ai la meme erreur.
En espérant avoir été plus clair...

PS: s'il y a une manière plus élégante que les maps pour effectuer le meme traitement, je suis preneur.

[Laurent Gomila]

Oups oui... J'ai répondu un peu précipitemment. Ceci devrait marcher :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

(this->*(item->second))("toto");

(à ce propos, pourquoi _item_map["port"] = &set_port; ou _item_map["port"] = set_port; ne compilent pas?)

Tout simplement parce que ce n'est pas la bonne syntaxe (même si certains compilos l'acceptent).

[b4u]

tiens c'est étonnant qu'on soit obligé de passer par le pointeur d'instance... il y a une raison particulière ou c'est simplement à savoir? (idem pour le besoin de nommer la classe dans l'affectation _item_map["port"] = &ServerConfig::set_port;)