Discussion :

[Baptiste Wicht]

Bonsoir,

Tout d'abord, je sais pas vraiment si ça s'appelle bien des références croisées, mais je pense que ca lui va bien.

Je suis en train de développer une application en utilisation le MVC, même le M(VC)C. C'est à dire que j'ai aussi un contrôleur purement Swing (ActionController) qui gère les actions des mes interfaces et un contrôleur de la vue générique (Controller).

Voilà mon problème : Au début de mon programme, je demande à mon programme de charger toutes les actions du programme.

Voilà en épuré la classe qui va faire cela :

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
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
public class ActionControllerManager {
	private static ArrayList<ActionController> controllers = null;
	private static MyLogger logger = MyLoggerFactory.getLogger(ActionControllerManager.class);
 
	static {
		controllers = new ArrayList<ActionController>();
 
		controllers.add(new AboutActionController());
		controllers.add(new ConfigActionController());
		controllers.add(new StatsActionController());
		controllers.add(new FilmActionController());
		controllers.add(new JThequeActionController());
	}
 
	/**
         * Retourne un controlleur en fonction du type passé en paramètre
         * @param controllerType Le type de controlleur voulu
         * @return Le controlleur adéquat
         */
	public static ActionController getController(int controllerType){
		logger.debug("Get controller " + controllerType);
		for(ActionController controller : controllers){
			if(controller.canControl(controllerType)){
				logger.debug("Return controller " + controller);
				return controller;
			}
		}
 
		logger.debug("Controller not found, return null");
 
		return null;
	}
 
	/**
         * Charge toutes les actions des différents controlleurs
         * @throws ActionLoaderException Cette exception est levée s'il y a eu une erreur durant le chargement des 
         * actions
         */
	public static void loadAllActions() throws ActionLoaderException{
		for(ActionController controller : controllers){
			boolean good = controller.loadActions();
 
			if(!good){
				ActionLoaderException exception = 
					new ActionLoaderException("Les actions n'ont pas pu être chargées correctement");
 
				if(controller.getException() != null){
					exception.initCause(controller.getException());
				}
 
				throw exception;
			}
		}
	}
}

Le problème c'est que lorsqu'il éxécuter la méthode loadAction de AboutActionActionController, ca va appeller le constructeur d'une action (AcDisplayLicence) qui lui va utiliser ControllerFactory :

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
public class ControllerFactory {
	private static ArrayList<Controller> controllers = null;
 
	static {
		controllers = new ArrayList<Controller>();
 
		controllers.add(new JThequeController());
		controllers.add(new UpdateController());
	}
 
	public static Controller getController(int controllerType){
		for(Controller controller : controllers){
			if(controller.canManage(controllerType)){
				return controller;
			}
		}
 
		return null;
	}
}

Voici le code de AcDisplayLicence :

Code :

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

1
2
3
4
5
6
7
8
9
10
public class AcDisplayLicence extends MyAction {
	private static final long serialVersionUID = -5130253480109588863L;
 
	private LicenceController controller = 
		(LicenceController)ControllerFactory.getController(Constantes.Controllers.LICENCE);
 
	public void actionPerformed(ActionEvent e) {
		controller.displayView();
	}
}

Malheureusement, le fait d'initialiser JThequeController initialise aussi l'interface graphique et cette interface a besoin de JThequeActionController pour récupérer les actions, malheureusement, a ce moment là de l'éxécution, la méthode loadAction n'es pas encore lancée, je récupère donc logiquement des actions encore nulles et mon interface graphique est toute vide...

Je sais pas si j'ai été bien clair, en plus le problème est assez compliqué... Mais quelqu'un aurait-il une idée ? Parce que là, j'ai un peu de peine... Mais je vais continuer à chercher.

Merci

[Baptiste Wicht]

Bon, j'ai trouvé une parade, il suffit de ne pas initialiser tout de suite mon Controller, mais à la première utilisation. Je suis sûr que les actions ne pourront pas être éxécutés avant la fin de la création de l'interface graphique donc c'est bon.

Donc, il faudrai que je fasse ça :

Code :

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

1
2
3
4
5
6
7
public class AcDisplayLicence extends MyAction {
	private static final long serialVersionUID = -5130253480109588863L;
 
	public void actionPerformed(ActionEvent e) {
		ControllerFactory.getController(Constantes.Controllers.LICENCE).displayView();
	}
}

Mais je pense qu'un problème pareil traduit tout de même un problème de conception...

[Sanguko]

D'une maniere générale, ce n'est pas tres bon de faire toutes les initialisations dans les blocs static, parce que :
- tu ne controles pas l'ordre d'execution de ces blocs
- la gestion des erreurs (et des exceptions) est plus compliquée, alors que c'est en général simple à faire dans le main.

C'est un peu la même chose avec les constructeurs. Si il y a beaucoup de choses à faire, il vaut mieux ajouter une methode d'initialisation. Le constructeur ne se charge de ne faire que le strict minimum.

[Baptiste Wicht]

D'une maniere générale, ce n'est pas tres bon de faire toutes les initialisations dans les blocs static, parce que :
- tu ne controles pas l'ordre d'execution de ces blocs
- la gestion des erreurs (et des exceptions) est plus compliquée, alors que c'est en général simple à faire dans le main.

C'est un peu la même chose avec les constructeurs. Si il y a beaucoup de choses à faire, il vaut mieux ajouter une methode d'initialisation. Le constructeur ne se charge de ne faire que le strict minimum.

Je suis assez d'accord avec, c'est en effet plus propre avec une méthode qui va initialiser le tout, mais l'avantage c'est qu'avec un bloc static, tu es sûr que ce soit initialisé dès qu'on utilise la classe alors que dans le cas d'une méthode tu dépends plus du code. Mais ca ne résouds pas mon problème...

[natha]

- tu ne controles pas l'ordre d'execution de ces blocs

Ah bon ?
Il y a une question dans le bouquin d'entrainement à la certif SCJP qui porte sur l'ordre d'exécution de blocs statics, non statics et du constructeur et la réponse n'est pas "impossible à déterminer". Ca dépendrait de la JVM sinon ?

[g0up1l]

le fait d'initialiser JThequeController initialise aussi l'interface graphique

Comment un contrôleur peut-il intitialiser une interface graphiqe ?

Normalement,c'est ton manager qui doit construire ton contrôleur, ton IHM et ton document puis effectuer les branchements: vue -> doc, vue->contrôleur et contrôleur->doc.

[Baptiste Wicht]

Comment un contrôleur peut-il intitialiser une interface graphiqe ?

Normalement,c'est ton manager qui doit construire ton contrôleur, ton IHM et ton document puis effectuer les branchements: vue -> doc, vue->contrôleur et contrôleur->doc.

C'est intéressant ce que tu me dis là

En fait, dans le constructeur du contrôleur, je fais :

Code :

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

setView(new JFrameJTheque());

Toi tu le ferais ou alors ?

[Sanguko]

Ah bon ?
Il y a une question dans le bouquin d'entrainement à la certif SCJP qui porte sur l'ordre d'exécution de blocs statics, non statics et du constructeur et la réponse n'est pas "impossible à déterminer". Ca dépendrait de la JVM sinon ?

Je suis d'accord que l'ordre entre bloc static, non static et constructeur est deterministe.

Mais est ce que tu es capable de donner l'ordre d'execution entre plusieurs blocs static ?
Par exemple si tu as

Code :

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

1
2
3
4
5
6
7
8
9
10
class A {
 static {
    ...
 } 
}
class B {
 static {
   ...
 } 
}

qui s'execute avant qui ?

Je pense qu'il n'y a pas de réponses simples, mais si tu as des explications, je suis preneur.

[natha]

qui s'execute avant qui ?
Je pense qu'il n'y a pas de réponses simples, mais si tu as des explications, je suis preneur.

Je ne m'y connais pas des masses, mais je pense passer la certif cette année alors j'ai fait le test du java.developpez.com ici :
http://brabant.developpez.com/java/c...ions/factices/
Tu vas à la question 3 (2.3)
Bon il n'y a pas de bloc static dans Oiseau mais à priori s'il y en avait ils seraient exécutés avant ceux de Rapace.

Désolé pour le Hors Sujet wichtounet...