In JavaScript, il ritorno da un'istruzione switch è considerato una pratica migliore rispetto all'utilizzo di break?

Opzione 1 - cambia usando return:

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

        default: return "";
    }    
}

Opzione 2: passare usando l'interruzione:

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

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

        case 2: 
            retVal = "Two";
            break;

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

    return retVal;
}

So che funzionano entrambi, ma è un'altra delle migliori pratiche? Tendo a gradire l'opzione 1: passare usando return best, in quanto è più semplice e pulito.

Ecco un jsFiddle del mio esempio specifico usando la tecnica menzionata nei commenti di @ ic3b3rg :

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;
}

Per chiamare la funzione, lo farei nei seguenti modi:

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

Il problema qui è che restituisce sempre indefinito. Immagino che sia perché passa nel valore reale dell'oggetto letterale e non nella proprietà. Cosa farei per risolvere questo problema usando la tecnica descritta nei commenti di @ ic3b3rg ?

Un'interruzione ti consentirà di continuare l'elaborazione nella funzione. Il solo ritorno dall'interruttore va bene se questo è tutto ciò che vuoi fare nella funzione.

Dipende, se la tua funzione consiste solo nell'istruzione switch, allora penso che vada bene. Tuttavia, se si desidera eseguire altre operazioni all'interno di quella funzione, probabilmente non è una grande idea. Potrebbe anche essere necessario prendere in considerazione le vostre esigenze in questo momento rispetto al futuro. Se si desidera modificare la funzione dall'opzione 1 all'opzione due, saranno necessari ulteriori refactoring.

Tuttavia, dato che all'interno delle dichiarazioni if ​​/ else è consigliabile eseguire le seguenti operazioni:

var foo = "bar";

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

Sulla base di ciò, si potrebbe sostenere che l'opzione 1 è una pratica migliore.

In breve, non esiste una risposta chiara, quindi finché il codice aderisce a uno standard coerente, leggibile e mantenibile, vale a dire non mescolare e abbinare le opzioni una e due in tutta l'applicazione, questa è la migliore pratica che dovresti essere a seguire.