Discussion :

[Phobizeur]

Bonjour,

Je rencontre actuellement un problème bloquant dont je ne trouve pas la solution, le but qu'un objet puisse envoyer un this ( donc un pointer vers lui même ) à un autre objet ( via son constructeur ) en argument.

voici ce que j'ai essayé en version simplifié qui ne fait apparaitre que le code du problème :

UnitTower.h

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
class UnitTower
{
public:
	UnitTower();
	~UnitTower();
	sf::Sprite getSprite();
	void update();
	void setTarget(UnitMobile);
	UnitMobile getTarget();
 
protected:
	UnitMobile cible;
	sf::Texture texture;
	sf::Sprite sprite;
};

Une fonction dans le UnitTower.cpp qui doit crée un objet et envoyer un pointer de lui même

Code :

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

void UnitTower::update(){ Shoot shoot(this) }

le constructeur de l'objet qui récupère l'autre

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
class Shoot
{
public:
	Shoot(UnitTower*);
	~Shoot();
	void fire();
 
protected:
	sf::Texture texture;
	sf::Sprite sprite;
	int dommage;
	UnitTower* tower;
	sf::Clock clock;
};

Et son Shoot.cpp

Code :

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

1
2
3
4
Shoot::Shoot(UnitTower* tOwer)
{
	tower = tOwer;
}

Je vous remercie d'avance et je continue à chercher

Cdlm,

Phobizeur

[KsassPeuk]

Lu'!

Et donc c'est quoi l'erreur ?

Deux trucs au passage quand même :

• changer la casse d'un caractère dans le mot pour lever une amibguité ça promet des jolis fails sur les fautes de frappes
• c'est normal que la cible soit modifiée en recevant une copie ?

[Phobizeur]

L'erreur :

1>UnitTower.cpp(46): error C2664: 'Shoot::Shoot(const Shoot &)' : cannot convert argument 1 from 'UnitTower *const ' to 'const Shoot &'

Pourtant c'est tout simple, lorsque je crée Shoot shoot(this), je veux récupérer la tower pour avoir ses variables dans shoot.

[ternel]

A priori, le code que tu montres n'est pas celui qui a levé l'erreur.
Nettoie ton projet (clean) et recommence.

Par ailleurs, utilise la liste d'initialisation du constructeur.

Code :

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

Shoot::Shoot(UnitTower* tower) : tower(tower) {}

le tower en rouge est le nom du champ, dans la classe.

Seule la liste d'initialisation est capable de remplir les champs constants et les références.

[Phobizeur]

J'ai viens d’essayer avec la liste d'initialisation et j'ai toujours la même erreur sauf que j'ai plus de détail ^^"

Le détail :

Reason: cannot convert from 'UnitTower *const ' to 'const Shoot'

Je ne vois pas pourquoi ça ne marche pas !

[JolyLoic]

Visiblement, tu essaye de passer à ton constructeur un pointeur constant sur une UnitTower, alors qu'il s'attend à avoir un pointeur non constant. Donc soit tu n'as pas besoin de modifier la UnitTower depuis la classe Shoot, et tu prend en paramètre un pointeur constant que tu stockes dans un pointeur constant, soit la modification est requise, et dans ce cas là, il faut passer un pointeur non constant au constructeur.