checkbox selection problems, help me trace my program

Question

1.31/5 (3 votes)

See more:

I have a problem on the selections of my checkboxes, I don't know how to construct the program which will make the selections of these txtboxes correct so here it is,
I'm have four checkboxes here they are: chkAll, chkActivity, chkItem and chkService

XML

<asp:CheckBox ID="chkAll" runat="server" Text="All" Checked="True" ForeColor="#333333" style="font-family: sans-serif" AutoPostBack="True" oncheckedchanged="chkAll_CheckedChanged"/>
    <asp:CheckBox ID="chkActivity" runat="server"  Text="Activity" ForeColor="#333333" style="font-family: sans-serif" AutoPostBack="True" oncheckedchanged="chkAll_CheckedChanged"/>
    <asp:CheckBox ID="chkItem" runat="server"  Text="Item" ForeColor="#333333" style="font-family: sans-serif" AutoPostBack="True" oncheckedchanged="chkAll_CheckedChanged"/>
    <asp:CheckBox ID="chkService" runat="server" Text="Service" ForeColor="#333333" style="font-family: sans-serif" AutoPostBack="True" oncheckedchanged="chkAll_CheckedChanged"/>

By default the checkboxes are selected or checked == "true",

C#

protected void Page_Load(object sender, EventArgs e)
   {
       chkAll.Checked = true;
       chkActivity.Checked = true;
       chkItem.Checked = true;
       chkService.Checked = true;

Now when I deselect the first one which is chkAll meaning clicking on it so it will deselect, it will make all the other 3 which are chkActivity, chkItem and chkService unchecked/deselected too.
Now my problem is how to select the individual textboxes after or a comnibations of any of the three checkboxes deselected awhile ago please help me, I would really aprreciate your response and thanks in advanced.
By the way I've tried using this nested loop thing but it doesn't work, anyway I'll just post it so maybe you could improve it;

C#

protected void chkAll_CheckedChanged(object sender, EventArgs e)
   {
       if (chkAll.Checked)
       {
           chkActivity.Checked = true;
           chkItem.Checked = true;
           chkService.Checked = true;
       }
       else
       {
           if (chkActivity.Checked == false)
           {
               chkActivity.Checked = false;
           }
           else if (chkItem.Checked == false)
           {
               chkItem.Checked = false;
           }
           else if (chkService.Checked == false)
           {
               chkService.Checked = false;
           }
           else
           {
               chkActivity.Checked = false;
               chkItem.Checked = false;
               chkService.Checked = false;
           }
       }
   }

Posted 13-Aug-13 6:14am

timJosh

Updated 13-Aug-13 6:21am

v4

Add a Solution

Comments

Richard C Bishop 13-Aug-13 12:19pm

The reason that is happening is because you are calling the same event handler on each check box. What code exists inside that event handler?

pradiprenushe 13-Aug-13 12:28pm

Not getting your requiement?

TryAndSucceed 13-Aug-13 12:44pm

Try not using oncheckedchanged event in all the 3 checkboxes except "All". I think it is causing an infinite loop.

3 solutions

Solution 3

I used this:

C#

function NotAllByService() {
       var chkAct = document.getElementById('<%=chkActivity.ClientID %>');
       var chkServ = document.getElementById('<%=chkService.ClientID %>');
       var chkIte = document.getElementById('<%=chkItem.ClientID %>');
       if (chkServ.checked == false) {
           document.getElementById('<%=chkAll.ClientID %>').checked = false;
       }
       else {
           if (chkAct.checked == true & chkServ.checked == true & chkIte.checked == true) {
               document.getElementById('<%=chkAll.ClientID %>').checked = true;
           } else {
               document.getElementById('<%=chkAll.ClientID %>').checked = false;
           }
       }
   }

   function NotAllByActivity() {
       var chkAct = document.getElementById('<%=chkActivity.ClientID %>');
       var chkServ = document.getElementById('<%=chkService.ClientID %>');
       var chkIte = document.getElementById('<%=chkItem.ClientID %>');
       if (chkAct.checked == false) {
           document.getElementById('<%=chkAll.ClientID %>').checked = false;
       }
       else {
           if (chkAct.checked == true & chkServ.checked == true & chkIte.checked == true) {
               document.getElementById('<%=chkAll.ClientID %>').checked = true;
           } else {
               document.getElementById('<%=chkAll.ClientID %>').checked = false;
           }
       }
   }

   function NotAllByItem() {
       var chkIte = document.getElementById('<%=chkItem.ClientID %>');
       var chkServ = document.getElementById('<%=chkService.ClientID %>');
       var chkAct = document.getElementById('<%=chkActivity.ClientID %>');
       if (chkIte.checked == false) {
           document.getElementById('<%=chkAll.ClientID %>').checked = false;
       }
       else {
           if (chkAct.checked == true & chkServ.checked == true & chkIte.checked == true) {
               document.getElementById('<%=chkAll.ClientID %>').checked = true;
           } else {
               document.getElementById('<%=chkAll.ClientID %>').checked = false;
           }
       }

   }

Posted 16-Aug-13 22:29pm

timJosh

Solution 1

Can I unsee it?!!! :-)

Really, it's painful to see. How it's possible to write such thing?

C#

if (chkActivity.Checked == false)
{
   chkActivity.Checked = false;
}

when essentially this does nothing?
And you should not write redundant "if"; when you have some Boolean value already it's

C#

bool condition = //...
//...
chkActivity.Checked = condition;

White the code in a non-nonsense, accurate way, and perhaps you won't need to ask such questions. :-)

—SA

Posted 13-Aug-13 6:42am

Sergey Alexandrovich Kryukov

Updated 13-Aug-13 6:43am

v2

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)

gaga blues · Accepted Answer · 2013-08-13T06:54:00

Good day sir. Instead of calling a postback when you check the chkAll checkbox, you can just check the chkActivity, chkItem etc etc through javascript;

change your chkAll to this

C#

<asp:checkbox id="chkAll" runat="server" checked="True" forecolor="#333333" text="All" style="font-family:sans-serif" onchange="searchCheckAll();" xmlns:asp="#unknown" />

then add this function between

JavaScript

<script type="text/javascript"> and </script>