Refactoriser du code existant

Version imprimable

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:

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

Un truc comme ça ?

Code:

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

24/04/2012, 14h01
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é ?
24/04/2012, 14h12
JoeChip

Ca dépend du degré de couplage désiré.
24/04/2012, 14h20
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 ?
24/04/2012, 14h27
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.
24/04/2012, 14h48
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.