Discussion :

[Tex-Twil]

Bonjour,
j'ai un problème de passage de Classe en paramètre.

J'ai une classe CVecteur3d et une classe CPoint. Je voudrait mettre un constructeur dans CVecteur3d qui construit cette classe à partir de la classe point.

Je me suis inspiré de ça
http://c.developpez.com/faq/cpp/?pag...sage_parametre

mais mon code ne marche pas:

Code :

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

1
2
3
4
 
	CVecteur3d();
	CVecteur3d(float x, float y, float z);
	CVecteur3d(const CPoint &p);

merci pour votre aide.

[jmv]

salut,

mais mon code ne marche pas:

Code :

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

1
2
3
4
 
	CVecteur3d();
	CVecteur3d(float x, float y, float z);
	CVecteur3d(const CPoint &p);

y a pas de code ici ! que des déclarations.
Si tu veux qu'on t'aide, montres-en plus

[Tex-Twil]

oui parce que c'est la déclaration qui ne marche pas:

Code :

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

1
2
3
 
e:\FAC - MASTER\INFOGRAPHIE\RayTraceVisual\CVecteur3d.h(20): error C2226: syntax error : unexpected type 'CVecteur3d'
e:\FAC - MASTER\INFOGRAPHIE\RayTraceVisual\CVecteur3d.h(20): error C2238: unexpected token(s) preceding ';'

[xxiemeciel]

Salut,

jmv a raison il en faut plus si tu veux de l'aide on ne peux rien dire a partir de si peu d'information. Montre la declaration complete de ta classe CVecteur3d.

XXiemeciel

[Tex-Twil]

ok justement je ne voulais pas trop en mettre pour vous éviter de tout lire.
Voici le .h

Point

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
 
#ifndef _CPoint_
#define _CPoint_
 
#include "CVecteur3d.h"
class CPoint
{
public:
	float x, y, z;
 
	CPoint();
	CPoint(float x, float y, float z);
 
	void operator = (CPoint p);
	CPoint operator + (CPoint &p);
	CPoint operator - (CPoint &p);
 
};
 
#endif

et Vecteur:

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
33
34
35
 
 
#ifndef _CVecteur3d_
#define _CVecteur3d_
 
#include "CPoint.h"
 
class CVecteur3d
{
private:
	float x,y,z;
	float norme;
 
	// Norme du vecteur
	void setNorme();
	void normalizer();
public:
 
	CVecteur3d();
	CVecteur3d(float x, float y, float z);
	//CVecteur3d(const CPoint &p);
 
	/* Get - Set */
	void setXYZ(float x, float y, float z);
	void setX(float x);
	void setY(float y);
	void setZ(float z);
 
	float getX();
	float getY();
	float getZ();
 
	void afficherConsole();
};
#endif

et le constructeur

Code :

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

1
2
3
4
5
6
7
 
CVecteur3d::CVecteur3d(const CPoint &p)
{
	this->setX(p.x);
	this->setY(p.y);
	this->setZ(p.z);
}

[Luther13]

C'est innopportum d'inclure "CVecteur3d.h" dans CPoint.h

[Tex-Twil]

Je sais pas ce que ça faisait ici mais en tout cas le probleme venait de cette inclusion !!!!

merci

[Luther13]

Un petit Tag résolu. Et hop c'est emballé !

[jmv]

tu n'as pas besoin d'inclure CVecteur3d.h dans CPoint.h (surtout que celui-ci inclu à son tout CVecteur3d.h)

l' operator=() de la classe CPoint me paraît inutile, celui par défaut ne convient pas ?

Dans CVecteur3d.h, tu peux, au lieu d'include CPoint.h, faire une déclaration anticipée de ta classe.

Code :

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

class CPoint;

et un conseil : ne stocke pas la norme, c'est une information redondante que tu va être obligé de mettre à jour sans cesse.

[edit] j'arrive trop tard [/edit]

[Tex-Twil]

ok merci pour les conseils.

Quel est l'avantage de faire une déclaration anticipée de la classe CPoint ?

[jmv]

Quel est l'avantage de faire une déclaration anticipée de la classe CPoint ?

éviter une inclusion, c'est tout.

Mais ça ne marche que si tu utilise uniquement des pointeurs ou références
sur CPoint.

[Tex-Twil]

j'ai un autre problème avec ce constructeur: quand j'essaye de lui passer un CPoint en paramètre il y a une erreur à la compilation:

Code :

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

1
2
3
4
5
6
7
8
9
 
bool CSphere::estIntercepte(CRayon r)
{
	bool touche;
	CPoint p ; 
	CVecteur3d vect(p);
 
	return touche;
}

Erreur:

Code :

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

1
2
 
RayTraceVisual fatal error LNK1120: 1 unresolved externals

.. je ne suis pas habitué aux passages par références

[Luther13]

Un résumé de la situation et de ton code source ou de la solution que tu as adoptée serait interessannt à ce stade.

[Tex-Twil]

ok. Je travaille sur un projet de Lancé de Rayon (Ray Trace)

A ce stade de projet, je suis en train de calculer si il y a ou non l'intersection entre un rayon et une sphere.

Un Rayon est caracterisé par un point de départ et un vecteur de direction. Le code ci dessus et le commencement de la méthode de la classe CSphere qui doit dire si un rayon touche cette sphère.

Mais même si j'essaye d'instancier un vecteur avec un point dans le main j'ai la même erreur. Il se trouve aussi que je pourrais très bien m'en passer de ce constructeur prenant un point comme paramètre mais j'aimerais quand même réussir à trouver d'où vient l'erreur.

[Luther13]

Il n'y a pas de raison que ca ne marche pas. Mais voici des pistes possibles.

- je vois 3 declarations constructeurs pour CPoint. le conctructeur par défaut que tu utilise a t-il été implémenté ?

- les paramètres du constructeur CPoint(float x, float y, float z) ne sont pas bien choisis: préférer d'autres noms différents de x,y,z

-Enfin, concernant le passage par référence, etant donné que la référence en paramètre est constante, les methodes CPoint:: utilisées doivent aussi êtres accessibles aux objets de type constant

[Tex-Twil]

oui le constructeur par défaut est implémenté.

Code :

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

1
2
3
4
5
6
7
 
CPoint::CPoint()
{
	this->x = 0;
	this->y = 0;
	this->z = 0;
}

Meme si j'utilise un autre constructeur, l'erreur est la meme

Code :

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

1
2
3
4
5
6
 
int main()
{
	CPoint p(1.0, 1.0, 1.0) ;   //= pos - r.getDepart();
	CVecteur3d vect(p);
}

Code :

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

1
2
3
4
 
RayTraceVisual error LNK2019: unresolved external symbol "public: __thiscall CVecteur3d::CVecteur3d(class CPoint const &)" (??0CVecteur3d@@QAE@ABVCPoint@@@Z) referenced in function _main
 
RayTraceVisual fatal error LNK1120: 1 unresolved externals

Pourquoi ne pas utiliser x y z comme params du constructeur ? je differencie bien les paramètre des attributs de la classe en les appelant
this->x, this->y ...

Je ne suis pas sur de comprendre ton dernier point [/code]

[Laurent Gomila]

L'erreur semble claire : l'éditeur de lien ne trouve pas le corps de ton constructeur de CVector3d prenant un point.

[Tex-Twil]

shame on me !!!

J'avais laissé le corps en commentaire, ha ha.