0

следующим кодом я создаю двумерный массив pictureBox.

private PictureBox[,] pics = new PictureBox[6, 6];

private void createMap() {

        for (int i = 0; i < 6; i++)
        {
            for (int j = 0; j < 6; j++)
            {
                pics[i, j] = new PictureBox();
                pics[i, j].Location = new Point(34 + 76 * j, 99 + 76 * i);
                pics[i, j].Size = new Size(70, 70);
                pics[i, j].Image = Properties.Resources.BackGroundLime;
                this.Controls.Add(pics[i, j]);
            }
        }
    }

Подскажите, как понять по какому pictureBox было нажато?

1
  • Ответы в общем правильные. Но я бы советовал использовать событие MouseClick вместо Click. Так в аргументах события больше информации передаётся. Commented 29 июн. 2020 в 11:52

2 ответа 2

1
private void Form1_Load(object sender, EventArgs e)
    {
        for (var i = 0; i < 6; i++)
        {
            for (var j = 0; j < 6; j++)
            {
                pics[i, j] = new PictureBox();
                pics[i, j].Location = new Point(34 + 76 * j, 99 + 76 * i);
                pics[i, j].Size = new Size(70, 70);
                pics[i, j].Image = Properties.Resources.BackGroundLime;
                pics[i, j].MouseClick += PictureBoxes_Click;
                pics[i, j].Name = $"Блок #{i},{j}";
                Controls.Add(pics[i, j]);
            }
        }
    }
private void PictureBoxes_Click(object sender, EventArgs e)
    {
        MessageBox.Show(((PictureBox) sender).Name);
    }
3
  • Код Happy_Sweetice подошел, все работает )))
    – vladdemo1
    Commented 29 июн. 2020 в 11:41
  • Что за название такое: Form1_Click? Ведь не форму кликаем, а пикчабокс. Я бы назвал метод PictureBoxes_Click. Обратите внимание на множественное число, ведь этот обработчик сразу для всех пикчабоксов. Commented 29 июн. 2020 в 11:51
  • @AlexanderPetrov , Исправил название, а так это набросок из VS. Commented 29 июн. 2020 в 12:08
0
for (int i = 0; i < 6; i++)
{
    for (int j = 0; j < 6; j++)
    {
        pics[i, j] = new PictureBox();
        pics[i, j].Location = new Point(34 + 76 * j, 99 + 76 * i);
        pics[i, j].Size = new Size(70, 70);
        pics[i, j].Image = Properties.Resources.BackGroundLime;
        pics[i, j].Click += (object sender, EventArgs args) => 
            MessageBox.Show($"Вы нажали на pics[{i}, {j}]");
        this.Controls.Add(pics[i, j]);
    }
}
1
  • 2
    Лучше заменить лямбду на нормальный метод. Не исключено, что эти динамически созданные контролы будут удаляться, тогда нужно будет отписаться от событий. От лямбды не отписаться. Commented 29 июн. 2020 в 11:33

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.