Solved

GridView with a Checkbox

Posted on 2009-05-15
8
585 Views
Last Modified: 2013-11-07
Hello Experts,
I have a GridVeiw which is populated and Binded on Form Load. When the user selects a checkbox from the grid and clicks submit button it cant find the Checkbox which is checked. Therefore chbTemp is always retruning false.
Your help would be appreciated  thanks in advance.
D
protected void Button1_Click(object sender, EventArgs e)
        {
string text = String.Empty;
                    for (int i = 0; i < GrdVw.Rows.Count; i++)
            {
                CheckBox chbTemp = GrdVw.Rows[i].FindControl("Chk") as CheckBox;
                if (chbTemp.Checked)
                {
                    text = (GrdVw.Rows[i].Cells[1].Text);
                }
            }
            txtBox.Text = text;
            }
 
<asp:GridView ID="GrdVw" runat="server" AutoGenerateColumns="False">
    <Columns>
        <asp:TemplateField HeaderText="insert">
            <ItemTemplate>
                <asp:CheckBox ID="Chk"  runat="server"/>
            </ItemTemplate>
        </asp:TemplateField>        
    </Columns>
</asp:GridView>

Open in new window

0
Comment
Question by:drgorsia
  • 3
  • 2
  • 2
  • +1
8 Comments
 
LVL 39

Accepted Solution

by:
abel earned 250 total points
ID: 24394075
You shouldn't have to loop through all the rows to find your checkbox. If the checkbox is checked, then the checkBox_Changed event will fire. You can use that instead.

If you don't want that, and the button is on a different location then the current row (if it is on the current row you can use NamingContainer), and you want, say, a lot of checked checkboxes treated in one go, then your method is good.

But, it seems that your gridview is not filled anymore. And all you are looking at is an empty set of datarows. the rows that you do find are the headerrows. Can you try the following code to find out whether that's the case (paste the output of the output window here, please)?

foreach (GridViewRow row in GrdVw.Rows)
{
    if (row.RowType == DataControlRowType.DataRow)
    {
        // found a datarow, it must have a checkbox:
        CheckBox chbTemp = row.FindControl("Chk") as CheckBox;
        Debug.Print("datarow found, has checkbox: " + (chbTemp != null)); 
    }
}

Open in new window

0
 
LVL 3

Expert Comment

by:channa_m
ID: 24395195
Try the following code, this is from one of my project, which works perfectly.
         for (int i = 0; i < gv.Rows.Count; i++)//loop the GridView Rows
            {
                CheckBox cbApprove = (CheckBox)gv.Rows[i].Cells[0].FindControl("cbApprove"); //find the CheckBox
                if (cbApprove != null)
                {
                    if (cbApprove.Checked)
                    {
                         Response.write("Checked");
                    }
                }
            }
 
 
 
 
   <asp:TemplateField HeaderText="Select">
                    <ItemTemplate>
                        <asp:CheckBox ID="cbApprove" runat="server" />
                    </ItemTemplate>
                </asp:TemplateField

Open in new window

0
 
LVL 39

Expert Comment

by:abel
ID: 24396215
@channa_m: never use Response.write, unless your whole page is created with Response.Writes (it should be Write, not write, btw). Instead, use a Literal or a Label and write your data there.

In the event that you mix Response.Write with ASPX code, the .Write either comes totally at the end, or totally at the beginning (other situations also possible) and that means before the <html>: wrong markup and often browsers don't even show it....
0
Master Your Team's Linux and Cloud Stack

Come see why top tech companies like Mailchimp and Media Temple use Linux Academy to build their employee training programs.

 
LVL 3

Expert Comment

by:channa_m
ID: 24396354
Abel: I know Response.Write is not the best approach, thanks for the info though.
0
 
LVL 7

Expert Comment

by:ajolly
ID: 24401071
Check if you have set the EnableViewState property true for your gridview and checkbox.
0
 
LVL 39

Expert Comment

by:abel
ID: 24401918
@ajolly: agreed and good point, but do note (@OP) that the default is TRUE for this. So if you don't set it explicitly, it is on.
0
 
LVL 7

Expert Comment

by:ajolly
ID: 24401999
yes, its true by default.

but still we don't see the code for the page and the web.config.
If you set the enableviewstate property for the page or in web.config for all pages, it will be false.
In that case you will have to explicitly enable it for the control/page you want.
0
 

Author Closing Comment

by:drgorsia
ID: 31587283
Abel thanks for your help....used your method and it worked fine. Shame is that the project has been put on hold. Hopefully it will kick start soon...thanks for all the help.
D
0

Featured Post

Webinar: Aligning, Automating, Winning

Join Dan Russo, Senior Manager of Operations Intelligence, for an in-depth discussion on how Dealertrack, leading provider of integrated digital solutions for the automotive industry, transformed their DevOps processes to increase collaboration and move with greater velocity.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Problem Hi all,    While many today have fast Internet connection, there are many still who do not, or are connecting through devices with a slower connect, so light web pages and fast load times are still popular.    If your ASP.NET page …
Calculating holidays and working days is a function that is often needed yet it is not one found within the Framework. This article presents one approach to building a working-day calculator for use in .NET.
I've attached the XLSM Excel spreadsheet I used in the video and also text files containing the macros used below. https://filedb.experts-exchange.com/incoming/2017/03_w12/1151775/Permutations.txt https://filedb.experts-exchange.com/incoming/201…

820 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question