[STL]Problème itérateur avec list

Version imprimable

Hello,

Je débute avec les conteneurs et mon compilateur me met une floppée d'erreurs dans cette fonction:
Code:

1 2 3 4 5 6 7 8 9 10 // Dessine la liste des briques void DrawBricks(SDL_Surface* MainScreen) { list<CBrick*>::iterator i; for(i = m_brickList.begin(); i != m_brickList.end(); i++) { SDL_FillRect(MainScreen, i->m_Position, i->m_Color); } }
La liste est déclarée comme ceci:

Code:

list <CBrick*> m_brickList;

Et pourtant j'ai aussi un code avec itérateur pour détruire les objets et là il n'y a pas d'erreurs :(:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 CLevel::~CLevel() { // Destruction de la liste list<CBrick*>::iterator i; for(i = m_brickList.begin(); i != m_brickList.end(); i++) { delete *i; cout << "Brique effacée de la mémoire" << endl; } m_brickList.erase(m_brickList.begin(), m_brickList.end() ); }
Ce n'est pas comme ça qu'on utilise l'itérateur? Je n'ai pas trouvé d'exemple similaire dans la FAQ ou ailleurs...

03/10/2005, 16h48
la drogue c'est mal

c'est quoi les erreurs ?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
 
f:\documents and settings\jérémy.jeremy-portable\mes documents\visual studio projects\pong\level.h(49): error C2227: la partie gauche de '->m_Position' doit pointer vers un class/struct/union
f:\documents and settings\jérémy.jeremy-portable\mes documents\visual studio projects\pong\level.h(49): error C2839: type de retour 'std::list<_Ty>::_Tptr' non valide pour l'opérateur surchargé 'operator ->'
        with
        [
            _Ty=CBrick *
        ]
f:\documents and settings\jérémy.jeremy-portable\mes documents\visual studio projects\pong\level.h(49): error C2227: la partie gauche de '->m_Position' doit pointer vers un class/struct/union
f:\documents and settings\jérémy.jeremy-portable\mes documents\visual studio projects\pong\level.h(49): error C2839: type de retour 'std::list<_Ty>::_Tptr' non valide pour l'opérateur surchargé 'operator ->'
        with
        [
            _Ty=CBrick *
        ]
f:\documents and settings\jérémy.jeremy-portable\mes documents\visual studio projects\pong\level.h(49): error C2227: la partie gauche de '->m_Color' doit pointer vers un class/struct/union
f:\documents and settings\jérémy.jeremy-portable\mes documents\visual studio projects\pong\level.h(49): error C2839: type de retour 'std::list<_Ty>::_Tptr' non valide pour l'opérateur surchargé 'operator ->'
        with
        [
            _Ty=CBrick *
        ]
f:\documents and settings\jérémy.jeremy-portable\mes documents\visual studio projects\pong\level.h(49): error C2839: type de retour 'std::list<_Ty>::_Tptr' non valide pour l'opérateur surchargé 'operator ->'
        with
        [
            _Ty=CBrick *
        ]
f:\documents and settings\jérémy.jeremy-portable\mes documents\visual studio projects\pong\level.h(49): error C2227: la partie gauche de '->m_Color' doit pointer vers un class/struct/union

En gros que l'itérateur est faux je pense... il n'arrive pas à accéder à mon objet qui pourtant doit marcher...

essai ca :

Code:

1
2
3
4
5
6
7
8
9
10
 
 
void DrawBricks(SDL_Surface* MainScreen) 
{ 
   list<CBrick*>::iterator i; 
   for(i = m_brickList.begin(); i != m_brickList.end(); i++) 
   { 
      SDL_FillRect(MainScreen, (*i)->m_Position, (*i)->m_Color); 
   } 
}

Code:

1
2
3
4
for(i = m_brickList.begin(); i != m_brickList.end(); i++)
   {
      SDL_FillRect(MainScreen, (*i)->m_Position, (*i)->m_Color);
   }

03/10/2005, 17h13
Fiquet

Ah d'accord merci :D je n'avais plus pensé qu'il fallait deréférencer 2 fois l'itérateur le genre de truc trop con...

Mais au fait y'a-t-il un sens de déclarer une liste de pointeurs si on y accède comme ceci? Quand l'objet est il copié?
03/10/2005, 17h25
Laurent Gomila

L'intérêt de stocker des pointeurs est justement qu'il n'y a aucune copie. Après à toi de voir si c'est cohérent avec tes classes et leur sémantique.

Pour la frime, sans boucles du tout:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
namespace 
{
  /** Dessine une seule brique */
  void DrawBrick(CBrick * brick, SDL_Surface * scr)
  {
    SDL_FillRect(scr, i->m_Position, i->m_Color); 
  }
 
  /** Detruit l'objet pointé */
  struct DeleteObject
  {
    template<typename T>
    void operator()(const T* ptr) const
    {
      delete ptr;
    }
  };
}
 
void 
CLevel::DrawBricks(SDL_Surface * MainScreen)
{
  for_each(m_brickList.begin(), m_brickList.end(),
           bind2nd(ptr_fun(DrawBrick), MainScreen));
}
 
void
CLevel::~CLevel()
{
  for_each(m_brickList.begin(), m_brickList.end(),
           DeleteObject());
}