Question

我正在使用API，它以奇怪的格式向我发送数据。或者至少对我而言。数据是商店的营业时间，我希望在前端以我需要的方式匹配它们（应用程序是用Angular编写的，使用两个API，所以我需要让API使用我的逻辑，我不想适应他们的格式）

这是数据来自API的格式：

var operatingHoursArray = [
    {Weds: true, End: "17:00", Start: "09:00"},
    {Tue: true, End: "17:00", Start: "09:00"},
    {Thur: true, End: "17:00", Start: "09:00"},
    {Sun: false, End: "", Start: ""},
    {Sat: true, End: "17:00", Start: "09:00"},
    {Mon: true, End: "17:00", Start: "09:00"},
    {Fri: true, End: "17:00", Start: "09:00"}
]

奇怪的结构IMO，我本来希望将本周的日子作为对象，然后将开放和关闭的时间藏在其中。我的应用程序（AngularJS）要求数据采用以下格式：

var formattedHours = {
    Sunday: 'Closed',
    Monday: 'Closed',
    Tuesday: 'Closed',
    Wednesday: 'Closed',
    Thursday: 'Closed',
    Friday: 'Closed',
    Saturday: 'Closed'
};

Times默认为'Closed'，这是我用来匹配星期几的代码和我需要的格式：

var daysOfWeek = [
    { sform: 'Mon', lform: 'Monday' },
    { sform: 'Tue', lform: 'Tuesday' },
    { sform: 'Weds', lform: 'Wednesday' },
    { sform: 'Thur', lform: 'Thursday' },
    { sform: 'Fri', lform: 'Friday' },
    { sform: 'Sat', lform: 'Saturday' },
    { sform: 'Sun', lform: 'Sunday' }
];

// Loop through the operating hours for the dealer
for (var i = operatingHoursArray.length - 1; i >= 0; i--) {
    // Loop through the property names for each day, getting the first property name (the day of week)
    for (property in operatingHoursArray[i]) {
        // Loop through the days of the week
        for (var v = daysOfWeek.length - 1; v >= 0; v--) {
            // If the day of the week (array) matches the property name, get the details
            if(daysOfWeek[v].sform == property && operatingHoursArray[i][property] === true) {
                formattedHours[daysOfWeek[v].lform] = operatingHoursArray[i].Start + ' - ' + operatingHoursArray[i].End;
            }
        };
        break; // Forces loop to stop after first property
    }
};

这真的很快变得非常讨厌，但凭借我的知识（noob级别），我不确定如何使这更有效率。它正在满足我的需求，但是有更好的方法来编写代码吗？目前它必须运行49次才能检查一周中的每一天。同样，一些商店不提供7天的工作时间，而只提供他们开放的时间。我无法更改formattedHours的结构，因为其他API依赖于相同的结构。

Answer 1

将daysOfWeek更改为对象：

var daysOfWeek = {
    Mon: "Monday",
    Tue: "Tuesday",
    ...
};

然后，您只需访问daysOfWeek[property]，而不是通过循环搜索daysOfWeek。

Answer 2

从短到长日的名称开始。

var daysOfWeek = {
    'Mon': 'Monday',
    'Tue': 'Tuesday',
    'Weds': 'Wednesday',
    'Thur': 'Thursday',
    'Fri': 'Friday',
    'Sat': 'Saturday',
    'Sun': 'Sunday'
];

for (var i = operatingHoursArray.length - 1; i >= 0; i--) {
    // Loop through the property names for each day, getting the first property name (the day of week)
    for (property in operatingHoursArray[i]) {
        var dow = daysOfWeek[property];
        if(dow && operatingHoursArray[i][property]) {
            formattedHours[dow] = operatingHoursArray[i].Start + ' - ' + operatingHoursArray[i].End;
            break;
        }
    }
}

Answer 3

我会带你的有序集合并迭代它。对于每个项目，我会循环operatingHoursArray，直到找到当前daysOfWeek的一个，然后构建您的字符串。

使用.forEach()和.every()清理一下。 .every()用于内部循环，以便我们可以在找到匹配后暂停迭代。

var daysOfWeek = [
    { sform: 'Mon', lform: 'Monday' },
    { sform: 'Tue', lform: 'Tuesday' },
    { sform: 'Weds', lform: 'Wednesday' },
    { sform: 'Thur', lform: 'Thursday' },
    { sform: 'Fri', lform: 'Friday' },
    { sform: 'Sat', lform: 'Saturday' },
    { sform: 'Sun', lform: 'Sunday' }
];

daysOfWeek.forEach(function(day) {
    operatingHoursArray.every(function(item) {
         if (item.hasOwnProperty(day.sform)) {
             day.formatted = item.Start + ' - ' + item.End;
             day.start = item.Start;
             day.end = item.End;
             day.open = item[day.sform]
             return false;
         }
    });
});

请注意，我正在将格式化字符串以及原始数据添加到daysOfWeek个对象中。

这样您就可以保留您定义的顺序。因此，现在您可以按索引查找，并以可靠的顺序进行迭代，这在使用for-in时是不可能的。

JavaScript：优化循环内循环内的循环

3 个答案: