Как это бросать InvalidCastException

Question

Следующий код надрез бросает InvalidCastException на петле Еогеасп:

Невозможно привести объект типа «System.Windows.Forms.StatusStrip» до типа «System.Windows .Forms.GroupBox.

Я просто не понимаю, как это возможно .... Затем я снова нуб, поэтому его, вероятно, что-то глупое.

 private void doSlide(GroupBox MoveThis) 
    { 
     //location 12,27 
     var t = Task.Factory.StartNew(() => 
      { 
       ExecuteSecure(() => 
        { 
          foreach (GroupBox box in this.Controls) 
          { 
           if (box != MoveThis) 
           { 
            box.Left = (-1) * box.Width; 
           } 
           else 
           { 
            do 
            { 
             if (box.Left > 12) 
              box.Left--; 
             else 
              box.Left++; 
            } 
            while (box.Left != 12); 
           } 
          } 

        }); 
      }); 
    } 

Вот код для Execute Secure

private void ExecuteSecure(Action a) 
     { 
      if (InvokeRequired) 
       BeginInvoke(a); 
      else a(); 
     } 

В основном у меня есть форма с фиксированным размером и несколько коробок групп на форме, только один из которых виден в любой заданной точке. Когда нам нужно сделать новую видимость GroupBox, мы вызываем DoSlide (GroupBox) и указываем групповой ящик, который мы хотим сделать видимым. Затем предполагается, что каждый GroupBox переместит форму в местоположение (-Box.Width, 27), за исключением указанной формы, которая получает слайд (увеличенный или уменьшенный box.left) в представлении.

Answer 1

Вы хотите использовать

this.Controls.OfType<GroupBox>() 

в вашем Еогеасп. This.Controls возвращает все элементы управления, а не только GroupBoxes. Метод расширения OfType<T> фильтрует коллекцию по указанному типу.

Answer 2

Когда вы Переберите каждый элемент управления, вы должны быть определяющим типа управления, а не только при условии, что все они GroupBox объектов

Answer 3

Коллекция this.Controls содержит все ваши формы управления первого уровня, так что не каждый контроль внутри является GroupBox. Вы должны получить только GroupBox управления

Изменение цикла, используя следующий синтаксис

foreach (GroupBox box in this.Controls.OfType<GroupBox>()) 
{ 
    .... 
} 

Answer 4

Не каждый элемент управления в Controls коллекции является GroupBox (очевидно), но foreach пытается бросить их в линию.

Таким образом, изменить свою петлю на что-то вроде этого:

foreach (var control in this.Controls) 
{ 
    if(control is GroupBox) 
    { 
     if (control != MoveThis) 
     { 
      control.Left = (-1) * control.Width; 
     } 
     else 
     { 
      do 
      { 
       if (control.Left > 12) 
        control.Left--; 
       else 
        control.Left++; 
      } 
      while (control.Left != 12); 
     } 
    } 
} 

EDIT: Стоит отметить, что это потенциально может быть очень медленным, так как вы оценить тип каждого элемента управления в форме. Код, который предлагает Джон Крафт и Стив, должен фильтровать коллекцию управления только теми, которые являются GroupBox, так что это, вероятно, даст лучшую производительность ...