Discussion :

[tamerla]

Salut,
j'ai ce code :

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
36
 
class Panier {
   Fruit[] t;
 
   public Panier() {
      this.t = new Fruit[0];
   }
 
   public Panier(Panier p) {
      this.t = new Fruit[p.t.length];
      for(int i = 0; i < p.t.length; i++) {
         this.t[i] = p.t[i];
      }
   }
 
   public Panier(Fruit[] f) {
      this.t = new Fruit[f.length];
      for(int i = 0; i < f.length; i++) {
         this.t[i] = f[i];
      }
   }
 
   public Panier(Fruit f, Panier p) {
      if(p==null) {
         this.t = new Fruit[1];
         this.t[0] = f;
      }
      else {
         this.t = new Fruit[p.t.length+1];
         for(int i = 0; i < p.t.length; i++) {
            this.t[i] = p.t[i];
         }
         this.t[p.t.length] = f;  
      }
   }
}

Mon probleme est au niveau des deux constructeurs ayant pour arguments panier p et l'autre Fruit[] f, en fait je ne comprends vraiment la différence entre les deux,
qu est ce que ca change de mettre un objet de type panier en argument par rapport à lattribut Fruit[] f ?
merci de m'éclairer

[tchize_]

Ca change que l'appelant n'a pas besoin d'aller sortir lui même les Fruit du Panier, chose qu'il ne sait peut être pas faire d'ailleurs.


Note que le code aurait pu être plus concis:

Code :

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

1
2
3
   public Panier(Panier p) {
      this(p.t);
   }

[tamerla]

ok merci , et qu'est ce que ca veut dire

Code :

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

this(p.t);

exactement?

[tchize_]

ça délègue à un autre constructeur

[kolodz]

Cela fait appel au polymorphisme du constructeur.

Code :

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

this(p.t);

va faire appel au constructeur correspondant (car t est de type Fruit[]) :

Code :

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

 public Panier(Fruit[] f)

Note : Il faudrait malgré tout vérifier si p n'est pas null avant. :

Code :

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

1
2
3
4
5
      if(p==null) {
         this(p.t);
      } else {
         this();
      }

D'aileurs, il est possible de faire la même simplification sur :

Code :

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

public Panier(Fruit f, Panier p)

En utilisant :

Code :

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

this(p);

Puis, un ajout.

Cependant, je ne pense pas que la structure tableau ne me semble pas forcément indiqué dans ton cas. Car, l'ajout et la suppression d'élément nécessite la création d'un nouveau tableau. Un objet List me semble plus adapté.

Cordialement,
Patrick Kolodziejczyk.
source :
http://fr.wikibooks.org/wiki/Program.../Polymorphisme

[tchize_]

Note : Il faudrait malgré tout vérifier si p n'est pas null avant. :

Code :

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

1
2
3
4
5
      if(p==null) {
         this(p.t);
      } else {
         this();
      }

Et non, ce code ne marchera pas. Les appels this() et super() doivent être la première instruction d'un constructeur. Ils ne peuvent pas être ailleurs. Pour tester le null, au mieux on peux faire:

Code :

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

this(p==null?null:p.t);

[kolodz]

En effet.
Merci du rappel.