Как сократить блок case case, преобразовав число в название месяца?

Есть ли способ написать это на меньшем количестве строк, но при этом легко читаемым?

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

Определите массив, затем перейдите по индексу.

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

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

как насчет того, чтобы вообще не использовать массив :)

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

согласно этому ответу Получить название месяца от Даты от Дэвида Стори

Попробуй это:

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

Обратите внимание, что это mmможет быть целое число или строка, и это все равно будет работать.

Если вы хотите, чтобы несуществующие ключи приводили к пустой строке ''(вместо undefined), добавьте эту строку:

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

JSFiddle .

Вместо этого вы можете создать массив и найти название месяца:

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


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

См. Ответ @CupawnTae для рационального использования кода || ''

Быть осторожен!

То, что должно немедленно вызвать тревогу, - это первая строка: var month = '';- почему эта переменная инициализируется пустой строкой, а не nullили undefined? Возможно, это было просто привычкой или копированием / вставкой кода, но если вы не знаете этого наверняка, игнорировать это при рефакторинге кода небезопасно.

Если вы используете массив названий месяцев и измените свой код, var month = months[mm-1];вы измените поведение, потому что теперь для чисел за пределами диапазона или нечисловых значений monthбудет undefined. Вы можете знать, что это нормально, но во многих ситуациях это было бы плохо.

Например, предположим, что вы switchнаходитесь в функции monthToName(mm), и кто-то вызывает вашу функцию следующим образом:

var monthName = monthToName(mm);

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

Теперь, если вы monthName[mm-1]перейдете на использование массива и вернетесь , вызывающий код больше не будет работать должным образом, и он будет отправлять undefinedзначения, когда должен отображать предупреждение. Я не говорю, что это хороший код, но, если вы точно не знаете, как этот код используется, вы не можете делать предположений.

Или, может быть, исходная инициализация была там, потому что некоторый код ниже по строке предполагает, что monthэто всегда будет строка, и делает что-то вроде month.length- это приведет к возникновению исключения для недопустимых месяцев и потенциально полностью уничтожит вызывающий скрипт.

Если вы делаете знать весь контекст - например , все это ваш собственный код, и никто еще никогда не собирается использовать его, и вы доверяете себе , не забывайте , что вы сделали изменения когда - нибудь в будущем - это может быть безопасным , чтобы изменить поведение вроде этого, но очень много ошибок возникает из-за такого предположения, что в реальной жизни вам гораздо лучше программировать в защите и / или тщательно документировать поведение.

Ответ Wasmoo дает все правильно (РЕДАКТИРОВАТЬ: ряд других ответов, включая принятый, теперь тоже исправлен) - вы можете использовать months[mm-1] || ''или, если вы предпочитаете, с первого взгляда сделать более очевидным, что происходит, что-то вроде:

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

var month;

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

Для полноты картины хочу дополнить текущие ответы. По сути, вы можете опустить breakключевое слово и напрямую вернуть соответствующее значение. Эта тактика полезна, если значение не может быть сохранено в предварительно вычисленной справочной таблице.

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

Еще раз, использование справочной таблицы или функций даты более кратко и субъективно лучше .

Вы можете сделать это с помощью массива:

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

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

Вот еще один вариант, который использует только 1 переменную и по-прежнему применяет значение по умолчанию, ''когда mmнаходится за пределами диапазона.

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

Вы можете записать его как выражение вместо переключателя, используя условные операторы:

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

Если вы раньше не видели связанных условных операторов, это может показаться труднее читать. Написание его в виде выражения делает один аспект даже более заметным, чем исходный код; ясно, что целью кода является присвоение значения переменной month.

Основываясь на предыдущем ответе Cupawn Tae, я бы сократил его до:

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

В качестве альтернативы, да, я ценю, менее читаемый:

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

Как и @vidriduch, я хотел бы подчеркнуть важность i20y («интернационализации») кода в современном контексте и предложить следующее краткое и надежное решение вместе с унитарным тестом.

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

Я стараюсь оставаться как можно ближе к исходному вопросу, то есть преобразовывать числа от 1 до 12 в названия месяцев, не только для одного особого случая, но и возвращаться undefinedв случае неверных аргументов, используя некоторую ранее добавленную критику и содержание других ответы. (Изменение с undefinedна ''тривиально, если требуется точное соответствие.)

Я бы выбрал решение wasmoo , но настроил его так:

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

На самом деле это тот же самый код, но с другим отступом, что IMO делает его более читабельным.