Should I loop

[Steve Bishop]

I have 10 fields each with it's own row in a table. Based on another
fields value within the same row for each, I would like to hide or show
the row.

Example: I have 10 rows with size and order qty data:
Row 1 thru 4 are taken with sizes large, X-Large, XX-Large and
XXX-Large.

Rows 5 thru 10 don't have size values populated, so I want to hide the
row. I know how to show and hide elements by id including TR's, but I
was wondering if there is an efficient way to use a loop for this vs.
using a big ass IF statement. Help/Examples appreciated. Thanks.

*** Sent via Developersdex http://www.developersdex.com ***
Don't just participate in USENET...get rewarded for it!

[Brian Genisio]

Steve Bishop wrote:

I have 10 fields each with it's own row in a table. Based on another
fields value within the same row for each, I would like to hide or show
the row.

Example: I have 10 rows with size and order qty data:
Row 1 thru 4 are taken with sizes large, X-Large, XX-Large and
XXX-Large.

Rows 5 thru 10 don't have size values populated, so I want to hide the
row. I know how to show and hide elements by id including TR's, but I
was wondering if there is an efficient way to use a loop for this vs.
using a big ass IF statement. Help/Examples appreciated. Thanks.

You really need to show code, so I will need to make assumptions...

assuming that the data is in an array myArray. Since I dont know what
type of data you have in the array, I am assuming it is cell data.
////////////////////////////////////
for(i=0; i<myArray.length; i++)
{
if(myArray[i].innerText == "")
// hide
else
// dont hide
}
////////////////////////////////////

If you need anything more specific, you will _have_ to give us example
code to modify.

Brian

[Steve Bishop]

Thanks Brian. Help appreciated. This is embarrassing, but here I go:
if (myform2.itm_005.value == "")
{
document.getElementById("q5").style.visibility="hi dden"
document.getElementById("q6").style.visibility="hi dden"
document.getElementById("q7").style.visibility="hi dden"
document.getElementById("q8").style.visibility="hi dden"
document.getElementById("q9").style.visibility="hi dden"
document.getElementById("q10").style.visibility="h idden"
}
if (myform2.itm_006.value == "")
{
document.getElementById("q6").style.visibility="hi dden"
document.getElementById("q7").style.visibility="hi dden"
document.getElementById("q8").style.visibility="hi dden"
document.getElementById("q9").style.visibility="hi dden"
document.getElementById("q10").style.visibility="h idden"
}
if (myform2.itm_007.value == "")
{
document.getElementById("q7").style.visibility="hi dden"
document.getElementById("q8").style.visibility="hi dden"
document.getElementById("q9").style.visibility="hi dden"
document.getElementById("q10").style.visibility="h idden"
}
if (myform2.itm_008.value == "")
{
document.getElementById("q8").style.visibility="hi dden"
document.getElementById("q9").style.visibility="hi dden"
document.getElementById("q10").style.visibility="h idden"
}
if (myform2.itm_009.value == "")
{
document.getElementById("q9").style.visibility="hi dden"
document.getElementById("q10").style.visibility="h idden"
}
if (myform2.itm_010.value == "")
{
document.getElementById("q10").style.visibility="h idden"
}
*** Sent via Developersdex http://www.developersdex.com ***
Don't just participate in USENET...get rewarded for it!

[Brian Genisio]

Steve Bishop wrote:

Thanks Brian. Help appreciated. This is embarrassing, but here I go:
if (myform2.itm_005.value == "")
{
document.getElementById("q5").style.visibility="hi dden"
document.getElementById("q6").style.visibility="hi dden"
document.getElementById("q7").style.visibility="hi dden"
document.getElementById("q8").style.visibility="hi dden"
document.getElementById("q9").style.visibility="hi dden"
document.getElementById("q10").style.visibility="h idden"
}
<SNIP>....
if (myform2.itm_010.value == "")
{
document.getElementById("q10").style.visibility="h idden"
}

Assuming you know the number of total lines (number_of_cells) ... I am
short-circuiting it. I am also assuming you index 1 to 10, instead of 0
to 10 I havent tested this, so there may be typos or something, but you
will get the idea...

Oh yeah, this also assumes a broser that uses getElementById. Older
browsers may need some other method...

Also note, that your method for making things hidden will not work on
all browsers.

////////////////////////////////////////////////
var number_of_cells = 10;

for(i=1; i<=number_of_cells; i++) {
if(document.getElementById("itm_" + i).value == "")
document.getElementById("q" + i).style.visibility="hidden";
}
/////////////////////////////////////////////////