Asp.net Linkbutton loop

Question

I am having a problem with dynamic link buttons which I need help with. I am creating a dynamic asp.net table based on records in a datatable. I am also using dynamic linkbuttons.

protected System.Web.UI.WebControls.LinkButton lb;    

protected override void OnInit(EventArgs e)
    {
        // Build controls before page load
        lb = new LinkButton();
        lb.Text = "Update Image";

        // LinkButton obj for updating record
        lb.ID = "UpdateImg";
        lb.Click += new EventHandler(UpdateImg);
        this.Controls.Add(lb);
        base.OnInit(e);
    }

I create a new linkbutton instance OnInit so I can add it to the page before page load.

foreach (DataRow r in tb.Rows) // Create new row foreach row in table
            {
                TableRow tr = new TableRow();

                // Build cells
                TableCell c1 = new TableCell();
                TableCell c2 = new TableCell();
                TableCell c3 = new TableCell();
                TableCell c4 = new TableCell();
                TableCell c5 = new TableCell();
                TableCell c6 = new TableCell();

                c1.Controls.Add(new LiteralControl(r["Image_id"].ToString()));
                tr.Cells.Add(c1);
                c2.Controls.Add(new LiteralControl(r["Image_name"].ToString()));
                tr.Cells.Add(c2);
                c3.Controls.Add(new LiteralControl(r["Alt_text"].ToString()));
                tr.Cells.Add(c3);
                c4.Controls.Add(new LiteralControl("<input id=\"" + r["Image_id"].ToString() + "\" type=\"checkbox\"" + "\"></input>"));
                tr.Cells.Add(c4);


                LinkButton lbcopy = new LinkButton();
                lbcopy = lb;
                lbcopy.ID = "UpdateImg" + i;
                i++;
                c5.Controls.Add(lbcopy);
                tr.Cells.Add(c5);


                c6.Controls.Add(new LiteralControl("<a href=\"javascript:void(0);\" onclick=\"DeleteImage('" + r["Image_id"].ToString() + "','" + r["Image_name"].ToString() + "');\"><img src=\"../images/clipboard/del.png\" id=\"" + r["Image_id"].ToString() + "\" width=\"20\" height=\"20\" BORDER=0></a>"));
                tr.Cells.Add(c6);
                tblImageLibrary.Rows.Add(tr); // Assign tr to table

I then use a foreach loop to iterate through each row in the datatable so I can build each table row and add the cells to each row. The problem I am having is the linkbutton appears in the very last row. Possibly because there is only one linkbutton object and through each loop iteration its being moved to the next row?

I am new to asp.net so go easy on me.

Crab Bucket · Accepted Answer · 2011-12-13 12:30:16Z

I appreciate what you are trying to do but could I suggest another way. Try using a ListView (.Net 3.5+) and putting the linkButton in each row. The linkButton can be then hidden as required by hooking into the ItemDataBound event. Also the button can have command arguments embedded into it at this point.

 protected void MyListView_ItemDataBound(object sender, ListViewItemEventArgs e)
 {
     LinkButton linkButton = (LinkButton)e.Item.FindControl("linkButtonId");
     System.Data.DataRowView rowView = e.Item.DataItem as System.Data.DataRowView;

     //.. can hide and show depending on data
     linkButton.Visible = rowView["SomeData"].ToString() == "SomeValue";

     //.. or set command arg
     linkButton.CommandArgument = rowView["SomeMoreData"].ToString();
}

The link button will then trigger the ItemCommand event. The row of the data and any command arguments can be retrieved from the event.

 protected void MyListView_OnItemCommand(object sender, ListViewCommandEventArgs e)
 {
     if(e.CommandArgument == "SomeValue")
     {
       //.. do something
     }
 }

Similar methods can be used on Repeater and GridView controls which work with earlier versions of .Net (1.0 and 2.0 respectively). You might even find the GridView more useful as the edit button is more out of the box.

Your way of dynamically constructing a table will get problematic with events and maybe ViewState. Leveraging .Net grid controls would be a lot easier.

Manoj Savalia · Accepted Answer · 2011-12-13 11:32:18Z

0

I think that is not possible that you want because if you want to add Control dynamically then you must have loop for each record and then you add it in table.

answered Dec 13, 2011 at 11:32

Manoj Savalia

1,4023 gold badges13 silver badges37 bronze badges

2 Comments

nick gowdy Over a year ago

Is there an alternative that I can use? The linkbutton is for editing a record in the datatable, which is why I want each row to have a linkbutton. So the user can edit that particular row.

Manoj Savalia Over a year ago

there is not any method which is used directly. So you have to use loop for if.

Dan An · Accepted Answer · 2011-12-13 12:10:56Z

0

Are you trying to render a table with linkbuttons on each row? take look about asp:listview or asp:gridview.

If you can get a DataTable or a DataSet from your database then you just need to bind it to your listview or gridview. e.g.

myListView.datasource = myDataSet myListView.databind()

what else you need is to define layout and specify column name on ur aspx pages.

answered Dec 13, 2011 at 12:10

Dan An

4249 silver badges27 bronze badges

Collectives™ on Stack Overflow

Asp.net Linkbutton loop

3 Answers 3

Comments

2 Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related