Discussion :

[arthurembo]

slt à toute la communauté,
beh voilà j'ai un problème pas très simple. Dans le cadre d'un project j'ai écris une classe défini à peu près comme suit:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
class Scan
{
  private:
     std::vector< LaserRowSet > pointList; //!< stores all the points
     unsigned int pointCount; //!< buffers the number of points
     bool pointMemoryResponsability; 
  public:
     Scan(TimeIndex t);
     void add(Point *point, unsigned int laserRow=0, float yaw=0);
}

j'ai aussi défini une classe iterator pour itérer sur mes éléments avec les fonctions:

Code :

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

1
2
3
 
iterator begin();  //!< get Iterator that points to the first Point, i.e. the first Point in the first laser row
iterator end();

.

Je peux donc ajouter des points dans ma classe en faisant tout simplement:

Code :

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

1
2
3
4
 
Scan s1(0);
Point p1(2,2,2);
s1.add(&p1, 5);

Maintenant je veux afficher tous les points contenus dans mon objet scan, pour cela je fais:

Code :

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

1
2
3
4
5
6
7
 
Scan::iterator s1it = s1.begin();
for (s1it = s1.begin(); s1it != s1.end(); s1it++){
	Point *pt = *s1it;
	cout << "Point:" << *pt;
	ptCount++;
}

ca marche à merveille. Le problème est, je veux en fait utiliser boost_foreach, mais ca na marche pas:

Code :

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

1
2
3
4
 
BOOST_FOREACH(Point* pt, s1){
    cout << "Point:" << *pt;
   }

Je n'ai vraiment pas grande idée où sa coince. Quelqu'un m'a parlé d'iterator adaptator mais je n'ai aucune idée de comment on l'utilise.
Je travaille avec Eclipse sous linux et voilà l'érreur que j'ai:

Code :

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

1
2
3
4
5
6
7
8
9
 
Scan.cpp:201:   instantiated from here
/usr/include/boost/range/const_iterator.hpp:36: error: no type named ‘const_iterator’ in ‘class Scan’
/usr/include/boost/foreach.hpp: In function ‘typename boost::foreach_detail_::foreach_reference<T, C>::type boost::foreach_detail_::deref(const boost::foreach_detail_::auto_any_base&, boost::foreach_detail_::type2type<T, C>*) [with T = Scan, C = mpl_::bool_<false>]’:
Scan.cpp:201:   instantiated from here
/usr/include/boost/foreach.hpp:636: error: invalid initialization of non-const reference of type ‘Point*&’ from a temporary of type ‘Point*’
make[1]: Leaving directory `/home/simo/code/DataContainers3D'
make[1]: *** [debug/Scan.o] Error 1
make: *** [debug] Error 2

J'attends impatienment vos suggestions et merci d'avance.
PS: désolé des quelques fautes, j'écris avec clavier allemand.

[Arzar]

Salut!
Le mot clé ici c'est type.

error: no type named ‘const_iterator’ in ‘class Scan’

Et dans la doc de boost :

The support for STL containers is very general; anything that looks like an STL container counts. If it has nested iterator and const_iterator types and begin() and end() member functions, BOOST_FOREACH will automatically know how to iterate over it

Il faut donc que ta classe suive ce synopsis :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
class Scan
{
public:
typedef "..." iterator;
typedef "..." const_iterator;
iterator begin();
iterator end();
const_iterator begin() const;
const_iterator end() const;
...
}

[loufoque]

En fait, il faut que le type satisfasse les pré-requis de Range.
Tu regardes le concept et tu vérifies que tu l'implémentes correctement...

[arthurembo]

Merci pour vos réponses et excusez-moi de la réponse tardive. C'étais bel et biedn ca le problème. Mais bon je vais essayer de trouver une alternative car j'ai pas vraiment envie de reécrire ma Klasse iterator et la nommer const_iterator, pour moi ca ferai de la duplication de code. Je jette un coup d'oeil sur les iterators facades, il parait que ca peut resoudre aussi mon problème.

Merci

[Matthieu Brucher]

Tu ajoutes un typedef, ça peut être pas mal (typedef iterator<boost::add_const<CetteClasse>::type> const_iterator ou un truc du genre).

[arthurembo]

quel effet à cet typdef iterator...?, apparenment elle crée un const_iterator à partir de ma classe itérator...?, quelle est la valeur de type?

[Matthieu Brucher]

Il te fat un type const_iterator qui soit un itérateur sur le conteneur constant. D'où le fait de faire un typedef pour le créer sans réécrire le code.

[arthurembo]

donc en fait ma classe iterator{..} (c'est comme ca que j'ai nommé ma classe) doit etre un template d'après cette écriture?

doit ajouter un include<boost...>? ou un truc semblable afin que ca marche?

Merci

[Matthieu Brucher]

donc en fait ma classe iterator{..} (c'est comme ca que j'ai nommé ma classe) doit etre un template d'après cette écriture?

doit ajouter un include<boost...>? ou un truc semblable afin que ca marche?

Merci

Oui et oui (regarde dans la doc de MPL pour Boost).

[Alp]

http://www.boost.org/doc/libs/1_37_0...add_const.html

[arthurembo]

ok,
j'ai écris comme cela chez moi.

Code :

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

1
2
3
 
boost::add_const<iterator>::type const_iterator;
const_iterator begin();

Bon beh j'ai l'erreur:

Code :

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

1
2
 
‘const_iterator’ does not name a type

La première approche (typedef iterator<boost::add_const<iterator>::type> const_iterator) était une bonne approche, mais en fait pour cela il faudrait que ma classe iterator soit un template, ce dont je ne veux pas.

En fait je veux utiliser le BOOST_FOREACH pour itérer sur ma classe mais le problème effectivement est que je dois d'une manière définir un const_iterator mais je n'ai pas envie de réecrire une classe const_iterator du moment où il possède les mêmes fonctions que la classe iterator. Quelqu'un aurait il une meilleure solution?

Merci

[Alp]

Dans ce cas, ta classe iterator doit mettre à disposition une version const par le biais d'un typedef, que tu pourras utiliser de l'extérieur.

[arthurembo]

beh,
comment le faire concrètement?. C'est en fait ca mon problème. Désolé des questions mais je suis novice dans la STL et BOOST.

[Matthieu Brucher]

Utilise typedef.