Обработчик кнопок

@TwoSided · Регистрация: 08.04.2010

Author24 — интернет-сервис помощи студентам

Доброго времени суток. столкнулся с проблемой, суть в том, что я создаю n кнопок в цикле, а как сделать для них разные обработчики не знаю? точнее разные в них только ID. буду крайне благодарен за помощь.

C#

            for (int i = 1; i < 3; i++)
            {
                myCommandqwe.CommandText = "SELECT x FROM city WHERE id =" + i.ToString();
                xs = myCommandqwe.ExecuteScalar().ToString();
                x = Convert.ToInt16(xs);
                myCommandqwe.CommandText = "SELECT y FROM city WHERE id =" + i.ToString();
                ys = myCommandqwe.ExecuteScalar().ToString();
                y = Convert.ToInt16(ys);
                Button b = new System.Windows.Forms.Button();
                b.Location = new System.Drawing.Point(x, y);
                b.Name = "Label" + i.ToString();
                b.Text = "Label" + i.ToString();
                b.ForeColor = System.Drawing.Color.Yellow;
                this.Controls.Add(b);
                pictureBox1.SendToBack();
                b.Click  += new System.EventHandler(this.bcl);
                
            }

@Mikant · 08.04.2010, 11:20

C#

            for (int i = 0; i < b.Length; i++) {
                int ii = i;
                b[i].Click += delegate(object _sender, EventArgs _e) {
                    MessageBox.Show("Кнопка №" + ii + "нажата");
                };
            }

@Sergei · 08.04.2010, 11:20

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

@I2um1 · 08.04.2010, 11:30

Если я верно понял, то надо разные обработчики событий вызывать в этом месте:

Сообщение от TwoSided

C#

1	b.Click += new System.EventHandler(this.bcl);

Как вариант можна создать массив такого делегата:

C#

1	delegate void MyDel(object sender, EventArgs e);

Как-то так для трех кнопок:

C#

1	MyDel[] d = new MyDel[3];

А после присвоить нужные функции делегатам, например:

C#

1 2	for (int i = 0; i < 3; i++) d[i] = (a, b) => { MessageBox.Show(i.ToString()); };

И в итоге обработчик событий привязать так:

C#

1	b.Click += this.d[i];

Надеюсь, все понятно?

Добавлено через 1 минуту

Сообщение от Sergei

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

Неплохой вариант!

@Mikant · 09.04.2010, 12:55

Blood-Angel, проверь свой код и удивись, почему мессаджбокс всегда показывает число 3

-> замыкания <-

Добавлено через 4 минуты
Sergei, и в моем случае и в случае Blood-Angel создается только ОДИН обработчик события и потом цепляется. это несмотря на то, что даже объявление происходит в цикле. см. ту же ссылку

@I2um1 · 10.04.2010, 10:05

Сообщение от Mikant

Blood-Angel, проверь свой код и удивись, почему мессаджбокс всегда показывает число 3

Все просто:

C#

1	b.Click += this.d[i];

Дело в том, что я присвоил все три делегата одной и той же кнопке - я видел код иначе.

Вот потому и выдает MessageBox всегда три.

Надо так:

C#

1 2	for (int i = 0; i < 3; i++) b[i].Click += this.d[i];

@Mikant · 10.04.2010, 13:49

Не по теме:

Blood-Angel, а вот и нет. даже если ты присвоишь 3 разных делегата трем разным кнопкам все равно будет 3 выводиться. это интересный нюансег. вставь мой код в проект и закомменть int ii = i; увидишь что получится (ну остальные переименуй чтоль)

@I2um1 · 11.04.2010, 17:18

Не по теме:
Mikant, действительно, будем знать.

@Sergei 1512 / 779 / 103 Регистрация: 22.04.2008 Сообщений: 1,610
	08.04.2010, 11:20	3
	Вам не надо делать обработчики для каждой кнопки, а сделайте один для всех только в нем используйте проверки какая кнопка нажата и в зависимости от этого и выполняете нужные действия 1

@Mikant 1319 / 992 / 127 Регистрация: 08.12.2009 Сообщений: 1,299
	09.04.2010, 12:55	5
	Blood-Angel, проверь свой код и удивись, почему мессаджбокс всегда показывает число 3 -> замыкания <- Добавлено через 4 минуты Sergei, и в моем случае и в случае Blood-Angel создается только ОДИН обработчик события и потом цепляется. это несмотря на то, что даже объявление происходит в цикле. см. ту же ссылку 1

@Mikant
	10.04.2010, 13:49 #7
	Не по теме: Blood-Angel, а вот и нет. даже если ты присвоишь 3 разных делегата трем разным кнопкам все равно будет 3 выводиться. это интересный нюансег. вставь мой код в проект и закомменть int ii = i; увидишь что получится (ну остальные переименуй чтоль) 0

@I2um1 Злой няш 2136 / 1505 / 565 Регистрация: 05.04.2010 Сообщений: 2,881
	11.04.2010, 17:18	8
	Не по теме: Mikant, действительно, будем знать. 0