Manipuler les données d'un JTable

**lefou123456** · 21/04/2017, 02h23

Bonjour,

J'ai créé un JTable dont les données sont bien sûr gérées par un tableau d'Objects, cependant, pour un calcul je dois affecter à une liste toutes les valeurs de la première ligne du tableau; je me dis que je devrais faire un cast mais la JVM n'aime pas trop ça

.

Code :

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

list[i] = Double.parseDouble(tab[0][i]);

**joel.drigo** · 21/04/2017, 11h08

Salut,

Tout d'abord le code que tu montres ne caste pas un Object en Double, mais fait une conversion String vers Double, et c'est pour ça que ça ne compile pas. Pour caster en Double :

Code :

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

list[i] = (Double)tab[0][i];

ou, selon le type de list (si c'est double[]) (mais les deux fonctionnent que le type soit Double[], ou double[])

Code :

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

list[i] = (double)tab[0][i];

Bien sûr, ça ne fonctionnera que si la donnée dans le tableau est bien un Double, et pas une String, auquel cas il faudrait faire :

Code :

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

list[i] = Double.parseDouble((String)tab[0][i]);

Ensuite, je suppose que ce que tu montres est le tableau que tu utilises pour initialiser la JTable, et non le contenu de la JTable elle-même. Ils peuvent être égaux, mais rien ne peut l'assurer à 100% dans tous les cas. Maintenant tout dépend ce que tu as à faire au juste. Soit le calcul est antérieur à l'initialisation de la JTable (par exemple, par ce qu'on a besoin de ce résultat pour initialiser la JTable), et il serait préférable de travailler sur des données typées, en amont de la mise en tableau pour initialiser la JTable, soit il est postérieur, donc dépendant des calculs dans la JTable, et donc, il serait préférable d'utiliser directement la valeur provenant de la JTable (ou du modèle) :

Code :

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

... = (Double)table.getValueAt( table.convertRowIndexToView(ligne), table.convertColumnIndexToView(colonne));

**lefou123456** · 21/04/2017, 11h54

Merci beaucoup pour cette réponse complète, effectivement je cherchais à faire des calculs selon les données inscrites dans la JTable après son initialisation !

**lefou123456** · 21/04/2017, 12h14

Le problème persiste cependant, ClassCastException est toujours attrapé avec le message : "java.lang.String can't be cast to java.lang.Double "
Peu importe la façon dont j'essais...

Code :

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

list[i] = Double.parseDouble((String)table.getValue(table.convertRowIndexToView(0), table.convertColumnIndexToView(i));

Code :

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

list[i] = (Double)table.getValueAt(table.convertRowIndexToView(0), table.convertColumnIndexToView(i));

Le même résultat...

**joel.drigo** · 21/04/2017, 12h23

Tu es sûr que c'est bien sur cette ligne l'exception ?
Dans ton modèle ou tableau, la donnée, c'est du String ou du Double ? Et dans ton modèle, tu as rédéfini la méthode getColumnClass ? De quelle manière ? Par exemple, si ta donnée est du String, et que tu as redéfini cette méthode pour que la colonne soit traitée comme du Double, alors tu auras une ClassCastException, mais pas dans la ligne d'appell du getValueAt(), dans l'exécution de la JTable elle-même, au niveau du rendu de la colonne.

**lefou123456** · 21/04/2017, 12h29

En fait les messages d'erreur m'indiquent deux endroits, mais je commence à me dire que c'est effectivement pas ça l'erreur :

Exception in thread "AWT-EventQueue-0" java.lang.ClassCastException: java.lang.String cannot be cast to java.lang.Double
at com.louloux.view.Window.calculate(Window.java:105)
at com.louloux.view.Window$CalculateListener.actionPerformed(Window.java:130)
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)
at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(Unknown Source)
at java.awt.Component.processMouseEvent(Unknown Source)
at javax.swing.JComponent.processMouseEvent(Unknown Source)
at java.awt.Component.processEvent(Unknown Source)
at java.awt.Container.processEvent(Unknown Source)
at java.awt.Component.dispatchEventImpl(Unknown Source)
at java.awt.Container.dispatchEventImpl(Unknown Source)
at java.awt.Component.dispatchEvent(Unknown Source)
at java.awt.LightweightDispatcher.retargetMouseEvent(Unknown Source)
at java.awt.LightweightDispatcher.processMouseEvent(Unknown Source)
at java.awt.LightweightDispatcher.dispatchEvent(Unknown Source)
at java.awt.Container.dispatchEventImpl(Unknown Source)
at java.awt.Window.dispatchEventImpl(Unknown Source)
at java.awt.Component.dispatchEvent(Unknown Source)
at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
at java.awt.EventQueue.access$500(Unknown Source)
at java.awt.EventQueue$3.run(Unknown Source)
at java.awt.EventQueue$3.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
at java.awt.EventQueue$4.run(Unknown Source)
at java.awt.EventQueue$4.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
at java.awt.EventQueue.dispatchEvent(Unknown Source)
at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.run(Unknown Source)

Quant au code, le voici :

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
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
public class Window extends JFrame implements Observer {
 
	public static int height = 600;
	public static int width = (int) (1.618 * height);
	private double[] list = {};
 
	private JPanel container = new JPanel();
	private JPanel center = new JPanel();
	private JPanel west = new JPanel();
	private JButton button = new JButton("Calculate");
 
	String[] title = { "", "", "", "", "", "",
			"", "Total" };
 
	Object[][] data = { { "1", "1", "1", "1", "1", "1", "1", "" }, { "", "", "", "", "", "", "", "" },
			{ "", "", "", "", "", "", "", "" }, { "", "", "", "", "", "", "", "" }};
 
	private JTable table = new JTable(data, title);
	private JScrollPane scrollPane = new JScrollPane(table);
 
	private AbstractControler controler;
 
	public Window(AbstractControler controler) {
		this.setSize(width, height);
		this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		this.setLocationRelativeTo(null);
 
		initComponents();
 
		this.controler = controler;
		this.setVisible(true);
 
	}
 
	public void initComponents() {
 
		center.setLayout(new BorderLayout());
		center.add(scrollPane, BorderLayout.CENTER);
		west.add(button);
		west.setPreferredSize(new Dimension(150, 600));
		west.setBackground(Color.BLUE);
		container.setLayout(new BorderLayout());
		container.add(west, BorderLayout.WEST);
		container.add(center, BorderLayout.CENTER);
		this.setContentPane(container);
 
		button.addActionListener(new CalculateListener());
 
		initTable();
 
	}
 
	public void initTable() {
		int colWidth = 100;
		int rowHeight = 50;
 
		table.setFont(new Font("Arial", Font.BOLD, 15));
		TableColumn col;
		for (int i = 0; i < table.getColumnCount(); i++) {
			col = table.getColumnModel().getColumn(i);
			col.setPreferredWidth(colWidth);
		}
		for (int i = 0; i < table.getRowCount(); i++) {
			table.setRowHeight(i, rowHeight);
		}
 
		DefaultTableCellRenderer custom = new DefaultTableCellRenderer();
		custom.setHorizontalAlignment(JLabel.CENTER);
		for (int i = 0; i < table.getColumnCount(); i++) {
			table.getColumnModel().getColumn(i).setCellRenderer(custom);
		}
 
	}
 
	public void calculate() {
		double value = 0;
		for(int i = 0; i < data[0].length - 1; i++){
			if(table.getValueAt(0, i).equals(""))
				value = 0;
			else
				value = (Double)table.getValueAt(table.convertRowIndexToView(0), table.convertColumnIndexToView(i));
 
			this.list[i] = value;
		}
 
		for(double number : list){
			System.out.println(number);
		}
 
//		controler.setList(this.list);
	}
 
	@Override
	public void update(double[] list) {
		for(int i = 0; i < list.length; i++){
			table.setValueAt(String.valueOf(list[i]), 0, i);
		}
	}
 
	// CLASS //
 
	public class CalculateListener implements ActionListener {
 
		@Override
		public void actionPerformed(ActionEvent arg0) {
			calculate();
 
		}
 
	}
 
}

**joel.drigo** · 21/04/2017, 15h24

La ligne 105 c'est value = (Double)table.getValueAt(table.convertRowIndexToView(0), table.convertColumnIndexToView(i)); ? Et ça te fait aussi un cast

Ensuite, on voit que tes données, c'est du String. Donc, c'est sûr, un cast en Double ne fonctionnera pas. Mais pourquoi ne pas utiliser des doubles directement ? Si le souci, c'est que le DefaultTableCellRenderer ne gère pas bien le cas, tu peux le modifier légèrement :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
DefaultTableCellRenderer custom = new DefaultTableCellRenderer() {
protected void setValue(Object value) {
                if ( value instanceof Double ) {
                      Double val = (Double)value;
                      if ( val==0 ) super.setValue(null);
                      else super.setValue(decimalFormat.format(val)); // decimalFormat un DecimalFormat avec le pattern d'affichage souhaité
                }
                else {
                     super.setValue(value);
                }
	}
}

Tu gères toutes tes données en Double, plus de cast, de Double.parseDouble, ou de String.valueOf() à faire.

Par ailleurs, dans calculate(), de la même façon qu'il est préférable d'accèder aux données de la JTable directement, il est préférable d'utiliser table.getRowCount() plutôt que data[0].length.

A priori, si je comprends bien, tu cherches à avoir une ligne de totaux en bas de la JTable, en utilisant la dernière ligne pour afficher les résulats. Malheureusement, si c'est un moyen assez simple de gérer le truc, d'autant qu'il n'y a pas de moyen "natif" de gérer un footer dans une JTable, ce n'est pas un moyen très efficace : déjà ça t'inderdit les tris, les filtres. Ensuite, en cas de trop de lignes dans la JTable, les totaux sont invisibles. Si la JTable doit être éditable, il faut gérer le cas de façon particulière. Etc, etc.
Une solution peut être d'utiliser le JideScrollPane (faisant partie JIDE Common Layer, licence GPL) qui permet d'avoir un footer.

Exemple (sans gestion des filtres) :

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
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
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
public class JTableWithFooterExemple {
 
	public static void main(String[] args) {
 
		JFrame frame = new JFrame("Exemple");
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
 
		String[] cols = {"Col1","Col2","Col3"};
 
		Object[][] datas = new Object[30][cols.length];
 
		for(int i=0; i<datas.length; i++) {
			datas[i][0]=i%2==0?"toto":null;
			for(int j=1; j<datas[i].length; j++) {
				datas[i][j]=ThreadLocalRandom.current().nextDouble(-100,100);
			}
		}
 
 
		DefaultTableModel model = new DefaultTableModel(datas, cols) {
			public java.lang.Class<?> getColumnClass(int columnIndex) {
				switch(columnIndex) {
				case 0:
					return String.class;
				default:
					return Double.class;
				}
			}
		};
		JTable table = new JTable(model);
		table.setFillsViewportHeight(true);
 
		JideScrollPane scrollPane = new JideScrollPane(table);
		scrollPane.setColumnHeaderView(table.getTableHeader());
 
		JTable footer = new JTable(new FooterTableModel(model, 
				new FooterTableModel.ColumnFooter<String, Long>(0, String.class, s-> s.filter(Objects::nonNull).filter(x->!x.isEmpty()).count()),
				new FooterTableModel.ColumnFooter<Double, Double>(1, Double.class, s-> s.mapToDouble(d->d.doubleValue()).sum()),
				new FooterTableModel.ColumnFooter<Double, Double>(2, Double.class, s-> {
					OptionalDouble od = s.mapToDouble(d->d.doubleValue()).average(); 
					if ( od.isPresent() ) {
						return od.getAsDouble();
					}
					return null;
				})
				)); 
		footer.setFillsViewportHeight(false);
		footer.setRowSelectionAllowed(false);
		footer.setColumnSelectionAllowed(false);
		footer.setBorder(BorderFactory.createEmptyBorder(3,0,0,0));
		JPanel panelFooter = new JPanel(new BorderLayout());
		panelFooter.setBorder(BorderFactory.createMatteBorder(3, 0, 0, 0, Color.RED));
		panelFooter.add(footer);
		scrollPane.setColumnFooterView(panelFooter);
		footer.setColumnModel(table.getColumnModel());
 
		frame.add(scrollPane);
 
 
		frame.setSize(640, 400);
		frame.setLocationRelativeTo(null);
		frame.setVisible(true); 
 
	}
 
	public static class FooterTableModel extends AbstractTableModel implements TableModelListener {
 
		final private AbstractTableModel tableModel;
		final private Map<Integer, ColumnFooter<?,?>> columnFooters;
		private Object[] data;
 
		public FooterTableModel(AbstractTableModel tableModel, ColumnFooter<?,?>...footers) {
			this.tableModel=tableModel;
			this.columnFooters=Arrays.stream(footers).collect(Collectors.toMap(ColumnFooter::getColumnIndex, Function.identity()));
			init();
			tableModel.addTableModelListener(this);
		}
 
		private void init() {
			data = new Object[tableModel.getRowCount()];
			updateAllColumns();
		}
 
		private void updateAllColumns() {
			columnFooters.values().forEach(this::updateColumn);
		}
 
		@SuppressWarnings({ "rawtypes", "unchecked" })
		private void updateColumn(ColumnFooter footer) {
			columnAsStream(footer.getColumnIndex()).forEach(System.out::println);
			setValueAt(footer.compute(columnAsStream(footer.getColumnIndex()).map(footer::cast)), 0, footer.getColumnIndex());
		}
 
		public void updateColumn(int columnIndex) {
			ColumnFooter<?, ?> footer = columnFooters.get(columnIndex);
			if ( footer==null ) {
				setValueAt(null, 0, columnIndex);
			}
			else {
				updateColumn(footer);
			}
		}
 
		private Stream<?> columnAsStream(int columnIndex) {
			Iterable<?> iter = new Iterable<Object>() {
				@Override
				public Iterator<Object> iterator() {
					return new Iterator<Object>() {
 
						private int rowIndex=0;
						@Override
						public boolean hasNext() {
							return rowIndex<tableModel.getRowCount();
						}
 
						@Override
						public Object next() {
							if ( hasNext() ) {
								return tableModel.getValueAt(rowIndex++, columnIndex);
							}
							throw new NoSuchElementException();
						}
 
					};
				}
			};
			return StreamSupport.stream(iter.spliterator(), false);
		}
 
		@Override
		public int getColumnCount() {
			return tableModel.getColumnCount();
		}
 
		@Override
		public int getRowCount() {
			return 1;
		}
 
		@Override
		public Object getValueAt(int rowIndex, int columnIndex) {
			if ( rowIndex!=0 ) {
				throw new IndexOutOfBoundsException();
			}
			return data[columnIndex];
		}
 
		@Override
		public void setValueAt(Object value, int rowIndex, int columnIndex) {
			if ( rowIndex!=0 ) {
				throw new IndexOutOfBoundsException();
			} 
			data[columnIndex]=value;
			fireTableCellUpdated(rowIndex, columnIndex);
		}
 
		@Override
		public void tableChanged(TableModelEvent event) {
			switch (event.getType()) {
			case TableModelEvent.ALL_COLUMNS:
				init();
				break;
			case TableModelEvent.INSERT:
				updateAllColumns();
				break;
			case TableModelEvent.UPDATE:
				int columnIndex = event.getColumn();
				updateColumn(columnIndex);
			default:
				break;
			}
		}
 
		public static class ColumnFooter<T, R> {
 
			private final Class<T> c;
			private final Function<Stream<T>, R> function;
			private final int columnIndex;
 
			public ColumnFooter(int columnIndex, Class<T> c, Function<Stream<T>,R> function) {
				this.columnIndex=columnIndex;
				this.c=Objects.requireNonNull(c);
				this.function=Objects.requireNonNull(function);
			}
 
			public R compute(Stream<T> values) {
				return function.apply(values);
			}
 
			public int getColumnIndex() {
				return columnIndex;
			}
 
			public T cast(Object value) {
				return c.cast(value);
			}
 
		}
 
	}
 
 
}

**lefou123456** · 21/04/2017, 16h34

Mais le problème est forcement celui-ci; si je retire cette ligne d'affectation, ça marche parfaitement !

**joel.drigo** · 21/04/2017, 16h48

Il est impossible d'avoir le message "java.lang.String cannot be cast to java.lang.Double" avec l'instruction suivante puisqu'il n'y a pas de cast en Double dans cette instruction. :
avec list[i] = Double.parseDouble((String)table.getValue(table.convertRowIndexToView(0), table.convertColumnIndexToView(i));.

Et avec le code que tu as montré, il ne peut y avoir d'erreur de cast, les données étant des String. Il pourrait y avoir des erreurs de conversion en Double si tu ne n'avais pas le test d'égalité à "". Dans ton code, c'est sûr, il y aura des ArrayIndexOutOfBoundException (list n'est jamais dimensionné autrement qu'à 0). Mais des "java.lang.String cannot be cast to java.lang.Double", à part si tu gardes value = (Double)table.getValueAt(table.convertRowIndexToView(0), table.convertColumnIndexToView(i));, je ne vois pas comment.

Mais pour éviter tout problème, le plus simple est de travailler avec les bons types, au lieu de faire des aller-retours avec des String.

**lefou123456** · 21/04/2017, 17h11

Je viens d'essayer de changer le type du tableau en Double, mais ça ne change rien...

**joel.drigo** · 21/04/2017, 17h16

Tu peux me montrer le code que ça donne... Et, sinon, tu recompiles bien à chaque modif pour retester ?

**lefou123456** · 21/04/2017, 17h22

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
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
package com.louloux.view;
 
import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Dimension;
import java.awt.Font;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
 
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTable;
import javax.swing.table.DefaultTableCellRenderer;
import javax.swing.table.TableColumn;
 
import com.louloux.controler.AbstractControler;
import com.louloux.observer.Observer;
 
public class Window extends JFrame implements Observer {
 
	public static int height = 600;
	public static int width = (int) (1.618 * height);
	private double[] list = {};
 
	private JPanel container = new JPanel();
	private JPanel center = new JPanel();
	private JPanel west = new JPanel();
	private JButton button = new JButton("Calculate");
 
	String[] title = { "", "", "", "", "", "", "", "Total" };
 
	Double[][] data = { { .0, .0, .0, .0, .0, .0, .0, .0 }, { .0, .0, .0, .0, .0, .0, .0, .0 },
			{ .0, .0, .0, .0, .0, .0, .0, .0 }, { .0, .0, .0, .0, .0, .0, .0, .0 },
			{ .0, .0, .0, .0, .0, .0, .0, .0 } };
 
	private JTable table = new JTable(data, title);
	private JScrollPane scrollPane = new JScrollPane(table);
 
	private AbstractControler controler;
 
	public Window(AbstractControler controler) {
		this.setSize(width, height);
		this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		this.setLocationRelativeTo(null);
 
		initComponents();
 
		this.controler = controler;
		this.setVisible(true);
 
	}
 
	public void initComponents() {
 
		center.setLayout(new BorderLayout());
		center.add(scrollPane, BorderLayout.CENTER);
		west.add(button);
		west.setPreferredSize(new Dimension(150, 600));
		west.setBackground(Color.BLUE);
		container.setLayout(new BorderLayout());
		container.add(west, BorderLayout.WEST);
		container.add(center, BorderLayout.CENTER);
		this.setContentPane(container);
 
		button.addActionListener(new CalculateListener());
 
		initTable();
 
	}
 
	public void initTable() {
		int colWidth = 100;
		int rowHeight = 50;
 
		table.setFont(new Font("Arial", Font.BOLD, 15));
		TableColumn col;
		for (int i = 0; i < table.getColumnCount(); i++) {
			col = table.getColumnModel().getColumn(i);
			col.setPreferredWidth(colWidth);
		}
		for (int i = 0; i < table.getRowCount(); i++) {
			table.setRowHeight(i, rowHeight);
		}
 
		DefaultTableCellRenderer custom = new DefaultTableCellRenderer();
		custom.setHorizontalAlignment(JLabel.CENTER);
		for (int i = 0; i < table.getColumnCount(); i++) {
			table.getColumnModel().getColumn(i).setCellRenderer(custom);
		}
 
	}
 
	public void calculate() {
		double value = 0;
 
		for (int i = 0; i < data[0].length - 1; i++) {
			if (table.getValueAt(0, i).equals(null))
				value = 0;
			else
				value = (Double) table.getValueAt(table.convertRowIndexToView(0), table.convertColumnIndexToView(i));
 
			this.list[i] = value;
		}
 
		for (double number : list) {
			System.out.println(number);
		}
 
		// controler.setList(this.list);
	}
 
	@Override
	public void update(double[] list) {
		for (int i = 0; i < list.length; i++) {
			table.setValueAt(String.valueOf(list[i]), 0, i);
		}
	}
 
	// CLASS //
 
	public class CalculateListener implements ActionListener {
 
		@Override
		public void actionPerformed(ActionEvent arg0) {
			calculate();
 
		}
 
	}
 
}

Oui oui, à chaque modification...

**joel.drigo** · 21/04/2017, 17h28

Si j'exécute ton code (en virant AbstractController et Observer que je n'ai pas), alors j'obtiens comme je le disais :

Exception in thread "AWT-EventQueue-0" java.lang.ArrayIndexOutOfBoundsException: 0

at Window.calculate(Window.java:100)
at Window$CalculateListener.actionPerformed(Window.java:122)
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)
at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(Unknown Source)
at java.awt.Component.processMouseEvent(Unknown Source)
at javax.swing.JComponent.processMouseEvent(Unknown Source)
at java.awt.Component.processEvent(Unknown Source)
at java.awt.Container.processEvent(Unknown Source)
at java.awt.Component.dispatchEventImpl(Unknown Source)
at java.awt.Container.dispatchEventImpl(Unknown Source)
at java.awt.Component.dispatchEvent(Unknown Source)
at java.awt.LightweightDispatcher.retargetMouseEvent(Unknown Source)
at java.awt.LightweightDispatcher.processMouseEvent(Unknown Source)
at java.awt.LightweightDispatcher.dispatchEvent(Unknown Source)
at java.awt.Container.dispatchEventImpl(Unknown Source)
at java.awt.Window.dispatchEventImpl(Unknown Source)
at java.awt.Component.dispatchEvent(Unknown Source)
at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
at java.awt.EventQueue.access$500(Unknown Source)
at java.awt.EventQueue$3.run(Unknown Source)
at java.awt.EventQueue$3.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
at java.awt.EventQueue$4.run(Unknown Source)
at java.awt.EventQueue$4.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
at java.awt.EventQueue.dispatchEvent(Unknown Source)
at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.run(Unknown Source)

Et c'est dû au dimensionnement du tableau list (qui est de taille 0).

**lefou123456** · 21/04/2017, 17h29

Ah ok, donc je devrais préciser la taille de la liste dans sa déclaration ?

**joel.drigo** · 21/04/2017, 17h33

Au moins au moment de l'initialiser. Comme ça, même si d'aventures des lignes étaient ajoutées ou supprimées dans la JTable, ça continuerait à fonctionner.

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
public void calculate() {
		double value = 0;
 
		this.list = new double[table.getRowCount()-1];
		for (int i = 0; i < this.list.length; i++) {
			if (table.getValueAt(0, i).equals(null))
				value = 0;
			else
				value = (Double) table.getValueAt(table.convertRowIndexToView(0), table.convertColumnIndexToView(i));
 
			this.list[i] = value;
		}
 
		for (double number : list) {
			System.out.println(number);
		}
 
		// controler.setList(this.list);
	}

Pour moi, il n'a aucune raison d'être en attribut. Visiblement, il est prévu de le passer au controleur qui je suppose va le retourner via l'observer, qui va l'affecter à la JTable par update(), donc en passant un paramètre. Fais-en une variable locale à calculate().

**lefou123456** · 21/04/2017, 17h42

Super, maintenant quand je clique sur le bouton ça marche, mais lorsque je veux modifier le contenue d'une cellule depuis la fenêtre ça ne fonctionne pas...
Je crois que lorsque je modifie la valeur ça envoi un String alors que le programme attend un Double...

**joel.drigo** · 21/04/2017, 17h56

Le problème vient de l'éditeur par défaut, qui ne gère que des JTextField, JCheckbox, ou JComboBox, donc du String ou du Boolean.

La première solution, sans trop modifier le code serait de redéfinir l'éditeur :

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
DefaultCellEditor editor = new DefaultCellEditor(new JTextField()) {
			@Override
			public Object getCellEditorValue() {
				String s = (String)super.getCellEditorValue();
				if ( s.isEmpty() ) { 
					return 0d;
				}
				try {
					return Double.parseDouble(s);
				}
				catch (NumberFormatException e) { 
					return 0d;
				}
			}
		};

		custom.setHorizontalAlignment(JLabel.CENTER);
		for (int i = 0; i < table.getColumnCount(); i++) {
			table.getColumnModel().getColumn(i).setCellRenderer(custom);
			table.getColumnModel().getColumn(i).setCellEditor(editor);
		}

L'autre solution serait de passer un modèle où tu indiques que le type est Double, ce qui va permettre à la JTable de choisir un éditeur qui gère le Double. L'avantage, c'est qu'on ne pourra pas sortir de la cellule tant que la saisie n'est pas un double. Par exemple, au plus simple :

Code :

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

1
2
3
4
5
6
private TableModel tableModel = new DefaultTableModel(data, title) {
		public java.lang.Class<?> getColumnClass(int columnIndex) {
			return Double.class;
		};
	};
	private JTable table = new JTable(tableModel);

La première solution à l'avantage de permettre avec plus de boulot de faire un éditeur dédié, qui ne permettra pas de saisir autre chose qu'un double, mais avec le code que je t'ai donné, si on saisit des lettres, la saisie est prise en compte et on peut sortir du champ, mais la valeur est forcée à 0.

A noter, qu'en toute cohérence, il faudrait que lorsqu'on clic sur le bouton, la saisir soit forcée à s'arrêter :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
public class CalculateListener implements ActionListener {
 
		@Override
		public void actionPerformed(ActionEvent arg0) {
			TableCellEditor editor = table.getCellEditor();
			if ( editor!=null ) {
				editor.stopCellEditing();
			}
			calculate();
 
		}
 
	}

Mais là, le souci, c'est si la saisie est erronée, on fait le calcul avec l'ancienne valeur et la cellule reste en édition. On peut annuler le calcul en revanche :

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
	public class CalculateListener implements ActionListener {
 
		@Override
		public void actionPerformed(ActionEvent arg0) {
			TableCellEditor editor = table.getCellEditor();
			if ( editor!=null ) {
				editor.stopCellEditing();
				if ( table.isEditing() ) {
					JOptionPane.showMessageDialog(table, "Error: current input is not a number","Calculate",JOptionPane.ERROR_MESSAGE);
					return;
				}
			}
			calculate();
 
		}
 
	}

**lefou123456** · 21/04/2017, 18h17

Merci beaucoup pour toute cette aide que vous m'avez apporté; il ne me reste plus qu'à finir les algorithmes de calcul, donc pas grand chose.
En dehors de ça tout est parfait !

Manipuler les données d'un JTable

Composants Java

Discussions similaires

Partager

Partager