classe vecteur3D, passage par référence

Version imprimable

J'ai fait une classe vecteur3d qui permet de renvoyer le vecteur qui a la norme maximum. L'un est passage par référence, l'autre par valeur.
Qu'en pensez vous?

Code:

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
 
class vecteur3d {
private:
    float x,y,z;
public:
 
      vecteur3d(float,float,float);
      vecteur3d();
      int vecteurnormmax(vecteur3d);
};
 
vecteur3d::vecteur3d(){
x=y=z=0;}
 
vecteur3d::vecteur3d(float t,float tt,float ttt){
x=t;y=tt;z=ttt;}
 
int vecteur3d::vecteurnormmax(vecteur3d v){
double norme1= sqrt(x*x+y*y+z*z);
double norme2= sqrt(v.x*v.x+v.y*v.y+v.z*v.z);
int a=1;
if (norme2<norme1) a=2;
return a;
}
 
void main(){
vecteur3d X(2,1,3);
vecteur3d Y(1,4,2);
cout<<Y.vecteurnormmax(X);
 
}

Passage par référence:

Code:

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
 
class vecteur3d {
private:
    float x,y,z;
public:
 
      vecteur3d(float,float,float);
      vecteur3d();
      int vecteurnormmax(vecteur3d * );
};
 
vecteur3d::vecteur3d(){
x=y=z=0;}
 
vecteur3d::vecteur3d(float t,float tt,float ttt){
x=t;y=tt;z=ttt;}
 
int vecteur3d::vecteurnormmax(vecteur3d *v){
double norme1= sqrt(this->x*this->x+this->y*this->y+this->z*this->z);
double norme2= sqrt(v->x*v->x+v->y*v->y+v->z*v->z);
int a=1;
if (norme2<norme1) a=2;
return a;
}
 
void main(){
vecteur3d X(2,1,3);
vecteur3d Y(1,4,2);
cout<<Y.vecteurnormmax(&X);
 
}

20/05/2007, 18h24
Bakura

Salut !

Corrigez moi si je me trompe, mais d'après moi tu ne passes pas par référence ici, tu passes un pointeur vers un objet. Le passage par référence se fait comme ceci :

int vecteur3d::vecteurnormmax(vecteur3d &v)

Quand tu appeleras la fonction, tu ne passeras donc pas l'adresse de l'objet mais une référence vers l'objet :

int machin = v.vecteurnormmax (v2); // Pas de &v2

C'est la meilleure solution, ça t'évite d'avoir un appel au constructeur quand tu passes par valeur, et ça t'évite la syntaxe des pointeurs :

double norme2= sqrt(v.x*v.x+v.y*v.y+v.z*v.z); // En gros tu utilises comme si tu passais par valeur, sauf que c'est avec une référence ^^.

Deuxième remarque, les this->x, c'est inutile ! Fait tout simplement x.

Sinon, personnellement je trouve ça assez étrange comme syntaxe pour déterminer quel est le vecteur à la plus grande norme de faire une fonction membre, je trouve ça plus logique de faire une syntaxe comme ceci :

PlusGrandeNorme (v1, v2), plutôt que v1.PlusGrandNorme (v2)
20/05/2007, 18h46
deubelte

[/QUOTE]
Corrigez moi si je me trompe, mais d'après moi tu ne passes pas par référence ici, tu passes un pointeur vers un objet. Le passage par référence se fait comme ceci :int vecteur3d::vecteurnormmax(vecteur3d &v)
[/QUOTE]

Non, par référence, ca se fait avec *. Avec &, cela s'appelle par adresse.

[/QUOTE]
Sinon, personnellement je trouve ça assez étrange comme syntaxe pour déterminer quel est le vecteur à la plus grande norme de faire une fonction membre, je trouve ça plus logique de faire une syntaxe comme ceci :

PlusGrandeNorme (v1, v2), plutôt que v1.PlusGrandNorme (v2)
[/QUOTE]

C'était un exo ou il fallait queje fasse une fonction membre qui compare les normes.
20/05/2007, 18h47
JolyLoic

Citation:

Envoyé par Bakura

Le passage par référence se fait comme ceci :

int vecteur3d::vecteurnormmax(vecteur3d &v)

[...]
C'est la meilleure solution, ça t'évite d'avoir un appel au constructeur quand tu passes par valeur, et ça t'évite la syntaxe des pointeurs :

Ce n'est pas la meilleure solution, la meilleure solution est le passage par référence constante, qui a l'avantage de permettre de passer des objets constants, mais aussi des temporaires :

Code:

v.vecteurnormmax(v2+v3);

Ne marcherait pas avec des références non constantes.
20/05/2007, 19h07
Bakura

Citation:

Envoyé par JolyLoic

Ce n'est pas la meilleure solution, la meilleure solution est le passage par référence constante, qui a l'avantage de permettre de passer des objets constants, mais aussi des temporaires :

Code:

v.vecteurnormmax(v2+v3);

Ne marcherait pas avec des références non constantes.

Oui exact, en plus c'est ce que je fais à chaque fois en plus :/.

Citation:

Non, par référence, ca se fait avec *. Avec &, cela s'appelle par adresse.

Non, le passage par référence se fait avec l'opérateur & :).

int * i = &machin; // La c'est l'adresse

mais :

void MaFonction (const monObjet & obj); // La c'est par référence
21/05/2007, 13h44
smashy
le mieux a mon avis , c est (compilation des differentes reponses)
Code:

1 2 3 4 5 6 7 8 class vector3d { public: friend int vectormax(const vector3d & src1,const vector3d & src2){ ... } };
note:

tes sqrt sont inutiles dans ta comparaison car c est une fonction strictement croissante
21/05/2007, 13h57
deubelte

Citation:

tes sqrt sont inutiles dans ta comparaison car c est une fonction strictement croissante

Exact!!!

Que signifie le mot clé: Friend?
21/05/2007, 14h18
Laurent Gomila

Citation:

Envoyé par deubelte

Que signifie le mot clé: Friend?

Cela rend la fonction amie de ta classe, ce qui signifie qu'elle aura accès à ses données membres privées. Ce n'est pas obligatoire car j'imagine qu'il y a des accesseurs publiques pour accéder à x, y et z, ou directement à la norme.