Update label C#

Question

Hello

When the page first load i have a label who has 0 or 1. Look at the code and you will se what i trying to do. But it don't work because the page allready loaded.

protected void rptBugStatus_ItemDataBound(object sender, RepeaterItemEventArgs e)
    {
        if (e.Item.ItemType == ListItemType.Item || e.Item.ItemType == ListItemType.AlternatingItem)
        {
            Label lblName = e.Item.FindControl("lblBugStatus") as Label;
            if (lblName.Text == "1")
            {
                lblName.Text = lblName.Text + "Under arbete";
            }
            else if (lblName.Text == "0")
            {
                lblName.Text = "Fixad";
            }
            else { }
        }
    }

Answer 1

where you put your code? and be cautious e.Item.FindControl("lblBugStatus") as Label; may return null

Answer 2

You really want to avoid this type of coding if at all possible. This will quickly grow into an unmaintainable web site.

Query the underlying data instead of GUI elements.

Answer 3

it's just work fine to fine the control. But i figure out another way to get out the data i want. That i'am going to is fix the xml structur better.

Answer 4

You may have resolved this by now, but I think the problem with the code as you posted it is this:

Label lblName = e.Item.FindControl("lblBugStatus") as Label;

Because you are referencing a control in a repeater, the controls inside each repeater item are named dynamically based on their context. Most likely the name of the control is something like:

"rptBugStatus$repeaterItem0$lblBugStatus"

To find out the exact name, hard code a value, run the page in a browser, and look at the HTML output (via "View Source" in the browser menu). You should be able to scroll down and see your repeater (it will be rendered as a <table> tag), its items, and the controls living inside each item. The IDs/Names will be set and you can copy/paste them into your FindControl method.

Hope this helps,

Nate

Answer 5

Ok, I am taking the assumption you have set this label (assuming to be in the repeater, from your code) to have a value - I have tried databinding to something exactly the same, using the below code.

protected override void OnPreRender(EventArgs e)
{
    base.OnLoad(e);
    List<string> strList = new List<string>();
    strList.Add("1");
    rptBugStatus.DataSource = strList;
    rptBugStatus.DataBind();
}

protected void rptBugStatus_ItemDataBound(object sender, RepeaterItemEventArgs e)
{
    if (e.Item.ItemType == ListItemType.Item || e.Item.ItemType == ListItemType.AlternatingItem)
    {
        Label lblBugStatus = e.Item.FindControl("lblBugStatus") as Label;
        // have added this just so we are actualyl setting the text property on the bug status
        // - i have assumed you do this.
        lblBugStatus.Text = e.Item.DataItem.ToString();

        if (lblBugStatus.Text.Equals("1"))
        {
            lblBugStatus.Text = lblBugStatus.Text + "Under arbete";
        }
        else if (lblBugStatus.Text.Equals("0"))
        {
            lblBugStatus.Text = "Fixad";
        }
    }
}

With the aspx being

    <asp:Repeater runat="server" ID="rptBugStatus" OnItemDataBound="rptBugStatus_ItemDataBound">

    <ItemTemplate>
        <asp:Label ID="lblBugStatus" runat="server"></asp:Label>
    </ItemTemplate>
    </asp:Repeater>

And I have no problems.

• I get the below on the page.

1Under arbete

I think you are going to need to post more code if you have hidden any away from us :)

Tim