[1.5] Vector et héritage

**Piolet** · 14/09/2007, 09h49

Bonjour,

alors, je vais essayer de faire clair.
Mon problème est le suivant :

J'ai une classe, Controller pour ne pas la nommer, qui fait plein de choses, gère des évènements sur ses propriétés etc...bref...qq truc sympas qui m'evite de les refaires a chaque fois. Cette classe n'est pas abstraite, (mais a priori, si besoin elle peut le devenir

J'ai une autre classe, ControllerList qui, elle, hérite de Vector<Controller>.
j'ai réimplémenté qq méthodes, comme add et remove, pour y rajouter un comportement particulier.

tout ca marche très bien, mais la ou ca fait ce que je veux, c'est quand je fais une classe qui hérite de mon Controller, par exemple MyController, sur laquelle je rajoute qq méthodes mais le fonctionnement de l'objet reste le meme que celui d'un Controller, et une autre classe qui hérite de mon ControllerList, MyControllerList.

En fait j'aimerais que MyControllerList soit en fait un ControllerList<MyController> pour que quand je fasse ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
MyControllerList myList = new MyControllerList();
MyController controller = new MyController();
myList.add(controller);
 
//a la récupération je veux pouvoir récupérer un MyController directement, 
//sans cast, comme ceci :
controller = myList.get(0);
 
//or la je suis obligé de casté, car MyControllerList héritant de ControllerList,
//il me renvoie constament un Controller
controller = (MyController) myList.get(0);

voila...en gros je veux éviter le cast, en profitant de l'héritage de ControllerList.

Je sais pas si j'ai été tres clair

voila merci d'avance

**le y@m's** · 14/09/2007, 10h06

Depuis Java 1.5 il est possible de modifier le type de retour d'une méthode redéfinie, cf La covariance dans le JDK1.5.

Il te suffit donc de redéfinir la méthode get() de la classe MyControllerList.

Code :

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

1
2
3
4
@Override
public MyController get(int index) {
    return (MyController) super.get(index);
}

Par contre attention aux diverses méthodes d'ajout qui permettent toujours d'ajouter un Controller qui n'est pas un MyController ce qui déclencherait une erreur lors de la récupération de l'élément.

**Piolet** · 14/09/2007, 10h12

je voulais justement eviter de redefinir chaque méthode de l'héritant, je pensais qu'avec la definition bien faite, ca suffirait a faire en sorte que le get me renvoie le bon objet.

mais si c'est la seule méthode...

**le y@m's** · 14/09/2007, 10h40

Ce que tu peux faire c'est laisser les générics au type ControllerList.

Code :

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

ControllerList<T extends Controller> extends Vector<T>

Code :

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

MyControllerList extends ControllerList<MyController>

Ce qui donne

Code :

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

1
2
3
4
MyControllerList myList = new MyControllerList();
MyController controller = new MyController();
myList.add(controller);
controler = myList.get(0);

ou si tu veux pouvoir étendre MyControllerList :

Code :

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

MyControllerList<T extends MyController> extends ControllerList<T>

Ce qui donne

Code :

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

1
2
3
4
MyControllerList<MyController> myList = new MyControllerList<MyController>();
MyController controller = new MyController();
myList.add(controller);
controler = myList.get(0);

**Piolet** · 14/09/2007, 10h44

voila..c'est ca que j'ai essayé et qui n'a pas marché....mais ma syntaxe ne devait pas etre bonne...

merci bcp...j'essaie ca des que possible

**Piolet** · 14/09/2007, 12h11

alors voila ce que ca me fait, parce que moi, dans mon ControllerList j'ai redefini qq methodes comme :

Code :

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

1
2
3
4
5
6
7
 
@Override
public Controller remove(int index) {
   Controller controller = this.get(index);
   controller.setState(State.DELETED);
   return controller;
}

alors que j'ai bien mis la signature de ControllerList comme tu précisais :

Code :

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

1
2
 
ControllerList<T extends Controller> extends Vector<T>

il va me donner ca comme erreur

Code :

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

1
2
3
Multiple markers at this line
-the return type is incompatible with Vector<T>.remove(int)
-overrides java.util.Vector<T>.remove

et si je dis a ma méthode remove de renvoyer un objet de type T, bah...il admet pas que je fasse :

Code :

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

1
2
3
4
5
6
7
 
@Override
public T remove(int index) {
   Controller controller = this.get(index);
   controller.setState(State.DELETED);
   return controller;
}

parce que controller n'est pas de type T..normal

**adiGuba** · 14/09/2007, 12h16

Salut,

Si tu utilises le type abstrait T tu dois l'utiliser partout :

Code :

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

1
2
3
4
5
6
7
 
@Override
public T remove(int index) {
   T controller = this.get(index);
   controller.setState(State.DELETED);
   return controller;
}

a++

**Piolet** · 14/09/2007, 12h25

ah bah oui...ca passe mieux

nickel merci bcp

**Piolet** · 14/09/2007, 12h44

alors vais faire mon chieur

ca marche nickel dans la déclaration, par contre a l'utilisation...j'ai qq soucis...

alors comment expliquer

par exemple :

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
 
//Controller est un peu comme ca
class Controller {
   private Hashtable<String, Controller> _children;
   private Hashtable<String, ControllerList> _childrenList;
 
   public void addChildren(String name, Contoller controller) {
      this._children.put(name, controller);
   }
 
   public void addChildrenList(String name, ContollerList controllerList) {
      this._childrenList.put(name, controllerList);
   }
 
}

Code :

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

1
2
3
4
5
6
 
class Consultations<T extends Consultation> extends ControllerList<T> { 
// Consulation héritant de Controller
 
//qq méthodes bateau de calcul etc...
}

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
class Patient extends Controller { //(donc c'est un MyController)
   public Patient() {
      super.addChildrenList("consultations", new Consultations());
      super.addChildren("coordinate", new Coordinate());
   }
 
   public Consultations getConsultations() {
      return (Consultations) super.getChildrenList("consultations");
   }
}

et la j'ai plein de warning au niveau de l'ajout de mes consultations dans Patient, mais aussi au niveau de la liste de consulation dans Consultations...il considère ca comme un objet.

moi en gros...je veux faire un truc super générique a la base, en l'occurence ma classe Controller, et au premier plan, faire qq chose de commun, mais reprenant tout le tps le meme traitement. Donc, oui mes signatures sont différentes, mais pour le traitement que fait Controller, tous mes objets héritant peuvent au finale redevenir des Controller classiques...mais la..je bloque un peu

**le y@m's** · 14/09/2007, 14h33

Premier point, tu dois spécifier le type de tes classes utilisant les générics.
Par exempleau lieu de :

Envoyé par Piolet

Code :

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

private Hashtable<String, ControllerList> _childrenList;

il faut

Code :

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

private Hashtable<String, ControllerList<Controller>> _childrenList;

Ceci est valable pour toutes les classes utilisant les générics.

Ensuite pour la méthode

Code :

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

1
2
3
public Consultations<Consultation> getConsultations() {
    return (Consultations<Consultation>) super.getChildrenList("consultations");
}

Tu auras toujours un warning car tu cast un ControllerList<Controller> en Consultations<Consultation>.
Ce "problème" a déjà été abordé plusieur fois sur le forum. Voir également dans la FAQ :