Discussion :

[Mattgic]

Bonjour à tous

Petite question mais délicate....
J'ai une IHM composée de trois panels.
1er panel: un TreeView,
2ème panel: un graphique
3ème panel: une ListView.
(Le tout dans une winforms)

Tout ce petit monde s'est enregistré auprès du sujet (pattern observer).
Quand le sujet change je parcours donc ma lise "d'observers" et lance un "DoRefresh" sur chacun d'eux! jusqu'à là tout va bien !!

Code :

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

1
2
3
4
5
 
 foreach (IOBserver obs in obsList)
 {
      obs.DoRefresh();
 }

Dés qu'un des panel effectue sont "DoRefresh" il va donc recharger ses données, éventuellement faire des traitements au cas où, puis afficher le resultat!

Ca pourrait être bien, mais le refresh général est long et du coup faut patienter (1 à 2sec - oui je suis pressé). Le traitement pour chaque panel est en soin rapide, mais bon tant que chaque composant fasse son petit traitement, redéssine (par ex: le graphique) bah c'est long!

Cependant, l'idée m'est venue de créer un Thread dans ma boucle de refresh!

Code :

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

1
2
3
4
5
6
 
 foreach (IOBserver obs in obsList)
 {
      Thread th = new Thread(new ThreadStart(obs.DoRefresh));
      th.Start();
 }

boom ! eh oui le fameux cross-thread machin. Pareit que seul le thread qui a créé l'IHM peut le modifier!

Bon d'accord mais alors pour faire en sorte d'avoir quand même l'impression que chaque "Observer" se met à jour tout seul sans attendre que son camarade ait terminé on fait comment !!??

La solution de mettre des Invoke ou Begin/EndInvoke (asynchrone) ne marche pas plus.. en fait je n'ai alors plus d'exception mais le temps est multiplié par 10!

alors comment faire ?
merci !!

[tomlev]

Je pense qu'il manque des éléments pour qu'on puisse t'aider... En quoi consistent tes objets qui implémentent IObserver ? Quel est le "sujet" dont tu parles ?

[DonQuiche]

Bonjour. Ton panneau ne devrait contenir que du code UI, aucun traitement de récupération/transformation de données. Si seules les opérations UI sont réalisées sur le thread UI et que le reste est fait dans un autre thread, tu devrais avoir aucun problème, à moins d'une UI assez monstrueuse.

[Mattgic]

Je pense qu'il manque des éléments pour qu'on puisse t'aider... En quoi consistent tes objets qui implémentent IObserver ? Quel est le "sujet" dont tu parles ?

Bonjour,
Merci de vos réponses,
Le sujet représente un modèle de stockage de DVD.
Treeview: vue hiérarchique,
Graphique: fréquence,
Listview: auteur, info...

Il est vrai que chaque composant panel ("user control") font des requêtes dans un fichier XML pour trouver les infos .

Quand par exemple un DVD est sélectionné dans le Treeview, alors le graph va chercher les infos sur la fréquence , la listview va chercher les infos etc..

Donc d'après DonQuiche je devrais faire mes recherches traitement dans un thread à part et lancer après le refresh UI après ?

[tomlev]

Bonjour,
Merci de vos réponses,
Le sujet représente un modèle de stockage de DVD.
Treeview: vue hiérarchique,
Graphique: fréquence,
Listview: auteur, info...

OK, donc les observers, c'est les contrôles ?

En gros, toutes les tâches de récupération de données peuvent être faites dans un worker thread, et toutes celles qui accèdent à l'UI doivent être faites dans le thread de l'UI. Ce qui implique qu'il faut bien séparer les 2...

[Mattgic]

Oui les observers sont des controls !
Donc je dois utiliser un thread pour chaque traitement , puis quand les thread ont fini je "demande" à l'ui de faire un refresh avec les nouvelles données ?

[tomlev]

Donc je dois utiliser un thread pour chaque traitement , puis quand les thread ont fini je "demande" à l'ui de faire un refresh avec les nouvelles données ?

Oui, ou alors un thread pour tous les traitements, à toi de voir... mais n'oublie pas que lancer de nombreux threads est couteux en ressources

[Mattgic]

Ok merci, je vais tenter !

[Mattgic]

Ok ! problème à priori résolu!
Voici ce que je fais suivant vos conseils:

Dans le sujet j'ai toujours

Code :

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

1
2
3
4
5
 
foreach (IOBserver obs in obsList)
 {
      obs.DoRefresh();
 }

et là maintenant ça change !

dans chaque observer (implémentant donc DoRefresh) j'utilise un BackgroundWorker

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
 
public void DoRefresh()
{
	BackgroundWorker worker = new BackgroundWorker();
	worker.DoWork += new DoWorkEventHandler(DoCalculation);
	worker.RunWorkerCompleted += new RunWorkerCompletedEventHandler(DoRepaint);
	worker.RunWorkerAsync();
}
 
//Méthode lancée de manière asynchrone !
public void DoCalculation(object sender, DoWorkEventArgs args)
{
	//Calculs divers
}
 
//Méthode quand tout est terminé !
public void DoRepaint(object sender, RunWorkerCompletedEventArgs args)
{
	//Une fois mes calculs réalisés, je peux faire des refresh sur mes composants graphiques
}

Voilà !
Merci de votre aide