Organisation du code

Version imprimable

29/03/2011, 09h07
RomG7

Organisation du code

Bonjour à tous,

Je reviens vers vous car j'ai des problème dans l'organisation de mon code.
Je souhaite développer une application java + MySQL avec le pattern MVC.

J'ai découpé mon code en 3 parties :
Pour commencer, j'ai créé la "Frame" avec une barre de menu.
Ensuite au nord de la Frame, j'ai construit une barre d'outil me permettant de faire une recherche avec un ou plusieurs critères.
Enfin au centre de la Frame, j'ai mis en place différents "Panel" qui doivent s'initialiser en fonction des critères de recherche.

Chaque élément possède son propre controller :
- un controller principal pour la Frame
- un controller pour le panel de recherche
- un controller pour chaque panel au centre de la fenêtre.

L'objectif étant que sur changement d'un critère de recherche, le panel situé au centre de la fenêtre s'actualise automatiquement.
Le problème ici est que le controller de la Frame ne connais pas le panel affiché au centre de la fenêtre.
Des pistes, des solutions?
29/03/2011, 09h39
rtg57

Bonjour,

vous pouvez passer des reférences des autres classes aux différents constructeurs.

Je ne sais pas si cela va vous être utilise, mais regardez du côté de ce post:http://www.developpez.net/forums/d98...s-classe-mere/
A l'aide de ce système, j'ai réussi à donner à tous les composants des références sur des classes qui les intéressent.

@ bientôt...
29/03/2011, 10h00
RomG7

Merci pour ce tuyau, je ne connaissais pas cette écriture.

Mais dans mon cas, le controller de la Frame construit le controller du panel de recherche ainsi que le ou les controller panel central (panelCentral1, panelCentral2, ...) avec this en paramètre correspondant au controller de la frame.

Lorsque l'utilisateur va modifier un critère de recherche, comment le controller de la frame peut-il informer le panel central en cours d'exécution?

C'est un peu confus pour moi, le panel central connais la classe mère car il a sa référence en paramètre, mais la classe mère ne connais pas forcément le panel central en cours.
29/03/2011, 10h05
william44290

met ton code, on pourra surement mieux t'aider.

Mon code est assez long mais je vais essayer de le simplifier.
voici le code du controller de la fenêtre :

Code:

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
31
32
33
34
35
36
37
38
39
40
41
42
43
44
public class Controller { 
	private Connex connexion;
	private Model model;
	private View view;
 
 
	// CONTROLLER
	public Controller() {
		try {
			this.connexion = new Connex(); 
		} 
		catch (SQLException e) {}
		catch (Exception e) {}
 
		this.model = new Model();
		this.view = new View(this);
 
		this.view.setPanelRecherche(new ControlRecherche (this));
	}
 
 
	// GETTER AND SETTER
	public Connex getConnexion() {
		return connexion;
	}
 
	public Model getModel() {
		return model;
	}
 
 
	// EVENEMENT
	public void displayPanelCentral1 () {
		ControllerCentral1 controlCentral1 = new ControllerCentral1 (this);
		view.setPanelCentral(controlCentral1.getView());
	}
 
        public void displayPanelCentral2 () {
		ControllerCentral2 controlCentral2 = new ControllerCentral2 (this);
		view.setPanelCentral(controlCentral2.getView());
	}
 
        ...
}

Voici le code de mon controller frontal qui contruit la classe View qui étend de JFrame.
Ensuite les autres controller contruisent des vues qui étendent de JPanel.

29/03/2011, 10h46
william44290

bon, c'est un peu synthétique et on ne dispose pas du détail. Mais dans l'esprit de ce que tu as fait : puisque view et en charge de setter les panels, view pourrais aussi getter les références aux panels et par là propager des notifications particulières.

En java il existe de multiple façon de procéder pour obtenir un résultat. IL est surtout question de choix d'implémentation ce référent à une école de pensée, un choix d'outils ou plus communément d'utilisation de Design pattern.

Désolé, en l'état difficile de te dire : tu doit faire comme ceci ou comme cela.

Merci pour votre réponse.
Effectivement, je me suis orienté sur le pattern observer qui me permettra d'observer le changement sur les critères de recherche.
Voici le modèle de mon controller principal

Code:

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
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
public class Model {
	private int idService;
	private EventListenerList listeners;
 
 
	// CONTRUCTEUR
	public Model(){
		this(0);
	}
 
	public Model(int idService){
		super();
 
		this.idService = idService;
 
		listeners = new EventListenerList();
	}
 
 
	// GETTER AND SETTER
	public int getIdService() {
		return idService;
	}
 
	public void setIdService(int idService) {
		this.idService = idService;
 
		fireServiceChanged();
	}
 
 
	// OBSERVER
	public void addServiceListener(ServiceListener listener){
		listeners.add(ServiceListener.class, listener);
	}
 
	public void removeServiceListener(ServiceListener l){
		 listeners.remove(ServiceListener.class, l);
	}
 
	public void fireServiceChanged(){
		ServiceListener[] listenerList = (ServiceListener[])listeners.getListeners(ServiceListener.class);
 
		for(ServiceListener listener : listenerList){
			listener.serviceChanged(new ServiceChangedEvent(this, getIdService()));
		}
	}
}

Voici comment je l'ai utilisé dans mon controller principal.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
        public void displayPanelCentral1 () {
	     ControllerCentral1 controlCentral1 = new ControllerCentral1 (this);
             model.addServiceListener(controlCentral1.getView());
	     view.setPanelCentral(controlCentral1.getView());
	}
 
        public void displayPanelCentral2 () {
	     ControllerCentral2 controlCentral2 = new ControllerCentral2 (this);
             model.addServiceListener(controlCentral2.getView());
	     view.setPanelCentral(controlCentral2.getView());
	}
 
        ....

Je réussi ainsi à intercepter les évènements sur le panel de recherche.

Mais, car toujours un mais, lorsque j'ouvre par exemple le panelCentral1 puis le panelCentral2, et que je modifie le critère de recherche, le panelCentral1 et le panelCentral2 sont actualisé alors que je souhaiterai raffraichir uniquement le panelCentral en cours d'affichage

29/03/2011, 11h20
william44290

Cela est, pour le moment normal, les deux panels sont abonnés au même controleur dont la mission est de propager l'événement par le fireServiceChanged() a tous les objets abonnés. Donc tu dois ajouter des éléments discriments de source et de traitements pour arriver à tes fins. Les questions sont toujours quoi, oû et comment.
29/03/2011, 11h29
RomG7

Merci pour le temps que tu consacres à me répondre.

Je ne comprends pas ce que tu veux dire par

Citation:

Donc tu dois ajouter des éléments discriments de source et de traitements pour arriver à tes fins

Pour arriver à mes fins, il faudrait que chaque fois que j'ouvre un panelCentral, que je remove le panel central précédent en appelant la méthode removeServiceListener?
Le problème est que je ne peux pas remove le panel central précédent car je les construits à la volé à chaque ouverture et que je perds ainsi leur référence.

Peut être faut il que je créé toutes les instances des panel centraux à la contruction du controller principal OU bien créer une classe abstraite commune à tous les panel centraux???
Tous ça est assez confus dans mon esprit
29/03/2011, 11h39
william44290

Citation:

Envoyé par RomG7

Mais, car toujours un mais, lorsque j'ouvre par exemple le panelCentral1 puis le panelCentral2, et que je modifie le critère de recherche, le panelCentral1 et le panelCentral2 sont actualisé alors que je souhaiterai raffraichir uniquement le panelCentral en cours d'affichage

Alors au finale il y a trois panels : "panelCentral1","panelCentral2" et "panelCentral"

"panelCentral1" et "panelCentral2" sont abonnés et pas "panelCentral".

Votre énoncé n'est pas claire....

Non, en fait j'ai x panel central. Pour simplifier je les ai appelé PanelCentral1, PanelCentral2, ... ensuite le controller de la frame est en charge de setter ces panels dans la view avec la méthode "setPanelCentral".

Voici cette méthode dans la classe View
Code:

1 2 3 4 5 6 7 public void setPanelCentral(JPanel panelCentral ) { this.remove(this.panelCentral ); this.panelCentral = panelCentral ; this.add(panelCentral , BorderLayout.CENTER); this.panelCentral .revalidate(); }

tu pourrais ajouter un boolean getActifService(id) qui test ton panel avec un isVisible() par exemple.

et dans le :

Code:

1
2
3
4
5
6
7
8
9
10
 
	public void fireServiceChanged(){
		ServiceListener[] listenerList = (ServiceListener[])listeners.getListeners(ServiceListener.class);
 
		for(ServiceListener listener : listenerList){
                        if (getActifService(id)) {
			listener.serviceChanged(new ServiceChangedEvent(this, getIdService()));
                       }
		}
	}

Effectivement cela peut être une idée. Merci beaucoup pour tous ces conseils.
Ce n'est pas facile pour moi d'exprimer mes problèmes.

Peut être aussi en créant toutes les instances de mes panels du centre directement dans le controller principal pour mémoriser leur référence :

Code:

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
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
 
public class Controller { 
	private Connex connexion;
	private Model model;
	private View view;
	private  ControllerCentral1 controlCentral1;
        private  ControllerCentral2 controlCentral2
 
	// CONTROLLER
	public Controller() {
		try {
			this.connexion = new Connex(); 
		} 
		catch (SQLException e) {}
		catch (Exception e) {}
 
		this.model = new Model();
		this.view = new View(this);
 
		this.view.setPanelRecherche(new ControlRecherche (this));
 
                // création de toutes les instances de mes panel centraux pour mémoriser leur référence
                controlCentral1 = new ControllerCentral1 (this);
                controlCentral2 = new ControllerCentral2 (this);
	}
 
 
	// GETTER AND SETTER
	public Connex getConnexion() {
		return connexion;
	}
 
	public Model getModel() {
		return model;
	}
 
 
	// EVENEMENT
	public void displayPanelCentral1 () {
             // supprime tous les listener du modele
             model.removeAllServiceListener();
             model.addServiceListener(controlCentral1.getView());
	     view.setPanelCentral(controlCentral1.getView());
	}
 
        public void displayPanelCentral2 () {
             // supprime tous les listener du modele
             model.removeAllServiceListener();
             model.addServiceListener(controlCentral2.getView());
	     view.setPanelCentral(controlCentral2.getView());
	}
 
        ...
}

A force de m'interroger il me vient une autre idée, peut être directement dans la méthode setPanelCentral de la classe View en lui donnant en paramètre le modèle :
Code:

1 2 3 4 5 6 7 8 9 10 public void setPanelCentral(JPanel panelCentral, Model model) { model.removeServiceListener(this.panelCentral); model.addServiceListener(panelCentral); this.remove(this.panelCentral ); this.panelCentral = panelCentral ; this.add(panelCentral , BorderLayout.CENTER); this.panelCentral .revalidate(); }

29/03/2011, 13h05
william44290

Il ne faut pas perdre de vue que ce pattern a pour avantage de maintenir le découplage. En basculant une référence du model vous couplé plus fortement votre application.

Personnellement j'utilise une frame dans laquelle j'instancie des panels. Je n'utilise pas ma frame comme controleur.
29/03/2011, 13h37
RomG7

En fait je suis auto-didacte et passionné par le développement en java. J'ai appris beaucoup de chose avec les nombreux tutoriels présent sur ce site.
J'ai voulu me perfectionner en utilisant le design pattern MVC pour bien séparer les différentes couchent (le modèle, la vue et le contrôleur).

C'est pour cette raison que j'ai associé un contrôleur pour chaque élément graphique : un contrôleur pour le composant JFrame, un contrôleur par composant JPanel ...

Mais j'avoues que j'arrive à un stade ou je me mélange un peu les pinceaux. Je fais un pas devant et trois derrière.

Ce petit exercice me permettrait d'avancer vers la bonne direction
29/03/2011, 13h52
william44290

Il me semble que la mode depuis un moment en rapport avec le découplage c'est spring du coup un certain nombres de design patern présentés en java sont quelques peu obsolètes.