> function checkForNoSelections(){
>
> var chkd = 0;
> var e = document.viewItems;
>
> for(var i = 0; i < e.atccNum.length; i++){
> if(e.atccNum[i].checked == true){
> chkd = ++chkd;
> }
> }
> if(chkd > 0){
> e.submit();
> return true;
> }
> else{
> alert("You must select an item to display.");
> return false;
> }
> }
That's a bit odd - you typically don't use the assignment and increment
operators together that way, since you really only need one or the other.
The statement
++chkd;
is the same as
chkd = chkd + 1;
in the specific code you have above.
I would use something like this instead:
function checkForNoSelections() {
var e = document.viewItems;
for (var i = 0; i < e.atccNum.length; i++) {
if (e.attcNum[i].checked) {
return true;
}
}
alert('You must select an item to display.');
return false;
}
<form action="..." onsubmit="return checkForNoSelections();">
....
<input type="submit">
</form>
Dave Watts, CTO, Fig Leaf Software
http://www.figleaf.com/
Fig Leaf Software provides the highest caliber vendor-authorized
instruction at our training centers in Washington DC, Atlanta,
Chicago, Baltimore, Northern Virginia, or on-site at your location.
Visit http://training.figleaf.com/ for more information!
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~|
Logware (www.logware.us): a new and convenient web-based time tracking
application. Start tracking and documenting hours spent on a project or with a
client with Logware today. Try it for free with a 15 day trial account.
http://www.houseoffusion.com/banners/view.cfm?bannerid=67
Message: http://www.houseoffusion.com/lists.cfm/link=i:4:212460
Archives: http://www.houseoffusion.com/cf_lists/threads.cfm/4
Subscription: http://www.houseoffusion.com/lists.cfm/link=s:4
Unsubscribe:
http://www.houseoffusion.com/cf_lists/unsubscribe.cfm?user=11502.10531.4
Donations & Support: http://www.houseoffusion.com/tiny.cfm/54
