选项1 -使用返回开关:

function myFunction(opt) {
  switch (opt) {
    case 1: return "One";
    case 2: return "Two";
    case 3: return "Three";

    default: return "";
  }    
}

选项2 -开关使用断路:

function myFunction(opt) {
  let retVal = "";

  switch (opt) {
    case 1: 
      retVal = "One";
      break;
    case 2: 
      retVal = "Two";
      break;
    case 3: 
      retVal = "Three";
      break;
  }

  return retVal;
}

我知道这两种方法都有效,但其中一种是最佳实践吗?我倾向于选择1 -切换使用返回最好,因为它更干净和更简单。


下面是我使用@ic3b3rg评论中提到的技术的特定示例的jsFiddle:

let SFAIC = {};

SFAIC.common = {
  masterPages: {
    cs: "CS_",
    cp: "CP_"
  },
  contentPages: {
    cs: "CSContent_",
    cp: "CPContent_"    
  }
};

function getElementPrefix(page) {
  return (page in SFAIC.common.masterPages)
    ? SFAIC.common.masterPages[page]
    : (page in SFAIC.common.contentPages)
      ? SFAIC.common.contentPages[page]
      : undefined;
}

要调用该函数,我将以以下方式执行:

getElementPrefix(SFAIC.common.masterPages.cs);
getElementPrefix(SFAIC.common.masterPages.cp);
getElementPrefix(SFAIC.common.contentPages.cs);
getElementPrefix(SFAIC.common.contentPages.cp);

这里的问题是它总是返回undefined。我猜这是因为它传递的是对象字面量的实际值而不是属性。我该如何使用@ic3b3rg评论中描述的技术来修复这个问题?


当前回答

都不是,因为对于一个非常简单的任务来说,这两个都太啰嗦了。你可以这样做:

const result = ({
  1: 'One',
  2: 'Two',
  3: 'Three'
})[opt] ?? 'Default'    // opt can be 1, 2, 3 or anything (default)

当然,这也适用于字符串,两者的混合或不带默认大小写:

const result = ({
  'first': 'One',
  'second': 'Two',
  3: 'Three'
})[opt]                // opt can be 'first', 'second' or 3

解释:

它通过创建一个对象来工作,其中选项/case是键,结果是值。通过将选项放入括号中,您可以通过括号符号访问与表达式匹配的键的值。

如果括号内的表达式不是有效键,则返回undefined。我们可以通过使用空合并运算符??并返回一个默认值。

例子:

console.log('使用有效的大小写:',({ 1:“一个”, 2:“两个”, 3:“三” }) [1] ? ?“违约”) console.log('使用无效的大小写/默认:',({ 1:“一个”, 2:“两个”, 3:“三” }) [7] ? ?“违约”) .as-console-wrapper {max-height: 100% !important;

其他回答

暂停将允许您继续在函数中处理。如果你想在函数中做的只是从switch中返回就可以了。

这要看情况,如果你的函数只包含switch语句,那么我认为没问题。然而,如果你想在这个函数中执行任何其他操作,这可能不是一个好主意。您还可能需要考虑您现在和将来的需求。如果您希望将函数从选项一更改为选项二,则需要进行更多的重构。

然而,考虑到在if/else语句中,执行以下操作是最佳实践:

var foo = "bar";

if(foo == "bar") {
    return 0;
}
else {
    return 100;
}

基于此,可以认为选择一是更好的做法。

简而言之,没有明确的答案,所以只要您的代码遵循一致的、可读的、可维护的标准——也就是说,不要在整个应用程序中混合使用选项一和选项二,这就是您应该遵循的最佳实践。

都不是,因为对于一个非常简单的任务来说,这两个都太啰嗦了。你可以这样做:

const result = ({
  1: 'One',
  2: 'Two',
  3: 'Three'
})[opt] ?? 'Default'    // opt can be 1, 2, 3 or anything (default)

当然,这也适用于字符串,两者的混合或不带默认大小写:

const result = ({
  'first': 'One',
  'second': 'Two',
  3: 'Three'
})[opt]                // opt can be 'first', 'second' or 3

解释:

它通过创建一个对象来工作,其中选项/case是键,结果是值。通过将选项放入括号中,您可以通过括号符号访问与表达式匹配的键的值。

如果括号内的表达式不是有效键,则返回undefined。我们可以通过使用空合并运算符??并返回一个默认值。

例子:

console.log('使用有效的大小写:',({ 1:“一个”, 2:“两个”, 3:“三” }) [1] ? ?“违约”) console.log('使用无效的大小写/默认:',({ 1:“一个”, 2:“两个”, 3:“三” }) [7] ? ?“违约”) .as-console-wrapper {max-height: 100% !important;