Dynamically Creating Events

Question

1.00/5 (2 votes)

See more:

Hi all,
Please see below code

XML

<asp:Repeater ID="rptrAddPhotos" runat="server" DataSourceID="AddPhotosDS" OnItemDataBound="rptrAddPhotos_itemDataBound">
<ItemTemplate>                                        
   <asp:LinkButton ID= "lnkAddedPhoto" runat="server">
            XXXXXX
   </asp:LinkButton>
</ItemTemplate>
</asp:Repeater>

C#

protected void rptrAddPhotos_itemDataBound( object sender, RepeaterItemEventArgs e ) {
  Label AddImgDescription = (Label)e.Item.FindControl( "AddImgDescription" );
  lnkAddedPhoto.Click += new EventHandler(lnkAddedPhoto_onClick );
}

protected void lnkAddedPhoto_onClick( object sender, EventArgs e ) { 
		
}

Event 'lnkAddedPhoto_onClick'is not hitting. Please help me

Thanks
Manu v nath

Posted 17-Jun-13 3:32am

Manu V Nath

Updated 17-Jun-13 18:06pm

v5

Add a Solution

Comments

Richard C Bishop 17-Jun-13 10:13am

If you are not dynamically creating the "InkAddedPhoto" button, there is no reason to add an event handler dynamically.

Sergey Alexandrovich Kryukov 17-Jun-13 11:00am

First of all, even if the event is added using the designer, it is exactly as "dynamic" as when it is added in code. During runtime, it won't change the scenario. Adding event to the button created in designer-generated code still may make sense, just to make code more neat, flexible, etc. Coding style, not functionality...
—SA

Richard C Bishop 17-Jun-13 11:08am

Indeed, my usage of the word "dynamically" was probably not conveying my message as it were meant. Adding an event handler with code is not necessary if the control is not being generated by code is what I was trying to get across to the OP.

Sergey Alexandrovich Kryukov 17-Jun-13 11:58am

Agree, but as I say, there can be other reasons. For example, in windows application, I never add event handlers by the designer: it provides obsolete and redundant syntax which lacks the elegance of anonymous methods, and it make the code less supportable. The designer is generally overused, so people do a lot of repetitive manual work.
—SA

Richard C Bishop 17-Jun-13 12:01pm

Interesting, thank you for the perspective.

Sergey Alexandrovich Kryukov 17-Jun-13 12:53pm

You are welcome.
—SA

Tadit Dash (ତଡିତ୍ କୁମାର ଦାଶ) 17-Jun-13 10:49am

lnkAddedPhoto.Click += new EventHandler(lnkAddedPhoto_onClick );

Which LinkButton it will point to? You need to find it for every row and then you can do it.
Like you have found the control AddImgDescription, just like that find this control and then attach event.

2 solutions

Solution 1

Creating dynamic events and controls always creates problem with viewstate. Create your events inside your aspx.

Posted 17-Jun-13 13:28pm

Christian Graus

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

_Amy · Accepted Answer · 2013-06-17T18:22:00

You need to find LinkButton from ItemTemplate of Repeater.
Try this:

C#

protected void rptrAddPhotos_itemDataBound( object sender, RepeaterItemEventArgs e ) {
    Label lnkAddedPhoto= (Label)e.Item.FindControl( "lnkAddedPhoto" );
    lnkAddedPhoto.Click += new EventHandler(lnkAddedPhoto_onClick );
}

protected void lnkAddedPhoto_onClick( object sender, EventArgs e ) {

}

--Amit