Surcharge d'opérateur == sur une pair

Version imprimable

Bonjour !

Voici le début du problème, j'ai une multimap<long, Event>, et je cherche a supprime une entrée grace a une fonction en passant comme argument une reference de Event
Code:

1 2 3 4 5 6 7 8 9 10 void EventFactory::DeleteEvent(const Event& ev) { for(auto itr = _events.begin(); itr != _events.end(); ++itr) { if(itr->second == ev) { _events.erase(itr); } } }
Erreur sur la comparaison:

Citation:

D:\C++\projects\Servers\Shared\EventFactory.cpp|19|error: no match for 'operator==' in 'itr.std::_Rb_tree_iterator<_Tp>::operator-> [with _Tp = std::pair<const long int, Event>]()->std::pair<const long int, Event>::second == ev'|

Evidemment, je n'ai jamais fait ca :x
Donc je tente:
Code:

1 2 3 4 5 6 7 bool pair<long, Event>::operator==(const pair<long, Event> &comp) const { { return ((this.second.Repeat() == comp.second.Repeat()) && (this.second.Delay() == comp.second.Delay()) && (this.second.Repeated() == comp.second.Repeated()) && (this.first == comp.first)); }
Evidemment:

Citation:

D:\C++\projects\Servers\Master\..\Shared\Event.h|36|error: too few template-parameter-lists|

Comment faire pour que cela marche? :mouarf:

merci!

Bonjour,
essaye ceci

Code:

1
2
3
4
5
6
7
8
template<>
bool pair<long, Event>::operator==(const pair<long, Event> &comp) const {
{
    return ((this.second.Repeat() == comp.second.Repeat())
            && (this.second.Delay() == comp.second.Delay())
            && (this.second.Repeated() == comp.second.Repeated())
            && (this.first == comp.first));
}

Edit: ha ben non je suis mauvais au réveil avant mon jus d'orange. :aie: (pourtant operator==(const, const) externe à la classe j'y ai pensé :oops:)

Perdu :)

Citation:

Envoyé par DakM

D:\C++\projects\Servers\Shared\EventFactory.cpp|19|error: no match for 'operator==' in 'itr.std::_Rb_tree_iterator<_Tp>::operator-> [with _Tp = std::pair<const long int, Event>]()->std::pair<const long int, Event>::second == ev'|

L'erreur de vient pas de std::pair<const long int, Event>, mais de std::pair<const long int, Event>::second, c'est à dire de Event.

Par contre, tu ne peux pas te contenter de fournir une définition pour une fonction membre : il faut que la classe déclare cette fonction. Et si c'était le cas (déclaration de la fonction sans sa définition), tu aurais un autre type d'erreur (link erreur, fonction non trouvée, en gros).

Il faut simplement ajouter l'opérateur == pour Event, mais en fonction libre :
Code:

1 2 3 4 5 6 bool operator== (const Event& e1, const Event& e2) { return ((e1.Repeat() == e2Repeat()) && (e1.Delay() == e2.Delay()) && (e1.Repeated() == e2.Repeated()) && (e1 == e2.first)); }
PS: D'où viennent ces events ? Tu utilises un framework particulier ? Si c'est le cas, c'est probablement normal que == n'existe pas. Si c'est toi qui a créé ce système d'events, c'est probablement une erreur :)
A priori, cette classe Event doit respecter la sémantique d'entité et donc ne pas définir d'opérateur == (ni d'opérateur = donc tu ne peux pas les mettre dans une map). Tu dois passer par des pointeurs (de préférence shared_ptr) et tester simplement que les pointeurs correspondent au même objet en mémoire
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 map<long, shared_ptr<Event>> _events; void EventFactory::DeleteEvent(Event* ev) { if (!ev) return; for(auto itr = _events.begin(); itr != _events.end(); ++itr) { if(itr->second == ev) { _events.erase(itr); } } }
PS2 : pour rappel, this est un pointeur :
Code:

1 2 3 4 this->second.Repeat() this->second.Delay() this->second.Repeated() this->first