Discussion :

[karl3i]

Bonjour à tous,

je souhaite manipuler une liste avec un itérateur "reverse".
Pas de problème avec le code suivant :

Code :

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

1
2
3
 
std::list<int>::reverse_iterator UnRIterateur;	
UnRIterateur = MaListe.rbegin();

Par contre j'ai voulu faire la même chose mais avec un itérateur constant cette fois-ci (pour protéger les données de la liste)

Code :

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

1
2
3
 
std::list<int>::const_reverse_iterator UnRIterateurConstant;
UnRIterateurConstant = MaListe.rbegin();

La deuxième ligne ne compile pas, j'obtiens l'erreur C2679. Je ne comprends pas pourquoi en fait, car dans la MSDN, rbegin() est bien cité comme pouvant rendre un itérateur constant.
Ce qui me surprend, c'est que la manipulation de begin() avec des itérateurs constants non "reverse" ne me pose pas de problème.

Si vous avez des pistes ! Merci...

Je suis sous VC6.

Guilhem.

[onap]

Tu peux nous donner l'erreur que donne ton compilo et non sont numero stp

[karl3i]

La voilà :

error C2679: binary '=' : no operator defined which takes a right-hand operand of type 'class std::reverse_bidirectional_iterator<class std::list<int,class std::allocator<int> >:
:iterator,int,int &,int *,int>' (or there is no acceptable conversion)

[onap]

Sous g++ 3.3 je ne rencontre aucun problème avec ton code. MaListe est-elle bien du type std::list<int> ?

[karl3i]

Salut,

oui elle l'est :

Code :

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

1
2
 
std::list<int> MaListe;

Je la remplis ainsi :

Code :

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

1
2
3
4
 
MaListe.push_back(38);				
MaListe.push_back(26);				
MaListe.push_back(73);

Après une recherche sur Google http://www.talkaboutprogramming.com/group/comp.lang.c++/messages/725240.html il semblerait que quelqu'un d'autre soit tombé sur cette erreur (sur une map cette fois).

Onap, si tu as un peu de temps, est-ce que tu pourrais essayer de compiler mon code, je l'ai mis sur http://guilhem-martin.org/pb_constreverse.txt pour voir ce qu'il donne avec ton compilo ?

Merci,

Guilhem.

[onap]

Une seul chose n'est pas conforme à la norme: la fonction main renvoie un int donc:

Code :

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

1
2
 
void main() ====> int main()

Sinon aucun problème a la compil, ni a l'execution
Je n'ai pas regarder tous les messages que ton code affiche (sorry mais pas le temps ce soir), mais tu peux les retrouver ici:
http://users.skynet.be/sb301801/

[karl3i]

Super, merci beaucoup.

La sortie est tout à fait correcte.

Le problème est donc du côté de VC6. Je n'utilise pas stlport, c'est peut-être bien la source de mon pb...

Guilhem.

[onap]

Manifestement c'est un bug sous VC6, moi je travail sous Builder au bloulot, j'essayerai lundi pour savoir.
Sinon essaie comme cela:

Code :

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

1
2
 
std::list<int>::const_reverse_iterator UnRIterateurConstant= MaListe.rbegin();

[JolyLoic]

Une question : As tu patché la bibliothèque de VC6 ? Celle livrée par défaut contient certains bugs. Tu peux trouver les patchs sur http://www.dinkumware.com/vc_fixes.html
Mais bon, je n'ai rien l'air d'y voir qui corresponde à ton problème.


J'ai essayer ton code avec VC++8.0. Il compile et tourne sans problème (quand on commente les deux lignes ou un commentaire indique que ça ne marche pas, et où le programme s'arrête avec une belle erreur).

[karl3i]

@onap

L'écriture condensée que tu proposes génère malheureusement une autre erreur :

error C2440: 'initializing' : cannot convert from 'class std::reverse_bidirectional_iterator<class std::list<int,class std::allocator<int> >::iterator,int,int &,int *,int>' to 'c
lass std::reverse_bidirectional_iterator<class std::list<int,class std::allocator<int> >::const_iterator,int,int const &,int const *,int>'
No constructor could take the source type, or constructor overload resolution was ambiguous

@JolyLoic
C'est la version livrée par défaut. Merci pour l'URL je vais appliquer le patch et voir ce que cela donne.

@tous
Au boulot je suis sous .Net, ça devrait donc mieux marcher (j'ai entendu dire que la stl sous .Net se rapprochait plus du standard).

[karl3i]

Pour info, le patch SP6 pour VC6 ne corrige pas le problème.

Guilhem.

[karl3i]

Pour finir, j'ai pu compiler & exécuter sans problème avec VC7.1.

J'ai rencontré par ailleurs un autre problème avec VC6, en utilisant remove_if :

Code :

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

1
2
 
MaListe.remove_if(  est_impaire<int>() );

avec est_impaire comme suit :

Code :

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

1
2
3
4
5
6
7
8
9
 
template <class T> class est_impaire : public std::unary_function<T, bool> 
{
public:
   bool operator( ) ( T& val ) 
   {
   return ( val % 2 ) == 1;		// Si le nombre est paire, le modulo 2 est égale à 0, l'égalité n'est pas vérifié et donc la valeur false est renvoyée
   }
};

me génère cette erreur :

error C2664: 'remove_if' : cannot convert parameter 1 from 'class est_impaire<int>' to 'class std::binder2nd<struct std::not_equal_to<int> >'
No constructor could take the source type, or constructor overload resolution was ambiguous

alors qu'avec VC7, aucun problème.

Merci à JolyLoic et onap pour leur aide,
Guilhem.

[Laurent Gomila]

En ce qui concerne la seconde erreur, c'est dû au fait que VC6 ne supporte pas les fonctions membres templates de classes templates. Ce sera pareil pour les constructeurs de conteneur prenant en entrée une paire d'itérateurs, les fonctions membres prenant un ofncteur, ...