数値を月名に変換するスイッチケースブロックを短縮する方法

これを少ない行で書く方法はありますが、それでも簡単に読むことができますか？

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

この回答に従って、David Storey からDateから月の名前を取得します

これを試して：

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またはor ではなく、なぜこの変数が空の文字列に初期化されるの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' :
  '';

これまでにチェーンされた条件演算子を見たことがない場合、これは最初は読みにくいように見えるかもしれません。式として記述すると、1つの側面が元のコードよりも見やすくなります。コードの目的が変数に値を割り当てることであることは明らかですmonth。

前のクポーンテの答えに基づいて、私はそれを次のように短縮します。

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つの特殊なケースだけでなく、1から12の数字を月の名前に変換しますがundefined、無効な引数の場合は、以前に追加されたいくつかの批判と他のコンテンツを使用して返します。答えます。（からundefinedへの変更''は、正確な一致が必要な場合に備えて簡単です。）

wasmooの解決策を探しますが、次のように調整します。

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

まったく同じコードですが、インデントが異なっており、IMOによって読みやすくなっています。