如何缩短开关案例块将数字转换为月份名称？

Question

有没有办法在更少的行上写这个,但仍然容易阅读？

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

Answer 1

定义一个数组,然后按索引获取.

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

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

Answer 2

怎么样不使用数组:)

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

根据这个答案从David Storey 获取Date的月份名称

Answer 3

试试这个:

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

请注意,它mm可以是整数或字符串,它仍然可以工作.

如果您希望不存在的键导致空字符串''(而不是undefined),则添加以下行:

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

JSFiddle.

Answer 4

您可以改为创建一个数组并查找月份名称:

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


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

请参阅@CupawnTae的答案,了解代码背后的理性 || ''

Answer 5

小心!

应立即触发警钟的事情是第一行: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
}

Answer 6

为了完整性,我想补充目前的答案.基本上,您可以省略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 '';
}

再一次,使用查找表或日期函数更简洁,主观上更好.

Answer 7

你可以使用数组来做到这一点:

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

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

Answer 8

这是另一个仅使用1个变量的选项,''当mm超出范围时仍然应用默认值.

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

Answer 9

您可以使用条件运算符将其写为表达式而不是开关:

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.

Answer 10

在Cupawn Tae回答之前,我将其简化为:

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

或者,是的,我欣赏,不太可读:

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

Answer 11

和 @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匹配，从到 的更改是微不足道的。）''