homepage Welcome to WebmasterWorld Guest from 54.166.8.138
register, free tools, login, search, pro membership, help, library, announcements, recent posts, open posts,
Become a Pro Member

Home / Forums Index / Code, Content, and Presentation / JavaScript and AJAX
Forum Library, Charter, Moderator: open

JavaScript and AJAX Forum

    
javascript functions
How to end function early
Birdman




msg:1483901
 12:38 pm on May 7, 2002 (gmt 0)

If anyone has a minute to look at this script, could you tell me how to make the function end after an alert. I set the select box description(size and color) as the selected values, but want the function to end if they didn't make a valid choice. I know it would be easy enough to just take that value out of the form, but I know there is a way to do it. Anyway, here it is:
<script language="JavaScript">
function buildOrder()
{
var picksize="Please choose a size."
var pickcolor="Please choose a color."
var dcolor=document.dressopts.colorchoice.value;
if(dcolor=="colors")
{alert(pickcolor)}
var dsize=document.dressopts.sizechoice.value;
if(dsize=="sizes")
{alert(picksize)}
if(dsize=="4")
{cost="119"}
if(dsize=="5")
{cost="119"}
if(dsize=="6")
{cost="119"}
if(dsize=="7")
{cost="119"}
if(dsize=="8")
{cost="119"}
if(dsize=="10")
{cost="129"}
if(dsize=="12")
{cost="129"}
if(dsize=="14")
{cost="129"}
var desc="Your dress size is " + dsize + ". " + dcolor + " is your color choice. Your final cost for this dress is " + cost + ". And remember shipping is absolutely free!!!";
alert(desc)
var sOrder = "https://www.paypal.com/cart/add=1&business=m00n106%40aol.com&item_name=Flower girl dress(" + dcolor + ")(size " + dsize + ")&item_number=fgd88030&amount=" + cost;
window.open(sOrder,'cartwin','width=600,height=400,scrollbars,location,resizable,status');
}
</script>

Thanks

 

korkus2000




msg:1483902
 1:10 pm on May 7, 2002 (gmt 0)

You can break up this large function into smaller functions. Alert functions and assignment functions. Call the assignment functions if you don't alert.

txbakers




msg:1483903
 1:27 pm on May 7, 2002 (gmt 0)

Like Korkus said, I would break it up into smaller functions.

You could create flags (boolean variables) that get set when an alert is given. Then you could check the flag - if false you know that they never had a valid selection.

You also might want to look into the "switch" command instead of the else-if in this case.

With switch you can set a default: which would return false. You can also set "break;" to get out of the routine.

Good luck with it. It sounds like a fun project.

Birdman




msg:1483904
 2:39 pm on May 7, 2002 (gmt 0)

>Good luck with it. It sounds like a fun project.

It is fun :) , but time consuming! :o It's my mothers bridal shop site. She hired someone to build it but they only gave her like three pages and no way to even order! :( I got the ftp access and started changing, but didn't realize what I wasw getting into. It'll probably have 400 or more pages, plus, as you can see there are lots of variables in the items. I had just started learning HTML not long before. By the time I finish this site I hopefully have a clue. Anyway thanks again. If, by chance you wanted to check it out, I put it in profile.

Global Options:
 top home search open messages active posts  
 

Home / Forums Index / Code, Content, and Presentation / JavaScript and AJAX
rss feed

All trademarks and copyrights held by respective owners. Member comments are owned by the poster.
Home ¦ Free Tools ¦ Terms of Service ¦ Privacy Policy ¦ Report Problem ¦ About ¦ Library ¦ Newsletter
WebmasterWorld is a Developer Shed Community owned by Jim Boykin.
© Webmaster World 1996-2014 all rights reserved