Question

背景：对于学校作业，我正在做一个益智游戏。玩家必须创建一个益智游戏并将其保存在文本文件中。首先，播放器输入图块（PictureBoxes）的行和列，程序将创建2D PictureBox布局。在哪个播放器选择工具（从ImageList中分配了图像的按钮）并单击图块之后，该图像就会出现在图块上。

方法：我有一个自定义类，该类继承自PictureBox类并具有ToolValue属性。 ToolValue是玩家选择并添加到该PictureBox的工具（为工具分配常数）。为了加载图像，我创建了一个新的事件处理程序，用于处理PictureBox控件的click事件，并在for循环内具有其他参数。新的事件处理程序将图像加载到图块上，并将自定义PictureBox（MyPictureBox）类的ToolValue属性设置为常量。我创建了PictureBox类引用的二维数组。

MyPictureBox[,] Tile;

        public void DrawALineOfPictureBoxes(int rowNumber, int columnCount, int rowCount, int leftPosition, int topPosition, int height, int width)
        {
            Tile = new MyPictureBox[rowCount, columnCount];
            for (int colNumber = 0; colNumber < columnCount; colNumber++)
            {
                Tile[rowNumber, colNumber] = new MyPictureBox();
                Tile[rowNumber, colNumber].Left = leftPosition;
                Tile[rowNumber, colNumber].Top = topPosition;
                Tile[rowNumber, colNumber].Height = height;
                Tile[rowNumber, colNumber].Width = width;
                Tile[rowNumber, colNumber].BorderStyle = BorderStyle.Fixed3D;
                Tile[rowNumber, colNumber].SizeMode = PictureBoxSizeMode.StretchImage;
                Tile[rowNumber, colNumber].ToolValue = 0;
                Tile[rowNumber, colNumber].Click += new EventHandler((sender,e) => LoadImage_Click(sender, e, rowNumber, colNumber));
                this.Controls.Add(Tile[rowNumber, colNumber]);

                leftPosition += width;
            }
        }
        /// <summary>
        /// Method executes when Generate Button is clicked  
        /// Method invokes DrawALineOfPictureBoxes method which generates a row of Picture boxes
        /// 
        /// </summary>
        /// <param name="sender"></param>
        /// <param name="e"></param>
        /// 
        private void PictureBoxGenerate_Click(object sender, EventArgs e)
        {
            try
            {
                int numRows = int.Parse(txtRowCount.Text);
                int numColumns = int.Parse(txtColumnCount.Text);
                int leftPos = 400;
                int topPos = 120;
                int height = 100;
                int width = 100;
                //loop after each row of picturebox is generated
                for (int rowNumber = 0; rowNumber < numRows; ++rowNumber)
                {
                    DrawALineOfPictureBoxes(rowNumber, numColumns, numRows, leftPos, topPos, height, width);
                    topPos += height;
                }
            }
            catch (FormatException)
            {
                MessageBox.Show("Please provide valid data for rows and columns (Both must be integers)","Sokoban", MessageBoxButtons.OK, MessageBoxIcon.Error);
            }
        }


        /// <summary>
        /// Method is executed when PictureBox cell is clicked and load image to that picture box through resources
        /// </summary>
        /// <param name="sender"></param>
        /// <param name="e"></param>
        private void LoadImage_Click(object sender, EventArgs e, int rowNumber, int colNumber)
        {
            string Content = rowNumber.ToString() + "," + colNumber.ToString();
            Console.WriteLine(Content);
            MyPictureBox pictureBox = sender as MyPictureBox;
            switch (imageType)
            {
                case ImageType.None:
                    //Empty the pictureBox Cell
                    pictureBox.Image = null;
                    ToolVal = 0;

                    break;
                case ImageType.Hero:
                    pictureBox.Image = Properties.Resources.Hero;
                    ToolVal = 1;
                    break;
                case ImageType.Wall:
                    pictureBox.Image = Properties.Resources.Wall;
                    ToolVal = 2;
                    break;
                case ImageType.Box:
                    pictureBox.Image = Properties.Resources.Box;
                    ToolVal = 3;
                    break;
                case ImageType.Destination:
                    pictureBox.Image = Properties.Resources.Destination;
                    ToolVal = 4;
                    break;
                default:
                    break;
            }
            //assigning values to Tile array
            Tile[rowNumber,colNumber].ToolValue = ToolVal;
        }

问题：我已传递pictureBox的行号和列号，单击它作为加载图像的EventHandler的参数。在事件处理程序内部，开关循环检查单击了哪个工具，并相应地将常数分配给toolValue属性。如果单击第一个图片框，则传递给事件处理程序的参数对于rowNumber来说是0，对于colNumber是0。事件处理程序参数应该相同，但colNumber参数不为0，而是3，即。生成的2d pictureBoxes的总列。

在此处显示输出>> Colnumber is 3

Answer 1

您的问题在以下几行：

Tile[rowNumber, colNumber].Click += new EventHandler((sender,e) => LoadImage_Click(sender, e, rowNumber, colNumber));

您正在使用传递rowNumber和colNumber的lambda表达式。触发图块的Click事件时，将执行lambda，并传递当时的rowNumber和colNumber值（而不是创建事件处理程序时的值）。因为您的循环已经完成，所以您传递的colNumber的值始终为3。

因此，您需要另一种机制来计算单击的图块的行和列。您可以根据传递的发送者来执行此操作：将其投射到Tile上，检索其位置并据此计算行和列。

如果您确实要传递实际的列号，则可以始终像这样“捕获”“ colNumber”的值：

Tile = new MyPictureBox[rowCount, columnCount];
        for (int colNumber = 0; colNumber < columnCount; colNumber++)
        {
            int actualColumn = colNumber;
            int actualRow = rowNumber; 

            Tile[rowNumber, colNumber] = new MyPictureBox();
            ...
            Tile[rowNumber, colNumber].Click += new EventHandler((sender,e) => LoadImage_Click(sender, e, actualRow, actualColumn));
            this.Controls.Add(Tile[rowNumber, colNumber]);

            leftPosition += width;
        }

对于创建磁贴的循环的每次迭代，您将创建一个新变量，其中包含磁贴的实际行和列。这些值不会更改，因为每个迭代都有其自己的版本。然后传递这些变量而不是rowNumber和colNumber。

Answer 2

我可以为您提供大部分一般建议：

首先，WindowsForms和其他GUI技术不是游戏开发的正确工具。它适用于纯粹的回合制，单人或多座位的多人游戏。如果您没有超过图形。基本上，旧的Solitair大约是一切可能的上限。对于认真的开发，您需要使用Game Loop。这是一个学校/学习项目，因此实际上可能会使项目复杂一些。

第二，避免在UI中存储数据。始终在某些集合后面的代码中包含数据（游戏状态）。用户界面只是该数据的表示，您可以定期对其进行更新。

对于传递给事件的参数，这里最有用的应该是sender。这是引发事件的实例。您可以按以下方式对此进行检查：

//Shared handler for OK, Apply and Cancel Buttons
if(sender == btnOK || sender == btnApply){

}
if(sender == btnOK || sender == btnCancel){

}

或者您可以对其进行投射，以获得对发件人属性的完全访问权限

PictureBox source = (PictureBox)sender;
source.ImageLocataion = "clicked.png";

笔记的一个特殊属性是Tag。它需要一个对象，目的似乎是识别仅在此名称/引用不起作用的情况下到达此处的UI元素。诸如图块编号/标识符之类的内容应在此处提供帮助。或数据库行的主键。

将参数传递给事件处理程序

2 个答案: