Discussion :

[olibara]

Bonjour

Je cherche la plus belle maniere de parcourir TOUS les éléments a tous les niveau d'un Toolstrip

Une methode récursive me semble de mise ici
Maintenant je rame un peu pour sub-iterer selon le Type (DropDownButton ou MenuItem)

Donc comme je me dis que ca pourrait servir a d'autre et que je n'ai pas trouvé d'exemple ici je cree le post en attentant la premiere bonne solution que je continue a chercher

[tomlev]

Une methode récursive me semble de mise ici

Effectivement. L'approche récursive peut causer des débordements de pile sur des hiérarchies très profondes, mais a priori ce n'est pas le cas ici... ou alors l'application sera un cauchemar à utiliser

Maintenant je rame un peu pour sub-iterer selon le Type (DropDownButton ou MenuItem)

Ben c'est pareil, non ? Les deux héritent de ToolStripDropDownItem, qui a une propriété DropDownItems.

Donc un truc comme ça devrait le faire:

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
public IEnumerable<ToolStripItem> EnumerateItems(ToolStripItemCollection items)
{
    foreach(var item in items)
    {
        yield return item;
 
        var ddItem = item as ToolStripDropDownItem;
        if (ddItem == null)
            continue;
 
        foreach(var subItem in EnumerateItems(ddItem.DropDownItems))
        {
            yield return subItem;
        }
    }
}

(pas testé...)

[olibara]

Merci TomLev

Tu m'a mis sur la bonne piste

Oui je ne pense pas spontanément à tester le null d'un cast

Code :

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

1
2
        var ddItem = item as ToolStripDropDownItem;
        if (ddItem == null) {...}

Finalement ca donne ca (testé)

Code :

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

    BrowseMenu(toolStripMenu.Items);

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
   private static void BrowseMenu(ToolStripItemCollection items)
    {
      foreach (ToolStripItem item in items)
      {
        var ddItem = item as ToolStripDropDownItem;
        if (ddItem == null)
        {
          continue;
        }
        BrowseMenu(ddItem.DropDownItems);
      }
    }

[tomlev]

Finalement ca donne ca (testé)

Code :

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

    BrowseMenu(toolStripMenu.Items);

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
   private static void BrowseMenu(ToolStripItemCollection items)
    {
      foreach (ToolStripItem item in items)
      {
        var ddItem = item as ToolStripDropDownItem;
        if (ddItem == null)
        {
          continue;
        }
        BrowseMenu(ddItem.DropDownItems);
      }
    }

Bah là tu parcours les items, mais tu ne fais rien avec... donc ça sert à quoi au final ?

[Nathanael Marchand]

On va dire que je chipote mais je prefère dans ce sens qui me semble plus fluide:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
private static void BrowseMenu(ToolStripItemCollection items)
{
      foreach (ToolStripItem item in items)
      {
        if (item is ToolStripDropDownItem)
        {
            var ddItem = item as ToolStripDropDownItem;
            BrowseMenu(ddItem.DropDownItems);
        }
      }
}

Ou tiens, du Linq! Ca faisait longtemps!

Code :

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

1
2
3
4
5
6
7
8
9
 
private static void BrowseMenu(ToolStripItemCollection col)
{
      var items = col.OfType<ToolStripDropDownItem>();
      foreach (var item in items)
      {
            BrowseMenu(item.DropDownItems);
      }
}

[olibara]

Bah là tu parcours les items, mais tu ne fais rien avec... donc ça sert à quoi au final ?

C'est le parcours qui me posait souci, donc j'ai isolé cette partie de code

Apres l'objetif c'est d'appeler une methode a chaque Item pour le rendre enable ou visible en fonction de son Nom et d'un parametre utilisateur accessible dans une base de donnée

Mais ca c'est du gateau

[olibara]

Merci TomLev et PitMaverick78

Voila me semble-il un petit post a garder sur un coin de table

[tomlev]

C'est le parcours qui me posait souci, donc j'ai isolé cette partie de code

Apres l'objetif c'est d'appeler une methode a chaque Item pour le rendre enable ou visible en fonction de son Nom et d'un parametre utilisateur accessible dans une base de donnée

Si tu veux faire quelque chose de réutilisable, tu peux passer en paramètre un Action<ToolStripItem>. A partir de ton code, ça donnerait ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
   private static void BrowseMenu(ToolStripItemCollection items, Action<ToolStripItem> action)
    {
      foreach (ToolStripItem item in items)
      {
        action(item);
 
        var ddItem = item as ToolStripDropDownItem;
        if (ddItem == null)
        {
          continue;
        }
        BrowseMenu(ddItem.DropDownItems);
      }
    }

Pour l'utilisation, c'est tout bête. Par exemple, pour afficher le nom de chaque item dans une MessageBox :

Code :

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

BrowseMenu(toolStripMenu.Items, item => MessageBox.Show(item.Name));

Ou, si l'action à effectuer est un peu longue pour une expression lambda :

Code :

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

1
2
3
4
5
6
7
8
BrowseMenu(toolStripMenu.Items, DoSomething);
 
...
 
void DoSomething(ToolStripItem item)
{
    ...
}