JavaScript:正确的 switch-case 语句

Question

我的代码是这样的:

switch(typeof src) {
    case 'string':
    case 'boolean':
        return String(src).toLowerCase().indexOf(value.toLowerCase()) > -1;
    case 'object':
        return true;
    default:
        return false;
}

但是代码检查器说:n 个无条件的“break”语句应终止每个非空的 switch 子句

我这样写是不是很正常？

switch(typeof src) {
    case 'string':
    case 'boolean':
        return String(src).toLowerCase().indexOf(value.toLowerCase()) > -1;
        break;
    case 'object':
        return true;
        break;
    default:
        return false;
        // in default i need it or not?
}

还是我漏掉了什么？

Answer 1

那么您的代码检查器就错了，因为 return 终止了 switch 语句所在的整个函数。
return 之后的任何内容(比如 break 它希望您将其放在那里)将无法访问并且永远不会执行。

这是否漂亮或良好做法值得商榷，但我会说没问题。

如果您没有return 语句或任何其他终止当前 block 或跳出当前 block 的语句(如循环内的continue)，则您通常想在每个case block 的末尾放置一个break 以防止漏电，但实际上在某些情况下您想要穿透。

所以您的代码检查器实际上更像是一个意见检查器。