Surchage opérateur d'assignation =

Version imprimable

Bonjour,

J'ai deux classes: class Point3D et class Vector3D. Je souhaiterais pouvoir convertir un objet d'un type vers l'autre (il suffit de recopier 3 valeurs en gros...). J'ai essayé de surcharger l'opérateur = dans les deux classes mais mon compilateur (Visual) me dit qu'il ne connaît pas Point3D dans la classe vecteur (pourtant j'ai bien #include "Point3D" dans ma classe Vecteur3D et #include "Vector3D" dans ma classe Point3D) mais on dirait que cette référence croisée, lui emmêle les pinceaux! Voici mes surcharges:
Code:

1 2 3 4 gPoint3D& operator =(const gVector3D& v); //Convert a 3D-vector to a 3D-point gVector3D& operator =(const gPoint3D& p); //affectation ==> erreur : "missing type specifier"/ donc il ne connaît pas gPoint3D
Est-ce que cela vous semble correct? Peut-on croiser les assignations?

Merci d'avance!
Cordialement,
Rodrigue

10/05/2006, 11h24
Matthieu Brucher

C'est normal si Vecteur3D.h inclut Point3D qui inclut Vecteur3D que lors de la déclaration de Point3D, il ne sache rien de Vecteur3D si tu as mis des #ifndef...
Fais une déclaration anticipée de la structure Vecteur3D et Point3D dans les headers opposés.
10/05/2006, 11h30
Rodrigue

Par "header opposés", tu veux dire que je mette:
#include "Point3D.h" avant #ifndef dans mon fichier "Vector3D.h" et,
#include "Vector3D.h" avant #ifndef dans mon fichier "Point3D.h"?
J'ai une erreur à la compilation: too many include files...
Ben ouais ça boucle! Point3D appelle, Vector3D qui appelle Point3D etc.
10/05/2006, 11h32
Rodrigue

Je ne dois pas mettre "extern class" qqchose... pour lui dire qu'elle existe?
10/05/2006, 11h45
Matthieu Brucher

Non, j'ai dit déclaration anticipée, donc un truc style :

Code:

class Pointeur3D;

dans Vecteur3D.h
Et tu as bien mis des protections contre les boucles d'include ? Avec les #ifndef ?
Par ce que vu ce que tu as indiqué, ce n'est pas le cas.
10/05/2006, 11h54
Rodrigue

Oui oui j'en ai...
en fait c'est :

Code:

#pragma once
10/05/2006, 11h58
Rodrigue
Magnifique ça fonctionne :yaisse2:, j'ai réussi à compiler :king: !
T'es le meilleur :ave:...

Encore un petit problème!
Quand j'essaye d'affecter un Point3D à un Vecteur3D, il ne veut pas 8O :

Citation:

cannot convert from 'gPoint3D' to 'gVector3D'

Voici mes déclarations dans ma classe Vector3D:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 gVector3D& operator =(const gPoint3D& p); //affectation /** *: = operator **/ gVector3D& gVector3D::operator =(const gPoint3D& p) { x = p.x; y = p.y; z = p.z; return (*this); }
Il devrait pouvoir me convertir mon Point3D sans problème, non?
En fait je voudrais être capable de caster mon Point3D dans un calcul où j'ai besoin d'un produit scalaire (qui n'est déclaré que pour mes vecteurs)...
10/05/2006, 12h08
Rodrigue
:mouarf3: Ne te dérange pas ... Il fallait définir une constructeur du type:
Code:

1 2 3 gVector3D(const gPoint3D& Point);
Encore Mille fois Merci :hola:!
10/05/2006, 12h46
Matthieu Brucher
Citation:

Envoyé par Rodrigue

Oui oui j'en ai...
en fait c'est :

Code:

#pragma once

Mais ce n'est pas portabl eet risqué.
Il faut faire des :
Code:

1 2 3 4 5 6 #ifndef DANSMONHEADER_MACHIN_H #define DANSMONHEADER_MACHIN_H //Mon code #endif
10/05/2006, 13h19
r0d

Personnellement, je conseille de mettre les deux.
cf. http://www.developpez.net/forums/showthread.php?t=16767
10/05/2006, 14h22
Rodrigue
Et zut! Je me suis amusé à changer tous mes
Code:

1 2 3 4 5 #ifndef #define ... #endif
en

Code:

#pragma once

Bon ben on recommence :aie: !

Merci pour l'astuce. Je vais mettre les deux alors...
10/05/2006, 14h43
Laurent Gomila

Bof... A mon avis, le temps gagné en plaçant des #pragma once supplémentaires ne doit pas être transcendant.
10/05/2006, 15h23
r0d

Ca dépend de la taille du projet, et, bien sûr, du compilo. Mais bon, c'est clair que tant que le projet ne contient pas plus de, disons, une centaine de fichiers, la différence doit être dérisoire.

P.S: je ne trouve pas de comparatif sur le net :?
11/05/2006, 02h25
loufoque

Citation:

Quand j'essaye d'affecter un Point3D à un Vecteur3D, il ne veut pas 8O :

Tu ne réalises probablement pas une vraie affection.
Quelle est l'opération en question ?
11/05/2006, 09h14
Rodrigue

Merci Loufoque pour ton aide mais la solution a déjà été trouvée (cf mon message de 12h08). Je pensais avoir mis le message a résolu, je m'excuse! - programmer nuit gravement à la santé mentale -.

J'ai créer une constructeur dans ma classe Vector3D qui prend en paramètres un Point3D. Cela me permet de caster un point3D en vector3D très facilement dans mes formules. J'ai fait l'inverse dans ma classe Point3D, comme ça je peux faire des conversions à la volée... C'est géant ;)

Merci beaucoup pour votre aide!