Discussion :

[Micke7]

Bonjour,

J'ai créé une classe qui hérite de la classe Vector dans laquelle j'ai redéfinit les méthodes add() et remove().

Voilà son code

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
20
21
22
23
24
25
26
27
28
29
30
 
public class VectorCouchesSupplementaires extends Vector<Layer>{
	/**
         * 
         */
	private static final long serialVersionUID = 4526120306651519039L;
 
	public VectorCouchesSupplementaires(){
	}
 
	public boolean remove(Layer layer){
		super.remove(layer);
		//Mise à jour du tableau des couches supplémentaires
		JTablePerso.updateTableaux();
 
		//Mise à jour de la table couches supplémentaires
		Execution.UpdateVisibiliteCouche(false,layer.getName().trim(),LayerName.siret,LayerName.mlsm);
		return true;
	}
 
	public boolean add(Layer layer){
		super.add(layer);
		//Mise à jour du tableu des couches supplémentaires
		JTablePerso.updateTableaux();
 
		//Mise à jour de la table couches supplémentaires
		Execution.UpdateVisibiliteCouche(true,layer.getName().trim(),LayerName.siret,LayerName.mlsm);
		return true;
	}
}

Ma question est assez simple, comment empêcher que dans une classe autre, il soit possible d'appeler la méthode monVecteurPerso.remove(parametre) avec un paramètre qui ne soit pas du type Layer? Cette interdiction fonctionne très bien avec la méthode add(), mais pour remove(), si le paramètre n'est pas de type Layer, c'est la méthode remove() de la classe Vector qui est appelée...

Merci beaucoup.

[tchize_]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
        @override
	public boolean remove(Object o){
                if (!o instanceof Layer)
                    throw new IllegalArgumentException("can only remove layers");
                Layer layer = (Layer)o;
		super.remove(layer);
		//Mise à jour du tableau des couches supplémentaires
		JTablePerso.updateTableaux();
 
		//Mise à jour de la table couches supplémentaires
		Execution.UpdateVisibiliteCouche(false,layer.getName().trim(),LayerName.siret,LayerName.mlsm);
		return true;
	}

[Micke7]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
        @override
	public boolean remove(Object o){
                if (!o instanceof Layer)
                    throw new IllegalArgumentException("can only remove layers");
                Layer layer = (Layer)o;
		super.remove(layer);
		//Mise à jour du tableau des couches supplémentaires
		JTablePerso.updateTableaux();
 
		//Mise à jour de la table couches supplémentaires
		Execution.UpdateVisibiliteCouche(false,layer.getName().trim(),LayerName.siret,LayerName.mlsm);
		return true;
	}

Merci pour ton aide tchize_, mais en fait cela ne correspond pas exactement à ce je que je souhaiterais. En fait je voudrais rendre impossible la compilation s'il existe quelque part dans l'appli, la méthode monVecteurPerso.remove(object) avec object qui n'est pas de type Layer.
Je suis assez clair dans mes explication ?

[Rei Ichido]

Il n'est pas possible de réduire la visibilité d'une méthode héritée d'une classe mère ; donc la méthode de Tchize est la seule viable.

Par contre tu peux la passer en Deprecated, et avoir en plus un remove qui prend en argument un Layer, ce qui fournira une information aux gens qui l'utiliseraient sur un Objet qu'il vaut mieux passer par la méthode qui prend en argument un Layer, tout en gardant la fonctionnalité.

Edit : et ça permet via les warnings de savoir où ça peut être utilisé dans l'appli.

[tchize_]

ces méthodes ne permettront de toutes facons pas de détecter ceci:

Code :

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

1
2
3
4
 
VectorCouchesSupplementaires vect = ...;
Vector v = vect;
v.remove(objet)

Au moins la surcharge avec un IllegalArgumentException permet de détecter l'erreur at runtime.

Note que vouloir désactiver une méthode du parent est généralement dangereux car, au final, tu ne respecte plus le contrat du parent. Dans ton cas, il vaudrait peut etre mieux avoir remove(object) qui traite spécialement le cas des layer et délèhue juste au parent els autres cas.

[dingoth]

Code :

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

1
2
3
public final void methodePlusSupportee () {
  throw new UnsupportedOperationException();
}

À la compilation, tu ne peux le faire, mais dans le code, c'est ce qu'il y a de plus propre (avec la doc adéquate, bien sur).

En clair:
* ajouter le keyword final
* lancer une UnsupportedOperationException();