Использование инструкции foreach и переключатель для проверенных нескольких узлов

Question

Я занимаюсь узлами, дочерними узлами, дочерними дочерними узлами в списке TreeView; не говоря уже о флажках. Моя цель - выбрать/проверить несколько узлов, и каждый узел, связанный с эксклюзивным кодом, будет выполняться одним щелчком кнопки. Я подумал об использовании инструкции foreach с помощью оператора switch и if. Предварител, чтобы пройти через узлы дерева, , если, чтобы проверить, проверен ли узел, и коммутатор для просмотра значений, соответствующих проверяемому узлу. Это моя идея, но я чувствую, что для нее лучше. Может ли кто-нибудь проверить программу ниже или предоставить более легкую, более чистую версию?

Моя идея: сделать это следующим образом:

foreach (TreeNode rootNodes in treeView1.Nodes) 
{ 
    foreach (TreeNode childNodes in rootNodes.Nodes) 
    { 
     if (childNodes.Checked == true) 
     { 
      switch (childNodes.Name.ToString()) 
      { 
       case "Trial A": 
        //execute code for Trial A 
        MessageBox.Show("A"); //trial 
        break; 
       case "Trial B": 
        //execute code for Trial B 
        MessageBox.Show("B"); //trial 
        break; 
       case "Trial C": 
        //execute code for Trial C 
        MessageBox.Show("C"); //trial 
        break; 
       default: 
        MessageBox.Show("error"); 
        break; 
      } 
     } 
    } 
} 

До сих пор, когда я попытался это, он работал. Когда я выбрал несколько узлов, каждый код узла работал по одному. На самом деле я очень обрадовался. Проблема кроется во мне, есть это ворчащее чувство, что эта программа может быть потенциально неправильной или я могу столкнуться с проблемами в будущем.

Я использую VS 2005, .NET 2.0, работающий на WinForms. Sidenote: Я проанализировал XML-файл в список древовидной структуры. Узлы соответствуют объектам в XML-файле.

Еще одно сообщение: Я все еще учусь. Я новичок. Определенно, новичок C#. Что касается других опытов программирования, я знаю немного C и C++ (но только основы, только для некоторых упражнений и проектов в школе).

Answer 1

Если я правильно понимаю, что вы можете иметь неопределенную иерархическую глубину узлов. В этом случае вам необходимо выполнить рекурсию:

foreach (TreeNode node in treeView1.Nodes) RecurseTree(node); 

private void RecurseTree(TreeNode node) 
{ 
    if (node.Checked == true) 
    { 
     switch (node.Name) 
     { 
      case "Trial A": 
       //execute code for Trial A 
       MessageBox.Show("A"); //trial 
       break; 
      case "Trial B": 
       //execute code for Trial B 
       MessageBox.Show("B"); //trial 
       break; 
      case "Trial C": 
       //execute code for Trial C 
       MessageBox.Show("C"); //trial 
       break; 
      default: 
       MessageBox.Show("error"); 
       break; 
     } 
    } 

    foreach (TreeNode childNode in node.Nodes) RecurseTree(childNode); 
}