Switch-Case for strings in Javascript not working as expected

Question

So I have this problem with strings and switch-case, and I'll try to keep it as simple as possible.

Here event.keyCode has the value "65", and is the result of a keydown event of 'a' (using JQuery).

if (event.keyCode == "65") {
   alert("hmmmm");
}

That works, but:

switch (event.keyCode) {
   case '65':
      alert("Yay!");
      break;
}

That doesn't. However this will work:

switch ('65') {
   case '65':
      alert("Yay!");
      break;
}

And if I do this:

var t = '65';
switch (t) {
   case '65':
      alert("Yay!");
      break;
}

It works. And then I tried this:

var t = event.keyCode;
switch (t) {
   case '65':
      alert("Yay!");
      break;
}

But it fails!

So why does it match in the if-block at the beginning, but not for the switch-case?

What is your context testing ?

Boris Guéry
– Boris Guéry

2010-04-04 00:41:45 +00:00
Commented Apr 4, 2010 at 0:41 — Boris Guéry
– Boris Guéry, Commented Apr 4, 2010 at 0:41

Matthew Flaschen · Accepted Answer · 2010-04-04 00:58:53Z

59

keyCode is an integer, not a string. When you use ==, the conversion is done implicitly. However, the switch uses the equivalent of ===, which doesn't allow implicit conversions. You can test this easily with:

switch (65) {
   case '65':
      alert("Yay!");
      break;
}

As expected, it does not alert.

This is stated in ECMAScript, 5th edition section 12.11 (switch statement). The interpreter will enter a case statement if "input is equal to clauseSelector as defined by the === operator". input is 65 (integer) and clauseSelector is '65' (string) in my above example, which are not ===.

edited Apr 4, 2010 at 0:58

answered Apr 4, 2010 at 0:39

Matthew Flaschen

286k53 gold badges524 silver badges554 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Coltin Over a year ago

You're right, I'm used to things crashing when I compare (int == string). Thanks!

Matthew Flaschen Over a year ago

@2-Stroker, just switch directly on the number. It may be faster too.

Collectives™ on Stack Overflow

Switch-Case for strings in Javascript not working as expected

1 Answer 1

2 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related