Discussion :

[nadong94]

Salut à tous,

Ma JTable est en fait un log de mon application, et donc il faut que les infos soit du plus récent au plus vieux.
Hors ça fonctionne une fois sur deux pour le premier bouton est 0 fois pour le deuxième, sachant qu'ils font appel à la même méthode et de la même façon.
La méthode addLog :

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
 
public void addLog(String message, infoMessage info)
{
	JTable log = IhmConverter.getInstance().getTblLog();
	DefaultTableModel model = (DefaultTableModel) log.getModel();
	TableCellRenderer renderer = new CustomTableCellRenderer();
	SimpleDateFormat dateFormat = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss");
	Date toDay = new Date();
 
	if (info.equals(infoMessage.info))
	{
		String val = dateFormat.format(toDay) +" - INFO : "+ message;
		model.addRow(new Object[]{val});
		model.fireTableDataChanged();
	}
	else if (info.equals(infoMessage.warning))
	{
		String val = dateFormat.format(toDay) +" - WARNING : "+ message;
		model.addRow(new Object[]{val});
		model.fireTableDataChanged();
	}
	else if (info.equals(infoMessage.error))
	{
		String val = dateFormat.format(toDay) +" - ERROR : "+ message;
		model.addRow(new Object[]{val});
		model.fireTableDataChanged();
	}
	((TableRowSorter<?>) log.getRowSorter()).toggleSortOrder(0); //0 = the first column
	log.repaint();
	log.setDefaultRenderer(String.class, renderer);
}

☼ Comme vous l'avez compris le but est que ça fonctionne quelque soit le nombre de fois que j’appelle la méthode.

Merci d'avance

[joel.drigo]

Salut,

Au lieu d'appeler toggleSortOrder() à chaque fois que tu ajoutes un message (ce qui, comme son nom l'indique, inverse le tri sur la colonne), tu l'appelles qu'une seule fois à la création de la JTable, ou mieux, tu affectes un Comparator qui inverse le tri.

Par ailleurs, inutile de faire des fireTableDataChanged(), le DefaultTableModel gère très bien tout seul les évenements. D'ailleurs, ce n'est pas un tableDataChanged mais rowsInserted qu'il faudrait envoyer.

[nadong94]

Au lieu d'appeler toggleSortOrder() à chaque fois que tu ajoutes un message (ce qui, comme son nom l'indique, inverse le tri sur la colonne), tu l'appelles qu'une seule fois à la création de la JTable, ou mieux, tu affectes un Comparator qui inverse le tri.

Comment je m'y prend pour faire un comparateur qui inverse le tri ?

Car quand je le met au moment où je construit le log ça ne marche pas correctement non-plus.

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
tblLog = new JTable();
DefaultTableModel model = (DefaultTableModel) tblLog.getModel();
model.addColumn("Messages");
tblLog.getColumnModel().getColumn(0)
	.setCellRenderer(new CustomTableCellRenderer());
tblLog.setAutoCreateRowSorter(true);
((TableRowSorter<?>) tblLog.getRowSorter()).toggleSortOrder(0);
tblLog.setEnabled(false);
tblLog.setBackground(Color.lightGray);
tblLog.setGridColor(Color.lightGray);
tblLog.setBorder(new LineBorder(new Color(0, 0, 0), 1, true));
tblLog.setSelectionMode(ListSelectionModel.MULTIPLE_INTERVAL_SELECTION);
tblLog.getSelectionModel().clearSelection();
// constructs the popup menu
popupMenu = new JPopupMenu();
menuItemCopyAll = new JMenuItem("Copy All Rows");
menuItemRemoveAll = new JMenuItem("Remove All Rows");
menuItemCopyAll.addActionListener(new Events());
menuItemCopyAll.setToolTipText("Click to copy all lines");
menuItemRemoveAll.addActionListener(new Events());
menuItemRemoveAll.setToolTipText("Click to remove all lines");
popupMenu.add(menuItemCopyAll);
popupMenu.add(menuItemRemoveAll);
// sets the popup menu for the table
tblLog.setComponentPopupMenu(popupMenu); 
tblLog.addMouseListener(new MouseReleased(tblLog));
elevator = new JScrollPane(tblLog,
	JScrollPane.VERTICAL_SCROLLBAR_AS_NEEDED,
	JScrollPane.HORIZONTAL_SCROLLBAR_AS_NEEDED);
elevator.getViewport().setBackground(Color.WHITE);

panLog = new JPanel();
panLog.setLayout(new BorderLayout());
panLog.add(elevator, BorderLayout.CENTER);

[joel.drigo]

Comment je m'y prend pour faire un comparateur qui inverse le tri ?

Ceci devrait fonctionner :

Code :

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

table.getRowSorter().setComparator(0, sorter.getComparator(0).reversed());

Car quand je le met au moment où je construit le log ça ne marche pas correctement non-plus.

Euh, mais ta première colonne est bien triante ascendante ? Genre c'est la date ou un timestamp, un id incrémetal... ?
Parce que sinon, ne mets pas de row sorter et insert simplement au début au lieu d'ajouter à la fin te ligne de log :

Code :

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

model.insertRow(0, ta ligne de log);

[nadong94]

C'est une table d'une seule colonne avec comme première chaine un SimpleDateFormat.
La deuxième méthode fonctionne très bien :

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
 
2016-04-29 11:02:31 - INFO : Link Excel sheet was created
2016-04-29 11:02:30 - INFO : Cable Excel sheet was created
2016-04-29 11:02:27 - INFO : Port Excel sheet was created
2016-04-29 11:02:27 - INFO : NetworkItem Excel sheet was created
2016-04-29 11:02:27 - INFO : NetworkList Excel sheet was created
2016-04-29 11:02:27 - INFO : HDMT_NEMO.xlsx was created
2016-04-29 11:02:01 - INFO : Link was created
2016-04-29 11:02:01 - INFO : Cable was created
2016-04-29 11:02:01 - INFO : Port was created
2016-04-29 11:02:01 - INFO : NetworkItem was created
2016-04-29 11:02:01 - INFO : NetworkList was created
2016-04-29 11:01:38 - INFO : DB Config Patch Id Definition was created
2016-04-29 11:01:38 - INFO : DB IP Definition was created
2016-04-29 11:01:38 - INFO : DB Logical Port Allocation was created
2016-04-29 11:01:37 - INFO : DB Connectivity Definition was created

Et pour tester la deuxième je déclare le sorte comment ?

Code :

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

1
2
 
tblLog.getRowSorter().setComparator(0, sorter.getComparator(0).reversed());

Avec ceeci ça ne marche pas, car il ne reconnait pas la méthods reversed() :

Code :

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

1
2
 
TableRowSorter<DefaultTableModel> sorter = new TableRowSorter<DefaultTableModel>();

[joel.drigo]

La deuxième méthode fonctionne très bien :

Problème réglé alors.

Et pour tester la deuxième je déclare le sorte comment ?

Ce serait un peu plus complexe, et inutilement plus complexe, mais faisable certes.

Code :

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

1
2
 
tblLog.getRowSorter().setComparator(0, sorter.getComparator(0).reversed());

Avec ceeci ça ne marche pas, car il ne reconnait pas la méthods reversed() :

C'est du Java 8.
Sinon il faudrait faire un truc comme ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
tblLog.getRowSorter().setComparator(0, new Comparator<Object>() {
 
			private final Comparator<Object> c = (Comparator<Object>)tblLog.getRowSorter().getComparator(0);
 
			@Override
			public int compare(Object o1, Object o2) {
				return -c.compare(o1, o2); // ou c.compare(o2,o1)
			}
 
		});

Seulement, ça ne peut pas fonctionner de toute manière parce que tes données ne sont pas directement triables (techniquement si, mais pas sémantiquement, ou fonctionnellement du moins).
Il faut donc les rendre triables en écrivant un comparateur qu'il le fait.
A partir de la chaîne, ça consisterait soit à traiter la chaîne :

• à couper la chaîne par un split sur espace,
• puis de prendre les 2 premières parties, les concaténer pour réobtenir un "datetime"
• faire un parsing avec le SimpleDateFormat utilisé pour générer ce "datetime"
• comparer les dates

Assez lourd donc.

Le plus simple serait de créer une classe qui représente une ligne de log, avec les différents attributs qui la caractérisent (date, type, message...). De faire un renderer qui fait la conversion en String pour l'afficher comme actuellement, et de faire un comparateur décroissant sur l'attribut date.

Le seul avantage de cette méthode serait de permettre à l'utilisateur de changer le tri (passer d'un ascendant à un descendant et inversement). Voire d'étendre les possibilités de tri aux autres attributs (type en particulier)

[nadong94]

Problème réglé alors.

Oui en effet merci bien

Le seul avantage de cette méthode serait de permettre à l'utilisateur de changer le tri (passer d'un ascendant à un descendant et inversement). Voire d'étendre les possibilités de tri aux autres attributs (type en particulier)

Dans mon cas cela n'est pas nécessaire, donc je vais opter pour la précédente façon