Рекурсивный метод расширения для искателя серебра

Question

Мне нужен метод расширения, чтобы обойти все текстовые поля на моей странице Silverlight. Предположим, я всегда использую сетку, тогда у меня есть:

public static IEnumerable<UIElement> Traverse(this UIElementCollection source, Func<Grid, UIElementCollection> fnRecurse)
    {
        foreach (UIElement item  in source)
        {
            yield return item;
            var g = source.OfType<Grid>();
            foreach (Grid itemsub in g)
                {
                    var t = fnRecurse(itemsub);
                      t.Traverse(fnRecurse);
                      yield return itemsub;

                };
        }

    }

Теперь я могу назвать это так:

baseLayout.Children.Traverse(x =>  x.Children ).OfType<TextBox>().ForEach(
                w =>
                {
                    //Text Box Extension Method, clears the text
                    w.reset();
                });

Это никогда не срабатывает. Я считаю, что это OfType, который не может различить элементы пользовательского интерфейса.

Как бы я это сделал? Я хочу сгладить визуальное дерево, а затем перебрать. Не просто текстовые поля, а просто получение всего, что я хочу. Я уступаю в неправильном месте или слишком часто?

Edit:

Текущий код

public static IEnumerable<UIElement> Traverse(this UIElementCollection source, Func<Grid, UIElementCollection> fnRecurse)
    {

        foreach (UIElement item  in source)
        {
           source.OfType<Grid>().Select(x => x.Children).ForEach(v => Traverse(v, fnRecurse));
           yield return item;
        }

    }

и сетка

<Grid x:Name="LayoutRoot">
    <Grid.RowDefinitions>
        <RowDefinition Height="42"/>
        <RowDefinition Height="42"/>
        <RowDefinition Height="45"/>
        <RowDefinition Height="43"/>
        <RowDefinition Height="47"/>
        <RowDefinition Height="46"/>
        <RowDefinition/>
        <RowDefinition Height="67"/>
    </Grid.RowDefinitions>
    <Button x:Name="saveAddressButton" Height="24" HorizontalAlignment="Left" Margin="8,0,0,8" VerticalAlignment="Bottom" Width="135" Content="Save and Add More" Click="saveClick" Style="{StaticResource SaveButton}" Foreground="White" Grid.Row="7"/>
    <Button x:Name="saveAddressButton_Copy" Height="24" HorizontalAlignment="Right" Margin="0,0,8,8" VerticalAlignment="Bottom" Width="81" Content="Clear" Click="clearClick" Style="{StaticResource SaveButton}" Foreground="White" Grid.Row="7"/>
    <Grid Height="30" VerticalAlignment="Top" Margin="8,9,8,0">
        <TextBlock Margin="8,0,0,0" Text="AddressLine1" FontSize="16" Foreground="White" HorizontalAlignment="Left" Width="121"/>
        <TextBox x:Name="inputAddressLine1" TextWrapping="Wrap" Margin="218,0,0,0"/>
    </Grid>
    <Grid Height="30" Margin="8,8,8,0" VerticalAlignment="Top" Grid.Row="2">
        <TextBlock Margin="8,0,0,0" Text="AddressLine2" FontSize="16" Foreground="White" HorizontalAlignment="Left" Width="146"/>
        <TextBox x:Name="inputAddressLine2" TextWrapping="Wrap" Margin="219,0,0,0"/>
    </Grid>
    <Grid Height="30" Margin="8,0,8,7" VerticalAlignment="Bottom" Grid.Row="1">
        <TextBlock Margin="8,0,0,0" Text="Post Code" FontSize="16" Foreground="White" HorizontalAlignment="Left" Width="107"/>
        <TextBox x:Name="inputPostCode" TextWrapping="Wrap" Margin="219,0,0,0"/>
    </Grid>
    <Grid Height="30" VerticalAlignment="Bottom" Margin="8,0,8,9" Grid.Row="4">
        <TextBox x:Name="inputCounty" TextWrapping="Wrap" Margin="220,0,0,0"/>
        <TextBlock Margin="8,0,0,0" Text="County" FontSize="16" Foreground="White" HorizontalAlignment="Left" Width="155"/>
    </Grid>
    <Grid Margin="8,8,8,5" Grid.Row="3">
        <TextBox x:Name="inputTown" TextWrapping="Wrap" Margin="219,0,0,0"/>
        <TextBlock Margin="8,0,0,0" Text="AddressLine2" FontSize="16" Foreground="White" HorizontalAlignment="Left" Width="165"/>
    </Grid>
    <Grid Margin="8" Grid.Row="5">
        <TextBlock Text="Number" FontSize="16" Foreground="White" Margin="8,0,0,0" HorizontalAlignment="Left" Width="178"/>
        <TextBox x:Name="inputNumber" TextWrapping="Wrap" Margin="220,0,0,0"/>
    </Grid>
</Grid>

По-прежнему можно получить только один уровень глубины, возвращая 6 сеток и одну кнопку!

Моя текущая функция перемещения:

public static IEnumerable<UIElement> Traverse(this UIElementCollection source)
    {
        source.OfType<Grid>().SelectMany(v => Traverse(v.Children));
        //This is the top level.
        foreach (UIElement item in source)
        {
            yield return item;
        }
    }

Это просто знает, что мы имеем дело с сетками, поэтому нет необходимости во втором аргументе. Я уступаю только из итератора, а не из первой строки, которая должна вызывать функцию Traverse с дочерними сетками.

Answer 1

Вот функция, которую я использую для доставки такого рода вещей: -

    public static IEnumerable<DependencyObject> Descendents(this DependencyObject root)
    {
        int count = VisualTreeHelper.GetChildrenCount(root);
        for (int i = 0; i < count; i++)
        {
            var child = VisualTreeHelper.GetChild(root, i);
            yield return child;
            foreach (var descendent in Descendents(child))
                yield return descendent;
        }
    }

При наличии этого метода расширения ваш код становится: -

 foreach(var txt in baseLayout.Descendents().OfType<TextBox>())
 {
     txt.reset();
 }

Обратите внимание, что избегание метода foreach является одним из вариантов выбора. Мне не нравится идея метода расширения LINQEsq, который фактически изменяет или делает что-то для своего приложения. Я предпочитаю использовать правильное значение foreach, чтобы фактически воздействовать на результаты запроса.

Редактировать для "наркоманов метода расширения" (если вы разумны, вы не будете делать это): -

 public static IEnumerable<T> ForEach(this IEnumerable<T> items, Action<T> fn)
 {
     foreach (T item in items)
        fn(item);
 }

Редактировать что не так с вашим кодом.

Ну, в первую очередь эта строка в вашем методе Traverse является основной причиной вашей проблемы: -

 t.Traverse(fnRecurse);

Возвращает IEnumerable<UIElement>, но вы ничего не делаете с ним, даже не сохраняете результат в переменной.

Также эта строка: -

var g = source.OfType<Grid>();

приведет к перечислению каждой найденной сетки для каждого найденного элемента UIE. Так, например, если источник содержит TextBox и 2 Grids, вышеуказанная строка вызывается 3 раза. Обе сетки будут проходить через внутренний foreach дважды.

Также эта строка: -

yield return itemsub;

скважина itemsub всегда является Grid и будет отфильтрована последующими TypeOf<TextBox>.

Следовательно, единственный TextBox, который этот код когда-либо будет возвращать, это любой TextBox, найденный в начальном UIElementCollection.