Problème fermeture de fenetre

Version imprimable

19/07/2011, 15h48
H4rDBuG

Problème fermeture de fenetre

Bonjour à tous,

Je reviens vers vous avec un ultime problème.
J'ai sur mon application, un JButton qui lors d'un clic dessus, ouvre une nouvelle fenêtre contenant une JTable chargé par des données d'une base de données.

Lorsque j'ouvre ma fenêtre pour la première fois au lancement de l'application, pas de soucis, le tableau se remplit bien, je la referme, et si je l'ouvre directement après, le tableau est remplit mais en double !

Le problème vient certainement de la méthode de fermeture de la fenêtre.

J'ai donc d'abord essayé :

Code:

frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

Sauf que toute l'application se ferme, j'ai donc essayé :

Code:

frame.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);

Mais la, la fenêtre se ferme, mais ne se "détruit" pas, je me retrouve toujours avec un tableau remplit en double ...

Pour le EXIT_ON_CLOSE, c'est plutôt normal d'obtenir ce résultat, mais je trouve ceci plus bizarre avec le DISPOSE_ON_CLOSE qui est censé détruire la fenêtre non ?

Je suis à cours d'idées :/
19/07/2011, 15h59
adiGuba

Salut,

Citation:

Envoyé par H4rDBuG

mais je trouve ceci plus bizarre avec le DISPOSE_ON_CLOSE qui est censé détruire la fenêtre non ?

Non : le dispose libère seulement les ressources de la fenêtre lié au système d'exploitation. En aucun cas il ne détruit l'objet Java. Si tu ré-affiches la même fenêtre ces ressources seront recréé...

D'après ce que tu dis, je suppose que tu crées la fenêtre/table une seule fois, et que tu y ajoutes des données à chaque fois que tu l'affiches.

:arrow: As-tu réellement besoin de conserver la fenêtre une fois fermée et de conserver sa référence ?

Il serait plus simple de ne pas la conserver une fois fermée, et de la recréer si besoin...

a++
19/07/2011, 16h02
H4rDBuG

Merci adiGuba de ta réponse, c'est exactement ce que je veux faire, mais je ne connais pas la méthode pour la supprimer à la fermeture ...
19/07/2011, 16h27
Ivelios

Un petit remove() ou removeAll() sur le panel contenant la JTable. Ensuite tu enchaîne avec le dispose et le tour devrait être joué ;)
19/07/2011, 16h45
adiGuba

Citation:

Envoyé par H4rDBuG

Merci adiGuba de ta réponse, c'est exactement ce que je veux faire, mais je ne connais pas la méthode pour la supprimer à la fermeture ...

Ben tu dois conserver la référence quelque part. Il suffit de ne pas le faire...

Après sans voir ton code c'est difficile de t'aider plus !

a++
19/07/2011, 16h47
H4rDBuG

Je vois pas trop ou placer le remove(), du fait que j'appelle la méthode setDefaultCloseOperation(), je ferme la fenêtre avec la croix rouge

Voila le code ma classe tabCategorie (qui ne contient que mon JTable avec 2 boutons

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
74
75
76
77
78
 
package gestionintendance;
 
import java.awt.*;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.print.PrinterException;
import java.io.File;
import java.io.IOException;
import javax.swing.*;
import javax.swing.table.*;
 
public class tabCategorie
{
    static String[] titreColonnes = {"NomCategorie"}; 
 
    public tabCategorie(Object[][] donnees, String[] titreColonnes,String titre)
    {
	JFrame frame = new JFrame(titre);
 
	DefaultTableModel model = new DefaultTableModel(donnees,titreColonnes);
	final JTable jt = new JTable(model);
 
        jt.getColumnModel().getColumn(0).setPreferredWidth(200);
 
        JButton print = new JButton("Imprimer");
 
        print.addActionListener(new ActionListener() 
        {
            @Override
            public void actionPerformed(ActionEvent evt) 
            {
                try 
                {                
                    jt.print();
                } 
                catch (PrinterException ex) 
                {
                    System.out.println(ex.getMessage());
                    System.out.println(ex);
                }
            }
	});
 
        JButton export = new JButton("Exporter vers excel");
 
        export.addActionListener(new ActionListener() 
        {
            @Override
            public void actionPerformed(ActionEvent evt) 
            {
                try 
                {                
                    String user = System.getProperty("user.name");
                    ExcelExporter exp = new ExcelExporter(); 
                    exp.exportTable(jt, new File("C:/Users/"+user+"/Desktop/fiche-categorie.xls"));
                } 
                catch (IOException ex) 
                {
                    System.out.println(ex.getMessage());
                    System.out.println(ex);
                }
            }
	});
 
        JPanel pan = new JPanel();
	JScrollPane pane = new JScrollPane(jt,JScrollPane.VERTICAL_SCROLLBAR_AS_NEEDED,JScrollPane.HORIZONTAL_SCROLLBAR_AS_NEEDED);
        pane.setPreferredSize(new Dimension(200,300));
        pan.add(pane);
        pan.add(print);
        pan.add(export);
	frame.add(pan);
	frame.pack();
        frame.setLocationRelativeTo(null);
	frame.setVisible(true);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    }
}

j'appelle l'ouverture de cette fenêtre de cette façon :

Code:

1
2
 
tabCategorie t = new tabCategorie(Categorie.getCategorie(Categorie.getAll()),tabCategorie.titreColonnes,"Consultation des categories");

20/07/2011, 02h34
atha2

Mise à part la façon dont tu as architecturé ton programme, je ne vois pas de problème.
Ton "truc" avec titreColonne est juste un peu bizarre. Pourquoi le passer en paramètre ??
Au pire tu peux toujours ajouter un WindowListener et sur l'evenement windowClosed, tu mets ton model à null. Mais si ça marche, je suis curieux de savoir pourquoi :aie:
20/07/2011, 09h03
H4rDBuG

Bah enfait, j'ai préféré créer une classe à part entière, car j'ai une fenetre pour les catégories, mais aussi pour les produits, les fournisseurs etc.
Quand au fait de passer les titres en paramètres, je trouve cela plus simple pour une méthode de ne lui passer qu'une variable, plutôt qu'un pavé de String[] ...

Comment aurais-tu structurer le programme ?
20/07/2011, 11h00
hwoarang

A mon avis, ton probleme ne vient pas de la fenetre mais plutot des données qui lui sont envoyées.

Est ce que tu pourrais montrer le code qui crée le fameux "Object[][] donnees" passé en parametre (en partant du resultset) ?
Je me demande si ce n'est pas stocké quelque part (par exemple un ArrayList) puis, à l'ouverture de la 2e fenetre, la requete serait executée une 2e fois et les données ajoutées...

Ah effectivement, une piste que je n'avais pas envisagée :

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
 
 static ArrayList<Categorie> getAll()
    {
        String sql = "select NomCateg from Categorie;";
        try
        {
            Statement stmt = (Statement) new BDD().connStocks.createStatement();
            ResultSet rs = stmt.executeQuery(sql);
 
            Categorie categ = null;
 
            while(rs.next())
            {
                categ = new Categorie();
                categ.setNomCateg(rs.getString(1));
                listeCategorie.add(categ);
            }
        }
        catch(SQLException e)
        {
            System.out.println(e);
        }
        return listeCategorie;     
    }
 
    private Object[] toTab()
    {
	Object[] ligne = new Object[1];
	ligne[0] = NomCateg;
        return ligne;
    }
    static Object[][] getCategorie(ArrayList<Categorie> alc)
    {
 
	int n=alc.size();
	Object[][] donnees =  new Object[n][]; 
	int i=0;
	for(Categorie c : alc)
        {
		donnees[i]=c.toTab();
		i++;
	}
	return donnees;
    }

Voilà les méthodes de la classe catégorie utiles au JFrame

20/07/2011, 11h32
Ivelios
Code:

listeCategorie.add(categ);

Tu ajoute bien un "categ" à ta liste mais il n'est jamais supprimé.
Et comme je suppose que listeCategorie est static et bien il utilise tout le temps le même.

Soit tu recrée une nouvelle liste à chaque fois, soit tu supprime les éléments de la liste avec d'en rajouter.
Code:

1 2 List<Categorie> listeCategorie = new ArrayList<Categorie>(); listeCategorie.add(categ);
Code:

1 2 listeCategorie.remove(); // Ou quelquechose comme ça listeCategorie.add(categ);
ps : Il y a pas mal de static dans ton code. Tu pourrais t'en passer, parce que le static ça peux vite devenir LE MAL :twisted:
20/07/2011, 11h41
adiGuba

+1

Dans getAll() tu rajoutes des éléments dans une "listeCategorie" dont on se sait rien ?
Elle est static ? Dans ce cas à chaque appel à la méthode tu rajouteras des données déjà présente :?

:arrow: La liste devrait être créé dans la méthode à chaque appel !

Sinon ta gestion de la BDD est catastrophique : tu ne libères jamais tes ressources !!!!

:faq: Comment libérer proprement les ressources ?

a++
20/07/2011, 11h49
H4rDBuG

Je viens de tester en créant la liste directement dans la méthode getAll() et ça marche très bien ;)

Merci beaucoup pour l'aide apportée.

Merci adiGuba, je vais regarder de plus près, je suis assez nouveau en java (moins d'un an d'éxpérience) et c'est la première VRAIE application que je réalise avec une base de données SQL ... Je vais regarder attentivement ton lien ;)

Coder c'est bien, Coder proprement, c'est mieux ;)