Discussion :

[JamesP]

Bonjour,

J'ai une question assez simple mais qui me turlupine ...

Question éthique et question performances, pour éviter de recopier du code dans différents constructeurs, mieux vaut-il faire :

Code :

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

1
2
3
4
5
6
7
8
public class MaClasse {
  public MaClasse() {
    new MaClasse( "Pas d'argument !");
  }
 
  public MaClasse( String maChaine) {
    System.out.println( maChaine );
  }

ou alors

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
public class MaClasse {
  public MaClasse() {
    affiche( "Pas d'argument !" );
  }
 
  public MaClasse( String maChaine ) {
    affiche( maChaine );
  }
 
  public void affiche ( String maChaine ) {
    System.out.println( maChaine );
  }

Vaut-il mieux faire une méthode qui fait la construction ou alors appeler les constructeurs entre eux ? Dans le cas n°1 est-ce que ca instancie 2 fois l'objet ?

Bien sur dans des cas avec de GROSSES CLASSES !!!

Merci de vos lumières ^^

[natha]

Ta solution 1 n'est pas correcte car tu construis un autre objet MaClasse pendant la construction de ton objet. L'output peut être correct mais le principe est faux.

Il y a pour moi 2 solutions propres :

La première : les constructeurs appellent le constructeur le plus détaillé :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
public class MaClasse {
  public MaClasse() {
    this( "Pas d'argument !" );
  }
 
  public MaClasse( String maChaine ) {
    affiche( maChaine );
  }
 
  private void affiche ( String maChaine ) {
    System.out.println( maChaine );
  }

La seconde utilise un Builder et ne vaut que si tu as beaucoup beaucoup de paramètres optionnels (je ne fais l'exemple qu'avec un paramètre) :

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
 
public class MaClasse {
  public MaClasse(Builder builder) {
    affiche(builder.chaine);
  }
  private void affiche ( String maChaine ) {
    System.out.println( maChaine );
  }
 
  public static class Builder {
    String chaine = "Pas d'argument !";
    public Builder maChaine(String maChaine) {
      this.chaine = maChaine; return this;
    }
  }
}

A construire comme suit :

Code :

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

1
2
MaClasse maClasse = new MaClasse.Builder();
MaClasse maClasse = new MaClasse.Builder().maChaine("ma chaine");

[adiGuba]

Salut,

+1 pour la solution 1 corrigé par natha !

Par contre la méthode affiche() devrait être déclaré en private de préférences (les constructeurs ne devrait pas appeler de méthodes non-private ou non-final pour éviter certain effet de bord)


a++

[natha]

Par contre la méthode affiche() devrait être déclaré en private de préférences (les constructeurs ne devrait pas appeler de méthodes non-private ou non-final pour éviter certain effet de bord)

Bonne remarque effectivement. J'me suis concentré sur son problème de constructeur

[JamesP]

Oki merci ...

Pour le "public" désolé j'ai tapé les 2 bouts de code vite fais ... Mais oui bien mettre un private !!!

Sinon j'avais jamais pensé au this( ... );

J'ai honte de moi la ... Comme quoi ...

En tous cas merci bcp !!! ^^