Java Script switch case statement is not working

Question

1.00/5 (1 vote)

See more:

JavaScript

function onChange(control, oldValue, newValue, isLoading) 
{
   if (isLoading || newValue == '') {
      return;
   }

   switch(newValue) {

        case 4:
            alert(newValue);
     /*        g_form.setDisplay('DLP_Scan',false);
            g_form.setDisplay('account_requesting',false);
            g_form.setDisplay('rule_change',false);
            g_form.setDisplay('report_requesting',false);
            g_form.setDisplay('change_need',true);*/

            break;

    case 2:
            alert(newValue);
        g_form.setDisplay('DLP_Scan',false);
            g_form.setDisplay('change_need',false);
            g_form.setDisplay('rule_change',false);
            g_form.setDisplay('report_requesting',false);
            g_form.setDisplay('account_requesting',true);
        break;
    case 5:
            alert(newValue);
        g_form.setDisplay('DLP_Scan',false);
            g_form.setDisplay('change_need',false);
            g_form.setDisplay('account_requesting',false);
            g_form.setDisplay('report_requesting',false);
            g_form.setDisplay('rule_change',true);
        break;
            case 3:
            alert(newValue);
        g_form.setDisplay('DLP_Scan',false);
            g_form.setDisplay('change_need',false);
            g_form.setDisplay('account_requesting',false);
            g_form.setDisplay('rule_change',false);
            g_form.setDisplay('report_requesting',true);
        break;
            case 6:
            alert(newValue);
        g_form.setDisplay('account_requesting',false);
            g_form.setDisplay('change_need',false);
            g_form.setDisplay('rule_change',false);
            g_form.setDisplay('report_requesting',false);
            g_form.setDisplay('DLP_Scan',true);
        break;

    }
    //Type appropriate comment here, and begin script below

    }

Posted 6-Apr-15 22:05pm

ajitdstar4u

Updated 6-Apr-15 22:19pm

Afzaal Ahmad Zeeshan

v2

Add a Solution

Comments

Afzaal Ahmad Zeeshan 7-Apr-15 4:17am

Why is it not working?

ajitdstar4u 7-Apr-15 4:25am

i checked the syntax, it is correct but still it is not working.
is anything wrong about it. i guess it is not due to proper numbering of case like first we have to write for case 1 then for case to etc . any help will be much appreciated.

Afzaal Ahmad Zeeshan 7-Apr-15 4:28am

You should check the console in that case. It would help you.

OriginalGriff 7-Apr-15 4:31am

So how do you know it's not working?
How are you calling it?
How are you testing it?
What does it do that you didn't expect, or not do that you did?

ajitdstar4u 7-Apr-15 5:29am

if the case statement execute the in UI some fields like g_form.setdisplay('DLP_Scan', true) should enable but it is not.

OriginalGriff 7-Apr-15 5:53am

And?
What are you passing it?
How are you passing the parameter values?
Is it doing anything?

1 solution

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)

Abhinav S · Accepted Answer · 2015-04-06T22:29:00

Variable newValue probably contains string.
The cases should be in quotes.
So your cases would be