Discussion :

[casafa]

Hello,

J'ai ce code :

Code :

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

entities[name] = CMediaManager::instance()->getMedia<CEntity>(filename);

où entities est une std::map et getMedia peut lever une exception.

Ce que je trouve bizzard : Si "getMedia" lève une exception, j'ai quand même un ajout dans la std::map qui se fait !

N'est-il pas censé d'abord évaluer l'expression de droite et si elle réussi (pas d'exception), il fait l'affectation ?!

Merci...

[JolyLoic]

Tu as écrit un truc genre :

Code :

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

f() = g();

Dans ce cas, rien n'oblige à évaluer f avant g ou l'inverse. L'évaluation de f, qui est chez toi entities[name] crée un élément dans le map, avec une valeur par défaut, et retourne une référence vers cet élément. Rien ne sera affecté à cette référence, car une exception sera lancée avant, mais il a déjà été créé.

Si tu es sur que l'élément n'existe pas déjà dans la map, le plus simple pour éviter son ajout en cas d'exception est d'écrire :

Code :

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

1
2
 
entities.insert(make_pair(name, CMediaManager::instance()->getMedia<CEntity>(filename)));

[casafa]

Merci