วิธีการย่อบล็อกกรณีสลับการแปลงตัวเลขเป็นชื่อเดือน

มีวิธีเขียนสิ่งนี้ในบรรทัดน้อยลง แต่ยังอ่านได้ง่ายหรือไม่?

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

ตามคำตอบนี้รับชื่อเดือนจาก Dateจาก David Storey

ลองสิ่งนี้:

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 ทำให้อ่านได้ง่ายขึ้น