熄灯 C# 多维数组

Question

我们有熄灯游戏要编程

        Button[,] lights = new Button[5,5];

            for (int i = 0; i < lights.GetLength(1); i++)
            {
                for (int j = 0; j < lights.GetLength(0); j++)
                {
                    lights[i, j] = new Button();
                    lights[i, j].Size = new System.Drawing.Size(50, 50);
                    lights[i, j].Click += (sender, args) => light_OnOff(lights[i,j], i, j);
                    lights[i, j].BackColor = Color.Yellow;
                    lightPanel.Controls.Add(lights[i, j]);

                    MessageBox.Show("I:"+Convert.ToString(i) + "J:" +Convert.ToString(j));
                }
            }
        }

        public void light_OnOff(object sender, int i, int j)
        {
            if(lights[i, j].BackColor == Color.Yellow)
            {
                lights[i, j].BackColor = Color.Black;
            }
        }
    }
}

我现在遇到的问题是它说它超出了Array范围（索引），但我不知道为什么。 也许你可以帮助我。

Answer 1

不要以您的方式创建事件，而是设置按钮的名称以将它们彼此分开。

        for (int i = 0; i < lights.GetLength(1); i++)
        {
            for (int j = 0; j < lights.GetLength(0); j++)
            {
                lights[i, j] = new Button();
                lights[i, j].Size = new System.Drawing.Size(50, 50);
                lights[i, j].Name = "button" + i.ToString() + j.ToString(); // set name like this
                lights[i, j].Click += autoGeneratedEventName_Click; //after "+=" hit tab twice visual studio will create event auto;
                lights[i,j].Location = new Point(40 + (j*70), 20 + (i * 70));
                lights[i, j].BackColor = Color.Yellow;
                this.panel1.Controls.Add(lights[i, j]);

                //MessageBox.Show("I:" + Convert.ToString(i) + "J:" + Convert.ToString(j));
            }
        } 

而在事件中，sender 将解决问题，您不需要将数组的索引发送到事件。 （您也可以从按钮的最后 2 个名称中找到索引）

private void autoGeneratedEventName_Click(object sender, EventArgs e)
        {
            Button b = sender as Button; //Clicked object is a Button
            if (b.BackColor == Color.Yellow) 
            {
                b.BackColor = Color.Black;
            }
            label1.Text = b.Name;
        }

结果;

希望有所帮助，

Answer 2

您正在处理代码中所谓的“闭包”。

lights[i, j].Click += (sender, args) => light_OnOff(lights[i,j], i, j);

变量 i 和 j 不会被 lamdba 表达式复制，而是作为引用处理，在这些循环结束后仍然存在。 所以当点击事件被触发时，它会使用 i & j (5) 的最新值，这会导致超出数组范围的异常。

要获得预期的行为，您可以将 i & j 复制到局部变量，如下所示：

int e = i; int f = j; 
lights[i, j].Click += (sender, args) => light_OnOff(lights, e, f);

阅读更多信息： http : //csharpindepth.com/Articles/Chapter5/Closures.aspx