Trong JavaScript, việc trả lại câu lệnh chuyển đổi có được coi là cách thực hành tốt hơn so với sử dụng break không?

Tùy chọn 1 - chuyển đổi sử dụng return:

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

        default: return "";
    }    
}

Tùy chọn 2 - chuyển đổi bằng cách sử dụng break:

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

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

        case 2: 
            retVal = "Two";
            break;

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

    return retVal;
}

Tôi biết rằng cả hai công việc, nhưng là một trong những thực hành tốt nhất? Tôi có xu hướng thích Tùy chọn 1 - chuyển đổi bằng cách sử dụng return tốt nhất, vì nó sạch hơn và đơn giản hơn.

Dưới đây là một jsFiddle về ví dụ cụ thể của tôi bằng cách sử dụng kỹ thuật được đề cập trong các nhận xét của @ 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;
}

Để gọi hàm, tôi sẽ làm như vậy theo các cách sau:

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

Vấn đề ở đây là nó luôn trả về không xác định. Tôi đoán rằng đó là vì nó vượt qua giá trị thực của đối tượng theo nghĩa đen chứ không phải thuộc tính. Tôi sẽ làm gì để khắc phục điều này bằng cách sử dụng kỹ thuật được mô tả trong nhận xét của @ ic3b3rg ?

Một ngắt sẽ cho phép bạn tiếp tục xử lý trong chức năng. Chỉ cần quay trở lại khỏi công tắc là ổn nếu đó là tất cả những gì bạn muốn làm trong chức năng.

Nó phụ thuộc, nếu chức năng của bạn chỉ bao gồm câu lệnh switch, thì tôi nghĩ rằng nó ổn. Tuy nhiên, nếu bạn muốn thực hiện bất kỳ hoạt động nào khác trong chức năng đó, có lẽ đó không phải là một ý tưởng tuyệt vời. Bạn cũng có thể phải xem xét các yêu cầu của bạn ngay bây giờ so với trong tương lai. Nếu bạn muốn thay đổi chức năng của mình từ tùy chọn một sang tùy chọn hai, sẽ cần thêm cấu trúc lại.

Tuy nhiên, do trong câu lệnh if / other, cách tốt nhất là thực hiện như sau:

var foo = "bar";

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

Dựa trên điều này, đối số có thể được đưa ra là lựa chọn đó là thực tiễn tốt hơn.

Nói tóm lại, không có câu trả lời rõ ràng, miễn là mã của bạn tuân thủ một tiêu chuẩn nhất quán, dễ đọc, có thể duy trì - nghĩa là không trộn lẫn và kết hợp các tùy chọn một và hai trong suốt ứng dụng của bạn, đó là cách tốt nhất bạn nên làm tiếp theo.