Calculation using switch case on radio Button

Cap2010 · Jul 10, 2000

I want calculate using switch statement unable to do it. What could be the error. Tried using if condition and for loop doesn't work. Want to calculate each location, position and display the total with respective location and position at the bottom What could be error and how to call all function in one function. function calc1() { function loc1() function loc2() } <HTML><HEAD> <SCRIPT language="JavaScript"> <!-- function position(pos){ var t=0; switch(pos) { case 10:{t=10; break; } case 9: {t=9; break; } case 8: {t=8; break; } case 7: {t=7; break; } case 6: {t=6; break; } case 5: {t=5; break; } case 4: {t=4; break; } case 3: {t=3; break; } default: alert("You Must Select&quot

; break; } } function location(loc){ var t1; switch(loc) { case 10:{t1=10; break; } case 9: {t1=9;  break; } case 8: {t1=8;  break; } case 7: {t1=7;  break; } case 6: {t1=6;  break; } case 5: {t1=5; break; } case 4: {t1=4; break; } case 3: {t1=3; break; } default: alert("You Must Select&quot

; break; } } function calc(testform){ var ttotal1; call function position(pos); {ttotal1 = parseInt(t); } function location(loc); {ttotal1  = ttotal1 + parseInt(t1);} } </script> </HEAD> <BODY BGCOLOR="#FFFFFF"> <form name="testform" > <table border=1> <tr><td> Q1 What is the position <INPUT TYPE="radio" NAME="pos" Value="10">North East <INPUT TYPE="radio" NAME="pos" Value="9">East <INPUT TYPE="radio" NAME="pos" Value="8">North <INPUT TYPE="radio" NAME="pos" Value="7">North West <INPUT TYPE="radio" NAME="pos" Value="6">South East <INPUT TYPE="radio" NAME="pos" Value="5">West <INPUT TYPE="radio" NAME="pos" Value="4">South <INPUT TYPE="radio" NAME="pos" Value="3">South West </td></tr> <tr><td> Q2 Where is the Location? <INPUT TYPE="radio" NAME="loc" Value="10">South East <INPUT TYPE="radio" NAME="loc" Value="9">North West <INPUT TYPE="radio" NAME="loc" Value="8">East <INPUT TYPE="radio" NAME="loc" Value="7">West <INPUT TYPE="radio" NAME="loc" Value="6">South <INPUT TYPE="radio" NAME="loc" Value="5">North <INPUT TYPE="radio" NAME="loc" Value="4">North East <INPUT TYPE="radio" NAME="loc" Value="3">South West </td></tr> <tr><td> <INPUT type=button value="Get your Location" onClick="calc(testform);"> <INPUT type=reset value=clear> </TD></TR> <TR colspan=2><TD align="center"> Result <INPUT name=result size=4> </TD></TR> </TBODY></TABLE></FORM> <table border="1" summary="Total Location"> <tr>Your Location <th>No.</th> <th>Questions</th> <th>Points</th> <th> Your Location</th> <tr><td align="right">1</td> <td>Your position is</td> <td align="right">10</td> <td align="right"> </td> </tr> <tr><td align="right">2</td> <td>Your Location is </td> <td align="right">10</td> <td align="right">  </td> </tr> <tr><td> </td> <td align="right">Total </td> <td align="right">100</td> <td align="right">  </td> </tr> </tr> </table></div> </BODY> </HTML>

carpeliam · Jul 10, 2000

there are no brackets in switch statements. So, instead of this: switch(loc) { case 10:{t1=10; break; } case 9: {t1=9;  break; } case 8: {t1=8;  break; } case 7: {t1=7;  break; } case 6: {t1=6;  break; } case 5: {t1=5; break; } case 4: {t1=4; break; } case 3: {t1=3; break; } default: alert("You Must Select&quot

; break; } you'd want this: switch(loc) { case 10: t1=10; break; case 9:  t1=9;  break; case 8:  t1=8;  break; case 7:  t1=7;  break; case 6:  t1=6;  break; case 5:  t1=5;  break; case 4:  t1=4;  break; case 3:  t1=3;  break; default: alert("You Must Select&quot

; break; } That should work... Liam Morley <a href=mailto:lmorley@wpi.edu>lmorley@wpi.edu</a> <a href=

http://www.wpi.edu/~lmorley/>

] :: imotic :: website :: [</a> "light the deep, and bring silence to the world. 
light the world, and bring depth to the silence.

palbano · Jul 10, 2000

Dear Liam and ALad, > there are no brackets in switch statements. While brackets are not required inside the cases of a switch statement they are allowed and should not cause any errors in the way ALad's original code is written. I have not check all the code but there are several problems here: function calc(testform){ var ttotal1; call function position(pos); {ttotal1 = parseInt(t); } function location(loc); {ttotal1  = ttotal1 + parseInt(t1);} } That code is just a mess. First: call function position(pos); I don't have any idea what the scripting engine might do with that expression! I would hope an error would be produced if there is no 'call' statement in Javascript which I don't think there is. What I think you want to do is this: postion(pos); Then you call your 'location' function location(loc); which defines a local variable named 't1' then you try to access the local variable in the outer scope: ttotal1  = ttotal1 + parseInt(t1); That's just not going to work. You need to do a little more studying of 'variable scope' in programming concepts. I would think the browser would generate an error at the above line if it ever gets to it. Good luck -pete

carpeliam · Jul 10, 2000

thanks, Pete... I didn't want to look through all that code, so I just put down the first thing that didn't look right. Goes back to the "necessary code to reproduce the problem" issue... Liam Morley <a href=mailto:lmorley@wpi.edu>lmorley@wpi.edu</a> <a href=

http://www.wpi.edu/~lmorley/>

] :: imotic :: website :: [</a> "light the deep, and bring silence to the world. 
light the world, and bring depth to the silence.

palbano · Jul 10, 2000

Liam, > Goes back to the "necessary code to reproduce the problem" issue THANKS! I didn't know anyone was listening! ;o) -pete

Cap2010 · Jul 11, 2000

Have a look at the Script only, have Improvised it. Only Trouble is it does not goes for calculation, it just displays the default message twice and ttotal=0.   <html> <script language="JavaScript"> var t=0, t1=0; function location1(x){ switch(x) { case 10:t=10; return t=10; break; case 9: t=9; return t=9;  break; case 8: t=8; return t=8; break; case 7: t=7; return t=7; break; case 6: t=6; return t=6; break; case 5: t=5; return t=5; break; case 4: t=4; return t=4; break; case 3: t=3; return t=3; break; default:    alert("You Must Select&quot

; break;         }         return t; } function location2

{ switch

{ case 10: t1=10; return t1=10; break; case 9: t1=9; return t1=9; break; case 8: t1=8; return t1=8; break; case 7: t1=7; return t1=7; break; case 6: t1=6; return t1=6; break; case 5: t1=5; return t1=5; break; case 4: t1=4; return t1=4; break; case 3: t1=3; return t1=3; break; default:     alert("You Must Select&quot

; break; } return t1; } function calc(testform){ var ttotal1; location1(testform.loc1) ttotal1 = parseInt(t); location2(testform.loc2) ttotal1 = ttotal1 + parseInt(t1); alert(" TOTAL -> " + ttotal1) return true; } </script>  <form name="testform" > <table BORDER > <tr> <td>Q1 What is the position <input TYPE="radio" NAME="loc1" Value="10">North East <input TYPE="radio" NAME="loc1" Value="9">East <input TYPE="radio" NAME="loc1" Value="8">North <input TYPE="radio" NAME="loc1" Value="7">North West <input TYPE="radio" NAME="loc1" Value="6">South East <input TYPE="radio" NAME="loc1" Value="5">West <input TYPE="radio" NAME="loc1" Value="4">South <input TYPE="radio" NAME="loc1" Value="3">South West </td> </tr> <tr> <td>Q2 Where is the Location? <input TYPE="radio" NAME="loc2" Value="10">South East <input TYPE="radio" NAME="loc2" Value="9">North West <input TYPE="radio" NAME="loc2" Value="8">East <input TYPE="radio" NAME="loc2" Value="7">West <input TYPE="radio" NAME="loc2" Value="6">South <input TYPE="radio" NAME="loc2" Value="5">North <input TYPE="radio" NAME="loc2" Value="4">North East <input TYPE="radio" NAME="loc2" Value="3">South West  </td> </tr>  <tr> <td> <input type=button value="Get your Location"      onClick="calc(testform);">         <input type=reset value=clear>  </td> </tr> <tr colspan="2">     <td ALIGN=CENTER>Result<input name=result size=4></td> </tr> </table> </form> </center> </body> </html>

carpeliam · Jul 11, 2000

arrgh

) If I'm only supposed to be looking at the script, don't post what I don't need to see... sorry, I don't mean to complain- but scrolling past unnecessary data alone can be painful... Liam Morley <a href=mailto:lmorley@wpi.edu>lmorley@wpi.edu</a> <a href=

http://www.wpi.edu/~lmorley/>

] :: imotic :: website :: [</a> "light the deep, and bring silence to the world. 
light the world, and bring depth to the silence.

Cap2010 · Jul 13, 2000

Hi, Had given the HTML statement to try it out and see the message.  Nothing much about it. Awaiting, Lad

Tek-Tips is the largest IT community on the Internet today!

Members share and learn making Tek-Tips Forums the best source of peer-reviewed technical information on the Internet!

Calculation using switch case on radio Button

Cap2010

Programmer

carpeliam

Programmer

palbano

Programmer

carpeliam

Programmer

palbano

Programmer

Cap2010

Programmer

carpeliam

Programmer

Cap2010

Programmer

Similar threads

Part and Inventory Search

Sponsor