JavaScript中switch语句使用大于小于判断的代码问题排查

先看一下你提供的代码片段：

let userName = 'roy';
const userQuestion = 'Do you do CrossFit?';
userName ? console.log('Hello ' + userName + ' !') : console.log('Hello!');
console.log(`The user asked: ${userQuestion}`);
let randomNumber = Math.floor(Math.random() * 8);
let eightBall = '';
console.log(randomNumber);
switch (randomNumber) {
  case 0: eightBall = 'It is certain'; break;
  case 1: eightBall = 'It is defidedly so'; break;
  case randomNumber >= 1: eightBall = 'we ...'
}

核心问题：switch语句的case用法错误 #

JavaScript的switch语句是严格匹配机制：它会把switch()里的表达式（这里是randomNumber）和每个case后面的值做===（严格相等）比较。你写的case randomNumber >= 1是一个条件判断，它的结果是布尔值true或false，而randomNumber是0-7的数字，数字和布尔值永远不会严格相等，所以这个case永远不会被触发。

另外，这个case还缺少break语句，即使它能匹配，也会导致代码继续往下执行（如果后面有其他case的话），引发逻辑混乱。

修复方案 #

根据你的需求，看起来是想让randomNumber为2-7时，eightBall赋值为'we ...'。有两种常见的修复方式：

方式1：用default处理剩余所有情况

default是switch的默认分支，当没有任何case匹配时就会执行它，适合处理“其他所有情况”：

switch (randomNumber) {
  case 0: 
    eightBall = 'It is certain'; 
    break;
  case 1: 
    eightBall = 'It is decidedly so'; // 顺便修正拼写错误：defidedly → decidedly
    break;
  default:
    eightBall = 'we ...'; 
    break;
}

方式2：多个case合并（fall through）

如果需要明确指定哪些值对应这个分支，可以把多个case写在一起，利用fall through特性（不写break，让代码流到下一个case）：

switch (randomNumber) {
  case 0: 
    eightBall = 'It is certain'; 
    break;
  case 1: 
    eightBall = 'It is decidedly so'; 
    break;
  case 2:
  case 3:
  case 4:
  case 5:
  case 6:
  case 7:
    eightBall = 'we ...'; 
    break;
}

额外小提示 #

原代码里case 1的字符串拼写有误：It is defidedly so应该是It is decidedly so，建议修正这个拼写错误，避免输出内容不准确。

内容的提问来源于stack exchange，提问作者user7496931