what is wrong with this java script function?

-=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- (c) WidthPadding Industries 1987 0\|541\|0 -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=- -=+=-
Socoder -> C/C++/C#/Other -> what is wrong with this java script function?
Sat, 12 Jun 2010, 09:00
ShadowIce	I can't get the function getchecked() to show an alert box... <script> function getCheckedValue(radioObj) { if(!radioObj){ return ""; } var radioLength = radioObj.length; if(radioLength == undefined){ if(radioObj.checked){ return radioObj.value; }else{ return ""; } } for(var i = 0; i < radioLength; i++) { if(radioObj[i].checked) { return radioObj[i].value; } } return ""; } } function getchecked(){ var thecode=getCheckedValue(document.forms['radioExampleForm'].elements['code']); if(thecode!==""){ if(thecode=="HTML"){ alert('Checked value is: '+thecode+'.'); //playSound("test1.mp3"); }else if(thecode=="PHP"){ //playSound("test2.mp3"); alert('Checked value is: '+thecode+'.'); }else{ //playSound("test3.mp3"); alert('Checked value is: nothing'); } } } </script> <label for="code1"><input type="radio" value="HTML" name="code" id="code1"> HTML</label> <label for="code2"><input type="radio" value="PHP" name="code" id="code2"> PHP</label> <label for="code3"><input type="radio" value="MySQL" name="code" id="code3"> MySQL</label> <input type="button" onclick="getchecked()" value="Show Checked Value"> --v thanks!
Sat, 12 Jun 2010, 09:10
shroom_monk	On the line var thecode=getCheckedValue(document.forms['radioExampleForm'].elements['code']); --v you're asking for a form 'radioExampleForm', when the form doesn't exist. You need to use <form> tags to get it to work! On a side note, you have an extraneous } symbol before the line function getchecked(){. Just so you know. -=-=- A mushroom a day keeps the doctor away... Keep It Simple, Shroom!
Sat, 12 Jun 2010, 09:19
ShadowIce	i fgt. to add the form. <form name="radioExampleForm" method="get" action=""> it still wont work
Sat, 12 Jun 2010, 09:29
shroom_monk	Really? I added in the form using your line there, and it worked. Except for the SQL option, because you left that out of the code. Here's your code, working, with the <form> lines, the SQL recognition added, the extra } removed and proper indenting: <script> function getCheckedValue(radioObj) { if(!radioObj){ return ""; } var radioLength = radioObj.length; if(radioLength == undefined){ if(radioObj.checked){ return radioObj.value; }else{ return ""; } } for(var i = 0; i < radioLength; i++) { if(radioObj[i].checked) { return radioObj[i].value; } } return ""; } function getchecked(){ var thecode=getCheckedValue(document.forms['radioExampleForm'].elements['code']); if(thecode!==""){ if(thecode=="HTML"){ alert('Checked value is: '+thecode+'.'); //playSound("test1.mp3"); }else if(thecode=="PHP"){ //playSound("test2.mp3"); alert('Checked value is: '+thecode+'.'); }else if(thecode=="MySQL"){ //playSound("test2.mp3"); alert('Checked value is: '+thecode+'.'); }else{ //playSound("test3.mp3"); alert('Checked value is: nothing'); } } } </script> <form name="radioExampleForm" method="get" action=""> <label for="code1"><input type="radio" value="HTML" name="code" id="code1"> HTML</label> <label for="code2"><input type="radio" value="PHP" name="code" id="code2"> PHP</label> <label for="code3"><input type="radio" value="MySQL" name="code" id="code3"> MySQL</label> <input type="button" onclick="getchecked()" value="Show Checked Value"> </form> --v -=-=- A mushroom a day keeps the doctor away... Keep It Simple, Shroom!