Simplifying this JavaScript-switch

Question

I would like some advice on how to slim down this switch:

switch (lotUser | winningLot) {
        case lotUser === winningLot[0]:
        case lotUser === winningLot[1]:
        case lotUser === winningLot[2]:
        case lotUser === winningLot[3]:
        case lotUser === winningLot[4]:
        case lotUser === winningLot[5]:
        case lotUser === winningLot[6]:
        case lotUser === winningLot[7]:
            console.log("You win!");
            break;
        default:
            console.log("You do not win!");
            break;
    }

Instead of

case lotUser === winningLot[0]:

I wrote the script to be:

switch (lotUser | winnendLot) {
    case lotUser === winnendLot[0|1|2|3|4|5|6|7]:
        console.log("You win!");
        break;
    default:
        console.log("You do not win!");
        break;
}

I just don't know if this works the way I want it to work. It needs to check if the generated lotUser is equal to one of the values in an array (winningLot). If the lotUser equals one or more of the values in the winningLot array, it should output "You win!". Could someone please confirm that my code does the description I gave?

Answer 1

What about Array.prototype.indexOf()?

if (winnedLot.indexOf(lotUser) !== -1) {
  console.log("Won!");
}
else {
  console.log("Lost!");
}

It searches the array for the first occurrence of the value stored in lotUser and returns its respective index.

Since you do not need to count the occurrences, this should be the best way.

---

If you want to count them, use a loop:

var count = 0;

for (var i=0, len=winnedLot.length; i<len; i++) {
  if (winnedLot[i] === lotUser) {
    count++;
  }
}

Answer 2

You can simply use indexOf:

if(winningLot.indexOf(lotUser) >= 0) {
    ...
} else {
    ...
}

Answer 3

Well for starters you're using switch incorrectly. The value to compare goes in the switch(...) part, and the possible values are listed by each case ...:

Anyway, that aside, all you want is to check if lotUser is in the winnendLot array. Easy:

// assuming supporting browser:
if( winnendLot.indexOf(lotUser) > -1) console.log("You win!");

// full browser support:
var winner = false, l = winnendLot.length, i;
for( i=0; i<l; i++) {
    if( winnendLot[i] === lotUser) {
        winner = true;
        break;
    }
}
if( winner) console.log("You won!");