JavaScriptでは、switchステートメントから戻ることは、breakを使用するよりも良い方法と考えられていますか？

オプション1-リターンを使用して切り替える：

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

        default: return "";
    }    
}

オプション2-ブレークを使用して切り替える：

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つはベストプラクティスですか？私はオプション1を好む傾向があります-よりクリーンでシンプルなので、リターンを使用して切り替えるのが最善です。

@ ic3b3rgのコメントで言及されている手法を使用した私の特定の例のjsFiddleは次のとおりです。

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のコメントに記載されている手法を使用してこれを修正するにはどうすればよいですか？

ブレークにより、関数の処理を続行できます。それがあなたが関数でやりたいことがすべてであれば、スイッチから戻るだけで結構です。

場合によっては、関数がswitchステートメントのみで構成されている場合は問題ないと思います。ただし、その関数内で他の操作を実行したい場合、それはおそらく素晴らしいアイデアではありません。また、現在と将来の要件を考慮する必要がある場合もあります。関数をオプション1からオプション2に変更する場合は、さらにリファクタリングが必要になります。

ただし、if / elseステートメント内では、次のことを行うことがベストプラクティスです。

var foo = "bar";

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

これに基づいて、オプション1の方が優れているという主張をすることができます。

簡単に言うと、明確な答えはありません。コードが一貫性​​があり、読みやすく、保守可能な標準に準拠している限り、つまり、アプリケーション全体でオプション1と2を混在させたり一致させたりしないでください。それがベストプラクティスです。以下。