Discussion :

[gloglo]

J’ai ce code que je souhaite améliorer, merci de bien vouloir m’aider

Application 1
L’application 1 contient deux package pacInterface1 et pacObject1. Le premier est composé d’interface et le deuxième de classes qui implémentent ces interfaces. Ainsi on y trouvera une classe par interface.

Exemple ObjectPack1 implements InterfacePack1.

Application 2 (celle qui m’intéresse) contient pareil deux packages pacInterface2 et pacObject2.
Voici ce que j’ai :

Code :

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

InterfacePack2 extends InterfacePack1 (De l’application 1).

Et

Code :

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

1
2
3
4
5
6
7
8
9
ObjectPack2 implements InterfacePack2 {
	public ObjectPack1 objectPack1  ;
 
	public objectPack2 (objectPack1  objectPack1 ) {
       		 super(objectPack1 );
      		  this. objectPack1  = objectPack1 ;
 	   }
 
}

Ici, ce n’est pas logique qu’il y est une agrégation et un héritage au même temps, il faudra virer l’un ou l’autre.
Que pensez vous, je vire l’agrégation ou l’héritage ???

Merci

[Invité]

salut,

Exemple ObjectPack1 implements InterfacePack1.

Application 2 (celle qui m’intéresse) contient pareil deux packages pacInterface2 et pacObject2.
Voici ce que j’ai :
InterfacePack2 extends InterfacePack1 (De l’application 1).
Et
ObjectPack2 implements InterfacePack2 {
public ObjectPack1 objectPack1 ;

public objectPack2 (objectPack1 objectPack1 ) {
super(objectPack1 );
this. objectPack1 = objectPack1 ;
}

}

bon il existe plusieurs solutions.
ici si ObjectPack2 et ObjectPack1 implimentent les fonctions de l'interface InterfacePack1 de la même façon il serais logique de supprimer l'héritage car il y aura une redondonce et tu crée une instance de ObjectPack1 dans ObjectPack2 ou bien tu peut fusionner les 2 interfaces InterfacePack1 et InterfacePack2et les 2 classes ObjectPack2 et ObjectPack1.
sinon s'ils implimentent les fonctions de InterfacePack1 de manière différentes alors là tu peut faire ça

Code :

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

 ObjectPack2 implements InterfacePack2, InterfacePack1

en tout cas ça reste difficile de répondre car ça dépend de ta conception.

[gloglo]

le truc c'est que ici, en faisant cet héritage dans mon application2, je suis obligé de ré implémenter des méthodes de interfacePack1 dont je n'est pas besoin et du coup je fais par exemple ce ci

Code :

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

1
2
public void setX(int x0){
}

ou encore

Code :

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

1
2
3
public int getX(){
  return 0;
}

je me dis que si j'utilise l'agrégation et si j'enleve l'héritage alors de cette façon j'implémente uniquement les méthodes dont j'ai besoin sauf que je ne sais pas côté conception si c'est mieux ou non.

[Rayndell]

Pour éviter de ré-implémenter les méthodes de l'interface 1 il faut de toutes façons supprimer l'héritage. L'idéal pour moi serait de faire comme ceci :
- Créer une classe abstraite contenant les méthodes communes à ObjectPack1 et ObjectPack2,
- Faire hériter ces deux classes de la classe abstraite (en surchargeant éventuellement les méthodes de la classe maman),
- Implémenter les classes concrètes par leur interface correspondante.

[gloglo]

cette classe abstraite je la crée dans l'application 1 ??
en fait l'application 1 est une application qui commune à plusieurs applications dont la 2, c'est comme une sorte d'interface netre les differentes applications du SI et donc je ne pourrais pas y toucher car ceci impactera les autres applications qui sont gérés par les autres personnes.

[Rayndell]

La classe abstraite est propre à ton application, donc crée là plutôt dans la 2. Mais le fait de la créer dans la 1 n'influera pas sur l'application 1, au pire elle ne lui servira à rien. Mais on ne sait jamais, peut-être que d'autres personnes en auront besoin. A toi de voir.