Discussion :

[SrK]

Salut,

Je voulais factoriser du code existant.

Comment procéderiez vous ?

J'avais quasi le même traitement exécuté à deux trois choses près.

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
public class One {
  private A a = new A();
  private B b = new B();
}
 
public class Two {
  private A a = new A();
  private Bdetailed b = new Bdetailed();
  private String other;
}
 
public class B {
  private final Custom custom = new Custom();
}
 
public class Bdetailed {
  private final Custom custom = new Custom();
  private final Custom custom2 = new Custom();
}

Je voulais donc faire de l'héritage avec Bdetailed qui hériterai B pour éviter de dupliquer du code.
Puis faire de l'héritage de Two qui hériterai de One.

Le truc c'est que je ne sais pas vraiment quoi faire des attributs des classes parentes ? je mets protected ou je mets un accesseur ? ( pour les attributs de B et Bdetailed )

Pour one et two, vu que A et B seront communes, je peux avoir la base dans la classe One, et juste rajouter mon attribut de type String dans B ?

Merci

[JoeChip]

Un truc comme ça ?

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
public class One {
  protected A a = new A();
  protected B b = new B();
}
 
public class Two extends One{
  private String other;
  public Two(){
    b = new Bdetailed();
  }
}
 
public class B {
  protected final Custom custom = new Custom();
}
 
public class Bdetailed extends B{
  private final Custom custom2 = new Custom();
}

[SrK]

Oui, c'est exactement ca, merci.

J'hésite entre ca et mettre des accesseurs pour accéder aux champ "protected" qui serait alors privé ?

[JoeChip]

Ca dépend du degré de couplage désiré.

[SrK]

En gros, si je veux donner tous les droits à la classe fille je mets l'attribut en protected, sinon je gère les accesseurs entre private/protected en fonction des besoins ?

[JoeChip]

Lis ça :

http://codemonkeyism.com/generation-...ramming-style/

En gros, c'est tellement facile (avec Refactor dans Eclipse, par exemple) d'ajouter des accesseurs si un jour tu en as besoin, que ça ne vaut plus trop la peine de le faire systématiquement.

[SrK]

Ok ca marche.

Par contre, le fait de ne pas spécifier le type hérité dans la classe B, ca force à caster à chaque appel.