rendre une std::list circulaire

Bonjour,
En travaillant sur un des points de la génération de ville je suis tombé sur un problème vraiment basique que je n'arrive pas à résoudre.
Je ne comprend vraiment pas pourquoi cela ne fonctionne pas pourtant rien de bien compliqué normalement.
J'ai une liste qui représente un polygone convexe ou concave ce n'est pas important mais une liste de point représente un ensemble de segment et je veux parcourir tous les segments donc les points consécutifs 2 à 2 et et c'est simple théoriquement j'ai 2 itérateurs dont un en avance d'un cran sur l'autre. Je quitte mon do while quand le premier itérateur est a end(). Et si celui en avance est à end() je le fais pointé sur begin :

Code:

---

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

glBegin(GL_LINES); it0 = it1 = il.data.begin(); it1++; do { glVertex3f(it0->x - DX, 0.0f, it0->z - DZ); glVertex3f(it1->x - DX, 0.0f, it1->z - DZ); it0++; if (it1 != il.data.end()) it1++; else it1 = il.data.begin(); } while (it0 != il.data.end()); glEnd();

---

Mais lorsque je fais celà le dernier segment me donne la première valeur bonne et la seconde aléatoire mais pas la première valeur de ma liste.
Ou me suis-je trompé ?

PS : il contient data qui est une liste de fvector2d qui contient en autre 2 float x et z.

Merci

Bonjour,
Il me semble que tu devrais d'abord incrémenter it1 puis le passer à begin s'il est en fin de liste :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

it0 = it1 = il.data.begin(); while (it0 != il.data.end()) { ++it1; if (it1 == il.data.end()) it1 = il.data.begin(); glVertex3f(it0->x - DX, 0.0f, it0->z - DZ); glVertex3f(it1->x - DX, 0.0f, it1->z - DZ); ++it0; }

---

Merci :)

Penses à :resolu:

Comme le précise 3DArchi, ++it est plus efficace que it++ : pas de recopie inutile.

Ok merci j'oublie toujours le [résolu]
Et j'ai pris par habitude laissé le compilateur optimisé. Mais je ne suis pas sur qu'il le fasse pour les types non interne au langage comme les itérateurs...

Là c'est pas une question du compilateur, il peut rien y faire il fait ce que tu lui demandes et en l'occurence la version post et la version préfixée ne font pas exactement la même chose.
La version postfixée te renvoie une copie et est donc plus lourde.

Oui je sais mais comme d'habitude typiquement dans un for (i = 0; i < K; i++) ou ++i rend au même sauf que le compilateur choisit l'un plutôt que l'autre.
Bref en tout cas le problème est résolu et ne se posera plus :D

En modifiant le type de primitive en GL_LINE_LOOP, tu peux supprimer le test de l'itérateur en fin de liste. C'est OpenGL qui fera le boulot de bouclage sur le premier point.