Discussion :

[TocTocKiéLà?]

Bonjour,
avant de commencer je tiens à préciser que je n'ai pas l'habitude d'utiliser les classes conteneurs.
je ne comprends pas cette partie de code dans engine_event.h:

Code :

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

1
2
3
4
5
6
7
 
bool operator==(const engine_event& e){
return (type == e.type) && 
(s_data == e.s_data) && 
(i_data == e.i_data) && 
(v_data == e.v_data);
}

avant ça on définis des conteneurs associatif map :

Code :

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

1
2
3
4
5
 
int type;
std::map<std::string, std::string> s_data;
std::map<std::string, int> i_data;
std::map<std::string, serializable_vector3df> v_data;

operator est donc de type booléen type un entier
mais je comprends pas ce que fait :

Code :

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

operator==(const engine_event& e)

fait-il une comparaison ? au moment de l'implémentation même ? au moment où on l'appelle ?

et ça, ça fait quoi ?

Code :

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

1
2
3
4
return (type == e.type) && 
(s_data == e.s_data) && 
(i_data == e.i_data) && 
(v_data == e.v_data);

j'ai vraiement l'impression que l'implémentation as été fait dans le header même

[nouknouk]

avant ça on définis des conteneurs associatif map :

Les deux parties sont indépendantes.

fait-il une comparaison ? au moment de l'implémentation même ? au moment où on l'appelle ?

operator== permet de surcharger l'opérateur '==' dans le code qui utilise la classe (ici 'engine_event', j'imagine).

C'est une fonction comme une autre, on l'évalue à chaque fois qu'elle est appelée au cours de l'exécution du programme. La seule différence est que ça permet de redéfinir un opérateur qui fait partie du langage c++, l'opérateur '=='. Ca permet d'avoir du code plus clair et plus concis.

Ainsi, on pourra utiliser dans notre code:

Code :

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

if (engineEvent1 == engineEvent2) [...]

Ce code étant exactement équivalent à l'appel:

Code :

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

if (engineEvent1.operator==(engineEvent2))

De même, tu peux très bien renommer la fonction de 'operator==' en un nom 'classique', par exemple 'isMyObjectEqualTo' et t'en servir dans un test:

Code :

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

if (engineEvent1.isMyObjectEqualTo(engineEvent2)) [...]

... ça fera exactement la même chose.

et ça, ça fait quoi ?

Ici la fonction semble simplement tester que les données membres de chaque instance (s_data, i_data et v_data) sont elle-même égales.
On considère donc que deux instances de cette classe sont égales quand leurs trois données membres ci-dessus sont égales.

j'ai vraiement l'impression que l'implémentation as été fait dans le header même

oui. C'est juste pour suggérer au compilateur que cette fonction aurait intérêt à être 'inline'.



Pour finir, et comme dans l'immense majorité des questions 'de base', tu aurais pu trouver les réponses toi-même: developpez.com propose plein de , dont celle-ci. Autant s'en servir

[TocTocKiéLà?]

merci, la première partie de ton message est clair pour moi.
Par contre, que veut dire suggérer ? ça veut dire que le compilateur peut décider ?

enfin pour les FAQs, je les utilise beaucoup, quand je sais ce que je cherche, dans ce cas là, je ne savais pas qu'il s'agissait d'une surcharge d'opérateur, je me suis quand même tapé toute la lecture des classes conteneur de la STL http://cpp.developpez.com/cours/stl/#LIII avant de venir poster

[yamashi]

Oui de nos jours on préfère laisser le compilateur choisir pour ce qui est inline, il est générallement meilleurs que nous pour décider si il faut le faire ou pas.

[nouknouk]

Par contre, que veut dire suggérer ? ça veut dire que le compilateur peut décider ?

Effectivement, le compilateur est à priori le mieux placé pour savoir si une fonction a intérêt a être 'inlinée' ou pas. C'est donc lui qui décide au final ; le mot clef 'inline' ou la déclaration du corps de la fonction dans la définition de la classe (comme ton exemple) ne sont que des suggestions et le compilateur se réserve le droit de les suivre ... ou pas.

C'est d'ailleurs valable pour nombre d'autres optimisations qui sont faites par le compilo sans demander l'avis du développeur (hors flags 'globaux' stipulés lors de l'exécution du compilo pour forcer explicitement l'optimisation ou la non-optimisation de certains types d'écritures, exemple).

[TocTocKiéLà?]

merci de vos réponses , je mets le sujet en résolu.
Pour ce qui est du lien concernant l'optimisation, je le garde de côté pour l'instant, car c'est quelque chose que j'aimerais également découvrir, mais pour l'instant je préfère continuer la lecture du code de Pierre Schwartz.

[khayyam90]

et ça, ça fait quoi ?

Code :

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

1
2
3
4
return (type == e.type) && 
(s_data == e.s_data) && 
(i_data == e.i_data) && 
(v_data == e.v_data);

J'ai simplement défini l'opérateur == d'un engine_event comme étant le == entre tous les champs du engine_event

j'ai vraiement l'impression que l'implémentation as été fait dans le header même

En effet, la classe engine_event est l'une des rares qui n'a pas de fichier .cpp associé. Ce n'était même pas spécialement pour des questions d'inline. Juste par simplicité