Conversions

:pingoin: Bonsoir,
J'aimerais ecrire une fonction Clean:

Code:

---

void Clean(CPieces& aMove);

---

Son but est de passer en revue des objets CPieces d'une liste:

Code:

---

list <CPieces> Recipient;

---

et d'en suprimer certains suivant un critère quelconque.
J'ai donc ecrit:

Code:

---

for_each(Recipient.begin(),Recipient.end(),Clean);

---

Mais je récupère le message suivant à la compilation:
C:\...\EchecEtMat\Logic.cpp(91) : error C2664: 'for_each' : cannot convert parameter 3 from 'void (class CPieces &)' to 'void (__thiscall *)(class CPieces &)'
None of the functions with this name in scope match the target type
Je rame un peu (pour pas dire beaucoup) avec les subtilités de la conversion.Le principe général ne m'est pas étranger mais ce genre de finesses me laissent perplexe....

Si tu utilises VC6, c'est probablement un bogue du compilateur qui n'implémente pas le standard correctement. Je crois qu'en retournant un int avec ta fonction Clean(), ça contourne le problème. Par exemple:

Code:

---

1 2 3 4 5 6 7

int Clean(CPieces& aMove) { ...   return (0);   }

---

Cependant, ce n'est pas une bonne idée de supprimer des éléments avec l'algorithme for_each(), vois plutôt remove().

Ok
J'essaye et je te tiend au courrant.
:merci:

Clean ne peut pas être une fonction membre, du moins pas dans ton cas.

Sinon effectivement, pour supprimer des éléments d'un conteneur tu as l'option erase + remove_if qui marche bien.

:pingoin: Bonjour,
Je crois bien que je vais utiliser remove_if comme vous me le dites parceque ça coince toujours :arf:

ps:Et si je déclare Clean comme fonction membre de CPieces?

Je pense que le mieux est effectivement d'utiliser remove_if + erase, avec un foncteur qui va bien.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

struct Clean { bool operator ()(const CPiece& p) const { if (blah blah) return true; else return false; } };   Recipient.erase(std::remove_if(Recipient.begin(), Recipient.end(), Clean()), Recipient.end());

---

Sinon, si Clean était une fonction membre de CPiece, il faudrait utiliser std::mem_fun_ref.

L'idiome erase-remove est bien, mais pas pour les listes. Il sert avec les conteneurs dont les objets sont consécutifs en mémoire.

C'est vrai que les listes possèdent la fonction membre remove_if, je n'y pensais plus. Par contre on peut toujours utiliser le erase-remove_if pour les vieux compilos (VC6) qui ne supportent pas les fonctions membres templates de classes template.

:pingoin: Bonsoir,
Je suis tres impressionné par les réponses... :hola: Il va me faloir un petit peu de temps pour les digérer et les mettre en oeuvre mais quelque chose me dit que ça va marcher....
:merci: