fun with checkboxes

[payne747]

Hi everyone,

Can anyone help me with the following code, for some reason I can't
understand if I'm being thick of if javascript is so backwards!

HTML defines some checkboxes such as:

<form name="players_f orm">
<input type="checkbox" name="check1" value="blabla"
onclick="Update Players()" />
<input type="checkbox" name="check2" value="blabla"
onclick="Update Players()" />
.... (10 checkboxes in total)
</form>

The UpdatePlayers() function looks like this:

function UpdatePlayers()
{
for(i=0; i<10; i++) // always will be 10 checkboxes so run 10
times.
{
if (document.playe rs_form.check[i].checked == true)
{
players++;
}
else {
players--;
}
}

document.all.ou tput.innerHTML = players;
}

This bit of code is supposed to return the amount of boxes checked via
the 'players' variable. However the error thrown back is:

document.player s_form.check - null or unknown object

For some reason, the variable 'i' is not being appended to the check
name. Does anyone know of a better way to do this? Basically the aim is
to just dynamicaly count the amount of checked boxes on a screen.

Thanks!

Payne747

[Randy Webb]

pa******@gmail. com said the following on 4/19/2006 7:25 AM:

Hi everyone,

Can anyone help me with the following code, for some reason I can't
understand if I'm being thick of if javascript is so backwards!
You are being thick.
HTML defines some checkboxes such as:

<form name="players_f orm">
<input type="checkbox" name="check1" value="blabla"
onclick="Update Players()" />
<input type="checkbox" name="check2" value="blabla"
onclick="Update Players()" />
.... (10 checkboxes in total)
</form>
checkboxes can share a common name.

<input type="checkbox" name="myCheckbo xes" value="blabla"
onclick="Update Players()" />

The UpdatePlayers() function looks like this:

function UpdatePlayers()
{
for(i=0; i<10; i++) // always will be 10 checkboxes so run 10
times.
{
if (document.playe rs_form.check[i].checked == true)
If you are intent on naming them the way you named them:

if (document.playe rs_form.element s['check'+i].checked)

The == true is superfluous and not needed. The .checked property is a
boolean.
{
players++;
}
else {
players--;
}
}

document.all.ou tput.innerHTML = players;

document.getEle mentById('outpu t').innerHTML = players;

--
Randy
comp.lang.javas cript FAQ - http://jibbering.com/faq & newsgroup weekly
Javascript Best Practices - http://www.JavascriptToolbox.com/bestpractices/

[Chris Diver]

> For some reason, the variable 'i' is not being appended to the check

name. Does anyone know of a better way to do this? Basically the aim is
to just dynamicaly count the amount of checked boxes on a screen.

Well it wont, your code is trying to access an element in an array
that doesn't exist. Try changing each check box to :

<input type="checkbox" name="check" etc.. />

This should create an array of checkboxes and your
other code *should* then work.

HTH

Chris

[payne747]

ah ha! figured it out finally thanks to you guys and a bit of pizza,
for reference, heres the final code to count the amount of checked
boxes:

function UpdatePlayers() {

for(i=0; i<10; i++)
{
if(document.pla yers_form.check[i].checked)
{
players++;
}

alert('Players = ' + players);
}

document.all.ou tput.innerHTML = players;
players = 0;

}

pa******@gmail. com wrote:

Hi everyone,

Can anyone help me with the following code, for some reason I can't
understand if I'm being thick of if javascript is so backwards!

HTML defines some checkboxes such as:

<form name="players_f orm">
<input type="checkbox" name="check1" value="blabla"
onclick="Update Players()" />
<input type="checkbox" name="check2" value="blabla"
onclick="Update Players()" />
... (10 checkboxes in total)
</form>

The UpdatePlayers() function looks like this:

function UpdatePlayers()
{
for(i=0; i<10; i++) // always will be 10 checkboxes so run 10
times.
{
if (document.playe rs_form.check[i].checked == true)
{
players++;
}
else {
players--;
}
}

document.all.ou tput.innerHTML = players;
}

This bit of code is supposed to return the amount of boxes checked via
the 'players' variable. However the error thrown back is:

document.player s_form.check - null or unknown object

For some reason, the variable 'i' is not being appended to the check
name. Does anyone know of a better way to do this? Basically the aim is
to just dynamicaly count the amount of checked boxes on a screen.

Thanks!

Payne747