Change Background Color of ASP:GridView Based On Text

Question

I am using the OnRowDataBound() event to change the Row.BackColor based on the text. Now my issue is, this is extremely slow. I have roughly 100 rows in the ASP:Gridview and only need to evaluate the first row or index[0]. What optimization techniques do you guru's have on how to speed this process up?

protected void GridView1_OnRowDataBound(object sender, GridViewRowEventArgs e)
{
    if (e.Row.RowType == DataControlRowType.DataRow)
    {
        TableCell cell = e.Row.Cells[1];
        string question = cell.Text;

        if (question == "View below to get data:" | question == "Check the state information below:" | question == "Check the state information below:")
        {
            e.Row.BackColor = System.Drawing.Color.Gray;
        }
    }
}

Answer 1

Generally, styling should be handled using client side methods such as using css/javascript so I would recommend you do some brushing up on that side. Never the less I was like you when I first started and just jumped straight into c# so here is a purely c# based solution :).

If it's only ever going to be the first row that needs to be shaded grey you can just reference that row when the binding in complete. The event you're currently subscribing to OnRowDataBound will fire for every row. There is another event that can be accessed once all rows are completed called OnDataBound.

First thing to do would be to register this event in the html markup of your gridview, using the same approach you used to register the OnRowDataBound event.

<asp:gridView ID="GridView1" runat="server" AutoGenerateColumns="false" OnDataBound="GridView1_OnDataBound"></asp:gridView>

Then you can code up the event in your c#.

protected void GridView1_OnDataBound(object sender, EventArgs e)
{
    if(GridView1.Rows.Count > 0) //Check there are rows bound
    {
        var Row = GridView1.Rows[0];// Get The First Row - you may have a header row. If this is the case change 0's to 1's!

        Row.BackColor = System.Drawing.Color.Gray;// Assign the back color
    }
}