Discussion :

[Krishna]

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

Code :

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

void Clean(CPieces& aMove);

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

Code :

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

list <CPieces> Recipient;

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

Code :

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

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....

[axile]

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 :

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

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().

[Krishna]

Ok
J'essaye et je te tiend au courrant.

[Laurent Gomila]

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.

[Krishna]

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

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

[Laurent Gomila]

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

Code :

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

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.

[Luc Hermitte]

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.

[Laurent Gomila]

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.

[Krishna]

:pingoin: Bonsoir,
Je suis tres impressionné par les réponses... 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....