Help iterator

[Bombseb]

Bonjour,

J'ai un petit problème dans un prog que j'éssaye de faire, je débute en C++ et SFML donc désolé par avance si c'est une question de noob.

Voici mon code (épuré)

le .h :

Code: [Sélectionner]

class Mec
{
    public:
        Mec(float const& x, float const& y);

    private:
        sf::Sprite currentSprite;

        std::vector<sf::Sprite> idle_Right;
        std::vector<sf::Sprite>::iterator itSprite;
};


le .cpp :

Code: [Sélectionner]

Mec::Mec(float const& x, float const& y)
{
   itSprite = idle_Right.begin();
   *itSprite->setPosition (sf::Vector2f (x,y));
}
A la compilation il râle sur le setPosition :

void value not ignored as ought to be

Pourtant *itSprite est bien du type sf::Sprite non ?

Est-ce que quelqu'un a une idée svp ?

Merci d'avance

[Bombseb]

Bon j'ai trouvé, il fallais mettre ca :

(*itSprite).setPosition (sf::Vector2f (x,y));

Je ne comprend pas trop pourquoi....

[Laurent]

Parce que -> est prioritaire sur *. Donc ta première version était en fait :

*(itSprite->setPosition (sf::Vector2f (x,y)))

Sinon, tu peux aussi tout simplement écrire

itSprite->setPosition (sf::Vector2f (x,y))

Un itérateur a la même sémantique qu'un pointeur.

[Bombseb]

Salut et merci pour ta réponse,

Effectivement ta solution marche aussi...
Du coup je ne comprend plus trop, je vais m'informer un peu plus sur les iterateurs...

[Laurent]

Qu'est-ce que tu ne comprends pas ?

[Bombseb]

Nonnon c'est bon

Je viens de faire quelques tests sur l'iterator....c'est bon j'ai compris, on le manipule comme un pointeur
Je ne suis pas encore hype familier avec tout ca mais ca va venir, merci pour ton aide en tout cas 

[Laurent]

N'hésite pas à bien bosser tes bases de C++ avant d'essayer de faire des trucs compliqués. On voit trop de débutants qui foncent tête baissée dans l'élaboration d'un jeu avec SFML sans même connaître le C++, et... ils arrivent droit dans le mur

[Bombseb]

Bon alors j'ai encore un problème avec mon iterator 

Dans ma classe j'ai ces variables :

Code: [Sélectionner]

sf::Sprite *currentSprite;
std::vector<sf::Sprite> walk_Up;
std::vector<sf::Sprite>::iterator itSprite;

Et dans mon .cpp à un moment je fait ca :

Code: [Sélectionner]

itSprite = walk_Up.begin();
currentSprite = itSprite;
Et à la compil sur l'affectation de currentsprite il me dis ca :

Code: [Sélectionner]

/home/seb/Progs/SFML/2DISO/2DISO-build-desktop-Qt_4_8_1_dans_le_PATH__Syst_me__Release/../2DISO/mec.cpp:112: erreur : cannot convert 'std::vector<sf::Sprite>::iterator* {aka __gnu_cxx::__normal_iterator<sf::Sprite*, std::vector<sf::Sprite> >*}' to 'sf::Sprite*' in assignment
Pourtant je ne comprend pas, currentSprite est bien un pointeur vers un sf::Sprite et mon iterator est sensé être un pinteur vers un sf:sprite aussi... non ?

[Bombseb]

Je remarque que si je fait ca :

Code: [Sélectionner]

currentSprite = &(*itSprite);
Ca marche.... bizarre

[Laurent]

Un itérateur se comporte comme un pointeur, mais ce n'en est pas un. Pointeur et itérateurs sont deux types qui n'ont rien à voir, et il faut déréférencer comme tu le fais pour retrouver l'adresse de l'objet pointé par l'itérateur.

[Bombseb]

ok...

Bizarre quand même mais bon
Du coup j'ai préféré utiliser un simple compteur plutôt qu'un iterator au moins c'est plus simple

[Laurent]

C'est pas bizarre, ni compliqué. Au contraire, les itérateurs font tout pour que leur utilisation soit naturelle et proche des pointeurs, pour te faciliter la vie. Mais ce n'en sont pas (leur fonctionnement peut être plus complexe selon le type de conteneur), donc il faut une petite conversion pour avoir l'objet pointé puis son adresse. C'est tout.

Mais pourquoi est-ce que tu as besoin de stocker un itérateur ? Normalement tu ne les utilises que localement, pour itérer sur un conteneur ou trouver un élement.

[Bombseb]

C'était pour gérer l'animation d'un sprite en quatre étapes.
Je stocke mes quatre sprites dans un vector, et à chaque affichage j'incrément mon iterator pour passer au sprite suivant

[Laurent]

Là tu peux utiliser un indice, en plus le bouclage sera plus facile (tu as juste à faire un modulo).

[Bombseb]

Yep, c'est exactement ce que j'ai fait sauf que j'ai utilisé un and pour boucler en faisant ca :

        nAnimFrame++;
        nAnimFrame = nAnimFrame & 3;