Discussion :

[trolldom]

Bonjour,
Je développe une application winform où plusieurs fenêtres présentent les mêmes objets qui implémentent les interfaces qui vont bien pour le databinding. Pour assurer la synchronisation de mes vues et pour effectuer mes cas d'utilisateur j'ai un assez grand nombre de eventHandler. Je suis pas sûr de la qualité de ma manière de gérer mes eventHandler et voudrait donc avoir des avis.
Notemment dans une fenetre je crée des cycles d'évenement car j'ai des handler qui réagissent à la vue et d'autre au modèle comme illustré en psuedo-code ci après.

Code vb :

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

1
2
3
 
AddHandler CurrentUser.PropertyChanged, Addressof CurrentUser_PropertyChanged
AddHandler ComboPays.SelectedValueChanged, Addressof ComboPays_SelectedValueChanged

Je casse les cycles en "débranchant" les eventHandler qui m'embêtent avant un traitement, puis en les rebranchant après. Au besoin j'appelle une metode de raifraichissement de la vue.

Code vb :

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

1
2
3
4
5
6
7
 
sub ComboPays_SelectedValueChanged (...)
RemoveHandler CurrentUser.PropertyChanged, Addressof CurrentUser_PropertyChanged
CurrentUser.DoSomeThing(unObjet) 'un traitement
Update_CurrentVue()
AddHandler CurrentUser.PropertyChanged, Addressof CurrentUser_PropertyChanged
end sub

Cette métode marche bien pour l'instant.
Du fait que ce n'est pas toujours le même handler qui doit être débrancher/rebrancher,j'ai envie de regrouper tous mes handler pour un objet dans des métodes pour les débrancher/rebrancher. du style

Code vb :

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

1
2
sub AddHandler_pour_CurrentUser
sub RemoveHandler_pour_CurrentUser

Je pourrais ainsi me garantir d'avoir briser tous les cycles et que tout est reinitialiser correctement.
le nombre de handler par métode n'a rien d'affolant (3-4) mais je me demande si cette métode présente des inconvénients.
Il y a un cas où il y aurait de nombreux débrancher/rebrancher c'est quand je change l'instance "Parent" de la vue avec une structure hypothétique du style
MonParent
|-Une Collection Fille
|-Une Collection Fille
|-Un Objet Fils
|-encore d'autre

J'aimerai bien avoir des avis, en espérant que ce que j'ai décrit est compréhensible.
Dom
PS: je commence à regarder des découplements basés sur le pattern "Command" mais j'aimerai utiliser le moins d'abstraction possible.

[Piotrek]

Salut

Si j'ai bien compris, j'ai deja eu ce probleme. Ton code n'a rien de choquant en soi, mais je pense qu'il pourrait etre beaucoup plus harmonieux si tu ajoutais une propriete "IsUpdating" a chaqun de tes objets

Code :

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

1
2
3
4
5
6
sub ComboPays_SelectedValueChanged (...)
CurrentUser.IsUpdating = True
CurrentUser.DoSomeThing(unObjet) 'un traitement
Update_CurrentVue()
CurrentUser.IsUpdating = False
end sub

Chaque Set de champ testerai la valeur de Me.IsUpdating et ignorerait le traitemement. De plus, un appel + un test sur un bool devrait etre plus rapide qu'un abonnement/desabonnement a un evenement

[trolldom]

Salut,
Merci de ta réponse
En analysant un peu mieux mes évennements il était assez simple d'implémenter la solution que tu proposes et je vais donc l'utiliser.
De plus ca me permet, pour une de mes actions, de continuer à utiliser (ou abuser) les changements sur l'objet, via l'écoute de propertyChanged) au lieu de devoir surveiller tous les champs. :-)

Bonne journée
Dom