Problème pour rafraichir les composants

Version imprimable

30/09/2006, 17h45
agougeon

Problème pour rafraichir les composants

Bonjour,
Je debute en swing et j'essai de realiser une interface qui sert a renommer les fichiers.... en gros j'ai un JFileChooser qui sert pour choisir un repertoire et une JTable avec deux colonne : une avec l'ancien et l'autre avec le nouveau a saisir.
Au debut de l'applis, j'ai qu'un JMenuBar, mais quand je charge mon repertoire, mon JScrollPane contenant ma JTable ne s'affiche pas.
En fait je sais pas ou mettre le code, j'ai juste un JScrollPane en attribut prive de ma JFrame et dans l'actionPerformed de mon JMenuItem j'instancie ma JTable avec les bon composant. Je fais pourtant un repaint() de ma JFrame.
Je me pose souvent ces question.... dois je mettre tous mes composant graphique en attribut privé de ma Frame, quand dois je rafraichir pour impacter les modifications? avec quelles methodes?

Merci, si vous avez une adresse de site, je suis preneur.
30/09/2006, 17h55
sinok

Euh pour l'adresse de site: http://java.sun.com/docs/books/tutor...nts/table.html

Sinon fais voire le code où tu mets la JTable dans ton JScrollPane (ou celui ou tu mets ton JScrollPane dans ta fenêtre), ce sera plus facile pour te donner un coup de main

ok, c'est du code carrement bourin, mais si tu peux me dire comment organiser le code, si je dois ecrire d'autre classe qui etende certain composant ou autre....

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
79
80
81
82
83
84
85
86
 
public class JRenameFile extends JFrame {
 
	private String path;
	private JTable table;
	private JScrollPane jscroll;
 
	public void setPath(String path){
		this.path = path;
	}
 
	public String getPath(){
		return this.path;
	}
 
	public static Object[][] getListOfFileName(String path){
		File repertoire = new File(path);
 
		File[] list = repertoire.listFiles();
 
		Object[][] donnees = new Object[list.length][2];
 
		for(int i = 0 ; i < list.length ; i++){		
 
			donnees[i][0] = list[i].getName();
			donnees[i][1] = "";
 
		}		
 
		return donnees;		
	}
 
	public static String[] getColumnNames(){
		String[] columnNames = {"Nom du fichier","Renommer en :"};
		return columnNames;		
	}
 
 
	public JRenameFile(){
 
		jscroll = new JScrollPane();
		table = new JTable();
		jscroll.setViewportView(table);
		setTitle("Renommer simplement vos fichiers");
		setSize(800,600);
		GridBagLayout grid = new GridBagLayout();
		getContentPane().setLayout(grid);
		JMenuBar menu = new JMenuBar();
		JMenu file = new JMenu("File");
		JMenuItem load = new JMenuItem("load");
		load.addActionListener(new ActionListener(){
			public void actionPerformed(ActionEvent arg0) {
				JFileChooser fc = new JFileChooser();
				fc.setFileSelectionMode(JFileChooser.DIRECTORIES_ONLY);
				int returnVal = fc.showOpenDialog(JRenameFile.this);
		        if (returnVal == JFileChooser.APPROVE_OPTION) {
		            setPath(fc.getSelectedFile().getPath());
		            table = new JTable(getListOfFileName(getPath()),getColumnNames());
		            jscroll.setViewportView(table);
		            GridBagConstraints c1 = new GridBagConstraints();
		            c1.gridx = 0;
		            c1.gridy = 0;
		            c1.anchor = GridBagConstraints.EAST;
		            getContentPane().add(jscroll,c1);
		            getContentPane().repaint();
 
		        }
			}	
		});
 
 
		file.add(load);
		menu.add(file);
		setJMenuBar(menu);
 
	}
 
 
	public static void main(String[] args){
 
		JRenameFile frame = new JRenameFile();
		frame.setVisible(true);
 
	}
 
}

Merci beauccoup

30/09/2006, 18h38
sinok

fais un appel à revalidate() avant le repaint()

Merci, mais ça marche toujours pas... j'ai testé avec ça :

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
79
80
81
82
83
84
85
86
87
 
public class JRenameFile extends JFrame {
 
	private String path;
	private JTable table;
	private JScrollPane jscroll;
 
	public void setPath(String path){
		this.path = path;
	}
 
	public String getPath(){
		return this.path;
	}
 
	public static Object[][] getListOfFileName(String path){
		File repertoire = new File(path);
 
		File[] list = repertoire.listFiles();
 
		Object[][] donnees = new Object[list.length][2];
 
		for(int i = 0 ; i < list.length ; i++){		
 
			donnees[i][0] = list[i].getName();
			donnees[i][1] = "";
 
		}		
 
		return donnees;		
	}
 
	public static String[] getColumnNames(){
		String[] columnNames = {"Nom du fichier","Renommer en :"};
		return columnNames;		
	}
 
 
	public JRenameFile(){
 
		jscroll = new JScrollPane();
		table = new JTable();
		jscroll.setViewportView(table);
		setTitle("Renommer simplement vos fichiers");
		setSize(800,600);
		GridBagLayout grid = new GridBagLayout();
		getContentPane().setLayout(grid);
        GridBagConstraints c1 = new GridBagConstraints();
        c1.gridx = 0;
        c1.gridy = 0;
        c1.anchor = GridBagConstraints.EAST;
        getContentPane().add(jscroll,c1);
		JMenuBar menu = new JMenuBar();
		JMenu file = new JMenu("File");
		JMenuItem load = new JMenuItem("load");
		load.addActionListener(new ActionListener(){
			public void actionPerformed(ActionEvent arg0) {
				JFileChooser fc = new JFileChooser();
				fc.setFileSelectionMode(JFileChooser.DIRECTORIES_ONLY);
				int returnVal = fc.showOpenDialog(JRenameFile.this);
		        if (returnVal == JFileChooser.APPROVE_OPTION) {
		            table = new JTable(getListOfFileName(fc.getSelectedFile().getPath()),getColumnNames());
		            jscroll.add(table);
		            table.revalidate();
		            table.repaint();
		            jscroll.revalidate();
		            jscroll.repaint();
		        }
			}	
		});
 
 
		file.add(load);
		menu.add(file);
		setJMenuBar(menu);
 
	}
 
 
	public static void main(String[] args){
 
		JRenameFile frame = new JRenameFile();
		frame.setVisible(true);
 
	}
 
}

Je pense plus avoir un problème d'organisation du code...

y'a bcp de probleme.
je vais pas commencer a expliquer swing mais je vais juste te dire comment changer un peu pour que ca marche

tu dois pas recrer chaque fois ta table et la rajouter au scrollpane. une fois suffit.

si tu fais un truc comme ca devrait marcher?
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 jscroll = new JScrollPane(); table = new JTable(); jscroll.setViewportView(table); setTitle("Renommer simplement vos fichiers"); setSize(800,600); GridBagLayout grid = new GridBagLayout(); getContentPane().setLayout(grid); GridBagConstraints c1 = new GridBagConstraints(); c1.gridx = 0; c1.gridy = 0; c1.anchor = GridBagConstraints.EAST; getContentPane().add(jscroll, c1); JMenuBar menu = new JMenuBar(); JMenu file = new JMenu("File"); JMenuItem load = new JMenuItem("load"); load.addActionListener(new ActionListener(){ public void actionPerformed(ActionEvent arg0) { JFileChooser fc = new JFileChooser(); fc.setFileSelectionMode(JFileChooser.DIRECTORIES_ONLY); int returnVal = fc.showOpenDialog(JRenameFile.this); if (returnVal == JFileChooser.APPROVE_OPTION) { setPath(fc.getSelectedFile().getPath()); table.setModel(new DefaultTableModel(getListOfFileName(getPath()),getColumnNames()); } } }); file.add(load); menu.add(file); setJMenuBar(menu);
a la place de recrer la table chaque fois, change juste le model ;-)

en plus c'est tres rare ou tu a besoin d'appeler toi meme les methode (re)validate(), (re)paint(), doLayout()...

ensuite, ta GridBagConstraint comme tu l'a mis elle sert a rien.
tu aurais le meme resulltat si tu fais juste un
Code:

1 2 3 getContentPane().setLayout(grid); getContentPane().add(jscroll);

Bon j'ai corrigé et maintenant ça marche....
Merci, je regarde ton code...
Voici le mien :

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
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
 
import java.awt.GridBagConstraints;
import java.awt.GridBagLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.io.File;
import java.util.Enumeration;
import java.util.Vector;
 
import javax.swing.JButton;
import javax.swing.JFileChooser;
import javax.swing.JFrame;
import javax.swing.JMenu;
import javax.swing.JMenuBar;
import javax.swing.JMenuItem;
import javax.swing.JScrollPane;
import javax.swing.JTable;
import javax.swing.KeyStroke;
import javax.swing.ListSelectionModel;
import javax.swing.event.TableColumnModelListener;
import javax.swing.table.DefaultTableModel;
import javax.swing.table.JTableHeader;
import javax.swing.table.TableColumn;
import javax.swing.table.TableColumnModel;
import javax.swing.tree.DefaultTreeModel;
 
public class JRenameFile extends JFrame {
 
	private String path;
	private JTable table;
	private JScrollPane jscroll;
	private DefaultTableModel model;
	private Vector<Vector> data;
	private JButton button;
 
	public void setPath(String path){
		this.path = path;
	}
 
	public String getPath(){
		return this.path;
	}
 
	public void setListOfFileName(String path){
		//table.removeAll();
		data.removeAllElements();
 
		System.out.println(data.size());
		File repertoire = new File(path);
 
		File[] list = repertoire.listFiles();
 
		String[][] donnees = new String[list.length][2];
 
		for(int i = 0 ; i < list.length ; i++){	
			Vector<String> d = new Vector<String>();
			if(list[i].isFile()){
				d.add(list[i].getName());
				d.add("");
				data.add(d);
			}
		}			
	}
 
	public JRenameFile(){
 
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		jscroll = new JScrollPane();
		data = new Vector<Vector>();
		button = new JButton("Vider");
    	model = new DefaultTableModel();
		model.addColumn("Nom du fichier");
		model.addColumn("Renommer en :");	 
		table = new JTable(model);
		jscroll.setViewportView(table);
		setTitle("Renommer simplement vos fichiers");
		GridBagLayout grid = new GridBagLayout();
		getContentPane().setLayout(grid);
        GridBagConstraints c1 = new GridBagConstraints();
        c1.gridx = 0;
        c1.gridy = 0;
        c1.anchor = GridBagConstraints.WEST;
        GridBagConstraints c2 = new GridBagConstraints();
        c2.gridx = 0;
        c2.gridy = 1;
        c2.anchor = GridBagConstraints.WEST;
        getContentPane().add(jscroll,c1);
        getContentPane().add(button,c2);
		JMenuBar menu = new JMenuBar();
		JMenu file = new JMenu("File");
		JMenuItem load = new JMenuItem("load");
		load.addActionListener(new ActionListener(){
			public void actionPerformed(ActionEvent arg0) {
				JFileChooser fc = new JFileChooser();
				fc.setFileSelectionMode(JFileChooser.DIRECTORIES_ONLY);
				int returnVal = fc.showOpenDialog(JRenameFile.this);
		        if (returnVal == JFileChooser.APPROVE_OPTION) {
		        	setListOfFileName(fc.getSelectedFile().getPath());	        	
		        	removeAll();
		        	for(int i = 0 ; i < data.size() ; i++){
		        		model.addRow(data.get(i));
		        	}
		        	model.fireTableDataChanged();
		        }
			}	
		});
 
		button.addActionListener(new ActionListener(){
			public void actionPerformed(ActionEvent arg0) {
				removeAll();
				model.fireTableDataChanged();
			}
 
		});
		file.add(load);
		menu.add(file);
		setJMenuBar(menu);
 
	}
 
	public void removeAll(){
		System.out.println("Row : " + model.getRowCount());
		int row = model.getRowCount();
		for(int i = 0 ; i < row ; i++){
			model.removeRow(0);
		}
		System.out.println("After : " + model.getRowCount());
	}
 
	public static void main(String[] args){
		JRenameFile frame = new JRenameFile();
		frame.pack();
		frame.setVisible(true);
	}
 
}

30/09/2006, 21h12
bbclone

t'es sur le bon chemin :)
utiliser des model c'est mieux ;)