Discussion :

[adaneels]

Bonjour tout le monde,

Dans une application WPF, j'ai une liste déroulante dans un onglet (on va parler en français d'abord, le code suivra ). Lorsque l'utilisateur sélectionne cet onglet, la liste déroulante s'initialise et se place sur le premier élément.
Hors, cette dernière action relance l'évènement tabControl1_SelectionChanged et l'action se répète indéfiniment jusqu'au dépassement de pile.

Pour plus de clarté, voici les bouts de code :
XAML :

Code :

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

1
2
3
4
5
6
7
8
<TabControl Name="tabControl1" SelectionChanged="tabControl1_SelectionChanged">
  <TabItem Header="Tab1">
    [...]
  </TabItem>
  <TabItem Header="Tab2">
    <ComboBox Name="cb1" />
  </TabItem>
</TabControl>

Code Behind (C#) :

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
private void tabControl1_SelectionChanged(object sender, SelectionChangedEventArgs e)
{
  if (tabControl1.SelectedIndex == 1) // si ont est sur Tab2
  {
    Dictionary<string, string> DicoCbItems = new Dictionary<string, string>();
    DicoCbItems.Add("lorem ipsum", "1");
    DicoCbItems.Add("oui oui", "2");
    DicoCbItems.Add("superman", "3");

    Binding BindCb = new Binding();
    BindCb.Source = DicoCbItems;
    cb1.DisplayMemberPath = "Key";
    cb1.SelectedValuePath = "Value";
    cb1.SetBinding(ComboBox.ItemsSourceProperty, BindCb);

    // prochaine instruction déclenche l'évènement tabControl1_SelectionChanged et 
    // ça tourne en boucle jusqu'au dépassement de pile
    cb1.SelectedIndex = 0; 
  }
}

Est-ce que quelqu'un sait comment faire autrement (proprement*) ?
Si vous savez pourquoi ça fait cela, je suis aussi preneur, voire même beaucoup plus intéressé que pour le comment!

* : par "proprement", je veux dire plus proprement qu'un flag "bool Tab2IsInitializing" pour ne faire l'évènement qu'une fois à la fois, solution que j'ai rapidement mis en place.

[seiryujay]

Normalement, l'évènement SelectionChanged d'un TabControl n'est lancé que lorsque l'onglet sélectionné change.
Or d'après ce que tu nous dit, c'est le fait de changer l'item sélectionné de la ComboBox qui rappelle la méthode tabControl1_SelectionChanged().

Est-ce que par hasard, tu n'aurais pas ajouté la méthode tabControl1_SelectionChanged() sur l'évènement SelectionChanged de ta ComboBox ?

[tomlev]

Est-ce que par hasard, tu n'aurais pas ajouté la méthode tabControl1_SelectionChanged() sur l'évènement SelectionChanged de ta ComboBox ?

Exactement ce que j'allais suggérer...

[adaneels]

Est-ce que par hasard, tu n'aurais pas ajouté la méthode tabControl1_SelectionChanged() sur l'évènement SelectionChanged de ta ComboBox ?

Non, aucun évènement d'implémenter sur la combobox.

Je n'ai jamais rencontré ce type de problème avant de me mettre dans la WCF mais depuis quelques semaines que je m'y mets, j'ai bcp de surprises...

Si ça peut aider, car je suppose que ce n'est pas forcément normal : lors du démarrage, le tabcontrol (de l'espace de noms System.Windows.Controls) a son SelectionChanged qui est levé plusieurs fois au démarrage alors que je ne touche jamais à son SelectedIndex

[ikeas]

y'a un comportement spécial des tabcontrol pour les tabitem
en fait il n'initialise les objets qui sont dedans seulement quand le tabitem est sélectionné
moi je te suggèrerais de faire un binding sur une collection, ce qui preserverait a la fois ce qu'il y a dedans et la sélection

[seiryujay]

Une question toute bête aussi : as-tu besoin de recharger le contenu de ta ComboBox à chaque fois que ton TabItem est activé ?
Si ce n'est pas utile, t'as plus de problème

[adaneels]

J'ai réessayé avec un nouveau projet application WPF (Visual Studio 2008) et j'ai toujours le pb.

Voilà les étapes pour reproduire :
- créer un nouveau projet applicationWPF
- mettre un tabcontrol sur Window1
- ajouter un onglet
- mettre un combobox sur un des onglets (le second dans mon exemple)
- implémenter l'évènement tabControl1_SelectionChanged avec le code suivant :

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
private void tabControl1_SelectionChanged(object sender, SelectionChangedEventArgs e)
    {
      Dictionary<string, string> DicoCbItems = new Dictionary<string, string>();
      DicoCbItems.Add("lorem ipsum", "1");
      DicoCbItems.Add("oui oui", "2");
      DicoCbItems.Add("superman", "3");
 
      Binding BindCb = new Binding();
      BindCb.Source = DicoCbItems;
      comboBox1.DisplayMemberPath = "Key";
      comboBox1.SelectedValuePath = "Value";
      comboBox1.SetBinding(ComboBox.ItemsSourceProperty, BindCb);
 
      // prochaine instruction déclenche l'évènement tabControl1_SelectionChanged et 
      // ça tourne en boucle jusqu'au dépassement de pile
      comboBox1.SelectedIndex = 0;
    }

Exécutez le projet et essayez de choisir autre chose que le premier élément ... impossible.

Comme si le Combobox hérite de l'évènement selectionChanged de son parent (c'est de la science fiction?)


---
Au cas où, le mini-projet en pièce jointe

Merci pour le temps consacré,
Arnaud

[adaneels]

Une question toute bête aussi : as-tu besoin de recharger le contenu de ta ComboBox à chaque fois que ton TabItem est activé ?
Si ce n'est pas utile, t'as plus de problème

Je préfèrerai parce que ça provient d'un fichier externe de configuration (qui n'est pas gros et qui ne change pas souvent donc je ne le recharge pas à chaque fois non plus), ça évite d'avoir à redémarrer l'application pour prendre en compte les modifications.

[seiryujay]

Ok, comportement intéressant...
Je te conseille de regarder cette discussion.

Et notamment ce passage :

This is because both ComboBox and TabControl are derived from Selector, and the SelectionChanged event is a routed event, so the child ComboBox' SelectionChanged event will be routed to its parent control TabControl. Just like mouse event can be routed from a child element to its parent element.

C'est le genre de truc qui paraît louche au premier abord mais qui a une explication rationnelle.

Donc en gros, tu peux :
- soit rajouter le test suivant au début de ton évènement :

Code :

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

if (sender == e.OriginalSource)

- soit rajouter une méthode sur l'évènement SelectionChanged de ta ComboBox qui fera (entre-autre)

Code :

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

e.Handled = true

afin de stopper la propagation de ton RoutedEvent

Voilà.

[ikeas]

tu as une solution simple c'est de mettre tout ton code dans le

Code :

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

1
2
3
4
private void tabItem2_GotFocus(object sender, RoutedEventArgs e)
    {
 
    }

avec

Code :

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

<TabItem Header="tabItem2" Name="tabItem2" GotFocus="tabItem2_GotFocus">

bien sur

maintenant il faut proteger le code pour le repassage dans le tab

bon sinon une autre alternative qui me parrait plus propre
dans ton tabitem tu met

Code :

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

1
2
3
4
5
6
 
<TabItem Name="tabItem2" GotFocus="tabItem2_GotFocus" Loaded="tabItem2_Loaded">
                <TabItem.Header >
                    <TextBlock Text="sdqsqsqsd"  MouseLeftButtonDown="TextBlock_MouseLeftButtonDown" />
                </TabItem.Header>
                <Grid>

puis tu met le code dans le TextBlock_MouseLeftButtonDown

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
 
 if (tabControl1.SelectedIndex == 1) return;
        Dictionary<string, string> DicoCbItems = new Dictionary<string, string>();
        DicoCbItems.Add("lorem ipsum", "1");
        DicoCbItems.Add("oui oui", "2");
        DicoCbItems.Add("superman", "3");
 
        Binding BindCb = new Binding();
        BindCb.Source = DicoCbItems;
        comboBox1.DisplayMemberPath = "Key";
        comboBox1.SelectedValuePath = "Value";
        comboBox1.SetBinding(ComboBox.ItemsSourceProperty, BindCb);
 
        comboBox1.SelectedIndex = 0;
        textBox1.Text = "sdfghjklm";

[adaneels]

Ok, comportement intéressant...

Pour l'instant, j'avoue que je prends ça plus comme quelquechose qui m'a fait perdre une petite journée.
Mais je suis à chaud là, c'est pour ça. Une clope et ça ira mieux (message pour les enfants et ados : fumer n'est pas bien, une fois que l'on a commencé, on se dit pendant des années qu'on va arrêter et on ne le fait jamais).

Je te conseille de regarder cette discussion.

Et notamment ce passage :

Donc en gros, tu peux :
- soit rajouter le test suivant au début de ton évènement :

Code :

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

if (sender == e.OriginalSource)

- soit rajouter une méthode sur l'évènement SelectionChanged de ta ComboBox qui fera (entre-autre)

Code :

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

e.Handled = true

afin de stopper la propagation de ton RoutedEvent

Voilà.

ho purée les saligots chez Microsoft.
Merci !!
Tu googleïses mieux que moi.

C'est le genre de truc qui paraît louche au premier abord mais qui a une explication rationnelle.

L'explication que j'ai lu dans la discussion ne me paraît pas rationnelle. Si on avait par défaut le comportement d'avant (l'évènement n'est pas automatiquement routé aux parents) mais qu'on a la possibilité de le faire, je dirai super. Mais là, moyen.
Cependant, j'aime bien ta façon de voir car sans aucun jugement de valeur.

Merci encore pour la solution

[adaneels]

tu as une solution simple c'est de mettre tout ton code dans le
private void tabItem2_GotFocus(object sender, RoutedEventArgs e)
{

}
avec
<TabItem Header="tabItem2" Name="tabItem2" GotFocus="tabItem2_GotFocus">
bien sur

maintenant il faut proteger le code pour le repassage dans le tab

Merci, mais ça posait le même problème.
Le

Code :

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

e.Handled = true

; fonctionne impeccable.

[ikeas]

en fait j'ai corrige mon message ave une alternative plus "sexy"
ce qu'on constate c'est qu'il manque un selected sur tabitem, ca doit pouvoir se rajouter
moi je trouve que c'est plus sympa d'impacter sur un event lié au tab item plutot qu'a un autre (enfin comme la combo par exemple)

[tomlev]

Effectivement, j'observe le même comportement... c'est surprenant
En fait il faut juste tester si le e.OriginalSource == tabControl1, et ça marche

[adaneels]

en fait j'ai corrige mon message ave une alternative plus "sexy"

Pour éviter le repassage (tien, j'en ai à faire ce soir d'ailleurs) dans l'évènement, j'avais au départ déclaré une variable globale : un booléen IsTab2Initialising

dans le code du tabcontrol1_selectedIndexChanged, j'avais mis :

Code :

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

1
2
3
4
5
6
7
If (IsTab2Initializing)
{
  return; 
}
IsTab2Initializing = true;
[... le traitement ...]
IsTab2Initializing = false;

moins bidouilleux je pense (mais c'est une histoire de goût)

maintenant, avec la compréhension des évènements routés en WPF, je dormirai moins bête ce soir.

merci à tous ceux qui ont répondu à l'appel. Un autre va suivre vu la réussite inattendue ici

[ikeas]

a mon avis ca serait interessant de creer un message selected sur le tabitem
ca simplifirais et de loin

[seiryujay]

en fait j'ai corrige mon message ave une alternative plus "sexy"
ce qu'on constate c'est qu'il manque un selected sur tabitem, ca doit pouvoir se rajouter
moi je trouve que c'est plus sympa d'impacter sur un event lié au tab item plutot qu'a un autre (enfin comme la combo par exemple)

Moi aussi ça me fait bizarre de rajouter un évènement vide qui affecte juste le e.Handled (c'est le genre de truc qui peut sauter par la suite si y'a pas un commentaire explicite du pourquoi de la méthode).

C'est pour ça que le test

Code :

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

if (sender == e.OriginalSource)

englobant le contenu de la méthode appelée sur l'évènement SelectionChanged du TabControl me paraît plus adapté...

Tu googleïses mieux que moi.

Des années de pratique ^^ (faut dire que j'ai galéré sur pas mal de problèmes aussi... )

L'explication que j'ai lu dans la discussion ne me paraît pas rationnelle.

Pourquoi ?
Tu comprends bien le comportement sur un évènement souris ? Celui-ci est propagé de l'élément qui reçoit le clic vers ses éléments parents.
Ben là c'est pareil mais avec le SelectionChanged d'un élément dérivant de Selector qui est propagé vers ses éléments parents dérivant de Selector (autrement dit TabControl, ListBox, ComboBox).

Si on avait par défaut le comportement d'avant (l'évènement n'est pas automatiquement routé aux parents) mais qu'on a la possibilité de le faire, je dirai super. Mais là, moyen.

Ca c'est une autre histoire. Là faut voir directement avec monsieur Microsoft

Cependant, j'aime bien ta façon de voir car sans aucun jugement de valeur.

Ca c'est parce que j'ai pas perdu une journée de boulot à batailler dessus :p

[ikeas]

tiens j'ai trouve encore mieux

Code :

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

1
2
 
<TabItem Name="tabItem2" Selector.Selected="_Selected">

et tu met ton code dans le _Selected

[seiryujay]

tiens j'ai trouve encore mieux

Code :

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

1
2
 
<TabItem Name="tabItem2" Selector.Selected="_Selected">

et tu met ton code dans le _Selected

Bien vu !

[adaneels]

tiens j'ai trouve encore mieux

Code :

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

1
2
 
<TabItem Name="tabItem2" Selector.Selected="_Selected">

et tu met ton code dans le _Selected

Ca ne fait que quelques semaines que je me suis remis au C# (la dernière fois remontant à 2005 avec du .NET 1.1 et un peu de 2.0 !).
Et je tiens à vous dire que vos solutions me paraissent monstrueuses. Y'a trop de choses qui ont changé en .NET 3.5 !

Mais ta réponse est juste. Adoptée.

et un double-résolu, un.