JavaScript - Error With If Statements

This may be a basic programing problem but I am writing a function to validate a form using a bunch of else-if statements and encounters a problem when I use nested if/else-if statements. It will not continue on to the next else-if, after it returns false.
Code:

// to check each entry on the form and alert user if entry is invalid.
function checkform(){
  var checkssn = /(\d{3})-(\d{2})-(\d{4})/;
  var checkphone = /(\d{3})-(\d{4})/;	
  var checkname = /[a-zA-Z]+(\.|,|\s)*/;
  var checkzip = /(^967|^968)\d{2}(-\d{4})?$/;
				
  // check to see if user have selected an Election
  if (!document.f1.elections[0].checked && !document.f1.elections[1].checked && !document.f1.elections[2].checked && !document.f1.elections[3].checked) {
	window.alert("Please select an Election!")
	return false;
// check to see if user entered a valid SSN
  } else if ( checkssn.test(document.f1.ssn.value) == false){		
	window.alert("[1]. Please enter a valid social security number in the format ddd-dd-ddd");
	return false;
				
// check to see if user entered a valid home telephone number or business telephone number
  }else if ( document.f1.home_phone.value == '' && document.f1.business_phone.value == '') {
	window.alert("[4]. Please enter a Home or Business telephone number!")	
	return false;
  } else if ( document.f1.home_phone.value != ''){ 
	if (checkphone.test(document.f1.home_phone.value) == false){		
                window.alert("[4]. Please enter a valid home phone number in the format ddd-ddd");
	        return false;
        }
  } else if ( document.f1.business_phone.value != ''){	
		if ( checkphone.test(document.f1.business_phone.value) == false){		
			window.alert("[4]. Please enter a valid business phone number in the format ddd-ddd");
			return false;						
		}	
 // check to see if user entered a valid Name
  }else if ( checkname.test(document.f1.lastname.value) == false){		
		window.alert("[5]. Last Name can only consist of letters, periods(.), commas(,) and spaces");
		return false;
}else if ( checkname.test(document.f1.firstname.value) == false){		
		window.alert("[5]. First Name can only consist of letters, periods(.), commas(,) and spaces");
		return false;

The problem occurs when it validates the phone numbers. When a valid number is entered, it will not move to the next else-if statement to validate the name. It's been years since I program in Java/C, so I'm a bit rusty.
Any help is appreciated.

-Alex

The issue I am having is with my sumAll function:

Code:

<script type="text/javascript">
/* <![CDATA[ */
   function addNumbers()
 {
   a = parseInt(document.getElementById('txtFirst').value);
   b = parseInt(document.getElementById('txtSecond').value);
   c = parseInt(document.getElementById('txtThird').value);
   d = parseInt(document.getElementById('txtFourth').value);
   ansE = document.getElementById("Sum");
   ansE.value = '$' + (a + b + c + d);

   x = (a + b + c + d);


   if ( x >= 100000) {
     document.getElementById('Bonus').value = '$' + 1200; }
   
   else if ( x > 50000 && x<100000) {
     document.getElementById('Bonus').value = '$' + 800; }

   else {
     document.getElementById('Bonus').value = '$' + 0; }


   
   if (document.getElementById('Blue').checked == true && document.getElementById('Gala').checked == true) {
     document.getElementById('Committee').value = '$' + 800; }

   else if (document.getElementById('Blue').checked == true && document.getElementById('Gala').checked == false) {
     document.getElementById('Committee').value = '$' + 350; }

   else if (document.getElementById('Blue').checked == false && document.getElementById('Gala').checked == true) {
     document.getElementById('Committee').value = '$' + 450; }

   else {
     document.getElementById('Committee').value = '$' + 0; }     
}

   function sumAll()
 {
   y = parseInt(document.getElementById('Bonus').value);
   z = parseInt(document.getElementById('Committee').value);
   total = document.getElementById("Total");
   total.value = (y + z);
}


   function funcCalc()
 {
   addNumbers();
   sumAll();
}


/* ]]> */
</script>

I'm really new to this and I'm trying to figure out how to sum the two values from if statements. ParseInt is returning NaN. Before that, it was adding my results together as strings. Is there an easy way to add these together without creating a very complicated if statement to total these up?

Thanks!

Hi

I need help to finish this code with using nested if statements

A price of a ticket to a passenger will be:
First Class 500
Economy Class (with meal) 400
Economy Class (without meal) 200
How I can write a JavaScript code according to the following specifications:
a. Read the class that the passenger wants to travel on.
b. If the class is the first class, print the price of ticket.
c. If the class is the economy class, ask the user if he/she wants a meal on
the flight. Then print the price of the ticket according to the response of
the passenger.
The program should simply accept one possible strings the user enters; it is not
required to work for all possible inputs such as First, first, FIRST or Yes , yes, YES.
This is the code which I have been trying
{
var inputNum = prompt("Enter the class you want\n first class?\neconomy? :");
if (isNaN(inputNum)) {
if (inputNum.match(/first class/)) {
document.write("<h1><center>your Ticket is 500<\center><\h1>");
}
else
{
prompt("<h1>if you want a meal on the flight press OK <\h1>");
document.write("<h1>your ticket is 400<h1>,");
}
}

Hi, been using javascript for a while now, and wondering how else this could be coded to be more effective and to 'work'.

The idea is that the user gets prompted, 'what is your birthday?' the user would put in 0121 or whatever their birthday is,
as 01 = month and 21 = day. I'm sure you'll understand what I mean....

Code:

<DOCTYPE html>
<html>
<script>
var userAnswer = prompt("When is your birthday? Example: Jan 21st = 0121")
</script>
<head>
</head>
<body>
<script>
if(userAnswer >= 0101){
alert('You are Capricorn!') };
if(userAnswer >= 0120){
alert('You are Aquarius!') };
if(userAnswer >= 0219){
alert('You are Pisces!') };
if(userAnswer >= 0321){
alert('You are Aries') };
if(userAnswer >= 0420){
alert('You are Taurus') };
if(userAnswer >= 0521){
alert('You are Gemini!') };
if(userAnswer >= 0621){
alert('You are Cancer!') };
if(userAnswer >= 0723){
alert('You are Leo!') };
if(userAnswer >= 0823){
alert('You are Virgo!') };
if(userAnswer >= 0923){
alert('You are Libra!') };
if(userAnswer >= 1023){
alert('You are Scorpio!') };
if(userAnswer >= 1122){
alert('You are Sagittarius!') };
if(userAnswer >= 1222){
alert('You are Capricorn!') }; 
</script>
</body>
</html>

Reply With Quote




01-18-2015, 10:01 AM #2
Mitchellwood101 View Profile View Forum Posts

New to the CF scene Join Date Nov 2014 Posts 7 Thanks 0 Thanked 0 Times in 0 Posts

And then it will alert the user of their star sign******





Reply With Quote

01-18-2015, 05:47 PM #3
Philip M View Profile View Forum Posts

Supreme Master coder! Join Date Jun 2002 Location London, England Posts 18,371 Thanks 204 Thanked 2,573 Times in 2,551 Posts

If you have been using Javascript for a while now, you should be aware that prompts and alerts are long obsolete, and I would advise that you should not be wasting your time learning this stuff. Use DOM methods to obtain input from and display messages to your users. Be aware that a value starting with 0 is a string, not a number, and so must be in quotes - that is why your code does not work.
In any case you need rigorous checking to ensure that the birthday entered in this form is valid, e.g. not 56th Febtember or "Mickey Mouse". You would do best to obtain the input from select lists for month and day, expressed as numbers. A tiny bit more sophistication would allow the user to select a month by name, where the text of the option is the month name, and the value of that option the month number.

Code:

<select id = "monthlist" onchange = "getSignFunction()">
<option value = "0"> Choose your birth month</option>
<option value = 1>January</option>
<option value =  2>February</option>
... and so on
</select>

Then you want something like this:-

Code:

function getSignFunction()"{
var sign = "";
var month = document.getElementById("monthlist").value;
var date = document.getElementById("datelist").value;
if (month == 0) {return}  //  invalid choice

if (month == 1 && date >=20 || month == 2 && date <=18) {sign = "Aquarius";}
if (month == 2 && date >=19 || month == 3 && date <=20) {sign = "Pisces";}
if (month == 3 && date >=21 || month == 4 && date <=19) {sign = "Aries";}
if (month == 4 && date >=20 || month == 5 && date <=20) {sign = "Taurus";}
if (month == 5 && date >=21 || month == 6 && date <=21) {sign = "Gemini";}
if (month == 6 && date >=22 || month == 7 && date <=22) {sign = "Cancer";}
if (month == 7 && date >=23 || month == 8 && date <=22) {sign = "Leo";}
if (month == 8 && date >=23 || month == 9 && date <=22) {sign = "Virgo";}
if (month == 9 && date >=23 || month == 10 && date <=22) {sign = "Libra";}
if (month == 10 && date >=23 || month == 11 && date <=21) {sign = "Scorpio";}
if (month == 11 && date >=22 || month == 12 && date <=21) {sign = "Sagittarius";}
if (month == 12 && date >=22 || month == 1 && date <=19) {sign = "Capricorn";}

alert (sign);  // alert used here just for testing purposes.  Crate as <span> and then use document.getElementById("spanid").innerHTML = sign to display the result to the user.

}

All advice is supplied packaged by intellectual weight, and not by volume. Contents may settle slightly in transit.

Here is the code, in full:


<htmll">

<head><title></title>

<script type="text/javascript">
<!-- HIDE FROM INCOMPATIBLE BROWSERS
function checkGrade(grade, value) {

switch (grade) {

case "A":
if(grade == '90')
window.alert("Your grade is excellent.");
break;
case "B":
window.alert("Your grade is good.");
break;
case "C":
window.alert("Your grade is fair.");
break;
case "D":
window.alert("You are barely passing.");
break;
case "F":
window.alert("You failed.");
break;
}
}
// STOP HIDING FROM INCOMPATIBLE BROWSERS -->
</script>

</head>
<body>

<form name="gradeForm"action="Your Grades">
<input type="text" name="grade" />
<input type="button" value="Check Grade" onclick="checkGrade(document.gradeForm.grade.value);" />

</form>

</body>

</html>

What's throwing me off is the "A" in the case. As well, "(document.gradeForm.grade.value)"

I need to make it a IF statement, but not sure how to call that function.

Give me an example for switch statements.

I had to create one that tests value of parameter using a text string,


I came up with this value Code:

 function createBar(partyType,percent){
switch(partyType) {
case D: <td class='dem'></td>
break;

Should I use " or ' between the dem?

I am using break statements.
do all breaks should end with string or do i need the bracket?

I hope this does not seem like i getting people to do homework for me again,, just want to verify if i did this area correctly?

Okay, so I got this code originally from another site. (A few months ago) I don't remember the site, and I was wondering if anyone could help me get it back how it was.

Originally I believe the if statements were like this:
Code:

if (hr == 6, 7, 8)
document.write("")

I want the same code to appear when any of these numbers appear.

Here is my current code:

Code:

document.write("<center><font size=+1>")
day = new Date()
hr = day.getMinutes()
document.write("")
if (hr == 0)
document.write("<script type='text/javascript' src='http://www.1800banners.com/adserver/adserver.js'></script> <script type='text/javascript'> var varsarray=[]; varsarray[0]='24047'; if(!token) {var token='0'} else {var token=token+1;} var loadtype='0'; var adtype='0'; var background_color='#9C9C9C'; var border_color='#9C9C9C'; var text_color='#060606'; var text_rollcolor='#000000'; var domain_color='#000000'; varsarray['interhours']=''; varsarray['interpages']=''; adserver_initialize(token,loadtype,adtype,background_color,border_color,text_color,text_rollcolor,domain_color,varsarray); </script>")
if (hr == 1)
document.write("")
if (hr == 2)
document.write("")
if (hr == 3)
document.write("")
if (hr == 4)
document.write("")
if (hr == 5)
document.write(")
if (hr == 6)
document.write("")
if (hr == 7)
document.write("")
if (hr == 8)
document.write("")
if (hr == 9)
document.write("")
if (hr == 10)
document.write("")
if (hr == 11)
document.write("")
if (hr == 12)
document.write("")
if (hr == 13)
document.write("")
if (hr == 14)
document.write("")
if (hr == 15)
document.write("")
if (hr == 16)
document.write("")
if (hr == 17)
document.write("")
if (hr == 18)
document.write("")
if (hr == 19)
document.write("")
if (hr == 20)
document.write("")
if (hr == 21)
document.write("")
if (hr == 22)
document.write("")
if (hr == 23)
document.write("")
if (hr == 24)
document.write("")
if (hr == 25)
document.write("")
if (hr == 26)
document.write("")
if (hr == 27)
document.write("<script type='text/javascript'> clicksor_enable_adhere = false; clicksor_default_url = ''; clicksor_banner_border = '#99CC33'; clicksor_banner_ad_bg = '#FFFFFF'; clicksor_banner_link_color = '#000000'; clicksor_banner_text_color = '#666666'; clicksor_banner_image_banner = true; clicksor_banner_text_banner = true; clicksor_layer_border_color = ''; clicksor_layer_ad_bg = ''; clicksor_layer_ad_link_color = ''; clicksor_layer_ad_text_color = ''; clicksor_text_link_bg = ''; clicksor_text_link_color = ''; clicksor_enable_text_link = false; </script> <script type='text/javascript' src='http://ads.clicksor.com/showAd.php?nid=1&amp;pid=154466&amp;adtype=2&amp;sid=233215&amp;zone=13447'></script> <noscript><a href='http://www.yesadvertising.com'>affiliate marketing</a></noscript>")
if (hr == 28)
document.write("<script type='text/javascript'> clicksor_enable_adhere = false; clicksor_default_url = ''; clicksor_banner_border = '#99CC33'; clicksor_banner_ad_bg = '#FFFFFF'; clicksor_banner_link_color = '#000000'; clicksor_banner_text_color = '#666666'; clicksor_banner_image_banner = true; clicksor_banner_text_banner = true; clicksor_layer_border_color = ''; clicksor_layer_ad_bg = ''; clicksor_layer_ad_link_color = ''; clicksor_layer_ad_text_color = ''; clicksor_text_link_bg = ''; clicksor_text_link_color = ''; clicksor_enable_text_link = false; </script> <script type='text/javascript' src='http://ads.clicksor.com/showAd.php?nid=1&amp;pid=154466&amp;adtype=2&amp;sid=233215&amp;zone=13447'></script> <noscript><a href='http://www.yesadvertising.com'>affiliate marketing</a></noscript>")
if (hr == 29)
document.write("")
if (hr == 30)
document.write("<script type='text/javascript' src='http://www.1800banners.com/adserver/adserver.js'></script> <script type='text/javascript'> var varsarray=[]; varsarray[0]='24047'; if(!token) {var token='0'} else {var token=token+1;} var loadtype='0'; var adtype='0'; var background_color='#9C9C9C'; var border_color='#9C9C9C'; var text_color='#060606'; var text_rollcolor='#000000'; var domain_color='#000000'; varsarray['interhours']=''; varsarray['interpages']=''; adserver_initialize(token,loadtype,adtype,background_color,border_color,text_color,text_rollcolor,domain_color,varsarray); </script>")
if (hr == 31)
document.write("")
if (hr == 32)
document.write("<script type='text/javascript'> clicksor_enable_adhere = false; clicksor_default_url = ''; clicksor_banner_border = '#99CC33'; clicksor_banner_ad_bg = '#FFFFFF'; clicksor_banner_link_color = '#000000'; clicksor_banner_text_color = '#666666'; clicksor_banner_image_banner = true; clicksor_banner_text_banner = true; clicksor_layer_border_color = ''; clicksor_layer_ad_bg = ''; clicksor_layer_ad_link_color = ''; clicksor_layer_ad_text_color = ''; clicksor_text_link_bg = ''; clicksor_text_link_color = ''; clicksor_enable_text_link = false; </script> <script type='text/javascript' src='http://ads.clicksor.com/showAd.php?nid=1&amp;pid=154466&amp;adtype=2&amp;sid=233215&amp;zone=13447'></script> <noscript><a href='http://www.yesadvertising.com'>affiliate marketing</a></noscript>")
if (hr == 33)
document.write("<script type='text/javascript'> clicksor_enable_adhere = false; clicksor_default_url = ''; clicksor_banner_border = '#99CC33'; clicksor_banner_ad_bg = '#FFFFFF'; clicksor_banner_link_color = '#000000'; clicksor_banner_text_color = '#666666'; clicksor_banner_image_banner = true; clicksor_banner_text_banner = true; clicksor_layer_border_color = ''; clicksor_layer_ad_bg = ''; clicksor_layer_ad_link_color = ''; clicksor_layer_ad_text_color = ''; clicksor_text_link_bg = ''; clicksor_text_link_color = ''; clicksor_enable_text_link = false; </script> <script type='text/javascript' src='http://ads.clicksor.com/showAd.php?nid=1&amp;pid=154466&amp;adtype=2&amp;sid=233215&amp;zone=13447'></script> <noscript><a href='http://www.yesadvertising.com'>affiliate marketing</a></noscript>")
if (hr == 34)
document.write("")
if (hr == 35)
document.write("<script type='text/javascript'> clicksor_enable_adhere = false; clicksor_default_url = ''; clicksor_banner_border = '#99CC33'; clicksor_banner_ad_bg = '#FFFFFF'; clicksor_banner_link_color = '#000000'; clicksor_banner_text_color = '#666666'; clicksor_banner_image_banner = true; clicksor_banner_text_banner = true; clicksor_layer_border_color = ''; clicksor_layer_ad_bg = ''; clicksor_layer_ad_link_color = ''; clicksor_layer_ad_text_color = ''; clicksor_text_link_bg = ''; clicksor_text_link_color = ''; clicksor_enable_text_link = false; </script> <script type='text/javascript' src='http://ads.clicksor.com/showAd.php?nid=1&amp;pid=154466&amp;adtype=7&amp;sid=233215&amp;zone=13447'></script> <noscript><a href='http://www.yesadvertising.com'>affiliate marketing</a></noscript>")
if (hr == 36)
document.write("")
if (hr == 37)
document.write("<script type='text/javascript'> clicksor_enable_adhere = false; clicksor_default_url = ''; clicksor_banner_border = '#99CC33'; clicksor_banner_ad_bg = '#FFFFFF'; clicksor_banner_link_color = '#000000'; clicksor_banner_text_color = '#666666'; clicksor_banner_image_banner = true; clicksor_banner_text_banner = true; clicksor_layer_border_color = ''; clicksor_layer_ad_bg = ''; clicksor_layer_ad_link_color = ''; clicksor_layer_ad_text_color = ''; clicksor_text_link_bg = ''; clicksor_text_link_color = ''; clicksor_enable_text_link = false; </script> <script type='text/javascript' src='http://ads.clicksor.com/showAd.php?nid=1&amp;pid=154466&amp;adtype=2&amp;sid=233215&amp;zone=13447'></script> <noscript><a href='http://www.yesadvertising.com'>affiliate marketing</a></noscript>")
if (hr == 38)
document.write("<script type='text/javascript'> clicksor_enable_adhere = false; clicksor_default_url = ''; clicksor_banner_border = '#99CC33'; clicksor_banner_ad_bg = '#FFFFFF'; clicksor_banner_link_color = '#000000'; clicksor_banner_text_color = '#666666'; clicksor_banner_image_banner = true; clicksor_banner_text_banner = true; clicksor_layer_border_color = ''; clicksor_layer_ad_bg = ''; clicksor_layer_ad_link_color = ''; clicksor_layer_ad_text_color = ''; clicksor_text_link_bg = ''; clicksor_text_link_color = ''; clicksor_enable_text_link = false; </script> <script type='text/javascript' src='http://ads.clicksor.com/showAd.php?nid=1&amp;pid=154466&amp;adtype=2&amp;sid=233215&amp;zone=13447'></script> <noscript><a href='http://www.yesadvertising.com'>affiliate marketing</a></noscript>")
if (hr == 39)
document.write("")
if (hr == 40)
document.write("<script type='text/javascript' src='http://adhitzads.com/138517'></script>")
if (hr == 41)
document.write("")
if (hr == 42)
document.write("<script type='text/javascript'> clicksor_enable_adhere = false; clicksor_default_url = ''; clicksor_banner_border = '#99CC33'; clicksor_banner_ad_bg = '#FFFFFF'; clicksor_banner_link_color = '#000000'; clicksor_banner_text_color = '#666666'; clicksor_banner_image_banner = true; clicksor_banner_text_banner = true; clicksor_layer_border_color = ''; clicksor_layer_ad_bg = ''; clicksor_layer_ad_link_color = ''; clicksor_layer_ad_text_color = ''; clicksor_text_link_bg = ''; clicksor_text_link_color = ''; clicksor_enable_text_link = false; </script> <script type='text/javascript' src='http://ads.clicksor.com/showAd.php?nid=1&amp;pid=154466&amp;adtype=2&amp;sid=233215&amp;zone=13447'></script> <noscript><a href='http://www.yesadvertising.com'>affiliate marketing</a></noscript>")
if (hr == 43)
document.write("<script type='text/javascript'> clicksor_enable_adhere = false; clicksor_default_url = ''; clicksor_banner_border = '#99CC33'; clicksor_banner_ad_bg = '#FFFFFF'; clicksor_banner_link_color = '#000000'; clicksor_banner_text_color = '#666666'; clicksor_banner_image_banner = true; clicksor_banner_text_banner = true; clicksor_layer_border_color = ''; clicksor_layer_ad_bg = ''; clicksor_layer_ad_link_color = ''; clicksor_layer_ad_text_color = ''; clicksor_text_link_bg = ''; clicksor_text_link_color = ''; clicksor_enable_text_link = false; </script> <script type='text/javascript' src='http://ads.clicksor.com/showAd.php?nid=1&amp;pid=154466&amp;adtype=2&amp;sid=233215&amp;zone=13447'></script> <noscript><a href='http://www.yesadvertising.com'>affiliate marketing</a></noscript>")
if (hr == 44)
document.write("")
if (hr == 45)
document.write("<script type='text/javascript' src='http://adhitzads.com/136216'></script>")
if (hr == 46)
document.write("")
if (hr == 47)
document.write("<script type='text/javascript'> clicksor_enable_adhere = false; clicksor_default_url = ''; clicksor_banner_border = '#99CC33'; clicksor_banner_ad_bg = '#FFFFFF'; clicksor_banner_link_color = '#000000'; clicksor_banner_text_color = '#666666'; clicksor_banner_image_banner = true; clicksor_banner_text_banner = true; clicksor_layer_border_color = ''; clicksor_layer_ad_bg = ''; clicksor_layer_ad_link_color = ''; clicksor_layer_ad_text_color = ''; clicksor_text_link_bg = ''; clicksor_text_link_color = ''; clicksor_enable_text_link = false; </script> <script type='text/javascript' src='http://ads.clicksor.com/showAd.php?nid=1&amp;pid=154466&amp;adtype=2&amp;sid=233215&amp;zone=13447'></script> <noscript><a href='http://www.yesadvertising.com'>affiliate marketing</a></noscript>")
if (hr == 48)
document.write("<script type='text/javascript'> clicksor_enable_adhere = false; clicksor_default_url = ''; clicksor_banner_border = '#99CC33'; clicksor_banner_ad_bg = '#FFFFFF'; clicksor_banner_link_color = '#000000'; clicksor_banner_text_color = '#666666'; clicksor_banner_image_banner = true; clicksor_banner_text_banner = true; clicksor_layer_border_color = ''; clicksor_layer_ad_bg = ''; clicksor_layer_ad_link_color = ''; clicksor_layer_ad_text_color = ''; clicksor_text_link_bg = ''; clicksor_text_link_color = ''; clicksor_enable_text_link = false; </script> <script type='text/javascript' src='http://ads.clicksor.com/showAd.php?nid=1&amp;pid=154466&amp;adtype=2&amp;sid=233215&amp;zone=13447'></script> <noscript><a href='http://www.yesadvertising.com'>affiliate marketing</a></noscript>")
if (hr == 49)
document.write("")
if (hr == 50)
document.write("<SCRIPT LANGUAGE='JavaScript1.1' SRC='http://bdv.bidvertiser.com/BidVertiser.dbm?pid=244169&bid=856524' type='text/javascript'></SCRIPT>")
if (hr == 51)
document.write("")
if (hr == 52)
document.write("<script type='text/javascript'> clicksor_enable_adhere = false; clicksor_default_url = ''; clicksor_banner_border = '#99CC33'; clicksor_banner_ad_bg = '#FFFFFF'; clicksor_banner_link_color = '#000000'; clicksor_banner_text_color = '#666666'; clicksor_banner_image_banner = true; clicksor_banner_text_banner = true; clicksor_layer_border_color = ''; clicksor_layer_ad_bg = ''; clicksor_layer_ad_link_color = ''; clicksor_layer_ad_text_color = ''; clicksor_text_link_bg = ''; clicksor_text_link_color = ''; clicksor_enable_text_link = false; </script> <script type='text/javascript' src='http://ads.clicksor.com/showAd.php?nid=1&amp;pid=154466&amp;adtype=2&amp;sid=233215&amp;zone=13447'></script> <noscript><a href='http://www.yesadvertising.com'>affiliate marketing</a></noscript>")
if (hr == 53)
document.write("<script type='text/javascript'> clicksor_enable_adhere = false; clicksor_default_url = ''; clicksor_banner_border = '#99CC33'; clicksor_banner_ad_bg = '#FFFFFF'; clicksor_banner_link_color = '#000000'; clicksor_banner_text_color = '#666666'; clicksor_banner_image_banner = true; clicksor_banner_text_banner = true; clicksor_layer_border_color = ''; clicksor_layer_ad_bg = ''; clicksor_layer_ad_link_color = ''; clicksor_layer_ad_text_color = ''; clicksor_text_link_bg = ''; clicksor_text_link_color = ''; clicksor_enable_text_link = false; </script> <script type='text/javascript' src='http://ads.clicksor.com/showAd.php?nid=1&amp;pid=154466&amp;adtype=2&amp;sid=233215&amp;zone=13447'></script> <noscript><a href='http://www.yesadvertising.com'>affiliate marketing</a></noscript>")
if (hr == 54)
document.write("")
if (hr == 55)
document.write("<form action='https://www.paypal.com/cgi-bin/webscr' method='post'><input type='hidden' name='cmd' value='_s-xclick'> <input type='hidden' name='hosted_button_id' value='8220588'> <input type='image' src='https://www.paypal.com/en_US/i/btn/btn_donateCC_LG.gif' border='0' name='submit' alt='PayPal - The safer, easier way to pay online!'> <img alt='' border='0' src='https://www.paypal.com/en_US/i/scr/pixel.gif' width='1' height='1'> </form>")
if (hr == 56)
document.write("")
if (hr == 57)
document.write("")
if (hr == 58)
document.write("")
if (hr == 59)
document.write("")

document.write("</font></center>")

Any help? Or do you need more information?

can someone see a way to minimize this down to one statement that makes it infinate?

if(x == i) || (x*1 == i) || (x*2 == i) || (x*3 == i) || (x*4 == i) || .............

x is a var user input. it is numeric.
i is the counter placed above the function.

I'm a little new to Javascript so I apologise if this is an easily resolved problem.

This is my Javascript code:
Code:

function display(){
		if(document.getElementById('one').style.display="none"){
		document.getElementById('one').style.display="block";
		}
		else if(document.getElementById('one').style.display="block"){
		document.getElementById('two').style.display="block";
		}
		else{
		document.getElementById('three').style.display="block";
		}
	}

This is my HTML:
Code:

<html>
....
<style>
#one,#two,#three {
display:none;
}
</style>
....
<ul>
		<li id="one">1</li>
		<li id="two">2</li>
		<li id="three">3</li>
		<li id="click" onclick="display()">+</li>
	</ul>

The idea is when I click the list item 'click' the display() function will be called and make list item 'one' appear. If list item 'one' is already visible however, list item 'two' will appear, etc. When I click it, list item one appears fine as it should, but if I click it again list item two does not appear, and I can't work out why. I could really do with some help, thank you in advance!

I have an order form which is validated by javascript...
To validate the fields I have used multiple if statements!
But the problem is it works well when checking the fields from top to bottom; but if the user leaves all the upper fields blanks and only inputs the last field, the script fails to validate all the upper fields..
Please help me improve my javascript...
The javascript code is

validatecalculate.js Code:

function setFocus()
  {
  document.getElementById("fname").focus()
  }


function IsNumeric(strString)
   //  check for valid numeric strings	
   {
   var strValidChars = "0123456789 ";//allowed numeric and space
   var strChar;
   var blnResult = true;

   if (strString.length == 0) return false;

   //  test strString consists of valid characters listed above
   for (i = 0; i < strString.length && blnResult == true; i++)
      {
      strChar = strString.charAt(i);
      if (strValidChars.indexOf(strChar) == -1)
         {
         blnResult = false;
         }
      }
   return blnResult;
   }

function isNaN(strelem){
	var numericExpression = /^[0-9]+$/;
	if(strelem.match(numericExpression)){
		return true;
	}else{
		return false;
	}
}

function validate_form() {
    missinginfo = ""; //alert("show hidden value " + (document.myform.totalsub.value));

  	if (document.getElementById("fname").value == "") {
        missinginfo += "\n     -  First Name Not Entered";
    	}

  	if (document.getElementById("lname").value == "") {
        missinginfo += "\n     -  Last Name Not Entered";
    	}

	if ((document.getElementById("email").value.indexOf("@") < 1) ||
        (document.getElementById("email").value.lastIndexOf(".") < 2))
	{
		if (document.getElementById("email").value == "")
		{
        	missinginfo += "\n     -  Email Address Not entered";
		}
		else{
		missinginfo += "\n     -  Invalid email Address.\n     >>email should have the format x@abc.yy";   
      
		}
	}

  	if (IsNumeric(document.getElementById("phone").value) == false)
	{
		if (document.getElementById("phone").value == "") {
 		missinginfo += "\n     -  Phone Number Not entered";
    		}
		else{
 		missinginfo += "\n     -  Invalid Phone Number.\n     >> Please enter numeric characters (space allowed)";}
	}

	var reg = /^\d{1,}\S\d{1,}\s[a-zA-Z]+\s[a-zA-Z]+$/;
	if (reg.test(document.getElementById("saddress").value)== false)
	{
		if (document.getElementById("saddress").value== "") {
        	missinginfo += "\n     -  Street Address Not entered";
    		}
		else{
  		missinginfo += "\n     -  Invalid Street Address.\n     >> Please use format 0/00 Tiger Street"; }
	} 
 
 	if (document.getElementById("suburb").value == "") {
        missinginfo += "\n     -  Suburb Not Entered";
    	}  
 
	var pval = /^\d{4}$/;   
	if (pval.test(document.getElementById("pcode").value)== false){ 
		if (document.getElementById("pcode").value== "") {
        	missinginfo += "\n     -  Postcode Not Entered";
    		}
		else{
		missinginfo += "\n     -  Invalid Post Code.\n     >> Please enter four numbers of your postcode"; }
	} 
 
   	if (document.getElementById("ccname").value == "") {
        missinginfo += "\n     -  Name on Credit Card not entered";
    	}  

	cval = /^\d{16}$/;
	if (cval.test(document.getElementById("ccnum").value)== false){ 
		if (document.getElementById("ccnum").value == "") {
        	missinginfo += "\n     -  Credit Card Number Not entered";
    		}  
		else {
		missinginfo += "\n     -  Invalid Credit Card Number.\n     >> Please enter 16 digits on your credit card. Ignore the dashes and spaces."; }
	}

	ymval=/^\d{2}$/;
	if (ymval.test(document.getElementById("edatemonth").value)== false) 
	{ 
		if (document.getElementById("edatemonth").value == "") {
        	missinginfo += "\n     -  Credit Card Expiry Month Not Entered";
		}
		else{
		missinginfo += "\n     -  Invalid Expiry Month.\n     >> Please enter 01 for Jan, 02 for Feb.... 11 for Nov & 12 for Dec."; }
	}

	
	if (ymval.test(document.getElementById("edateyear").value)== false){
		if (document.getElementById("edateyear").value == "") {
        	missinginfo += "\n     -  Credit Card Expiry Year Not Entered";
    		} 
		else{
		missinginfo += "\n     -  Invalid Expiry Year.\n     >> Please enter last two digits only. e.g.: 12 for 2012"; }
	} 

	cvvval=/^\d{3}$/;
	if (cvvval.test(document.getElementById("cvv").value)== false){
		if (document.getElementById("cvv").value == "") {
        	missinginfo += "\n     -  CVV Not Entered";
    		}
		else{
		missinginfo += "\n     -  Invalid CVV.\n     >> Please enter the 3 digits only"; }
	}

	else return true;    
  	if (missinginfo != "") {
        missinginfo ="______________________________________________\n" + "Form not Submitted:\n" +
        missinginfo + "\n______________________________________________" + "\nPlease check and submit again!";
        alert(missinginfo);
    	return false;
    	}
    	else return true;
}


function CalculateTotal() {
var bolt = parseInt(document.getElementById('qnty1').value);
var nut = parseInt(document.getElementById('qnty2').value);
var washer = parseInt(document.getElementById('qnty3').value);
document.getElementById("tot1").value = (bolt * 2.15).toFixed(2);
document.getElementById("tot2").value = (nut * 0.45).toFixed(2);
document.getElementById("tot3").value = (washer * 0.30).toFixed(2);
document.getElementById("totqnty").value = ((bolt) + (nut) + (washer));
document.getElementById("totamount").value = ((bolt * 2.15) + (nut * 0.45) + (washer * 0.30)).toFixed(2);
}

The html code with the form is Code:

<?xml version = "1.0" encoding = "utf-8" ?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"
"http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<!-- order.html
     The order page for 'Nuts 'R' Us'
     -->
<html xmlns = "http://www.w3.org/1999/xhtml">
<head>
<link rel="stylesheet" type="text/css" href="task2.css" />
<script type="text/javascript" src="validatecalculate.js"> </script>
 <title>Nuts 'R' Us - Order</title>
  </head>
  <body onload="return setFocus();">
<table class="table">
    <tr><td><h1> Nuts 'R' Us </h1></td></tr>
    <tr><td class="text2"> The best store in town.</td></tr>
    <tr><td>
	<table class="table2">
	<tr><td class="td1"><a href = "main.html">  Home  </a></td><td></td>
    <td class="td1"> <a href = "order.html">Order </a></td><td></td>
    <td class="td1"> <a href = "http://csusap.csu.edu.au/cgi-pub/rgauta01/process.cgi">Shopping-Cart </a></td></tr>
	</table></td>
     </tr>
     <tr><td>
	<hr />
	</td></tr>
	<tr><td><form action = "http://csusap.csu.edu.au/cgi-pub/rgauta01/process.cgi" method="post" id="myform">
  <table class="table">
<tr><td>First Name:</td>
	<td><input type = "text"  name ="fname"  id="fname" /></td></tr>
<tr><td>Last Name:</td>
	<td><input type = "text"  name ="lname" id="lname" /></td></tr>
<tr><td>email:</td>
	<td><input type = "text"  name ="email" id="email" /></td></tr>
<tr><td>Phone:</td>
	<td><input type = "text"  name ="phone" id="phone" /></td></tr>
<tr><td>Street Address:</td>
	<td><input type = "text"  name ="saddress" size ="50" id="saddress" /></td></tr>
<tr><td>Suburb:</td>
	<td><input type = "text"  name ="suburb" size ="30" id="suburb" /></td></tr>
<tr><td>Postcode:</td>
	<td><input type = "text"  name ="pcode" size ="30" id="pcode" /></td></tr><tr><td colspan="2" class="text3">Payment Details</td></tr> 
<tr><td colspan="2">Type of card: &nbsp;&nbsp;
<input type="radio" name="cardtype" value="visa" id="visa" checked="checked" />Visa
<input type="radio" name="cardtype" value="mastercard"  id="mastercard" />Mastercard
<input type="radio" name="cardtype" value="amex" id="amex" />Amex
<input type="radio" name="cardtype" value="diners"  id="diners" />Diners</td></tr> 
<tr><td>Name on Credit Card:</td>
<td><input type = "text"  name ="ccname" size ="50" id="ccname" /></td></tr>
<tr><td>Credit Card Number</td>
<td><input type = "text"  name ="ccnum" maxlength ="16" size ="50" id="ccnum" /></td></tr>
<tr><td colspan="2" class="text3">Expiry Date</td></tr>
<tr><td>Month(MM):</td>
<td><input type = "text"  name ="edatemonth" size ="5" maxlength ="2" id="edatemonth" /></td></tr> 
<tr><td>Year(YY):</td>
<td><input type = "text"  name ="edateyear" size ="5" maxlength ="2" id="edateyear" /></td></tr>
<tr><td>CVV:</td>
<td><input type = "text"  name ="cvv" size ="5" maxlength ="3" id="cvv" /></td></tr>
<tr><td colspan="2" class="text3">Order Details</td></tr>
<tr><td colspan="2"><table class="table3">
<tr><th>Item</th>
<th>Product Code</th>
<th>Diameter</th>
<th>Length</th>
<th>Colour</th>
<th>Unit</th> 
<th>Quantity</th>
<th>Price($)</th></tr>
<tr><td>Bolt</td><td>B113</td>
<td>9</td><td>50</td>
<td>Black</td>
<td>2.15</td>
<td class="td1"><input type ="text" name ="qnty1" size="3" onchange="CalculateTotal()" id="qnty1" value="0" /></td>
<td class="td1"><input type ="text" name ="tot1" size ="3" id="tot1" value="0.00" readonly="readonly" /></td></tr>
<tr><td>Nut</td>
<td>N234</td>
<td>5</td>
<td>N/A</td>
<td>Silver</td>
<td>0.45</td>
<td class="td1"><input type ="text" name ="qnty2" size ="3" onchange="CalculateTotal()" id="qnty2" value="0" /></td>
<td class="td1"><input type ="text" name ="tot2"  size ="3" id="tot2" value="0.00" readonly="readonly" /></td></tr>
<tr><td>Washer</td>
<td>W359</td>
<td>8</td>
<td>N/A</td>
<td>Silver</td>
<td>0.30</td>
<td class="td1"><input type ="text" name="qnty3" size ="3" onchange="CalculateTotal()" id="qnty3" value="0" /></td>
<td class="td1"><input type ="text" name ="tot3" size ="3" id="tot3" value="0.00" readonly="readonly"  /></td></tr>

 <tr><td colspan = "6" class="td2">Total:
</td><td class="td1"><input type ="text" name ="totqnty" size ="3" id="totqnty" value="0" readonly="readonly" /></td>
<td class="td1"><input type ="text" name ="totamount" size ="3" id="totamount" value="0.00" readonly="readonly" /></td></tr>
</table></td></tr>

<tr><td colspan="2" align="center"><input type = "reset"  value = "Reset" />&nbsp;
<input type = "submit"  value = "Submit" name="submit" onclick="return validate_form();"/></td></tr></table>

  </form>
	
	</td>
</tr>
<tr><td colspan="2">
<a href="http://validator.w3.org/check?uri=referer"><img
        src="http://www.w3.org/Icons/valid-xhtml10"
        alt="Valid XHTML 1.0 Strict" height="31" width="88" /></a>
</td></tr>
<tr><td class="td1">
<hr />
&copy; Nuts 'R' Us - 2011. All Rights Reserved.
	
	</td>
</tr>

   
  </table></body>
</html>

Currently I have the following javascript switch statement ...

Code:

		switch(firstchar){
		   case "0": case "1": case "2": case "3": case "4": case "5": case "6": case "7": case "8": case "9":
		   orderlist[0] += addthis; break;
		   case "a": case "A": orderlist[1] += addthis; break;
		   case "b": case "B": orderlist[2] += addthis; break;
		   case "c": case "C": orderlist[3] += addthis; break;
		   case "d": case "D": orderlist[4] += addthis; break;
		   case "e": case "E": orderlist[5] += addthis; break;
		   case "f": case "F": orderlist[6] += addthis; break;
		   case "g": case "G": orderlist[7] += addthis; break;
		   case "h": case "H": orderlist[8] += addthis; break;
		   case "i": case "I": orderlist[9] += addthis; break;
		   case "j": case "J": orderlist[10] += addthis; break;
		   case "k": case "K": orderlist[11] += addthis; break;
		   case "l": case "L": orderlist[12] += addthis; break;
		   case "m": case "M": orderlist[13] += addthis; break;
		   case "n": case "N": orderlist[14] += addthis; break;
		   case "o": case "O": orderlist[15] += addthis; break;
		   case "p": case "P": orderlist[16] += addthis; break;
		   case "q": case "Q": orderlist[17] += addthis; break;
		   case "r": case "R": orderlist[18] += addthis; break;
		   case "s": case "S": orderlist[19] += addthis; break;
		   case "t": case "T": orderlist[20] += addthis; break;
		   case "u": case "U": orderlist[21] += addthis; break;
		   case "v": case "V": orderlist[22] += addthis; break;
		   case "w": case "W": orderlist[23] += addthis; break;
		   case "x": case "X": orderlist[24] += addthis; break;
		   case "y": case "Y": orderlist[25] += addthis; break;
		   case "z": case "Z": orderlist[26] += addthis; break;
		   default: alert('Illegal Entry'); break;
		}

... that I wish to optimize so that I do not need to write out so many lines of code.

Also, I wish to take into the following as consideration ...
Code:

		   case "aa": case "Aa": case "aA": case "AA": orderlist[1] += addthis; break;

... for each two-character combination. (aa to zz)

Appreciate any help.

I need to create javascript that will create the ringup line (see code) based on accomodation selections. If they chose Wednesday as arrival and depart Thursday they will get the breakdown_wed_supplement value. If they arrive Wednesday and depart later than Thursday they will get charged for Wed and Thursday and finally if they arrive Thursday, they will get charge for Thursday. The Friday - Sunday values are working, but I can't get the Wed/Thurs to work. They were written by someone else and were working, but I needed to change it for the Thursday departure. I am really new to javascript and haven't got a clue where I went wrong.

Code:

		if (getControlValue(form.accommodation)==="Yes") {
			if (getControlValue(form.occupancy)==="Single") {
				total.ringup(conferencePrices.lodgingFeeConference,'Conference lodging for single person, Friday through Sunday','breakdown_lodging');
			} else if (getControlValue(form.occupancy)==="Double") {
				total.ringup(conferencePrices.lodgingFeeConferenceRoommate,'Conference lodging with roommate, Friday through Sunday','breakdown_lodging');
			}	
			if (form.arrivedate.selectedIndex===1 && form.departdate.selectedIndex===2) {
				if (getControlValue(form.roommate)==="Assign" || form.rm_arrive.selectedIndex===1 && form.rm_depart.selectedIndex===2) {
					total.ringup(conferencePrices.lodgingFeeNightlyRoommate,'Wednesday pre-conference lodging, with roommate','breakdown_wed_supplement');
				} else {
					total.ringup(conferencePrices.lodgingFeeNightly,'Wednesday pre-conference lodging, single person','breakdown_wed_supplement');
				}
			}
	
			if (form.arrivedate.selectedIndex===1 && form.departdate.selectedIndex===>2) {
				if (getControlValue(form.roommate)==="Assign" || form.rm_arrive.selectedIndex===1 && form.rm_depart.selectedIndex===>2) {
					total.ringup(conferencePrices.lodgingFeeNightlyRoommate,'Thursday pre-conference lodging, with roommate','breakdown_thurs_supplement');
					total.ringup(conferencePrices.lodgingFeeNightlyRoommate,'Wednesday pre-conference lodging, with roommate','breakdown_wed_supplement');
				} else {
					total.ringup(conferencePrices.lodgingFeeNightly,'Thursday pre-conference lodging, single person','breakdown_thurs_supplement');
					total.ringup(conferencePrices.lodgingFeeNightly,'Wednesday pre-conference lodging, single person','breakdown_wed_supplement');
				}
			}
			
			
			if (form.arrivedate.selectedIndex===2) {
				if (getControlValue(form.roommate)==="Assign" form.rm_arrive.selectedIndex===2) {
					total.ringup(conferencePrices.lodgingFeeNightlyRoommate,'Thursday pre-conference lodging, with roommate','breakdown_thurs_supplement');
				} else {
					total.ringup(conferencePrices.lodgingFeeNightly,'Thursday pre-conference lodging, single person','breakdown_thurs_supplement');
	}
	}
	

	}

Hello! I'd like to ask if it is possible to use conditional operators

var = condition ? var1 : var 2

to do the same job as the if-else statements I wrote in red below:

Thank you!

Code:

<p id="text">change text colour</p>
<br />
<a onclick="allsorting();">sort in both ascending & descending orders</a>
<div></div>
<script>
/* if & else */
function changetext(){
		document.getElementById('text').onclick = function (){
			swapcolour(this);
			}
		}
function swapcolour(text){
	 if(text.style.color == 'black'){
		text.style.color = 'red';
		}
	else {text.style.color = 'black'}
	} 
window.onload = changetext;
</script>

Hey guys, here's my code

Code:

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
	<title>Untitled Document</title>
	<script src="jquery-1.3.2.min.js" type="text/javascript"></script>

<style type="text/css">

#box {
background: red;
width: 300px;
height: 300px;
display: none;
}

#box2 {
background: green;
width: 300px;
height: 300px;
display: none;
}
	
	
</style>

<script type="text/javascript">

	$(function() { 
		$('a.about').click(function() {
			$('#box').stop().slideToggle(2000);
		});
	});

	$(function() { 
		$('a.contact').click(function() {
			$('#box2').stop().slideToggle(2000);
		});
	});

</script>
</head>


<body>

<div id="box"></div>
<div id="box2"></div>
<a href="#" class="about" />about</a>
<a href="#" class="contact" />contact</a>



</body>
</html>

Basically I want to make it so that if you click about, and about is open, once you click contact it closes about before opening contact? I'm not sure how to approach this though whether I should use if statements or something? I'm pretty new to jquery