Discussion :

[deubelte]

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 :

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
 
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 :

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
 
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);
 
}

[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)

[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.

[JolyLoic]

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 :

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

v.vecteurnormmax(v2+v3);

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

[Bakura]

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 :

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

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 :/.

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

[smashy]

le mieux a mon avis , c est (compilation des differentes reponses)

Code :

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

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

[deubelte]

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

Exact!!!


Que signifie le mot clé: Friend?