Question

var sorted = results.sort(function (a, b) {
    var nameA = "";
    var nameB = "";
    var str1 = a.date.toLowerCase() + " " + a.time;
    var str2 = b.date.toLowerCase() + " " + b.time;

    if (a.date.toLowerCase() != "" && a.time!= "") 
     nameA = Date.parse(new Date(str1))

    if (b.date.toLowerCase() != "" && b.time!= "") 
     nameB = Date.parse(new Date(str2))

    return nameA === nameB ? 0 : nameA < nameB ? -1 : 1;

});

Can anyone improve the code, if there any faults in the way i am sorting. Is there any better approach?
How can i implement http://underscorejs.org/#sortBy for the above sorting.

Answer 1

您代码中的一个主要瓶颈是您在每种操作中重新对名称 A/ nameB 进行重新比较。考虑到数组中的每个元素都可能参加多个比较, 这会有很多不必要的工作。您应该首先使用相同的算法构建排序索引, 然后用它进行排序。您也可以将排序键存储在同一对象中, 这样您就可以检查它是否在这里, 如果是的话, 不要重新计算它。

您的代码中另有两件事是绝对不必要的: .to LowerCase 与空字符串进行比较,并生成无用的 Data 对象,因为 Data.parse 已经明确指定为用字符串工作。

我制作了两种改良的版本——一种是辅助功能(更精细,更正确的意识形态)和内线——可以对某些联署材料引擎提供一些额外的加速。两者的运行速度都比原来的速度快大约3000倍:

内含 :

var sorted = results.sort(function (a, b) {
    var nameA = a.dateTimeMs
    var nameB = b.dateTimeMs
    if (!nameA && nameA !== "" && !nameA !== 0){
        if (a.date == "" || a.time == ""){
            nameA = a.dateTimeMs = ""
        } else {
            nameA = a.dateTimeMs = Date.parse(a.date.toLowerCase() + " " + a.time)
        }
    }
    if (!nameB && nameB !== "" && !nameB !== 0){
        if (b.date == "" || b.time == ""){
            nameB = b.dateTimeMs = ""
        } else {
            nameB = b.dateTimeMs = Date.parse(b.date.toLowerCase() + " " + b.time)
        }
    }

    return nameA === nameB ? 0 : nameA < nameB ? -1 : 1;
});

Answer 2

最好的办法是让您的 a 和 b 对象从一开始就计算出来。而不是在无需处理的情况下循环。或者按照 Oleg 的存储方式来保存它。

友情链接