Discussion :

[Flow_75]

bonjour,

J'ai un bout de code a vous soumettre.

J'ai une map (appelons la MAP) qui contient divers elements.

je fais :

Code :

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

1
2
3
 
if(MAP.find(element a effacer) != MAP.end()) // Verification de l'existence de l'element.
          MAP.erase(element a effacer);

Le probleme est que au moment du test (apres débuggage), le contenu de la map est correcte dans le sens ou le contenu est coherent.
Apres le erase, par contre, c'est le chaos dedans.
Je comprend pas car, au passage suivant dans la fonction. La map MAP est redevenue coherent et sans l'element effacé....

Merci de votre aide.

[Laurent Gomila]

Il faudrait voir plus de code, parce que là...

Quant au test de présence de l'élément, il est inutile (erase ne va pas crasher s'il ne trouve pas l'élément ).

[Flow_75]

quel code supplémentaire veut tu voir ?
Car je ne vois pas quoi rajouter qui pourrait aider...

[Flow_75]

quel code supplémentaire veut tu voir ?
Car je ne vois pas quoi rajouter qui pourrait aider...

[Laurent Gomila]

Oups, j'avais pas vu

Je sais pas, il faudrait voir un peu plus de contexte, parce que le code que tu donnes est correct tel quel. Si tu es motivé tu peux aussi tenter d'extraire un code minimal complet qui reproduit le problème.

[Flow_75]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
void DeleteArrow(std::string ArrowToDelete)
{
 
  if(ArrowMap.find(ArrowToDelete) == ArrowMap.end())
      return;
  else
    ArrowMap.erase(ArrowMap.find(ArrowToDelete));
 
 
... suite de traitement ...
 
// ArrowMap est une std::map<std::string,Arrow>

[frizou11]

Quant au test de présence de l'élément, il est inutile (erase ne va pas crasher s'il ne trouve pas l'élément ).

A ce propos, ça se passe de la même manière avec la méthode remove de la classe list ? ou il est nécessaire de tester auparavant si l'élément est bien dans la liste ?

[Laurent Gomila]

erase ou remove, quelque soit le conteneur, ont un comportement parfaitement défini si l'élément n'a pas été trouvé (il faut bien rechercher l'élément avant de le supprimer).

[frizou11]

erase ou remove, quelque soit le conteneur, ont un comportement parfaitement défini si l'élément n'a pas été trouvé (il faut bien rechercher l'élément avant de le supprimer).

Si je passe un élément en paramètre d'une fonction qui va supprimer cet élément d'une liste, j'ai le droit du faire direct un remove, ou je dois tester auparavant l'existence de cet élément dans la liste ?

[Laurent Gomila]

Si je passe un élément en paramètre d'une fonction qui va supprimer cet élément d'une liste, j'ai le droit du faire direct un remove, ou je dois tester auparavant l'existence de cet élément dans la liste ?

Mon message précédent n'était pas clair ?