CLASS VECTEUR 3D
barraq
Messages postés
89
Date d'inscription
mercredi 25 juin 2003
Statut
Membre
Dernière intervention
14 septembre 2007
-
2 mars 2005 à 21:58
brubru777 Messages postés 2 Date d'inscription lundi 7 novembre 2005 Statut Membre Dernière intervention 12 mai 2006 - 7 nov. 2005 à 02:18
brubru777 Messages postés 2 Date d'inscription lundi 7 novembre 2005 Statut Membre Dernière intervention 12 mai 2006 - 7 nov. 2005 à 02:18
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
https://codes-sources.commentcamarche.net/source/29879-class-vecteur-3d
https://codes-sources.commentcamarche.net/source/29879-class-vecteur-3d
7 nov. 2005 à 02:18
Pour un programme que j'ai fait récemment, j'ai eu besoin d'une classe vecteur. Mon code marchait plutôt bien à un petit détail près. J'ai eu un petit problème avec la surcharge des opérateurs +, -, etc. et << que j'ai redéfini pour afficher les vecteurs en texte sous la forme "(x, y, z)".
Dans un premier temps, j'ai défini les opérateurs arithmétiques de la manière suivante :
vecteur operator+(const vecteur &v) {
vecteur tmp;
tmp.x = x + v.x;
tmp.y = y + v.y;
tmp.z = z + v.z;
return tmp;
}
Cela marchait très bien pour les calculs mais pas pour l'affichage. C'est-à-dire que pour afficher la somme de deux vecteurs, je devais faire :
v = v1 + v2;
cout<<v;
Dans le cas où je mettais juste "cout<<(v1 + v2);", j'avais un message d'erreur.
Pour résoudre ce problème, j'ai redéfini les opérateurs comme cela :
vecteur &operator+(const vecteur &v) {
vecteur &tmp;
tmp->x = x + v.x;
tmp->y = y + v.y;
tmp->z = z + v.z;
return *tmp;
}
Cette solution semble fonctionner aussi bien pour le calcul que pour l' affichage. Cependant, je me demande si elle est bien correcte. Car si j'ai bien compris, on transmet juste l'adresse de la variable tmp et celle-ci est détruite juste après l'opération. Si c'est le cas, ça marcherait donc juste par hasard parce qu'aucun autre processus ne serait venu altérer la mémoire à cet endroit-là.
Est-ce que quelqu'un pourrait m'expliquer ce qui ne marche pas avec la première méthode et pourquoi la deuxième méthode qui me semble incorrecte marche?
D'avance merci.
5 mars 2005 à 11:19
4 mars 2005 à 22:57
4 mars 2005 à 22:57
#ifdef DEBUG
if(nul == 0)
cout << "Division de vecteur par 0!" << endl;
#endif
L'effet, ce sera d'avertir le programmeur pdt le débuggage, et puis ben pr le reste il n'aura qu'à faire le test lui même, ça me paraît plus logique.
4 mars 2005 à 20:55
float operator * (Vector3D Vector)
{
return (x * Vector.x + y * Vector.y + z * Vector.z);
}
met ca:
float operator * (const Vector3D & Vector)
{
return (x * Vector.x + y * Vector.y + z * Vector.z);
}
C'est beaucoup plus rapide, car tous ceux qui programment sur openGL ont besoin de performance.
De plus, j'ai fais ma propre classe vecteur 3D, en template, ce qui me permet de modifier le type de données de mes vecteurs, genre int, float, double, etc ..., ou autre. Ca peut présenter un intérêt.
De plus, si je vais vecteur / 0 ? il se passe quoi ? il faut donc gérer tous les cas possible pour rendre une classe de vecteur utilisable par tout le monde. Et dernière chose, je pense qu'il vaut mieux séparer en .h et .cpp ou .inl si tu déclares les fonctions en inline.
Dernières chose, si V est un vecteur,
X 4*V, ta classe le gère ? ou alors il faut mettre X V*4
Aller ++ :)
3 mars 2005 à 18:36
je corrige sa sur mon code alors.
et je regarde http://www.cppfrance.com/code.aspx?ID=29029
cool ;)
3 mars 2005 à 17:34
3 mars 2005 à 17:24
3 mars 2005 à 17:03
Par exemple:
classe string
{
char *p;
public:
string & operator = (const string & s )
{
delete []p; // si auto asignation on libere le buffer avant
// de copier sont contenu
// new[] + copie
}
};
}
3 mars 2005 à 16:50
Vector3D a(...);
a = a; //c'est légal; idiot mais légal
Ce serait intéressant de "templatiser" tout ça pour qu'on puisse choisir de travailler avec des int / float / double / classe perso / ...
3 mars 2005 à 14:46
Pour l'affectation, il ne faut pas oublier le test if( this == &Vector )
3 mars 2005 à 13:54
Vector3D& operator=(const Vector3D& Vector)
{
x = Vector.x;
y = Vector.y;
z = Vector.z;
return *this;
}
de même un contructeur de copie qui est très utile pour ce genre de classe :
Vector3D(const Vector3D& Vector)
{
*this = Vector;
}
tout simplement, l'opérateur = fera le reste
tu peux regarder mes sources
http://www.cppfrance.com/code.aspx?ID=29029
tu y trouvera peut-etre un peu d'inspiration ;-)
bon début en tout cas
3 mars 2005 à 10:28
et aussi un petit truc, dans la redéfinition de l'operateur = par exemple, j'ai utilisé Vector3D(x,y,z) a la place the "this" car en fait quand j'utilise "this" sa marche pas :s
si quelqu'un sai pourquoi ....
2 mars 2005 à 22:51
:)
2 mars 2005 à 21:58
produit scalaire et produit vectoriel en anglais :D
( et on se fou pas de ma gueule :D )