[RCP] Treeviewer non thread-safe ?

Version imprimable

08/11/2005, 16h52
Guildux

[RCP] Treeviewer non thread-safe ?

Bonjour,

Dans l'appli que je suis en train de développer sous Eclipse et utilisant RCP, j'ai besoin de faire un raffraichissement automatique du contenu d'un treeviewer contenu dans ma vue.

Pour cela, j'ai créé une classe qui hérite de java.util.TimerTask qui me lance un thread qui effectue mon raffraichissement de manière "cadencée".

Le probleme que je rencontre est que lorsque que met à jour les données de mon treeviewer (méthode refresh() ou setInput() ), une exception est lancée et mon thread est killé.

Les widgets RCP ne seraient-ils pas thread-safe ? Quelqu'un a-t-il déjà réussi a faire quelque chose de similaire ? Y'aurait-il un timer (ou des threads) spécifique à RCP ?

Je remercie d'avance toute personne pouvant m'apporter une réponse a ces questions.
08/11/2005, 19h09
Guildux

Après avoir quelque peu cherché, j'ai vu que le TreeViewer stocke les datas dans un CustomHashTable.

Cette classe n'hérite pas de HashTable ... mais cette derniere n'étant pas thread safe, quelqu'un sait-il si le comportement du CustomHashTable de RCP est le même dans un environnement multi-threadé ?

En ayant affiché l'Exception, voici la methode fautive :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
org.eclipse.swt.SWTException: Invalid thread access
	at org.eclipse.swt.SWT.error(SWT.java:2942)
	at org.eclipse.swt.SWT.error(SWT.java:2865)
	at org.eclipse.swt.SWT.error(SWT.java:2836)
	at org.eclipse.swt.widgets.Widget.error(Widget.java:395)
	at org.eclipse.swt.widgets.Widget.checkWidget(Widget.java:298)
	at org.eclipse.swt.widgets.Table.deselectAll(Table.java:944)
	at org.eclipse.jface.viewers.TableViewer.setSelectionToWidget(TableViewer.java:1011)
	at org.eclipse.jface.viewers.StructuredViewer.setSelectionToWidget(StructuredViewer.java:1496)
	at org.eclipse.jface.viewers.StructuredViewer.preservingSelection(StructuredViewer.java:1208)
	at org.eclipse.jface.viewers.StructuredViewer.refresh(StructuredViewer.java:1262)

Dans Widget.checkWidget, on trouve :

Code:

1
2
3
4
5
6
7
 
protected void checkWidget () {
	Display display = this.display;
	if (display == null) error (SWT.ERROR_WIDGET_DISPOSED);
	if (display.thread != Thread.currentThread ()) error (SWT.ERROR_THREAD_INVALID_ACCESS);
	if ((state & DISPOSED) != 0) error (SWT.ERROR_WIDGET_DISPOSED);
}

Il semblerait donc qu'aucun widget RCP ne soient thread-safe puisqu'il cherche systématiquement à redessiner le widget dans un thread qui n'est pas le thread principal ...

24/11/2005, 17h12
ninisan

En effet un thread externe ne peut pas acceder au thread principal

Pour solutionner :

display.asyncExec(new Refresher());

private class Refresher implements Runnable {
public void run() {
viewer.refresh();
}
}

Voila, j'espere que ca aura fait avancer la chose !

Virginie

Bonjour, je remonte ce message car j'ai un problème similaire.
J'arrive à mettre à jour une fois par seconde un TableTreeViewer à partir d'un thread non-UI.
Mais j'ai remarqué que mon programme consomme 80 ko de mémoire en plus chaque seconde. Je pense que ça vient du fait que je fais un refresh de mon TableTreeViewer.
Y a-t-il une solution pour rafraichir un Viewer à partir d'un timer afin de relancer le LabelProvider du Viewer ?
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 public void run() { while(boucle){ try { timer.sleep(1000); Display.getDefault().asyncExec(new Runnable(){ public void run() { viewer.refresh(); } }); } catch (InterruptedException e) { e.printStackTrace(); } } }