从 switch 语句中返回是否被认为是比使用 break 更好的做法？ [关闭]

Question

选项 1 - 使用 return 切换：

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

        default: return "";
    }    
}

选项 2 - 使用 break 切换：

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

    switch (opt) 
    {
        case 1: 
            retVal = "One";
            break;

        case 2: 
            retVal = "Two";
            break;

        case 3: 
            retVal = "Three";
            break;
    }

    return retVal;
}

我知道这两种方法都有效，但还有一种最佳做法吗？我倾向于选择选项 1 - 最好使用 return 进行切换，因为它更干净、更简单。

---

Here is a jsFiddle of my specific example using the technique mentioned in @ic3b3rg's comments：

var 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);

这里的问题是它总是返回未定义的。我猜这是因为它传递的是对象文字的实际值而不是属性。我将如何使用 @ic3b3rg 的 cmets 中描述的技术来解决此问题？

Answer 1

中断将允许您继续在函数中处理。如果您只想在函数中执行此操作，只需退出开关即可。

Answer 2

两者都不是，因为对于一个非常简单的任务来说，两者都非常冗长。 你可以这样做：

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

解释：

它通过创建一个对象来工作，其中选项/案例是键，结果是值。通过将选项放在括号中，您可以通过括号表示法访问与表达式匹配的键的值。

如果括号内的表达式不是有效的键，则返回undefined。我们可以使用nullish coalescing operator ?? 检测这种未定义的情况并返回一个默认值。

示例：

console.log('Using a valid case:', ({
  1: 'One',
  2: 'Two',
  3: 'Three'
})[1] ?? 'Default')

console.log('Using an invalid case/defaulting:', ({
  1: 'One',
  2: 'Two',
  3: 'Three'
})[7] ?? 'Default')

.as-console-wrapper {max-height: 100% !important;top: 0;}

Answer 3

这取决于，如果你的函数只包含 switch 语句，那么我认为它很好。但是，如果您想在该函数中执行任何其他操作，这可能不是一个好主意。您可能还必须考虑您现在和将来的需求。如果您想将功能从选项一更改为选项二，则需要进行更多重构。

但是，鉴于在 if/else 语句中，最好执行以下操作：

var foo = "bar";

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

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

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