Для каждого контура управления, затрагивающего только один элемент управления

Question

Я постараюсь сохранить это как можно более конкретным.

Это мой код:

foreach (Control c in this.Controls) 
{ 
    if (c.Height > 25) 
    { 
     c.BackColor = Color.Red; 
     pictureBox1.Location = new Point(x1, y1); 
     if (pictureBox1.Bounds.IntersectsWith(c.Bounds)) 
     { 
      isCollide = true; 
      label1.Text = "true"; 
      c.BackColor = Color.Green; 
     } 
     else 
     { 
      isCollide = false; 
      label1.Text = "false"; 
     } 

    } 
} 

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

Переменная «isCollide», которая имеет значение «true», просто делает ее так, что я не могу ее пропустить.

Проблема в том, что она затрагивает только 1 коробку изображений (стену). Тем не менее, я решил изменить цвет всех выбранных элементов управления на зеленый, чтобы проверить, какие элементы управления подвержены воздействию цикла, и все же это влияет на все стены, как это должно быть.

Мой вопрос:, почему цвет влияет на все элементы управления, через которые прошел мой код, но обнаружение конфликтов не работает на всех них? Это потому, что переменная isCollide установлена ​​одновременно на «ложь» и «истина» из-за столкновения с одной стеной и одновременно не сталкивается с другим?

Как я могу исправить эту проблему?

Это код, который использует «isCollide»:

public void move(string Direction) 
    { 
     if (Direction == "Up") 
     { 
      //b = x/a = y 
      reset(); 
      x1 = x; 
      y1 = y - 5; 
      isCollided(); 
      if (isCollide == false) 
      { 
       y -= 5; 
      } 


     } 

* Примечание: Не конец кода, аналогичный код был применен ко всем направлениям (сверху, слева, справа, снизу)

Answer 1

Конечное значение isCollide будет основано на столкновении последнего элемента управления, который повторяется в цикле foreach. Если вам не важно, с каким объектом сталкивается столкновение, я рекомендую выйти из цикла после обнаружения столкновения. Что-то вроде этого.

foreach (Control c in this.Controls) 
{ 
    if (c.Height > 25) 
    { 
     c.BackColor = Color.Red; 
     pictureBox1.Location = new Point(x1, y1); 
     if (pictureBox1.Bounds.IntersectsWith(c.Bounds)) 
     { 
      isCollide = true; 
      label1.Text = "true"; 
      c.BackColor = Color.Green; 
      break; 
     } 
     else 
     { 
      isCollide = false; 
      label1.Text = "false"; 
     } 
    } 
} 

Answer 2

Его, потому что isCollide является просто локальными переменной и не свойством вашего контроля, как ваш c.BackColor это.

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

public class Control 
{ 
    public bool isCollide {get; set;} 
    public string BackColor {get; set;} 
    public int Height {get; set;} 
} 

foreach (Control c in this.Controls) 
     { 
      if (c.Height > 25) 
      { 
       c.BackColor = Color.Red; 
       pictureBox1.Location = new Point(x1, y1); 
       if (pictureBox1.Bounds.IntersectsWith(c.Bounds)) 
       { 
        c.isCollide = true; 
        label1.Text = "true"; 
        c.BackColor = Color.Green; 
       } 
       else 
       { 
        c.isCollide = false; 
        label1.Text = "false"; 
       } 

      } 
     } 

Answer 3

Другой причиной может быть, что элементы управления на форме вложены в контейнеры (например, GroupBox). В этом случае вам придется рекурсивно проверять элементы в ихControl.