Wrox Programmer Forums

Need to download code?

View our list of code downloads.

Go Back   Wrox Programmer Forums > Web Programming > JavaScript > Javascript How-To
Password Reminder
Register
| FAQ | Members List | Calendar | Search | Today's Posts | Mark Forums Read
Javascript How-To Ask your "How do I do this with Javascript?" questions here.
Welcome to the p2p.wrox.com Forums.

You are currently viewing the Javascript How-To section of the Wrox Programmer to Programmer discussions. This is a community of tens of thousands of software programmers and website developers including Wrox book authors and readers. As a guest, you can read any forum posting. By joining today you can post your own programming questions, respond to other developers’ questions, and eliminate the ads that are displayed to guests. Registration is fast, simple and absolutely free .
DRM-free e-books 300x50
Reply
 
Thread Tools Search this Thread Display Modes
  #1 (permalink)  
Old October 27th, 2005, 05:44 AM
Friend of Wrox
Points: 8,019, Level: 38
Points: 8,019, Level: 38 Points: 8,019, Level: 38 Points: 8,019, Level: 38
Activity: 0%
Activity: 0% Activity: 0% Activity: 0%
 
Join Date: Jan 2005
Location: Mauchline, East Ayrshire, Scotland
Posts: 1,525
Thanks: 0
Thanked 0 Times in 0 Posts
Send a message via ICQ to crmpicco Send a message via AIM to crmpicco Send a message via MSN to crmpicco Send a message via Yahoo to crmpicco
Default build switch statement to make dynamic

I was wondering if anyone can see a way of tidying this up a bit? Maybe using a 'switch' statement. I have never used them before. Is that the best way to tackle this?

Code:
                else if (noa == 3)
                {
                    var temp = doco.elements["ad"+sNum].value;

                    if (activePassNo == 1)
                    {
                        if (adtStr.charAt(adtStr.length-1) == "~")
                        {
                            adtStr = adtStr.slice(0, -1);
                        }

                        doco.elements["ad"+sNum].value = adtStr + tempArr[1] + "~" + tempArr[2];
                    }
                    else if (activePassNo == 2)
                    {
                        if (adtStr.charAt(adtStr.length-1) == "~")
                        {
                            adtStr = adtStr.slice(0, -1);
                        }

                        doco.elements["ad"+sNum].value = tempArr[0] + "~" + adtStr + "~" + tempArr[2];
                    }
                    else if (activePassNo == 3)
                    {
                        if (adtStr.charAt(adtStr.length-1) == "~")
                        {
                            adtStr = adtStr.slice(0, -1);
                        }

                        doco.elements["ad"+sNum].value = tempArr[0] + "~" + tempArr[1] + "~" + adtStr;
                    }
                    else
                    {
                        if ((typeof(adtStr) != "undefined") && (typeof(tempArr[parseFloat(activePassNo)]) != "undefined"))
                        {
                            doco.elements["ad"+sNum].value = adtStr + tempArr[parseFloat(activePassNo)];
                        }
                        else if (typeof(adtStr) != "undefined")
                        {
                            doco.elements["ad"+sNum].value = tempArr[0] + "~" + adtStr;                    
                        }
                        else
                        {
                            doco.elements["ad"+sNum].value = adtStr;
                        }
                    }
                }
Picco

www.crmpicco.co.uk
__________________
_______________________
Ayrshire Minis - a Mini E-Community
http://www.ayrshireminis.com
http://www.crmpicco.co.uk
Reply With Quote
Reply


Thread Tools Search this Thread
Search this Thread:

Advanced Search
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is Off
HTML code is Off
Trackbacks are Off
Pingbacks are On
Refbacks are Off


Similar Threads
Thread Thread Starter Forum Replies Last Post
switch statement crmpicco Javascript How-To 3 March 9th, 2015 10:37 PM
Switch-statement grstad C# 3 June 11th, 2008 06:53 AM
Using Switch statement in XHTML Lizane Java Basics 1 June 15th, 2007 06:46 AM
switch statement seymour_glass C# 5 January 24th, 2007 12:08 AM
C# switch statement nalla General .NET 4 December 14th, 2005 05:31 AM



All times are GMT -4. The time now is 08:03 PM.


Powered by vBulletin®
Copyright ©2000 - 2019, Jelsoft Enterprises Ltd.
© 2013 John Wiley & Sons, Inc.