Wrox Programmer Forums
Go Back   Wrox Programmer Forums > Web Programming > JavaScript > Javascript How-To
| 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 software programmers and website developers including Wrox book authors and readers. New member registration was closed in 2019. New posts were shut off and the site was archived into this static format as of October 1, 2020. If you require technical support for a Wrox book please contact http://hub.wiley.com
  #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


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





Powered by vBulletin®
Copyright ©2000 - 2020, Jelsoft Enterprises Ltd.
Copyright (c) 2020 John Wiley & Sons, Inc.