ใน JavaScript การกลับมาจากคำสั่ง switch ถือเป็นการปฏิบัติที่ดีกว่าการใช้ตัวแบ่ง?

ตัวเลือกที่ 1 - สลับใช้การส่งคืน:

function myFunction(opt) 
{
    switch (opt) 
    {
        case 1: return "One";
        case 2: return "Two";
        case 3: return "Three";

        default: return "";
    }    
}

ตัวเลือก 2 - สลับการใช้ตัวแบ่ง:

function myFunction(opt) 
{
    var retVal = "";

    switch (opt) 
    {
        case 1: 
            retVal = "One";
            break;

        case 2: 
            retVal = "Two";
            break;

        case 3: 
            retVal = "Three";
            break;
    }

    return retVal;
}

ฉันรู้ว่าทั้งสองทำงานได้ แต่เป็นแนวปฏิบัติที่ดีที่สุดอีกข้อหนึ่งหรือไม่ ฉันมักจะชอบตัวเลือกที่ 1 - สลับใช้ผลตอบแทนที่ดีที่สุดเพราะสะอาดและเรียบง่ายขึ้น

นี่คือ jsFiddle ของตัวอย่างเฉพาะของฉันโดยใช้เทคนิคที่กล่าวถึงในความคิดเห็นของ @ ic3b3rg :

var SFAIC = {};

SFAIC.common = 
{
    masterPages: 
    {
        cs: "CS_",
        cp: "CP_"
    },

    contentPages: 
    {
        cs: "CSContent_",
        cp: "CPContent_"    
    }
};

function getElementPrefix(page) 
{
    return (page in SFAIC.common.masterPages)
        ? SFAIC.common.masterPages[page]
        : (page in SFAIC.common.contentPages)
            ? SFAIC.common.contentPages[page]
            : undefined;
}

ในการเรียกใช้ฟังก์ชั่นฉันจะทำในวิธีต่อไปนี้:

getElementPrefix(SFAIC.common.masterPages.cs);
getElementPrefix(SFAIC.common.masterPages.cp);
getElementPrefix(SFAIC.common.contentPages.cs);
getElementPrefix(SFAIC.common.contentPages.cp);

ปัญหาที่นี่คือมันส่งคืนไม่ได้กำหนดเสมอ ฉันเดาว่าเป็นเพราะมันผ่านไปในมูลค่าที่แท้จริงของวัตถุตามตัวอักษรไม่ใช่ทรัพย์สิน ฉันจะทำอย่างไรเพื่อแก้ไขปัญหานี้โดยใช้เทคนิคที่อธิบายไว้ในความคิดเห็นของ @ ic3b3rg

ตัวแบ่งจะอนุญาตให้คุณดำเนินการกับฟังก์ชันต่อไป เพียงแค่กลับออกจากสวิตช์ก็ใช้ได้ถ้านั่นคือทั้งหมดที่คุณต้องการทำในฟังก์ชั่น

ขึ้นอยู่กับว่าถ้าฟังก์ชั่นของคุณประกอบด้วยคำสั่ง switch เท่านั้นฉันคิดว่ามันดี อย่างไรก็ตามหากคุณต้องการดำเนินการอื่น ๆ ภายในฟังก์ชั่นนั้นอาจไม่ใช่ความคิดที่ดี คุณอาจต้องพิจารณาความต้องการของคุณตอนนี้กับในอนาคต หากคุณต้องการเปลี่ยนฟังก์ชั่นของคุณจากตัวเลือกที่หนึ่งเป็นตัวเลือกที่สองคุณจะต้องทำการปรับโครงสร้างใหม่อีกครั้ง

อย่างไรก็ตามในกรณีที่คำสั่ง if / else เป็นแนวปฏิบัติที่ดีที่สุดในการทำสิ่งต่อไปนี้:

var foo = "bar";

if(foo == "bar") {
    return 0;
}
else {
    return 100;
}

จากสิ่งนี้ข้อโต้แย้งอาจทำให้ตัวเลือกที่หนึ่งเป็นแนวปฏิบัติที่ดีกว่า

กล่าวโดยย่อไม่มีคำตอบที่ชัดเจนตราบใดที่รหัสของคุณเป็นไปตามมาตรฐานที่สอดคล้องกันอ่านได้และบำรุงรักษา - กล่าวคืออย่าผสมตัวเลือกและจับคู่หนึ่งและสองตลอดการสมัครของคุณนั่นคือแนวทางปฏิบัติที่ดีที่สุดที่คุณควรจะเป็น ดังต่อไปนี้