Не срабатывает лямбда-событие нажатия динамической кнопки

@Gamabunta · Регистрация: 23.08.2016

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

C#

protected void Button0_Click(object sender, EventArgs e)
{
    ButtonList.Add(new Button());
    ButtonList[NextButNum].ID = NextButNum.ToString();
    ButtonList[NextButNum].Text = "Нажми меня";
//Не срабатывает это событие
    ButtonList[NextButNum].Click += (s, e) =>
    {
        ButtonList.RemoveAt(Convert.ToInt16(((Button)sender).ID));
    };
    num++;
}

Подскажите в чём ошибка, заранее спасибо.

@OwenGlendower · 25.09.2016, 15:34

Gamabunta, попробуй так:

C#

protected void Button0_Click(object sender, EventArgs e)
{
    var btn = new Button { Text = "Нажми меня" };
    btn.Click += (s, e) =>
    {
        ButtonList.Remove((Button)sender);
    };
    ButtonList.Add(btn);
}

@Gamabunta · 25.09.2016, 15:49 **[ТС]**

OwenGlendower
Попробовал, даже прогнал дебаггером, событие вообще не достигается, ButtonList.Сount остаётся прежним

@Cupko · 25.09.2016, 18:10

а) Я так полагаю, вместо sender должен быть s, нет?
b) RemoveAt по индексу удаляет, а не по ID. У вас точно индекс с ID совпадает?

@Gamabunta · 25.09.2016, 18:35 **[ТС]**

Cupko
a) Вы правы
b) Всё верно, если вешаю на статическую кнопку - срабатывает.
Попробовал для проверки события так:

C#

ButtonList.Add(new Button { Text = "Нажми меня", ID = num.ToString() });
ButtonList[num].Click += (s, e) =>
{
    Label1.Text = "Event has taken place!";
};

Не сработало

@insite2012 · 25.09.2016, 20:01

Gamabunta, я в ASP.NET не особо силен, но помню из начальных глав, что конвейер ASP.NET работает немного не так, как мы привыкли в настольных приложениях, и вполне возможно, что эта проблема возникает именно из-за этого.
И именно это касается динамических контролов.

@Gamabunta · 25.09.2016, 20:09 **[ТС]**

insite2012
Так и есть, вот что нашёл:

This is a timing issue:
Page_Load is executed, page is rendered.
Dropdown is selected. Post back is send to the server. Page_Load is executed. Then the event is send to your dropdown instance. SelectedIndexChanged is executed. Your button is created and the page is rendered and send to the client.
Button is pushed. Post back is send to the server. Page_Load is executed. Asp tries to execute the event, but the button does not exist anymore. So the event is ignored.

If you're dynamically creating controls in WebForms, you always have to recreate them on every postback and before the ViewState is loaded

@insite2012 · 25.09.2016, 20:13

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

Так и есть

Ну да, я читал об этом.
Вам надо создавать контролы и подписку в событии Page_Load, насколько я помню.

@Gamabunta · 25.09.2016, 20:46 **[ТС]**

insite2012
Я пошёл другим путём и переопределил LoadViewState, но через Page_Load будет даже проще

@Gamabunta 35 / 35 / 16 Регистрация: 23.08.2016 Сообщений: 118
	25.09.2016, 15:49 [ТС]	3
	OwenGlendower Попробовал, даже прогнал дебаггером, событие вообще не достигается, ButtonList.Сount остаётся прежним 0

@Cupko 653 / 589 / 171 Регистрация: 17.07.2012 Сообщений: 1,670 Записей в блоге: 1
	25.09.2016, 18:10	4
	а) Я так полагаю, вместо sender должен быть s, нет? b) RemoveAt по индексу удаляет, а не по ID. У вас точно индекс с ID совпадает? 0

@insite2012 5534 / 4298 / 1217 Регистрация: 12.10.2013 Сообщений: 12,332 Записей в блоге: 2
	25.09.2016, 20:01	6
	Gamabunta, я в ASP.NET не особо силен, но помню из начальных глав, что конвейер ASP.NET работает немного не так, как мы привыкли в настольных приложениях, и вполне возможно, что эта проблема возникает именно из-за этого. И именно это касается динамических контролов. 0

@Gamabunta 35 / 35 / 16 Регистрация: 23.08.2016 Сообщений: 118
	25.09.2016, 20:09 [ТС]	7
	insite2012 Так и есть, вот что нашёл: This is a timing issue: Page_Load is executed, page is rendered. Dropdown is selected. Post back is send to the server. Page_Load is executed. Then the event is send to your dropdown instance. SelectedIndexChanged is executed. Your button is created and the page is rendered and send to the client. Button is pushed. Post back is send to the server. Page_Load is executed. Asp tries to execute the event, but the button does not exist anymore. So the event is ignored. If you're dynamically creating controls in WebForms, you always have to recreate them on every postback and before the ViewState is loaded 0

@insite2012 5534 / 4298 / 1217 Регистрация: 12.10.2013 Сообщений: 12,332 Записей в блоге: 2
	25.09.2016, 20:13	8
	Сообщение было отмечено Gamabunta как решение Решение Сообщение от Gamabunta Так и есть Ну да, я читал об этом. Вам надо создавать контролы и подписку в событии `Page_Load`, насколько я помню. 1

@Gamabunta 35 / 35 / 16 Регистрация: 23.08.2016 Сообщений: 118
	25.09.2016, 20:46 [ТС]	9
	insite2012 Я пошёл другим путём и переопределил LoadViewState, но через Page_Load будет даже проще 0

Не срабатывает лямбда-событие нажатия динамической кнопки

Решение