Discussion :

[houlouk]

Bonjour,
Je réalise un logiciel en java/swing en suivant le pattern mvc.
Je fais communiquer la vue et le modèle en utilisant cette méthode.
Le problème étant que lorsque le nombre de variables à écouter devient important, c'est assez lourd.
J'ai réussi à rendre les méthodes addListener, removeListener et getListeners génériques mais j'obtiens un warning (type safety) :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
	public void addListener(Class listenerClass,EventListener listener){
		listeners.add(listenerClass, listener);
 
	}
 
	public void removeListener(Class listenerClass, EventListener listener){
		listeners.remove(listenerClass, listener);
	}
 
	public SpeedSimulationListener[] getListener(Class listenerClass){
		return listeners.getListeners(listenerClass);
	}

Edit: j'ai finalement résolu ce problème

Pour les méthodes fire, je me demandais si il était possible de les ajouter automatiquement avec eclipse (comme on fait pour les getters et les setters).
Merci d'avance.

[tchize_]

Le warning viens du fait que ton paramètre Class n'est pas typé alors que c'est une paramètre générique:

Code :

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

public void addListener(Class<?> listenerClass,EventListener listener){

voir pour bien typer l'animal:

Code :

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

public void <T extends EventListener> addListener(Class<T> listenerClass,T listener){

Si tu as déjà ces méthodes add/remove/get sur un conteneur "listeners", tu peux juste faire dans le menu source => generate delegate methods

[houlouk]

Merci ça fonctionne

Pour générer automatiquement, je parlais des méthodes fire :

Par exemple si je déclare une variable dans mon modèle :

Code :

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

1
2
 
int speedSimulation;

Je voudrais pouvoir créer automatiquement la méthode (ainsi que les interfaces nécessaires)

Code :

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

1
2
3
4
5
6
7
 
protected void fireSpeedSimulationChanged(int speedSimulation) {
		for (SpeedSimulationListener listener : getSpeedSimulationListener(SpeedSimulationListener.class)){
			listener.speedSimulationChanged(speedSimulation);
		}
 
}

[tchize_]

Si tu commence à mettre un listener sur chaque propriété, tu ne va pas t'en sortir. Si tu veux surveiller l'état d'un objet, fait plutot un listener plus global genre SimulationStateChanged.

[houlouk]

C'est ce que je fais d'habitude mais du coup je suis obligé de relire tout l'état même ce qui n'a pas été modifié.

[tchize_]

Et si tu met un listener sur chaque propriété, tu va créer énormément de bruit dans ton code. Sans parler du spaghetti que ça va vite devenir avec les event qui modifient qui lance d'autres event qui modifient et tu va vite être coincé par des boucles d'event, donc encore plus de code pour en sortir. Et je ne parle même pas de la galère à maintenir chaque changement.

[houlouk]

Je parlais plus au niveau des performances mais c'est vrai que c'est plus simple comme ça.
Merci à toi.