JavaScript switch case using enum

Question

I have an "enum" declared like so:

var PlaceType = {
        PASSABLE_TERRAIN: 1,
        IMPASSABLE_TERRAIN: 0,
        SOMEWHAT_PASSABLE_TERRAIN: 2,
        PATH: 3
    };

and a function declared like this:

setPlaceType(placeType) {
        this.clear = false;
        this.placeType = placeType;

        alert("before switch "+(PlaceType.SOMEWHAT_PASSABLE_TERRAIN==this.placeType));
        switch(this.placeType) {
        case PlaceType.PASSABLE_TERRAIN: {
            alert("Case PASSABLE");
            break;
        }
        case PlaceType.IMPASSABLE_TERRAIN: {
            alert("Case IMPASSABLE");
            break;
        }
        case PlaceType.SOMEWHAT_PASSABLE_TERRAIN: {
            alert("Case SOMEWHAT_PASSABLE");
            break;
        }
        case PlaceType.PATH: {
            alert("Case PATH");
            break;
        }
        default: {
            alert("case default");
        }
        }
    }

if I call it like this:

setPlaceType(1);

I get the following alerts: "before switch true", "case default"

if I call it like this:

setPlaceType(2);

I get the following alerts: "before switch false", "case default"

In other words, the function is called with the proper argument, which, when doing (what it seems to me to be) the same comparison as the switch but via "==" I get correct behavior, but the switch never matches the values to the appropriate case. Does anybody know why?

It works for me: http://jsfiddle.net/Y2KEn/ are you certain you are passing an int? — scrappedcola, Oct 08 '12 at 14:25
Have you run this through a debugger to see what Javascript thinks about the runtime type of the comparison values in the Case blocks (or the inbound value, for that matter)? Just wondering if its a subtle coercion problem? (Shot in the dark) — David W, Oct 08 '12 at 14:25
Any chance you're calling this from an HTML page and passing in the value property of a input text box? That would surely send a string... — David W, Oct 08 '12 at 14:28

score 8 · Accepted Answer · edited Oct 08 '12 at 14:59

The comparison operator will cast both operands to strings if either operator is a string. If you pass in a string, you are comparing string == number which will cast the number to a string and, in the case of passing the string '2', it will be true.

switch case comparison uses the identity operator === and will fail if the operands are not the same type.

long story short, make sure you are always passing a number if your cases are comparing against numbers, you can double check like this:

setPlaceType(placeType) {
    if (typeof placeType !== 'number') {
        throw new Error('You must pass a number to setPlaceType!');
    }
    ...
}

also, you should be calling your function like this:

setPlaceType(PlaceType.PASSABLE_TERRAIN);

otherwise there's not really any point to using the "enumeration" (i use that term loosely) object.

I can be wrong but I think that in javascript, the type coercion always try to cast both types in number except if they are the same type; not in string. Please read this good article: http://javascriptweblog.wordpress.com/2011/02/07/truth-equality-and-javascript/ — Samuel, Feb 13 '13 at 23:36

score 2 · Answer 2 · answered Sep 13 '19 at 17:51

Another way to use enum for switch case:

   const PlaceType = Object.freeze({
            PASSABLE_TERRAIN: 1,
            IMPASSABLE_TERRAIN: 0,
            SOMEWHAT_PASSABLE_TERRAIN: 2,
            PATH: 3
        });

   function setPlaceType(placeType){
      return({
              0:"Case IMPASSABLE_TERRAIN",
              1:"Case PASSABLE_TERRAIN",
              2:"Case SOMEWHAT_PASSABLE_TERRAIN",
              3:"PATH"      
              }[placeType]);
     }

How do I use above scenario to handle undefined or value equal or greater than 4 — rusty, Oct 06 '20 at 11:53

score 1 · Answer 3 · edited May 23 '17 at 10:27

1

The matching case is determined using the === identity operator, not the == equality operator. The expressions must match without any type conversion. It would fail if you are passing in a string and not an integer.

setPlaceType(1);  //"Case PASSABLE"
setPlaceType("1");  //"case default"

Example running your code with the above lines: jsFiddle

So if you are saying it is failing, you are probably comparing a string to a number. Use parseInt.

this.placeType = parseint(placeType,10);

edited May 23 '17 at 10:27

Community

1
1

answered Oct 08 '12 at 14:24

epascarello

204,599
20
195
236

Thank you, that was it. However the fact that the "==" check passes but the switch check does not is really counter intuitive. I understand now that the switch actually makes a "===" check, but still. – Shivan Dragon Oct 08 '12 at 14:31
well, yes, 'cause it also explains about the "==" and "===", and I think it will be more helpful to some other mega noob like me if/when he/she stumbles over this question. – Shivan Dragon Oct 08 '12 at 14:37

bPratik · Answer 4 · 2012-10-08T14:38:12.290

When you are doing the comparison using == js is using type-coercion to cast the 2 operands to an intermediate type, string in this case, and thus compare them successfully.

So to get the effect to work with your switch statement, you will need to cast as such

this.placeType = parseint(placeType);

What you also got to learn here is that it is not really an ideal practice to compare 2 values in javascript using the == operator, instead use the === operator which also checks for the types to be the same. So in your case,

alert("before switch "+(PlaceType.SOMEWHAT_PASSABLE_TERRAIN==this.placeType));

would have failed if you would have used === as you are comparing an int and string

Working demo here: http://jsfiddle.net/pratik136/ATx8c/

score 1 · Answer 5 · edited May 23 '17 at 12:34

1

Refering to this => Switch-Case for strings in Javascript not working as expected

Switch do a ===, while if do a ==.

Hope this help! have a nice day

edited May 23 '17 at 12:34

Community

1
1

answered Oct 08 '12 at 14:31

JavaScript switch case using enum

5 Answers5