Jak uzyskać WSZYSTKIE kontrolki podrzędne formularza Windows Forms określonego typu (przycisk / pole tekstowe)?


120

Muszę uzyskać wszystkie kontrolki w formularzu typu x. Jestem prawie pewien, że kiedyś widziałem ten kod, który używał czegoś takiego:

dim ctrls() as Control
ctrls = Me.Controls(GetType(TextBox))

Wiem, że mogę iterować po wszystkich kontrolkach, które powodują, że dzieci używają funkcji rekurencyjnej, ale czy jest coś łatwiejszego lub prostszego, może na przykład następujący?

Dim Ctrls = From ctrl In Me.Controls Where ctrl.GetType Is Textbox

Odpowiedzi:


232

Oto inna opcja dla Ciebie. Przetestowałem to, tworząc przykładową aplikację, a następnie umieściłem GroupBox i GroupBox w początkowym GroupBox. W zagnieżdżonym GroupBox umieściłem 3 kontrolki TextBox i przycisk. To jest kod, którego użyłem (zawiera nawet rekursję, której szukałeś)

public IEnumerable<Control> GetAll(Control control,Type type)
{
    var controls = control.Controls.Cast<Control>();

    return controls.SelectMany(ctrl => GetAll(ctrl,type))
                              .Concat(controls)
                              .Where(c => c.GetType() == type);
}

Aby przetestować to w zdarzeniu ładowania formularza, chciałem policzyć wszystkie kontrolki wewnątrz początkowego GroupBox

private void Form1_Load(object sender, EventArgs e)
{
    var c = GetAll(this,typeof(TextBox));
    MessageBox.Show("Total Controls: " + c.Count());
}

I za każdym razem zwracał właściwą liczbę, więc myślę, że będzie to działać idealnie dla tego, czego szukasz :)


21
Zdefiniowana tutaj GetAll () jest bardzo dobrym kandydatem na metodę rozszerzającą do klasy Control
Michael Bahig

Podobał mi się sposób, w jaki używałeś wyrażeń lambda. Gdzie szczegółowo uczyć się wyrażeń lambda?
Aditya Bokade,

„System.Windows.Forms.Control.ControlCollection” nie zawiera definicji „Cast” i nie można znaleźć metody rozszerzenia „Cast” akceptującej pierwszy argument typu „System.Windows.Forms.Control.ControlCollection” (są brakuje dyrektywy using lub odwołania do zestawu?) „Jestem na .NET 4.5, a„ Controls ”nie ma funkcji / metody„ Cast ”/ czegokolwiek. czego mi brakuje?
soulblazer

2
@soulblazer Dodaj przestrzeń nazw System.Linq.
Ivan-Mark Debono

var allCtl = GetAll (this.FindForm (), typeof (TextBox)); // to jest kontrola użytkownika nic nie zwraca !!
bh_earth0

33

W C # (ponieważ oznaczyłeś go jako taki) możesz użyć wyrażenia LINQ w następujący sposób:

List<Control> c = Controls.OfType<TextBox>().Cast<Control>().ToList();

Edytuj dla rekursji:

W tym przykładzie najpierw tworzysz listę kontrolek, a następnie wywołujesz metodę, aby ją wypełnić. Ponieważ metoda jest rekurencyjna, nie zwraca listy, a jedynie ją aktualizuje.

List<Control> ControlList = new List<Control>();
private void GetAllControls(Control container)
{
    foreach (Control c in container.Controls)
    {
        GetAllControls(c);
        if (c is TextBox) ControlList.Add(c);
    }
}

Może to być możliwe w jednej instrukcji LINQ przy użyciu Descendantsfunkcji, chociaż nie jestem z nią tak zaznajomiony. Zobacz tę stronę, aby uzyskać więcej informacji na ten temat.

Edytuj 2, aby zwrócić kolekcję:

Jak zasugerował @ProfK, metoda, która po prostu zwraca żądane kontrolki, jest prawdopodobnie lepszą praktyką. Aby to zilustrować, zmodyfikowałem kod w następujący sposób:

private IEnumerable<Control> GetAllTextBoxControls(Control container)
{
    List<Control> controlList = new List<Control>();
    foreach (Control c in container.Controls)
    {
        controlList.AddRange(GetAllTextBoxControls(c));
        if (c is TextBox)
            controlList.Add(c);
    }
    return controlList;
}

Dzięki, C # lub VB jest dla mnie w porządku. Ale problem polega na tym, że Controls.OfType <TExtbox> zwraca tylko elementy podrzędne bieżącej kontrolki (w moim przypadku Form) i chcę, aby w jednym wywołaniu otrzymały WSZYSTKIE kontrolki w Formie "rekurencyjnie" (dzieci, podrzędne , sub-sub-childs, .....) w pojedynczej kolekcji.
Luis

Spodziewałbym się, że metoda o nazwie GetAllControls zwróci kolekcję kontrolek, które przypiszę do ControlList. Po prostu wydaje się lepszą praktyką.
ProfK

@ProfK Zgadzam się z tobą; odpowiednio zmieniając przykład.
JYelton

13

To jest ulepszona wersja rekurencyjnej GetAllControls (), która faktycznie działa na prywatnych zmiennych:

    private void Test()
    {
         List<Control> allTextboxes = GetAllControls(this);
    }
    private List<Control> GetAllControls(Control container, List<Control> list)
    {
        foreach (Control c in container.Controls)
        {
            if (c is TextBox) list.Add(c);
            if (c.Controls.Count > 0)
                list = GetAllControls(c, list);
        }

        return list;
    }
    private List<Control> GetAllControls(Control container)
    {
        return GetAllControls(container, new List<Control>());
    }

10

Połączyłem kilka poprzednich pomysłów w jedną metodę rozszerzającą. Korzyści są takie, że otrzymujesz poprawnie wpisany wyliczalny z powrotem, a dziedziczenie jest poprawnie obsługiwane przez OfType().

public static IEnumerable<T> FindAllChildrenByType<T>(this Control control)
{
    IEnumerable<Control> controls = control.Controls.Cast<Control>();
    return controls
        .OfType<T>()
        .Concat<T>(controls.SelectMany<Control, T>(ctrl => FindAllChildrenByType<T>(ctrl)));
}

5

Aby to zrobić, możesz użyć zapytania LINQ. Spowoduje to zapytanie o wszystko w formularzu typu TextBox

var c = from controls in this.Controls.OfType<TextBox>()
              select controls;

Dzięki, ale ten sam problem, co odpowiedź, zwraca tylko dzieci, ale nie dzieci podrzędnych itp., I chcę, aby wszystkie kontrolki zostały objęte. Jestem prawie pewien, że widziałem, że jest to możliwe z pojedynczym wywołaniem metody, która jest nowa w .NET 3.5 lub 4.0, pamiętaj, że widziałem to gdzieś w demonstracji
Luis

Ignorowanie braku rekursji, czy nie var c = this.Controls.OfType<TextBox>()dałoby tego samego wyniku?
CoderDennis

2
@Dennis: Tak, to kwestia preferencji (zazwyczaj). Zobacz stackoverflow.com/questions/214500/…, aby zapoznać się z interesującą dyskusją na ten temat.
JYelton

5

Może to starożytna technika, ale działa jak urok. Użyłem rekurencji, aby zmienić kolor wszystkich etykiet kontrolki. Działa świetnie.

internal static void changeControlColour(Control f, Color color)
{
    foreach (Control c in f.Controls)
    {

        // MessageBox.Show(c.GetType().ToString());
        if (c.HasChildren)
        {
            changeControlColour(c, color);
        }
        else
            if (c is Label)
            {
                Label lll = (Label)c;
                lll.ForeColor = color;
            }
    }
}

4

Chciałbym zmienić odpowiedź PsychoCoders: ponieważ użytkownik chce uzyskać wszystkie kontrolki określonego typu, możemy użyć typów generycznych w następujący sposób:

    public IEnumerable<T> FindControls<T>(Control control) where T : Control
    {
        // we can't cast here because some controls in here will most likely not be <T>
        var controls = control.Controls.Cast<Control>();

        return controls.SelectMany(ctrl => FindControls<T>(ctrl))
                                  .Concat(controls)
                                  .Where(c => c.GetType() == typeof(T)).Cast<T>();
    }

W ten sposób możemy wywołać funkcję w następujący sposób:

private void Form1_Load(object sender, EventArgs e)
{
    var c = FindControls<TextBox>(this);
    MessageBox.Show("Total Controls: " + c.Count());
}

to najlepsze (i najszybsze według moich testów) rozwiązanie moim zdaniem na tej stronie. Ale sugerowałbym zmianę kontrolek na tablicę: var enumerable = controls as Control [] ?? controls.ToArray (); a następnie zmień na: return enumerable.SelectMany (FindControls <T>) .Concat (enumerable) .Where (c => c.GetType () == typeof (T)). Cast <T> ();
Randall Flagg

Czy nie jest bardziej wydajne stosowanie .OfType<T>()metody Linq niż .Where(c => c.GetType() == typeof(T)).Cast<T>();uzyskanie tego samego efektu?
TheHitchenator

3

Nie zapominaj, że możesz również mieć TextBox w innych kontrolkach innych niż kontrolki kontenera. Możesz nawet dodać TextBox do PictureBox.

Więc musisz też sprawdzić, czy

someControl.HasChildren = True

w dowolnej funkcji rekurencyjnej.

Oto wynik, jaki uzyskałem z układu do testowania tego kodu:

TextBox13   Parent = Panel5
TextBox12   Parent = Panel5
TextBox9   Parent = Panel2
TextBox8   Parent = Panel2
TextBox16   Parent = Panel6
TextBox15   Parent = Panel6
TextBox14   Parent = Panel6
TextBox10   Parent = Panel3
TextBox11   Parent = Panel4
TextBox7   Parent = Panel1
TextBox6   Parent = Panel1
TextBox5   Parent = Panel1
TextBox4   Parent = Form1
TextBox3   Parent = Form1
TextBox2   Parent = Form1
TextBox1   Parent = Form1
tbTest   Parent = myPicBox

Spróbuj tego z jednym przyciskiem i jednym RichTextBox w formularzu.

Option Strict On
Option Explicit On
Option Infer Off

Public Class Form1

    Private Sub Button1_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles Button1.Click

        Dim pb As New PictureBox
        pb.Name = "myPicBox"
        pb.BackColor = Color.Goldenrod
        pb.Size = New Size(100, 100)
        pb.Location = New Point(0, 0)
        Dim tb As New TextBox
        tb.Name = "tbTest"
        pb.Controls.Add(tb)
        Me.Controls.Add(pb)

        Dim textBoxList As New List(Of Control)
        textBoxList = GetAllControls(Of TextBox)(Me)

        Dim sb As New System.Text.StringBuilder
        For index As Integer = 0 To textBoxList.Count - 1
            sb.Append(textBoxList.Item(index).Name & "   Parent = " & textBoxList.Item(index).Parent.Name & System.Environment.NewLine)
        Next

        RichTextBox1.Text = sb.ToString
    End Sub

    Private Function GetAllControls(Of T)(ByVal searchWithin As Control) As List(Of Control)

        Dim returnList As New List(Of Control)

        If searchWithin.HasChildren = True Then
            For Each ctrl As Control In searchWithin.Controls
                If TypeOf ctrl Is T Then
                    returnList.Add(ctrl)
                End If
                returnList.AddRange(GetAllControls(Of T)(ctrl))
            Next
        ElseIf searchWithin.HasChildren = False Then
            For Each ctrl As Control In searchWithin.Controls
                If TypeOf ctrl Is T Then
                    returnList.Add(ctrl)
                End If
                returnList.AddRange(GetAllControls(Of T)(ctrl))
            Next
        End If
        Return returnList
    End Function

End Class

2

Korzystanie z odbicia:

// Return a list with all the private fields with the same type
List<T> GetAllControlsWithTypeFromControl<T>(Control parentControl)
{
    List<T> retValue = new List<T>();
    System.Reflection.FieldInfo[] fields = parentControl.GetType().GetFields(System.Reflection.BindingFlags.NonPublic | System.Reflection.BindingFlags.Instance);
    foreach (System.Reflection.FieldInfo field in fields)
    {
      if (field.FieldType == typeof(T))
        retValue.Add((T)field.GetValue(parentControl));
    }
}

List<TextBox> ctrls = GetAllControlsWithTypeFromControl<TextBox>(this);

2

Oto moja metoda rozszerzenia Controlużywająca LINQ jako adaptacja wersji @PsychoCoder :

Zamiast tego wymaga listy typów, która pozwala nie potrzebować wielu wywołań, GetAllaby uzyskać to, czego chcesz. Obecnie używam go jako wersji przeciążonej.

public static IEnumerable<Control> GetAll(this Control control, IEnumerable<Type> filteringTypes)
{
    var ctrls = control.Controls.Cast<Control>();

    return ctrls.SelectMany(ctrl => GetAll(ctrl, filteringTypes))
                .Concat(ctrls)
                .Where(ctl => filteringTypes.Any(t => ctl.GetType() == t));
}

Stosowanie:

//   The types you want to select
var typeToBeSelected = new List<Type>
{
    typeof(TextBox)
    , typeof(MaskedTextBox)
    , typeof(Button)
};

//    Only one call
var allControls = MyControlThatContainsOtherControls.GetAll(typeToBeSelected);

//    Do something with it
foreach(var ctrl in allControls)
{
    ctrl.Enabled = true;
}

2

Czyste i łatwe rozwiązanie (C #):

static class Utilities {
    public static List<T> GetAllControls<T>(this Control container) where T : Control {
        List<T> controls = new List<T>();
        if (container.Controls.Count > 0) {
            controls.AddRange(container.Controls.OfType<T>());
            foreach (Control c in container.Controls) {
                controls.AddRange(c.GetAllControls<T>());
            }
        }

        return controls;
    }
}

Pobierz wszystkie pola tekstowe:

List<TextBox> textboxes = myControl.GetAllControls<TextBox>();

2

Możesz użyć poniższego kodu

public static class ExtensionMethods
{
    public static IEnumerable<T> GetAll<T>(this Control control)
    {
        var controls = control.Controls.Cast<Control>();

        return controls.SelectMany(ctrl => ctrl.GetAll<T>())
                                  .Concat(controls.OfType<T>());
    }
}

2

Oto moja metoda rozszerzenia. Jest bardzo wydajny i leniwy.

Stosowanie:

var checkBoxes = tableLayoutPanel1.FindChildControlsOfType<CheckBox>();

foreach (var checkBox in checkBoxes)
{
    checkBox.Checked = false;
}

Kod to:

public static IEnumerable<TControl> FindChildControlsOfType<TControl>(this Control control) where TControl : Control
    {
        foreach (var childControl in control.Controls.Cast<Control>())
        {
            if (childControl.GetType() == typeof(TControl))
            {
                yield return (TControl)childControl;
            }
            else
            {
                foreach (var next in FindChildControlsOfType<TControl>(childControl))
                {
                    yield return next;
                }
            }
        }
    }

jest to czystsza wersja, która jest leniwa, może być wyliczana i pobierana na żądanie.
Jone Polvora


1
public List<Control> GetAllChildControls(Control Root, Type FilterType = null)
{
    List<Control> AllChilds = new List<Control>();
    foreach (Control ctl in Root.Controls) {
        if (FilterType != null) {
            if (ctl.GetType == FilterType) {
                AllChilds.Add(ctl);
            }
        } else {
            AllChilds.Add(ctl);
        }
        if (ctl.HasChildren) {
            GetAllChildControls(ctl, FilterType);
        }
    }
    return AllChilds;
}

1
   IEnumerable<Control> Ctrls = from Control ctrl in Me.Controls where ctrl is TextBox | ctrl is GroupBox select ctr;

Wyrażenia lambda

IEnumerable<Control> Ctrls = Me.Controls.Cast<Control>().Where(c => c is Button | c is GroupBox);

Dodaj więcej do swojej odpowiedzi, wyjaśniając, co się dzieje i jaki ma to związek z pytaniem.
Fencer 04

0

Zmodyfikowałem z @PsychoCoder. Wszystkie kontrolki można teraz znaleźć (w tym zagnieżdżone).

public static IEnumerable<T> GetChildrens<T>(Control control)
{
  var type = typeof (T);

  var allControls = GetAllChildrens(control);

  return allControls.Where(c => c.GetType() == type).Cast<T>();
}

private static IEnumerable<Control> GetAllChildrens(Control control)
{
  var controls = control.Controls.Cast<Control>();
  return controls.SelectMany(c => GetAllChildrens(c))
    .Concat(controls);
}

0

To może zadziałać:

Public Function getControls(Of T)() As List(Of T)
    Dim st As New Stack(Of Control)
    Dim ctl As Control
    Dim li As New List(Of T)

    st.Push(Me)

    While st.Count > 0
        ctl = st.Pop
        For Each c In ctl.Controls
            st.Push(CType(c, Control))
            If c.GetType Is GetType(T) Then
                li.Add(CType(c, T))
            End If
        Next
    End While

    Return li
End Function

Myślę, że funkcja uzyskiwania wszystkich kontrolek, o których mówisz, jest dostępna tylko dla WPF .


0

Oto przetestowane i działające ogólne rozwiązanie:

Mam dużą liczbę kontrolek UpDownNumeric, niektóre w głównej formie, inne w skrzynkach grupowych w formularzu. Chcę, aby tylko ostatnia wybrana kontrolka zmieniła kolor z powrotem na zielony, dla którego najpierw ustawiłem wszystkie inne na biały, używając tej metody: (można również rozszerzyć na wnuki)

    public void setAllUpDnBackColorWhite()
    {
        //To set the numericUpDown background color of the selected control to white: 
        //and then the last selected control will change to green.

        foreach (Control cont in this.Controls)
        {
           if (cont.HasChildren)
            {
                foreach (Control contChild in cont.Controls)
                    if (contChild.GetType() == typeof(NumericUpDown))
                        contChild.BackColor = Color.White;
            }
            if (cont.GetType() == typeof(NumericUpDown))
                cont.BackColor = Color.White;
       }
    }   

To nie działa, jeśli formant podrzędny ma własne elementy podrzędne.
soulblazer

0

Możesz spróbować, jeśli chcesz :)

    private void ClearControls(Control.ControlCollection c)
    {
        foreach (Control control in c)
        {
            if (control.HasChildren)
            {
                ClearControls(control.Controls);
            }
            else
            {
                if (control is TextBox)
                {
                    TextBox txt = (TextBox)control;
                    txt.Clear();
                }
                if (control is ComboBox)
                {
                    ComboBox cmb = (ComboBox)control;
                    if (cmb.Items.Count > 0)
                        cmb.SelectedIndex = -1;
                }

                if (control is CheckBox)
                {
                    CheckBox chk = (CheckBox)control;
                    chk.Checked = false;
                }

                if (control is RadioButton)
                {
                    RadioButton rdo = (RadioButton)control;
                    rdo.Checked = false;
                }

                if (control is ListBox)
                {
                    ListBox listBox = (ListBox)control;
                    listBox.ClearSelected();
                }
            }
        }
    }
    private void btnClear_Click(object sender, EventArgs e)
    {
        ClearControls((ControlCollection)this.Controls);
    }

1
Samo wysłanie kodu pocztowego niewiele pomaga OP w zrozumieniu problemu lub rozwiązania. Powinieneś prawie ZAWSZE dołączyć jakieś wyjaśnienie, które ma towarzyszyć kodowi.
leigero

Pytanie nie mówiło nic o czyszczeniu formularza.
LarsTech

Tak, nie odpowiada na „pytanie”, ale jest miłym dodatkiem do niego. Dziękuję Ci!

0

Chociaż kilku innych użytkowników opublikowało odpowiednie rozwiązania, chciałbym opublikować bardziej ogólne podejście, które może być bardziej przydatne.

Jest to w dużej mierze oparte na odpowiedzi JYeltona.

public static IEnumerable<Control> AllControls(
    this Control control, 
    Func<Control, Boolean> filter = null) 
{
    if (control == null)
        throw new ArgumentNullException("control");
    if (filter == null)
        filter = (c => true);

    var list = new List<Control>();

    foreach (Control c in control.Controls) {
        list.AddRange(AllControls(c, filter));
        if (filter(c))
            list.Add(c);
    }
    return list;
}

0
    public static IEnumerable<T> GetAllControls<T>(this Control control) where T : Control
    {
        foreach (Control c in control.Controls)
        {
            if (c is T)
                yield return (T)c;
            foreach (T c1 in c.GetAllControls<T>())
                yield return c1;
        }
    }

0
    public IEnumerable<T> GetAll<T>(Control control) where T : Control
    {
        var type = typeof(T);
        var controls = control.Controls.Cast<Control>().ToArray();
        foreach (var c in controls.SelectMany(GetAll<T>).Concat(controls))
            if (c.GetType() == type) yield return (T)c;
    }

0

Dla każdego, kto szuka wersji VB kodu C # Adama napisanej jako rozszerzenie Controlklasy:

''' <summary>Collects child controls of the specified type or base type within the passed control.</summary>
''' <typeparam name="T">The type of child controls to include. Restricted to objects of type Control.</typeparam>
''' <param name="Parent">Required. The parent form control.</param>
''' <returns>An object of type IEnumerable(Of T) containing the control collection.</returns>
''' <remarks>This method recursively calls itself passing child controls as the parent control.</remarks>
<Extension()>
Public Function [GetControls](Of T As Control)(
    ByVal Parent As Control) As IEnumerable(Of T)

    Dim oControls As IEnumerable(Of Control) = Parent.Controls.Cast(Of Control)()
    Return oControls.SelectMany(Function(c) GetControls(Of T)(c)).Concat(oControls.Where(Function(c) c.GetType() Is GetType(T) Or c.GetType().BaseType Is GetType(T))
End Function

UWAGA: dodałem BaseType dopasowanie dla wszelkich pochodnych kontrolek niestandardowych. Jeśli chcesz, możesz to usunąć lub nawet ustawić jako parametr opcjonalny.

Stosowanie

Dim oButtons As IEnumerable(Of Button) = Me.GetControls(Of Button)()

0

Utwórz metodę

public static IEnumerable<Control> GetControlsOfType<T>(Control control)
{
    var controls = control.Controls.Cast<Control>();
    return controls.SelectMany(ctrl => GetControlsOfType<T>(ctrl)).Concat(controls).Where(c => c is T);
}

I użyj go jak

Var controls= GetControlsOfType<TextBox>(this);//You can replace this with your control

0

Używam VB, więc napisałem metodę rozszerzającą. To powoduje pobranie wszystkich elementów podrzędnych i podrzędnych kontrolki

Imports System.Runtime.CompilerServices
Module ControlExt

<Extension()>
Public Function GetAllChildren(Of T As Control)(parentControl As Control) As IEnumerable(Of T)
    Dim controls = parentControl.Controls.Cast(Of Control)
    Return controls.SelectMany(Of Control)(Function(ctrl) _
        GetAllChildren(Of T)(ctrl)) _
        .Concat(controls) _
        .Where(Function(ctrl) ctrl.GetType() = GetType(T)) _
    .Cast(Of T)
End Function

End Module

Następnie możesz go użyć w taki sposób, że „btnList” jest kontrolką

btnList.GetAllChildren(Of HtmlInputRadioButton).FirstOrDefault(Function(rb) rb.Checked)

W takim przypadku wybierze wybrany przycisk opcji.


-1

Po prostu:

For Each ctrl In Me.Controls.OfType(Of Button)()
   ctrl.Text = "Hello World!"
Next

Spowoduje to znalezienie tylko formantów bezpośrednio w kolekcji kontrolek „Ja” i nie znajdzie kontrolek Button, które znajdują się w jakichkolwiek kontenerach podrzędnych, tak jak próbował zasugerować plakat przez „WSZYSTKO”.
ChrisPBacon
Korzystając z naszej strony potwierdzasz, że przeczytałeś(-aś) i rozumiesz nasze zasady używania plików cookie i zasady ochrony prywatności.
Licensed under cc by-sa 3.0 with attribution required.