Discussion :

[Nicogo]

Bonsoir,

Alors, depuis hier, j'ai fait un sous-moteur à la lib SFML.
Sauf que je voudrais utiliser une liste (array sans limite) de type DrawUnit, qui est une class contenant une fonction drawOnTheWindow pour le moment.
Et j'ai une belle erreur de la part de Visual ... Donc si quelqu'un peux m'éclairer

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
 
 
#ifndef DEF_DRAWCLASS
#define DEF_DRAWCLASS
 
#include <SFML/Graphics.hpp>
#include <vector> 
 
#include "DrawUnit.h"
 
using namespace sf;
 
class DrawClass
{
private:
	std::vector<DrawUnit*> liste_to_draw;
public:
	DrawClass();
	~DrawClass();
 
	void drawAll(RenderWindow* window);
	int add_elem(DrawUnit* toAdd);
	void remove_elem(int index);
};
 
#endif

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
 
 
#include "DrawClass.h"
DrawClass::DrawClass()
{
 
}
DrawClass::~DrawClass()
{
 
}
 
void DrawClass::drawAll(RenderWindow* window)
{
	int i = 0;
	for (i = 0; i < (int)liste_to_draw.capacity(); i++)
	{
		liste_to_draw[i]->drawOnTheWindow();//Erreur ici
	}
 
}
int DrawClass::add_elem(DrawUnit* toAdd)
{
	liste_to_draw.push_back(toAdd);
	return liste_to_draw.end() - liste_to_draw.begin();
}
void DrawClass::remove_elem(int index)
{
	liste_to_draw.erase(liste_to_draw.begin() + index);
}

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
 
#ifndef DEF_DRAWUNIT
#define DEF_DRAWUNIT
 
#include <SFML/Graphics.hpp>
 
using namespace sf;
 
class DrawUnit
{
private:
 
public :
	int drawOnTheWindow();//RenderWindow* window);
};
 
#endif

Code :

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

1
2
3
4
5
6
7
8
 
 
#include "DrawUnit.h"
int drawOnTheWindow()//(RenderWindow* window)
{
	//window->clear();
	return 0;
}

L'erreur :

Code :

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

Error	1	error LNK2019: unresolved external symbol "public: int __thiscall DrawUnit::drawOnTheWindow(void)" (?drawOnTheWindow@DrawUnit@@QAEHXZ) referenced in function "public: void __thiscall DrawClass::drawAll(class sf::RenderWindow *)" (?drawAll@DrawClass@@QAEXPAVRenderWindow@sf@@@Z)	C:\Users\Nico\Documents\Visual Studio 2010\Projects\SFML\SFML\DrawClass.obj	SFML

ps : ça serais simpa aussi de dire s'il y a des trucs fais "bizarrement" car je passe du vb.net au C++, ça change ..

Merci

[koala01]

Salut,

L'erreur te dit que l'éditeur de liens ne trouve pas le code binaire correspondant à DrawUnit::drawOnTheWindow().

Selon moi, il y a trois solutions:

1. Soit VisualStudio s'est "emmêlé les pinceaux" et n'a pas recompilé DrawUnit.cpp après que tu aies changé le prototype de drawOnTheWindow
2. Soit tu as modifié le prototype de drawOnTheWindow dans le DrawUnit.h mais pas dans DrawUnit.cpp
3. Soit, enfin, DrawUnit.cpp ne fait pas partie du projet

Dans le premier cas, je te conseillerais de faire un "clear" de ton projet et de tout recompiler

Dans le deuxième cas, il faut corriger soit DrawUnit.h soit DrawUnit.cpp pour que les prototypes correspondent (et de préférence faire un clear du projet avant de tout recompiler )

Dans le troisième cas, il faut veiller à rajouter le fichier manquant à ton projet, avant de compiler à nouveau

Enfin, si tu es face à un projet important qui serait subdivisé en plusieurs "sous projet" et que DrawUnit.h / DrawUnit.cpp font partie d'un sous projet particulier, il serait bon de commencer par faire pareil avec le sous projet en question

[Nicogo]

Okk, Merci.

J'essaye tout ça demain et je vous redis quoi.

édit :

Et sinon le code reste "propre" ?

[koala01]

Ouppsss...

En relisant le code, je viens de trouver l'erreur...

Tu as défini

Code :

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

1
2
3
4
5
int drawOnTheWindow()//(RenderWindow* window)
{
	//window->clear();
	return 0;
}

qui défini une fonction libre nommée DrawOnTheWindow...

Or, tu déclare une fonction membre de la classe DrawUnit qui porte ce nom.

comme je présumes que tu souhaites effectivement qu'il s'agisse d'une fonction membre de ta classe, la définition devrait être pleinement qualifiée:

Code :

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

1
2
3
4
5
int DrawUnit::drawOnTheWindow()//(RenderWindow* window)
{
	//window->clear();
	return 0;
}

Okk, Merci.

Et sinon le code reste "propre" ?

Ben, à moins que la durée de vie de tes objet de type DrawUnit ne soit "gérée par ailleurs", tu risque d'avoir une belle fuite mémoire lors de la destruction de ton objet de type DrawClass.

Ce que tu places dans ta liste, c'est des pointeurs, qui correspondront de toutes évidences à des objets pour lesquels tu as eu recours à l'allocation dynamique.

Tu as donc accepté "de facto" de prendre seul la responsabilité de déterminer quand ces objets seront détruits.

Lorsque la liste sera détruite, tu perdra "toute référence" vers les objets qu'elle contient, et tu ne pourras donc plus y accéder pour détruire correctement les éléments pointés.

Je te laisses le soin de réfléchir à la manière de résoudre ce problème

[Kalith]

Et sinon le code reste "propre" ?

Non, tu as un bug ici :

Code :

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

1
2
3
4
5
6
7
8
9
void DrawClass::drawAll(RenderWindow* window)
{
	int i = 0;
	for (i = 0; i < (int)liste_to_draw.capacity(); i++)
	{
		liste_to_draw[i]->drawOnTheWindow();//Erreur ici
	}
 
}

std::vector::capacity() te donne le nombre maximum d'éléments que peux contenir ton std::vector avant qu'il ait besoin de ré-allouer de la mémoire. Toi tu veux plutôt le nombre total d'éléments qui sont stockés, et pour ça il faut utiliser std::vector::size().

Petite remarque en passant sur la boucle, tu peux écrire plus simplement :

Code :

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

for (int i = 0; i < (int)liste_to_draw.size(); i++)

Mais il est plus correct d'utiliser une variable non signée pour l'itération, en particulier std::size_t. D'une part ça t'évite le warning que Visual doit t'envoyer parce que tu compares une grandeur signée i et une grandeur non signée std::vector::capacity(), et d'autre part c'est plus correct car i n'a aucune raison d'être négatif :

Code :

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

for (std::size_t i = 0; i < liste_to_draw.size(); i++)

Sinon tu peux utiliser les itérateurs. La notation est un peu lourde, mais parfois plus pratique car la majorité des méthodes de std::vector (ou encore certaines fonctions de la bibliothèque standard) travaillent sur des itérateurs. Ici ça n'a pratiquement aucun intérêt, si ce n'est qu'il est un peu plus difficile d'écrire une sottise :

Code :

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

1
2
3
4
for (std::vector<DrawUnit*>::iterator i = list_to_draw.begin(); i != list_to_draw.end(); ++i)
{
    (*i)->drawOnTheWindow();
}

Et le top du top, si tu peux utiliser le C++11 :

Code :

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

1
2
3
4
for (DrawUnit* unit : list_to_draw)
{
    unit->drawOnTheWindow();
}

[r0d]

Bonjour,

au cas où tu cherches à améliorer ton code, tu devrais remplacer tous tes pointeurs par des références (références constantes dans certains cas). En tout cas, sur le code que tu as posté, je ne vois pas l'utilité des pointeurs, or, comme on dit: "les pointeurs, on ne les utilise que quand on ne peut pas s'en passer".

[darkman19320]

Bonjour,

Juste une petite remarque, évite d'utiliser "using namespace" dans les fichiers d'entête, ça pourrais poser problème dans d'autres projets.

[Nicogo]

Bonjour,

au cas où tu cherches à améliorer ton code, tu devrais remplacer tous tes pointeurs par des références (références constantes dans certains cas). En tout cas, sur le code que tu as posté, je ne vois pas l'utilité des pointeurs, or, comme on dit: "les pointeurs, on ne les utilise que quand on ne peut pas s'en passer".

Déso du double post, mais par curiosité, dans quel(s) cas est-on obliger d'utiliser un pointeur ?