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.

[Phobizeur]

J'ai modifié le prototype en :

Code :

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

Shoot(UnitTower const*);

et le constructeur en :

Code :

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

Shoot::Shoot(UnitTower const* tOwer) : tower(tOwer)

mais j'ai toujours l'erreur :

1>UnitTower.cpp(46): error C2664: 'Shoot::Shoot(const Shoot &)' : cannot convert argument 1 from 'UnitTower' to 'const Shoot &'
1>          Reason: cannot convert from 'UnitTower' to 'const Shoot'
1>          No user-defined-conversion operator available that can perform this conversion, or the operator cannot be called

En sachant que l'appelle du constructeur de Shoot se fait via une fonction de UnitTower.cpp :

Code :

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

Shoot shoot(this);

[Invité]

Bonjour,

As-tu, comme recommandé par leternel, nettoyé et recompilé ton projet ? Compiles/Travailles-tu aussi sur le bon fichier ?
Liste d'initialisation ou pas, le code que tu nous montres n'est pas en accord avec ces erreurs.

[Phobizeur]

J'ai clean et rebuilt... Toujours la même erreur...

J'ai upload le projet pour les courageux qui veulent jeter un œil :

Merci encore de vos réponses et de votre patience.

[Invité]

N'aurais-tu pas oublié de nous montrer un certain nombre d'autres erreurs ?

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
1>Shoot.h(8): error C2061: erreur de syntaxe*: identificateur 'UnitTower'
1>Shoot.h(16): error C2143: erreur de syntaxe*: absence de ';' avant 'const'
1>Shoot.h(16): error C4430: spécificateur de type manquant - int est pris en compte par défaut. Remarque*: C++ ne prend pas en charge int par défaut
1>UnitTower.cpp(27): warning C4244: 'initialisation'*: conversion de 'float' en 'int', perte possible de données
1>UnitTower.cpp(32): warning C4244: 'argument'*: conversion de 'int' en 'float', perte possible de données
1>UnitTower.cpp(36): warning C4244: 'argument'*: conversion de 'int' en 'float', perte possible de données
1>UnitTower.cpp(39): warning C4244: 'argument'*: conversion de 'int' en 'float', perte possible de données
1>UnitTower.cpp(43): warning C4244: 'argument'*: conversion de 'int' en 'float', perte possible de données
1>UnitTower.cpp(46): error C2664: 'Shoot::Shoot(const Shoot &)'*: impossible de convertir l'argument 1 de 'UnitTower *const ' en 'const Shoot &'
1>          Raison*: impossible de convertir de 'UnitTower *const ' en 'const Shoot'
1>          Aucun constructeur n'a pu prendre le type de source, ou la résolution de la surcharge du constructeur était ambiguë
1>  TowerNoob.cpp
1>Shoot.h(8): error C2061: erreur de syntaxe*: identificateur 'UnitTower'
1>Shoot.h(16): error C2143: erreur de syntaxe*: absence de ';' avant 'const'
1>Shoot.h(16): error C4430: spécificateur de type manquant - int est pris en compte par défaut. Remarque*: C++ ne prend pas en charge int par défaut
1>TowerNoob.cpp(14): warning C4244: 'argument'*: conversion de 'int' en 'float', perte possible de données
1>  Source.cpp
1>Shoot.h(8): error C2061: erreur de syntaxe*: identificateur 'UnitTower'
1>Shoot.h(16): error C2143: erreur de syntaxe*: absence de ';' avant 'const'
1>Shoot.h(16): error C4430: spécificateur de type manquant - int est pris en compte par défaut. Remarque*: C++ ne prend pas en charge int par défaut
1>TileMap.h(32): warning C4244: 'argument'*: conversion de 'unsigned int' en 'float', perte possible de données
1>TileMap.h(33): warning C4244: 'argument'*: conversion de 'unsigned int' en 'float', perte possible de données
1>TileMap.h(34): warning C4244: 'argument'*: conversion de 'unsigned int' en 'float', perte possible de données
1>TileMap.h(35): warning C4244: 'argument'*: conversion de 'unsigned int' en 'float', perte possible de données
1>TileMap.h(38): warning C4244: 'argument'*: conversion de 'unsigned int' en 'float', perte possible de données
1>TileMap.h(39): warning C4244: 'argument'*: conversion de 'unsigned int' en 'float', perte possible de données
1>TileMap.h(40): warning C4244: 'argument'*: conversion de 'unsigned int' en 'float', perte possible de données
1>TileMap.h(41): warning C4244: 'argument'*: conversion de 'unsigned int' en 'float', perte possible de données

Tes deux classes Shoot et UnitTower font référence l'une à l'autre. Voir ce sujet de la FAQ.

Prend l'habitude de commencer par la première erreur rencontrée par le compilateur, souvent celles qui suivent en découlent.