Comment raccourcir le bloc de cas de commutation en convertissant un nombre en nom de mois?

Existe-t-il un moyen d'écrire ceci sur moins de lignes, mais toujours facilement lisible?

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

Définissez un tableau, puis obtenez par index.

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

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

et ne pas utiliser du tout 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" }));

selon cette réponse Obtenir le nom du mois de Date de David Storey

Essaye ça:

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

Notez que cela mmpeut être un entier ou une chaîne et cela fonctionnera toujours.

Si vous voulez que les clés non existantes entraînent une chaîne vide ''(au lieu de undefined), ajoutez cette ligne:

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

JSFiddle .

Vous pouvez créer un tableau à la place et rechercher le nom du mois:

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


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

Voir la réponse de @CupawnTae pour le rationnel derrière le code || ''

Faites attention!

La chose qui devrait immédiatement déclencher des sonneries d'alarme est la première ligne: var month = '';- pourquoi cette variable est-elle initialisée à une chaîne vide, plutôt que nullou undefined? Il peut s'agir simplement d'une habitude ou d'un copier / coller de code, mais à moins que vous ne le sachiez avec certitude, il n'est pas prudent de l'ignorer lorsque vous refactorisez du code.

Si vous utilisez un tableau de noms de mois et que vous modifiez votre code, var month = months[mm-1];vous modifiez le comportement, car maintenant pour les nombres en dehors de la plage, ou les valeurs non numériques, monthseraundefined . Vous savez peut-être que c'est correct, mais il y a de nombreuses situations où ce serait mauvais.

Par exemple, disons que vous switchêtes dans une fonction monthToName(mm)et que quelqu'un appelle votre fonction comme ceci:

var monthName = monthToName(mm);

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

Maintenant, si vous passez à l'utilisation d'un tableau et au retour monthName[mm-1], le code d'appel ne fonctionnera plus comme prévu et il soumettra des undefinedvaleurs lorsqu'il est censé afficher un avertissement. Je ne dis pas que c'est un bon code, mais à moins que vous ne sachiez exactement comment le code est utilisé, vous ne pouvez pas faire d'hypothèses.

Ou peut-être que l'initialisation d'origine était là parce que du code plus loin dans la ligne suppose que ce monthsera toujours une chaîne et fait quelque chose commemonth.length - cela entraînera la levée d'une exception pour des mois invalides et potentiellement tuera complètement le script appelant.

Si vous ne connaissez tout le contexte - par exemple , il est tout votre propre code, et personne d'autre ne va jamais l' utiliser, et vous vous croyez ne pas oublier que vous avez fait le quelque temps de changement dans l'avenir - il peut être sûr de changer le comportement comme ça, mais tellement de bogues proviennent de ce genre d'hypothèse que dans la vraie vie, il vaut mieux programmer de manière défensive et / ou documenter le comportement à fond.

La réponse de Wasmoo est correcte (EDIT: un certain nombre d'autres réponses, y compris celle acceptée, ont maintenant été corrigées) - vous pouvez utiliser months[mm-1] || ''ou si vous préférez rendre plus évident en un coup d'œil ce qui se passe, quelque chose comme:

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

var month;

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

Par souci d'exhaustivité, j'aimerais compléter les réponses actuelles. Fondamentalement, vous pouvez ignorer le breakmot - clé et renvoyer directement une valeur appropriée. Cette tactique est utile si la valeur ne peut pas être stockée dans une table de recherche précalculée.

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

Encore une fois, utiliser une table de consultation ou des fonctions de date est plus succinct et subjectivement meilleur .

Vous pouvez le faire en utilisant un tableau:

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

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

Voici une autre option qui n'utilise qu'une seule variable et applique toujours la valeur par défaut ''lorsqu'elle mmest en dehors de la plage.

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

Vous pouvez l'écrire comme une expression au lieu d'un commutateur, en utilisant des opérateurs conditionnels:

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' :
  '';

Si vous n'avez pas vu d'opérateurs conditionnels chaînés auparavant, cela peut sembler plus difficile à lire au début. L'écrire comme une expression rend un aspect encore plus facile à voir que le code original; il est clair que l'intention du code est d'attribuer une valeur à la variable month.

En m'appuyant sur la réponse de Cupawn Tae, je la raccourcirais à:

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

Alternativement, oui, j'apprécie, moins lisible:

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

Comme @vidriduch, je voudrais souligner l'importance d'i20y ("internationalisabilité") du code dans le contexte actuel et suggérer la solution concise et robuste suivante avec le test unitaire.

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/>"));

J'essaie de rester aussi proche que possible de la question originale, c'est-à-dire de transformer les nombres 1 à 12 en noms de mois, non seulement pour un cas particulier, mais de revenir undefineden cas d'arguments invalides, en utilisant certaines des critiques précédemment ajoutées et le contenu d'autres réponses. (Le changement de undefinedà ''est trivial, au cas où une correspondance exacte serait nécessaire.)

Je choisirais la solution wasmoo , mais ajustez-la comme ceci:

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

C'est exactement le même code, vraiment, mais différemment en retrait, que l'OMI le rend plus lisible.