Come accorciare il blocco case switch convertendo un numero in un nome di mese?

C'è un modo per scrivere questo su meno righe, ma comunque facilmente leggibile?

var month = '';

switch(mm) {
    case '1':
        month = 'January';
        break;
    case '2':
        month = 'February';
        break;
    case '3':
        month = 'March';
        break;
    case '4':
        month = 'April';
        break;
    case '5':
        month = 'May';
        break;
    case '6':
        month = 'June';
        break;
    case '7':
        month = 'July';
        break;
    case '8':
        month = 'August';
        break;
    case '9':
        month = 'September';
        break;
    case '10':
        month = 'October';
        break;
    case '11':
        month = 'November';
        break;
    case '12':
        month = 'December';
        break;
}

Definisci un array, quindi ottieni per indice.

var months = ['January', 'February', ...];

var month = months[mm - 1] || '';

che ne dici di non usare affatto l'array :)

var objDate = new Date("10/11/2009"),
    locale = "en-us",
    month = objDate.toLocaleString(locale, { month: "long" });

console.log(month);

// or if you want the shorter date: (also possible to use "narrow" for "O"
console.log(objDate.toLocaleString(locale, { month: "short" }));

secondo questa risposta Ottieni il nome del mese da Date da David Storey

Prova questo:

var months = {'1': 'January', '2': 'February'}; //etc
var month = months[mm];

Nota che mm può essere un numero intero o una stringa e funzionerà comunque.

Se desideri che chiavi non esistenti risultino in una stringa vuota ''(invece di undefined), aggiungi questa riga:

month = (month == undefined) ? '' : month;

JSFiddle .

Puoi invece creare un array e cercare il nome del mese:

var months = ['January','February','March','April','May','June','July','August','September','October','November','December']


var month = months[mm-1] || '';

Vedi la risposta di @CupawnTae per il razionale dietro il codice || ''

Stai attento!

La cosa che dovrebbe immediatamente attivare i campanelli d'allarme è la prima riga: var month = '';- perché questa variabile viene inizializzata su una stringa vuota, anziché nullo undefined? Potrebbe essere stata solo un'abitudine o un codice copiato / incollato, ma a meno che tu non lo sappia per certo, non è sicuro ignorarlo quando esegui il refactoring del codice.

Se utilizzi un array di nomi di mesi e modifichi il codice, var month = months[mm-1];stai modificando il comportamento, perché ora per i numeri al di fuori dell'intervallo, o i valori non numerici, monthsarannoundefined . Potresti sapere che questo è ok, ma ci sono molte situazioni in cui questo sarebbe negativo.

Ad esempio, diciamo che sei switchin una funzione monthToName(mm)e qualcuno sta chiamando la tua funzione in questo modo:

var monthName = monthToName(mm);

if (monthName === '') {
  alert("Please enter a valid month.");
} else {
  submitMonth(monthName);
}

Ora, se si passa all'uso di un array e alla restituzione monthName[mm-1], il codice chiamante non funzionerà più come previsto e invierà undefinedvalori quando dovrebbe visualizzare un avviso. Non sto dicendo che questo sia un buon codice, ma a meno che tu non sappia esattamente come viene utilizzato il codice, non puoi fare supposizioni.

O forse l'inizializzazione originale era lì perché un po 'di codice più in basso presume che monthsarà sempre una stringa e fa qualcosa di similemonth.length : questo risulterà in un'eccezione generata per mesi non validi e potenzialmente ucciderà completamente lo script chiamante.

Se tu fai conoscere l'intero contesto - ad esempio, è tutto il proprio codice, e nessun altro è mai andare a usarlo, e si ritiene non te stesso dimenticare che hai fatto il cambiamento in futuro - può essere sicuro di cambiare il comportamento come questo, ma moltissimi bug derivano da questo tipo di presupposto che nella vita reale è molto meglio programmare in modo difensivo e / o documentare accuratamente il comportamento.

La risposta di Wasmoo ha ragione (EDIT: anche una serie di altre risposte, inclusa quella accettata, ora sono state corrette) : puoi usare months[mm-1] || ''o se preferisci rendere più ovvio a colpo d'occhio cosa sta succedendo, qualcosa come:

var months = ['January', 'February', ...];

var month;

if (mm >= 1 && m <= 12) {
  month = months[mm - 1];
} else {
  month = ''; // empty string when not a valid month
}

Per completezza vorrei integrare le risposte attuali. Fondamentalmente, puoi omettere la breakparola chiave e restituire direttamente un valore appropriato. Questa tattica è utile se il valore non può essere memorizzato in una tabella di ricerca precalcolata.

function foo(mm) {
    switch(mm) {
        case '1':  return 'January';
        case '2':  return 'February';
        case '3':  return 'March';
        case '4':  return 'April';
        // [...]
        case '12': return 'December';
    }
    return '';
}

Ancora una volta, l'uso di una tabella di ricerca o di funzioni di data è più succinto e soggettivamente migliore .

Puoi farlo usando un array:

var months = ['January', 'February', 'March', 'April', 
              'May', 'June', 'July', 'August', 
              'September', 'October', 'November', 'December'];

var month = months[mm - 1] || '';

Ecco un'altra opzione che utilizza solo 1 variabile e applica ancora il valore predefinito ''quando mmè al di fuori dell'intervallo.

var month = ['January', 'February', 'March',
             'April', 'May', 'June', 'July',
             'August', 'September', 'October',
             'November', 'December'
            ][mm-1] || '';

Puoi scriverlo come un'espressione invece di un interruttore, usando operatori condizionali:

var month =
  mm == 1 ? 'January' :
  mm == 2 ? 'February' :
  mm == 3 ? 'March' :
  mm == 4 ? 'April' :
  mm == 5 ? 'May' :
  mm == 6 ? 'June' :
  mm == 7 ? 'July' :
  mm == 8 ? 'August' :
  mm == 9 ? 'September' :
  mm == 10 ? 'October' :
  mm == 11 ? 'November' :
  mm == 12 ? 'December' :
  '';

Se non hai mai visto operatori condizionali concatenati prima, questo potrebbe sembrare più difficile da leggere all'inizio. Scriverlo come un'espressione rende un aspetto ancora più facile da vedere rispetto al codice originale; è chiaro che l'intenzione del codice è quella di assegnare un valore alla variabile month.

Basandosi sulla risposta di Cupawn Tae precedente, lo abbreverei a:

var months = ['January', 'February', ...];
var month = (mm >= 1 && mm <= 12) ? months[mm - 1] : '';

In alternativa, sì, apprezzo, meno leggibile:

var month = months[mm - 1] || ''; // as mentioned further up

var getMonth=function(month){
   //Return string to number.
    var strMonth = ['January', 'February', 'March',
             'April', 'May', 'June', 'July',
             'August', 'September', 'October',
             'November', 'December'
            ];
    //return number to string.
    var intMonth={'January':1, 'February':2, 'March':3,
             'April':4, 'May':5, 'June':6, 'July':7,
             'August':8, 'September':9, 'October':10,
             'November':11, 'December':12
            };
    //Check type and return 
    return (typeof month === "number")?strMonth[month-1]:intMonth[month]
}

Come @vidriduch, vorrei sottolineare l'importanza di i20y ("internazionalizzazione") del codice nel contesto odierno e suggerire la seguente soluzione concisa e robusta insieme al test unitario.

function num2month(month, locale) {
    if (month != Math.floor(month) || month < 1 || month > 12)
        return undefined;
    var objDate = new Date(Math.floor(month) + "/1/1970");
    return objDate.toLocaleString(locale, {month: "long"});
}

/* Test/demo */
for (mm = 1; mm <= 12; mm++)
    document.writeln(num2month(mm, "en") + " " +
                     num2month(mm, "ar-lb") + "<br/>");
document.writeln(num2month("x", "en") + "<br/>");
document.writeln(num2month(.1, "en") + "<br/>");
document.writeln(num2month(12.5, "en" + "<br/>"));

Cerco di rimanere il più vicino possibile alla domanda originale, cioè trasformare i numeri da 1 a 12 in nomi di mesi, non solo per un caso speciale, ma ci restituisco undefinedin caso di argomenti non validi, usando alcune delle critiche precedentemente aggiunte e contenuti di altri risposte. (La modifica da undefineda ''è banale, nel caso in cui sia necessaria la corrispondenza esatta .)

Vorrei la soluzione di wasmoo , ma regolatela in questo modo:

var month = [
    'January',
    'February',
    'March',
    'April',
    'May',
    'June',
    'July',
    'August',
    'September',
    'October',
    'November',
    'December'
][mm-1] || '';

È lo stesso identico codice, in realtà, ma con rientranza diversa, che IMO lo rende più leggibile.