Discussion :

[Fiquet]

Hello,

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

Code :

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

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 :

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

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 :

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

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...

[la drogue c'est mal]

c'est quoi les erreurs ?

[Fiquet]

Code :

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

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...

[la drogue c'est mal]

essai ca :

Code :

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

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); 
   } 
}

[Laurent Gomila]

Code :

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

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

[Fiquet]

Ah d'accord merci 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é?

[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.

[Jan Rendek]

Pour la frime, sans boucles du tout:

Code :

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

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());
}