Bagaimana cara mempersingkat blok saklar kasus yang mengubah angka menjadi nama bulan?

Adakah cara untuk menulis ini dengan lebih sedikit baris, tetapi masih mudah dibaca?

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

Tentukan array, lalu dapatkan dengan indeks.

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

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

bagaimana jika tidak menggunakan array sama sekali :)

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

sesuai jawaban ini Dapatkan nama bulan dari Tanggal dari David Storey

Coba ini:

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

Perhatikan bahwa mmbisa berupa integer atau string dan itu akan tetap berfungsi.

Jika Anda ingin kunci yang tidak ada menghasilkan string kosong ''(bukan undefined), tambahkan baris ini:

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

JSFiddle .

Anda dapat membuat array dan mencari nama bulan:

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


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

Lihat jawaban oleh @CupawnTae untuk alasan di balik kode || ''

Hati-hati!

Hal yang harus segera memicu lonceng alarm adalah baris pertama: var month = '';- mengapa variabel ini diinisialisasi ke string kosong, bukan nullatau undefined? Ini mungkin hanya kebiasaan atau salin / tempel kode, tetapi kecuali Anda tahu pasti, tidak aman untuk mengabaikannya saat Anda memfaktorkan ulang kode.

Jika Anda menggunakan array nama bulan dan mengubah kode var month = months[mm-1];Anda, Anda mengubah perilaku, karena sekarang untuk angka di luar rentang, atau nilai non-numerik, monthakan undefined. Anda mungkin tahu bahwa ini tidak masalah, tetapi ada banyak situasi di mana hal ini akan berdampak buruk.

Misalnya, Anda switchada dalam suatu fungsi monthToName(mm), dan seseorang memanggil fungsi Anda seperti ini:

var monthName = monthToName(mm);

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

Sekarang jika Anda mengubah menggunakan array dan mengembalikan monthName[mm-1], kode panggilan tidak akan berfungsi lagi sebagaimana mestinya, dan akan mengirimkan undefinednilai ketika seharusnya menampilkan peringatan. Saya tidak mengatakan ini adalah kode yang bagus , tetapi kecuali Anda tahu persis bagaimana kode itu digunakan, Anda tidak dapat membuat asumsi.

Atau mungkin inisialisasi asli ada di sana karena beberapa kode lebih jauh mengasumsikan bahwa monthakan selalu berupa string, dan melakukan sesuatu seperti month.length- ini akan menghasilkan pengecualian yang dilemparkan selama beberapa bulan yang tidak valid dan berpotensi mematikan skrip pemanggil sepenuhnya.

Jika Anda benar- benar mengetahui konteks keseluruhan - mis. Itu semua adalah kode Anda sendiri, dan tidak ada orang lain yang akan menggunakannya, dan Anda percaya pada diri sendiri tidak lupa Anda membuat perubahan di masa mendatang - mungkin aman untuk mengubah perilaku seperti ini, tapi soooo banyak bug berasal dari asumsi semacam ini bahwa dalam kehidupan nyata Anda jauh lebih baik memprogram secara defensif dan / atau mendokumentasikan perilaku secara menyeluruh.

Jawaban Wasmoo benar (EDIT: sejumlah jawaban lain, termasuk yang diterima, kini telah diperbaiki juga) - Anda dapat menggunakanmonths[mm-1] || '' atau jika Anda lebih suka membuatnya lebih jelas sekilas tentang apa yang terjadi, seperti:

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

var month;

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

Untuk kelengkapan, saya ingin melengkapi jawaban saat ini. Pada dasarnya, Anda dapat menghilangkan breakkata kunci dan langsung mengembalikan nilai yang sesuai. Taktik ini berguna jika nilai tidak dapat disimpan dalam tabel pencarian yang dihitung sebelumnya.

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

Sekali lagi, menggunakan tabel pencarian atau fungsi tanggal lebih ringkas dan lebih baik secara subyektif .

Anda bisa melakukannya menggunakan array:

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

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

Berikut opsi lain yang hanya menggunakan 1 variabel dan masih menerapkan nilai default ''saat mmberada di luar rentang.

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

Anda bisa menulisnya sebagai ekspresi alih-alih sakelar, menggunakan operator bersyarat:

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

Jika Anda belum pernah melihat operator bersyarat yang dirantai sebelumnya, ini mungkin tampak lebih sulit untuk dibaca pada awalnya. Menulisnya sebagai ekspresi membuat satu aspek lebih mudah dilihat daripada kode aslinya; jelas bahwa maksud kode adalah untuk memberikan nilai ke variabel month.

Berdasarkan jawaban Cupawn Tae sebelumnya, saya akan mempersingkatnya menjadi:

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

Atau, ya, saya menghargai, kurang terbaca:

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

Seperti @vidriduch, saya ingin menggarisbawahi pentingnya i20y ("kemampuan internasional") kode dalam konteks saat ini dan menyarankan solusi ringkas dan kuat berikut bersama dengan uji kesatuan.

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

Saya mencoba untuk sedekat mungkin dengan pertanyaan asli, yaitu mengubah angka 1 menjadi 12 menjadi nama bulan, tidak hanya untuk satu kasus khusus, tetapi kembali undefinedjika ada argumen yang tidak valid, menggunakan beberapa kritik yang ditambahkan sebelumnya ke dan isi dari yang lain jawaban. (Perubahan dari undefinedmenjadi ''sepele, jika diperlukan pencocokan tepat .)

Saya akan mencari solusi wasmoo , tetapi sesuaikan seperti ini:

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

Ini adalah kode yang persis sama, sungguh, tetapi berlekuk berbeda, yang membuatnya lebih mudah dibaca.