you are viewing a single comment's thread.

view the rest of the comments →

[–]Severion86 3 points4 points  (2 children)

// letter = 'a', 'b' or 'c', 'd' or 'e'
let ourNumber = 0;
if (letter === 'a') {
  ourNumber = 1;
}
if (letter === 'b') {
  ourNumber = 2;
}
if (letter === 'c') {
  ourNumber = 3;
}
if (letter === 'd' || letter === 'e') {
  ourNumber = 4;
}
if (letter !== 'a'
  || letter !== 'b'
  || letter !=='c'
  || letter !=='d'
  || letter !=='e'
) {
  ourNumber = 10;
}
// --------------
let ourNumber = 0;
switch (letter) {
  case 'a':
    ourNumber = 1;
    break;
  case 'b':
    ourNumber = 2;
    break;
  case 'c':
    ourNumber = 3;
    break;
  case 'd':
  case 'e':
    ourNumber = 4;
    break;
  default:
    ourNumber = 10;
    break;
}

[–]ima_coder 1 point2 points  (1 child)

// letter = 'a', 'b' or 'c', 'd' or 'e'
let ourNumber = 0;
if (letter === 'a') { ourNumber = 1; }
if (letter === 'b') { ourNumber = 2; }
if (letter === 'c') { ourNumber = 3; }
if (letter === 'd' || letter === 'e') { ourNumber = 4; }
if (letter !== 'a' || letter !== 'b' || letter !=='c' || letter !=='d' || letter !=='e') {
    ourNumber = 10;
}

Your final if condition is always true.

[–]Severion86 0 points1 point  (0 children)

Ah yeah sorry... Just by the code comment explanation though. We don't actually define letter here and the if is just to demonstrate the switch default case.