Multiple conditions in single case in Switch in JavaScript

Question

4.00/5 (2 votes)

See more:

I am using Switch statement in JavaScript,I need to apply single case on multiple cases

I am writing code as:

C#

function SelectDropDownValue(Service_Name, DefaultValue, Fieldname) {
    var result;
    switch (Service_Name) {      
        case ("SUBDIVIDED" || "ARB" || "MINING CLAIM" || "SECTIONAL" || "MAPS AND SUBDIVISIONS"):
            if (DefaultValue == "N" && Fieldname == "BACKTOBASEORDEROPTION") {
                result = "PLANT_BEGINNING";
                break;
            }
            else if (DefaultValue == "1") {
                result = "MOST_RECENT";
                break;
            }
            else if (DefaultValue == "2") {
                result = "SECOND_MOST";
                break;
            }
            else if(DefaultValue == "N" && Fieldname=="SORTINVESTIGATIVEOPTION" || Fieldname=="SORTORDEROPTION") 
            {
            result ="Newest";
            break;
            }//End       
        default: result = ""; break;
    } //end switch case
    return result;
}

Problem is arising is that the case is running only for Subdivided and not for other values

How can i do this in JS?

Posted 4-Apr-11 21:35pm

Syed Salman Raza Zaidi

Add a Solution

4 solutions

Solution 1

C#

function SelectDropDownValue(Service_Name, DefaultValue, Fieldname) {
    var result;
    switch (Service_Name) {
        case ("SUBDIVIDED"): 
        case ("ARB"): 
        case ("MINING CLAIM"): 
        case ("SECTIONAL"): 
        case ("MAPS AND SUBDIVISIONS"):
            if (DefaultValue == "N" && Fieldname == "BACKTOBASEORDEROPTION") {
                result = "PLANT_BEGINNING";
                break;
            }
            else if (DefaultValue == "1") {
                result = "MOST_RECENT";
                break;
            }
            else if (DefaultValue == "2") {
                result = "SECOND_MOST";
                break;
            }
            else if(DefaultValue == "N" && Fieldname=="SORTINVESTIGATIVEOPTION" || Fieldname=="SORTORDEROPTION")
            {
            result ="Newest";
            break;
            }//End
        default: result = ""; break;
    } //end switch case
    return result;
}

JavaScript uses C like syntax for switch statement as shown above and so just using a newline and another case should work fine.

Posted 4-Apr-11 21:43pm

Guyverthree

Comments

Manfred Rudolf Bihy 5-Apr-11 3:50am

I see you beat me to it! 5+

Solution 3

You'll have to do it like this:

C#

function SelectDropDownValue(Service_Name, DefaultValue, Fieldname) {
    var result;
    switch (Service_Name) {
        case "SUBDIVIDED":
        case "ARB": 
        case "MINING CLAIM":
        case "SECTIONAL":
        case "MAPS AND SUBDIVISIONS":
            if (DefaultValue == "N" && Fieldname == "BACKTOBASEORDEROPTION") {
                result = "PLANT_BEGINNING";
            }
            else if (DefaultValue == "1") {
                result = "MOST_RECENT";
            }
            else if (DefaultValue == "2") {
                result = "SECOND_MOST";
            }
            else if(DefaultValue == "N" && Fieldname=="SORTINVESTIGATIVEOPTION" || Fieldname=="SORTORDEROPTION")
            {
                result ="Newest";
            }//End
            break;
        default:
            result = "";
            break;
    } //end switch case
    return result;
}

That way it should work. BTW I moved all of your breaks out from the else-ifs because I like that style better and in this case there really isn't a difference.

Happy coding!

Posted 4-Apr-11 21:49pm

Manfred Rudolf Bihy

Updated 4-Apr-11 22:12pm

v2

Comments

Sergey Alexandrovich Kryukov 5-Apr-11 3:56am

Looks reasonable, a 5.
Happy coding to you, too!
--SA

Solution 4

Hi, what if you want to check multiple conditions using AND operator:

switch(true)<br />
{<br />
   case (val_1 == "12" && val_2 == "34"):<br />
   //statements;<br />
   break;<br />
}

Posted 13-Sep-11 1:06am

JayEnaR

Updated 13-Sep-11 1:07am

v2

Comments

Graham Breach 13-Sep-11 8:11am

Then you use an "if" and not a "switch/case".

"switch(true)" - ?????

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Sergey Alexandrovich Kryukov · Accepted Answer · 2011-04-04T21:45:00

Solution 2

This is the abuse of switch statement: it makes absolutely no sense if there is only one case + default. You also mix it with "if" statement. Change it all to "if", don't torture readability and common sense.

—SA

Posted 4-Apr-11 21:45pm

Sergey Alexandrovich Kryukov

Updated 4-Apr-11 21:51pm

v2

Comments

Manfred Rudolf Bihy 5-Apr-11 3:51am

You're right SA! 5+
Thanks for pointing that out.

Sergey Alexandrovich Kryukov 5-Apr-11 3:55am

Thank you, Manfred.
--SA

Guyverthree 5-Apr-11 3:56am

Indeed you are right and readability of code is important.

Sergey Alexandrovich Kryukov 5-Apr-11 10:59am

Thank you, Safar,
--SA