JTable, XML et modèles

Version imprimable

Bonjour,

je viens vers vous après plusieurs jours de recherche pour un problème qui est vraisemblablement traité souvent dans cette section.

J'ai un fichier XML qui est créé par mon application et qui doit être affiché dans une JTable. Dans l'ancienne version de ma JTable, je la créais sans passer par un modele et cela me convenait jusqu'à ce que je me rende compte que la JTable ne pouvait pas se rafraichir sans tableModel.
Depuis ce moment, j'ai un gros problème, je n'ai plus rien dans ma JTable :cry:

Je viens vers vous car je tourne en rond...

Voici le code de mes classes JTable et Model

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
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
 
 
public class remindEnCours {
 
 
 
    private  JScrollPane scrollPane;
    static JTable tableXML;
 
 
 
    public remindEnCours(FenetrePrincipale fenetre)
    {
        modeleTable monModele=new modeleTable();
        tableXML=new JTable(monModele);
        tableXML.setVisible(true) ;     
        centrerTable(tableXML);
        fenetre.getContentPane().add(new JScrollPane(tableXML), BorderLayout.CENTER);
    }
 
 
 
    private void centrerTable(JTable table) 
    {     
        DefaultTableCellRenderer custom = new DefaultTableCellRenderer();
        custom.setHorizontalAlignment(JLabel.CENTER);
        for (int i=0 ; i<table.getColumnCount() ; i++)
        table.getColumnModel().getColumn(i).setCellRenderer(custom);
        TableCellRenderer headerRenderer = table.getTableHeader().getDefaultRenderer();
        ((DefaultTableCellRenderer) headerRenderer).setHorizontalAlignment(DefaultTableCellRenderer.CENTER);
     }
 
    public class modeleTable extends DefaultTableModel {
 
 
        private Document document;
        private  Element racine;
        private  String[] columnNames={"Id", "Libellé de l'alerte", "Date", "Répétition", "Active"}; 
        private  Object[][] data;
 
        public modeleTable()
        {
        SAXBuilder sxb = new SAXBuilder();
        try {
            document = sxb.build(new File(".//files/Alertes.xml"));
        } catch (Exception e5) {
            e5.printStackTrace();
        }
 
        racine = document.getRootElement();
        System.out.println("racine " + racine);
        List listJob = racine.getChildren("alerte");
        FenetrePrincipale.barreEtat.setText(listJob.size()+" éléments chargés");
        Iterator i = listJob.iterator();
        while (i.hasNext())
        {
            Element courant = (Element) i.next();
            String[] rowData = {
                    courant.getAttributeValue("id"),
                    courant.getChild("libelle").getText(),
                    courant.getChild("date").getText(),
                    courant.getChild("frequence").getText(),
                    courant.getChild("active").getText()
            };
 
            fireTableDataChanged();
            addRow(rowData);
        }
        } 
 
    }
 
}

Merci d'avance!

07/04/2011, 14h11
Logan Mauzaize
1. columnNames et data ne sont pas utilisés
2. Tu "fire" avant de modifier tes données
3. addRow notifie déjà les changements aux tables
4. Inutile de notifier avant d'avoir construit le modèle.
5. Es-tu sûr que ton modèle est remplis ?
07/04/2011, 14h39
dimou59

Pour columNames et data, je les avais utilisé dans mes différents tests et en effet je ne les utilise pas maintenant.
Je ne comprends quand tu dis que je fire avant de modifier les données: fire n'indique pas simplement à la table que quand le xml change, il doit recharger la table?
Tu dis que je notifie avant de construire mon modèle... je suis un peu perdu :calim2:
07/04/2011, 14h49
Logan Mauzaize

Tu appeles la méthode "fireTableDataChanged" dans ton constructeur et avant d'appeler la méthode "addRow".

La méthode "addRow" appelle la méthode "fireTableRowsInserted"
07/04/2011, 14h57
dimou59

le fire ne sert à rien donc. Ou faut il placer le addRow afin que les changements du fichier XML soient pris en compte? J'ai essayé de suivre le tuto sur le site mais j'arrive pas à l'adapter à mon cas...

j'ai modifié mon code, les données apparaissent au lancement mais quand je rafraichis la table avec mon action, rien ne se passe:

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
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
 
public class remindEnCours {
 
 
 
    private JScrollPane scrollPane;
    static JTable table;
    private Vector columnNames;
    private Vector data;
    private Document document;
    private Element racine;
 
	public remindEnCours(FenetrePrincipale fenetre)
	{
		columnNames = new Vector();
        columnNames.add("Id");
        columnNames.add("Libelle");
        columnNames.add("Date");
        columnNames.add("Répétition");
        columnNames.add("Active");
        data = new Vector();
        table = new JTable(data, columnNames);
        SAXBuilder sxb = new SAXBuilder();
	    try {
	        document = sxb.build(new File(".//files/Alertes.xml"));
	    } catch (Exception e5) {
	        e5.printStackTrace();
	    }
	    racine = document.getRootElement();
	    List listJob = racine.getChildren("alerte");
	    Iterator i = listJob.iterator();
	    FenetrePrincipale.barreEtat.setText(listJob.size()+" éléments chargés");
	    while (i.hasNext())
	    {
	    	Element courant = (Element) i.next();
	    	String[] rowData = {
	    			courant.getAttributeValue("id"),
	    			courant.getChild("libelle").getText(),
	                courant.getChild("date").getText(),
	                courant.getChild("frequence").getText(),
	                courant.getChild("active").getText()
	    	};
	    	((DefaultTableModel) table.getModel()).addRow(rowData);
 
	    }
 
        centrerTable(table);
        fenetre.getContentPane().add(new JScrollPane(table), BorderLayout.CENTER);
 
	}
 
 
 
	public static void centrerTable(JTable table) 
    {     
    	DefaultTableCellRenderer custom = new DefaultTableCellRenderer();
    	custom.setHorizontalAlignment(JLabel.CENTER);
    	for (int i=0 ; i<table.getColumnCount() ; i++)
    	table.getColumnModel().getColumn(i).setCellRenderer(custom);
    	TableCellRenderer headerRenderer = table.getTableHeader().getDefaultRenderer();
    	((DefaultTableCellRenderer) headerRenderer).setHorizontalAlignment(DefaultTableCellRenderer.CENTER);
     }
 
 
 
 
 
 
}

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
 
private class SwingAction1 extends AbstractAction {
		public SwingAction1() {
			putValue(NAME, "Valider");
			putValue(SHORT_DESCRIPTION, "Valider");
		}
		public void actionPerformed(ActionEvent e) {
 
 
 
			try {
				alerte nouvelleAlerte=new alerte();
				ecrireID(fichierId);
			} catch (ParseException e1) {
				// TODO Auto-generated catch block
				e1.printStackTrace();
			}
 
 
			DefaultTableModel model=(DefaultTableModel)remindEnCours.table.getModel();
			model.fireTableDataChanged();
			remindEnCours.table.setModel(model);
			RemindMe.fenetre.getContentPane().add(new JScrollPane(remindEnCours.table), BorderLayout.CENTER);
			setVisible(false);
 
 
		}
	}

merci de votre aide!

08/04/2011, 14h45
Logan Mauzaize

Ton model n'est pas modifié ...
08/04/2011, 14h57
dimou59

Citation:

Envoyé par Nemek

Ton model n'est pas modifié ...

comment le modifier, n'est il pas modifié par l'action en faisant mon model.fireTableDataChanged();
08/04/2011, 15h07
Logan Mauzaize

les méthodes fireXXX de la classe AbstractDefaultTableModel servent à notifier les listener enregistrés que l'ensemble des données du modèle ont changé. Cela inclue également la JTable qui a enregistré un listener auprès du modèle.

Cela ne change en rien les données du modèle. D'ailleurs je ne vois pas comment la méthode "devinerait" comment elle doit modifier les données.

Citation:

Envoyé par Nemek

les méthodes fireXXX de la classe AbstractDefaultTableModel servent à notifier les listener enregistrés que l'ensemble des données du modèle ont changé. Cela inclue également la JTable qui a enregistré un listener auprès du modèle.

Cela ne change en rien les données du modèle. D'ailleurs je ne vois pas comment la méthode "devinerait" comment elle doit modifier les données.

ok, donc dans mon action j'ai rajouté le listage des données de mon xml à mon modele est je l'ai rechargé mais j'ai une erreur:
Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at creerUneTache$SwingAction1.actionPerformed(creerUneTache.java:601)
at javax.swing.AbstractButton.fireActionPerformed(Unknown Source)
at javax.swing.AbstractButton$Handler.actionPerformed(Unknown Source)
at javax.swing.DefaultButtonModel.fireActionPerformed(Unknown Source)
at javax.swing.DefaultButtonModel.setPressed(Unknown Source)
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 private class SwingAction1 extends AbstractAction { public SwingAction1() { putValue(NAME, "Valider"); putValue(SHORT_DESCRIPTION, "Valider"); } public void actionPerformed(ActionEvent e) { try { alerte nouvelleAlerte=new alerte(); ecrireID(fichierId); } catch (ParseException e1) { // TODO Auto-generated catch block e1.printStackTrace(); } DefaultTableModel model=(DefaultTableModel)remindEnCours.table.getModel(); racine = document.getRootElement(); List listJob = racine.getChildren("alerte"); Iterator i = listJob.iterator(); FenetrePrincipale.barreEtat.setText(listJob.size()+" éléments chargés"); while (i.hasNext()) { Element courant = (Element) i.next(); String[] rowData = { courant.getAttributeValue("id"), courant.getChild("libelle").getText(), courant.getChild("date").getText(), courant.getChild("frequence").getText(), courant.getChild("active").getText() }; model.addRow(rowData); } //model.fireTableDataChanged(); remindEnCours.table.setModel(model); RemindMe.fenetre.getContentPane().add(new JScrollPane(remindEnCours.table), BorderLayout.CENTER); setVisible(false); } }

08/04/2011, 15h55
Logan Mauzaize

Va voir la ligne 601 du fichier "creerUneTache.java" ;-)
08/04/2011, 16h31
dimou59

Citation:

Envoyé par Nemek

Va voir la ligne 601 du fichier "creerUneTache.java" ;-)

merci... j'avais oublié de parser le xml!!!
c'est bon ç amarche, je réafficher un jtable et donc il était vide!

en tout cas merci Nemek pour ta patience et tes bons conseils!
08/04/2011, 16h51
Logan Mauzaize

Remet le code "semi-complet", corrigé et clarifié :)

Citation:

Envoyé par Nemek

Remet le code "semi-complet", corrigé et clarifié :)

remplissage de mon jtable au lancement du programme

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
 
columnNames = new Vector();
        columnNames.add("Id");
        columnNames.add("Libelle");
        columnNames.add("Date");
        columnNames.add("Répétition");
        columnNames.add("Active");
        data = new Vector();
 
        //création de la table
        table = new JTable(data, columnNames);
 
        //on parse le xml
        SAXBuilder sxb = new SAXBuilder();
	    try {
	        document = sxb.build(new File(".//files/Alertes.xml"));
	    } catch (Exception e5) {
	        e5.printStackTrace();
	    }
	    racine = document.getRootElement();
	    List listJob = racine.getChildren("alerte");
	    Iterator i = listJob.iterator();
	    FenetrePrincipale.barreEtat.setText(listJob.size()+" éléments chargés");
	    //tant qu'il y a un élément suivant on crée une ligne de données et on l'affiche
	    while (i.hasNext())
	    {
	    	Element courant = (Element) i.next();
	    	String[] rowData = {
	    			courant.getAttributeValue("id"),
	    			courant.getChild("libelle").getText(),
	                courant.getChild("date").getText(),
	                courant.getChild("frequence").getText(),
	                courant.getChild("active").getText()
	    	};
	    	((DefaultTableModel) table.getModel()).addRow(rowData);
 
	    }
 
        centrerTable(table);
        fenetre.getContentPane().add(new JScrollPane(table), BorderLayout.CENTER);

Action qui met à jour les données du model de ma jtable

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 void actionPerformed(ActionEvent e) {
 
			//dans mon action je lance le meme traitement qu'à l'ouverture de mon programme
 
			try {
				alerte nouvelleAlerte=new alerte();
				ecrireID(fichierId);
			} catch (ParseException e1) {
				// TODO Auto-generated catch block
				e1.printStackTrace();
			}
 
			 SAXBuilder sxb = new SAXBuilder();
			    try {
			        document = sxb.build(new File(".//files/Alertes.xml"));
			        System.out.println("ok");
			    } catch (Exception e5) {
			        e5.printStackTrace();
			    }
			    //je récupère le model de ma jtable
			DefaultTableModel model=(DefaultTableModel)remindEnCours.table.getModel();
			 racine = document.getRootElement();
			    List listJob = racine.getChildren("alerte");
			    Iterator i = listJob.iterator();
			    FenetrePrincipale.barreEtat.setText(listJob.size()+" éléments chargés");
			    while (i.hasNext())
			    {
			    	Element courant = (Element) i.next();
			    	String[] rowData = {
			    			courant.getAttributeValue("id"),
			    			courant.getChild("libelle").getText(),
			                courant.getChild("date").getText(),
			                courant.getChild("frequence").getText(),
			                courant.getChild("active").getText()
			    	};
			    	//j'ajoute une ligne à mon modèle
			    	model.addRow(rowData);
 
			    }
			    //j'indique à ma table que les données de mon modèle ont changé
			model.fireTableDataChanged();
			//je réaffecte le modèle modifié à ma table
			remindEnCours.table.setModel(model);
			setVisible(false);
 
 
		}

encore merci Nemek, les modèles étaient un concept un peu opaque pour moi!