Discussion :

[Copernic09]

Bonjour à tous !

Application développée sous Eclipse avec Jigloo.

Je dois créer une interface graphique dans laquelle je distingue la partie interface (fichier appelé FInterface) de la partie traitement (FTraitement).
J'implémente de la manière suivante :
ma fonction Main instancie FTraitement et le constructeur de FTraitement instancie FInterface.

Première question : Y a-t-il un moyen plus propre pour pouvoir traiter les composants de FInterface dans FTraitement ?

Deuxième question : J'ai un bouton dans FInterface auquel j'ajoute un évenement clic mais je veux traiter cet évènement dans FTraitement ... une idée ?

Merci d'avance

à vous...

[g_rare]

La réponse : ("MVC" pour la question 1 et Swing pour la question2), et tu trouveras la !

[Copernic09]

Ok, je ne connaissais pas le MVC

Merci ! je vais me renseigner ... ca m'a pas l'air simple !

[leyee]

Ce n'est pas simple mais une fois que tu auras intégrer cela tu seras convaincu qu'il n'est pas raisonnable de concevoir une IHM autrement avec swing : c'est pas du temps perdu

[Copernic09]

Bon, je suis toujours dans l'étude de ce MVC, mais là; j'ai un problème que je n'arrive pas à résoudre...

Voici le code du Modèle :

Code :

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

1
2
3
4
5
6
7
 
	public void setModel(DefaultComboBoxModel f)
	{
		cb_final = f;
		setChanged();
		notifyObservers();
	}

Et voici le code du contrôleur :

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
 
Arr = new ArrayList();
 
ResultSet Lst_Table;
Statement Stmt_Lst;
int i = 0;
 
Stmt_Lst = (Statement) Connexion.Connec.getConnection().createStatement();
Lst_Table = (ResultSet) Stmt_Lst.executeQuery("SELECT Libelle_Service FROM service;");
Arr.add("");
 
while(Lst_Table.next())
{
	Arr.add(Lst_Table.getString(1));
	i++;
}
 
modele.setModel(new DefaultComboBoxModel(Arr.toArray()));

Et dans ce contrôleur, il me met une erreur à la dernière ligne :
Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException

Je suis bloqué, même si ça doit être un truc bidon...

[Magicmax]

Pour une NullPointer, toujours regarder les points dans la ligne incriminée

En l'occurence :

Code :

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

modele.setModel(new DefaultComboBoxModel(Arr.toArray()));

Arr est bien initialisée avant dans ton code mais qu'en est-il de modele ?

[Copernic09]

Voilà ce qu'il y a au dessus du code précédent :

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
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
 
package Planning;
 
import java.awt.*;
import java.awt.event.*;
import java.sql.SQLException;
import java.util.ArrayList;
 
import javax.swing.*;
 
import com.mysql.jdbc.ResultSet;
import com.mysql.jdbc.Statement;
 
/* On implémente ActionListener qui permet de traiter les événements du bouton */
public class Controleur_planning implements ActionListener,WindowListener {
 
	private Modele_planning modele;
	private planning vue;
	private ArrayList Arr;
 
	public Controleur_planning()
	{
		try
		{
			Arr = new ArrayList();
 
			ResultSet Lst_Table;
			Statement Stmt_Lst;
			int i = 0;
 
			Stmt_Lst = (Statement) Connexion.Connec.getConnection().createStatement();
			Lst_Table = (ResultSet) Stmt_Lst.executeQuery("SELECT Libelle_Service FROM service;");
			Arr.add("");
 
			while(Lst_Table.next())
			{
				Arr.add(Lst_Table.getString(1));
				i++;
			}
 
			modele.setModel(new DefaultComboBoxModel(Arr.toArray()));
		}
 
		catch (SQLException e) {
			e.printStackTrace();
		}	
	}
 
	/* Permettra de connaitre le modéle */
	public void setModele(Modele_planning m) {
		modele=m;
	}
 
	/* Permettra de connaitre la vue */
	public void setVue(planning v) {
		vue=v;
	}
 
	/* C'est ici que l'on traite l'action récupéré : implémentation dut à l'interface ActionListener */
	public void actionPerformed(ActionEvent e) {
	}
 
	/* WindowListener implémentation : pour fermer l'application avec la fenêtre */
	public void windowActivated(WindowEvent e) {
	}
 
	public void windowClosed(WindowEvent e) {	
	}
 
	public void windowClosing(WindowEvent e) {
		System.exit(0);
	}
 
	public void windowDeactivated(WindowEvent e) {
	}
 
	public void windowDeiconified(WindowEvent e) {
	}
 
	public void windowIconified(WindowEvent e) {
	}
 
	public void windowOpened(WindowEvent e)  
	{
		modele.setModel(new DefaultComboBoxModel(Arr.toArray()));
	}
 
 
}

[Magicmax]

Code :

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

private Modele_planning modele;

Ton objet de type Modele_planning est déclaré mais pas instancié.

Code :

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

modele.setModel(new DefaultComboBoxModel(Arr.toArray()));

Tu appelles une méthode de Modele_planning sur un objet qui n'est pas initialisé... NullPointerException.

[Copernic09]

Au début de mon constructeur du contrôleur, j'avais ça :

Code :

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

1
2
3
 
		modele=null;
		vue=null;

Et comme indiqué dans un tuto MVC, j'ai aussi un main comme suit :

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
 
package Planning;
 
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
 
public class Main_planning 
{
 
	public Main_planning()
	{
		try {
			UIManager.setLookAndFeel(
			UIManager.getCrossPlatformLookAndFeelClassName());
		}
		catch(Exception e) {
			System.out.println("Erreur : "+e);
		}
 
		/* Contient le modele */
		Modele_planning modele=new Modele_planning();
 
		/* Contient la vue */
		planning vue=new planning();
 
		/* Contient le controleur */
		Controleur_planning controleur=new Controleur_planning();
 
		/* Pour que le controleur connaisse le modele */
		controleur.setModele(modele);
 
		/* Pour que le controleur connaisse la vue*/
		controleur.setVue(vue);
 
		/* Pour que la vue connaisse le modele */
		vue.setModele(modele);
 
		/* Le controleur recevra les action de la vue */
		vue.setListener(controleur);
 
		/* La vue observera le modèle */
		modele.addObserver(vue);
 
		/* Et bien sure on affiche la vue */
		vue.affiche();
 
	}
}

[guigui5931]

Tu es sur que ce que tu veux faire ce n'est pas

Code :

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

1
2
 
setModel(new DefaultComboBoxModel(Arr.toArray()));

plutot que

Code :

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

1
2
 
modele.setModel(new DefaultComboBoxModel(Arr.toArray()));

[Copernic09]

Ben non, puisque cette ligne se trouve dans le contrôleur... je suis obligé d'entrer quelque chose du genre " modele. "

[guigui5931]

Je vais insister (au risque d'être relou) mais dans ta classe controleur tu a défini une méthode setModele qui te permet d'assigner un modèle à ton controleur (si j'ai bien suivi). Dans le constructeur de ton controleur tu fais un new DefaultComboBoxModel qui te permet de créer ton Modèle (toujours si j'ai bien suivi) donc ce qu'il te faut c'est assigner à ton contructeur le modèle que tu vient de créer (en utilisant la méthode setModel de ton controleur et non celle de ton modèle).

J'espère que je me fais comprendre (et que j'ai compris ton problème)

[Magicmax]

Surtout que tu le fais déjà dans ton main :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
/* Contient le modele */
		Modele_planning modele=new Modele_planning();
 
		/* Contient la vue */
		planning vue=new planning();
 
		/* Contient le controleur */
		Controleur_planning controleur=new Controleur_planning();
 
		/* Pour que le controleur connaisse le modele */
		controleur.setModele(modele);

Pourquoi le rajouter à nouveau dans ton constructeur du controlleur ?

[Copernic09]

C'est vrai, j'ai peut-être fait une confusion toute bête mais il ne faut pas confondre le modele du combobox et celui du MVC:

Code :

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

1
2
 
modele.setModel(new DefaultComboBoxModel(Arr.toArray()));

Ici, le setModel permet de copier le DefaultComboBoxModel dans le modele du MVC...puis je veux tout simplement mettre le modele de ComboBox à jour dans ma ComboBox de la Vue.

[Magicmax]

Dans ce cas là, au lieu de faire :

Code :

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

1
2
3
4
5
		/* Contient le controleur */
		Controleur_planning controleur=new Controleur_planning();
 
		/* Pour que le controleur connaisse le modele */
		controleur.setModele(modele);

Passe modele directement à ton constructeur :

Code :

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

Controleur_planning controleur=new Controleur_planning(modele);

De cette manière, tu pourras faire :

Code :

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

1
2
3
4
5
6
7
8
9
10
private Modele_planning modele;
	private planning vue;
	private ArrayList Arr;
 
	public Controleur_planning(Modele_planning m)
	{
		try
		{  
			Arr = new ArrayList();
                        this.modele = m;

et ton modele.setModel ne génèrera plus de NullPointer.

[Copernic09]

Bingo !

Merci beaucoup, même si je n'ai pas été forcemment très clair et que l'on s'est un peu embrouillé dans la discussion, au final ça fonctionne !

Bien compliqué tout ça...

Merci beaucoup !!!