JavaScript'te, switch ifadesinden çıkmak break kullanmaktan daha iyi bir uygulama mıdır?

Seçenek 1 - Dönüş kullanarak geçiş yapın:

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

        default: return "";
    }    
}

Seçenek 2 - Ara kullanarak geçiş:

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

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

        case 2: 
            retVal = "Two";
            break;

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

    return retVal;
}

Her ikisinin de işe yaradığını biliyorum, ama en iyi uygulamalardan biri daha mı? Seçenek 1'i beğenme eğilimindeyim - daha temiz ve daha basit olduğu için dönüşü en iyi şekilde kullanın.

@ İc3b3rg'nin yorumlarında bahsedilen tekniği kullanarak benim özel örneğimin bir 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;
}

İşlevi çağırmak için bunu aşağıdaki şekillerde yaparım:

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

Burada sorun her zaman tanımsız döndürüyor olmasıdır. Ben bunun özellik değil nesnenin gerçek değeri geçerken çünkü tahmin ediyorum. @ İc3b3rg'nin yorumlarında açıklanan tekniği kullanarak bunu düzeltmek için ne yapmalıyım ?

Bir ara, işlevde işlemeye devam etmenizi sağlar. İşlevde yapmak istediğiniz tek şey varsa, sadece anahtardan çıkmak iyidir.

Bu, eğer fonksiyonunuz sadece switch ifadesinden oluşuyorsa, bunun iyi olduğunu düşünüyorum. Ancak, bu işlev içinde başka işlemler yapmak istiyorsanız, muhtemelen iyi bir fikir değildir. İhtiyaçlarınızı, şimdi ve gelecekte de dikkate almanız gerekebilir. İşlevinizi birinci seçenekten ikinci seçeneğe değiştirmek isterseniz, daha fazla yeniden düzenleme gerekecektir.

Ancak, if / else ifadeleri dahilinde aşağıdakileri yapmak en iyi yöntemdir:

var foo = "bar";

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

Buna dayanarak, birinci seçeneğin daha iyi bir uygulama olduğu argümanı yapılabilir.

Kısacası, net bir cevap yoktur, bu nedenle kodunuz tutarlı, okunabilir, sürdürülebilir bir standarda bağlı olduğu sürece - yani uygulamanız boyunca bir ve iki seçenekleri karıştırmayın ve eşleştirmeyin, bu en iyi uygulama olmalıdır takip etme.