Discussion :

[atmakefka]

Bonjour,
J'ai défini une map d'objets complexes (des classes).

Code :

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

1
2
3
4
5
6
7
8
struct cmp
{
  bool operator()(const UInt128* val1, const UInt128* val2) const{
    return val1->compareTo(*val2);
  }
};
 
map<const UInt128*, const Contact*, cmp> contacts;

Les opérateurs ==, != <, >, <=, >= sont définis dans les classes UInt128 et Contact.

Le problème c'est que si je met 20 éléments dans ma map, lorsque je la parcours avec un itérateur de cette manière,

Code :

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

1
2
3
4
map<const UInt128*, const Contact*, cmp>::const_iterator it;
for (it = contacts_.begin(); it != contacts_.end(); it++){
      cout << "test" << endl;
}

je n'ai que deux fois l'affichage "test". Autrement dit,
la map n'est pas enitèrement parcourue alors qu'en affichant contacts.size() avant et après la boucle, j'ai bien 20 d'affiché...

Par contre, j'ai bien 20 fois "test" si j'enlève le comparateur cmp dans la définition de la map, c'est à dire si je fais:

Code :

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

map<const UInt128*, const Contact*> contacts;

J'ai besoin du comparateur cmp. D'ou peut venir le problème et comment le résoudre?

Merci !

[atmakefka]

Merci!

En fait j'ai dû mettre:

Code :

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

1
2
3
4
5
6
struct cmp
{
  bool operator()(const UInt128* val1, const UInt128* val2) const{
    return val1->compareTo(*val2) == -1;
  }
};

au lieu de

Code :

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

1
2
3
4
5
6
struct cmp
{
  bool operator()(const UInt128* val1, const UInt128* val2) const{
    return val1->compareTo(*val2);
  }
};

(compareTo renvoie -1, 0 ou 1 selon que l'objet est considéré comme étant plus petit, égal ou plus grand.


Edit : bizarre, un forumeur m'avait donné une réponse mais elle n'y est plus.

[jabbounet]

oui il y'avait quelque chose qui ne me plaisait pas dans ma réponse et j'étais en train de la réécrire.

plutot que

Code :

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

1
2
3
4
5
6
7
 
struct cmp
{
  bool operator()(const UInt128* val1, const UInt128* val2) const{
    return val1->compareTo(*val2) == -1;
  }
};

je mettrais

Code :

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

1
2
3
4
5
6
7
 
struct cmp
{
  bool operator()(const UInt128* val1, const UInt128* val2) const{
    return (val1->compareTo(*val2) < 0);
  }
};

surtout si le compareTo que tu utilise est celui-ci
http://msdn.microsoft.com/en-us/libr...to(VS.71).aspx

Ils ne spécifient pas les valeur de retour.

Return Value

A 32-bit signed integer that indicates the relative order of the comparands. The return value has these meanings:
Value Meaning

• Less than zero This instance is less than obj.
• Zero This instance is equal to obj.
• Greater than zero This instance is greater than obj.

[atmakefka]

D'accord, merci!