0

我正在为数百个类似页面编写 Greasemonkey 脚本。

并且想要在 $("#cont > h2")

所以我尝试了:

var 1stgroup = new Array(40)
1stgroup [0] = $("#cont > h2:contains('AA')")
1stgroup [1] = $("#cont > h2:contains('AB')")
1stgroup [2] = $("#cont > h2:contains('AC')")
...
1stgroup [39] = $("#cont > h2:contains('BN')")

for (i =0; i < 40 ; i++) {
    if (1stgroup.text())       {
    //do something here
    };
};

而不是第2~7组。

它有效,但看起来多余......

我该如何优化它?

对不起我的英语不好,感谢每一个回复。

4

1 回答 1

4
  • DOM 遍历是 JavaScript 中最慢的部分,因此尽可能少地遍历 DOM 是有意义的。
  • :contains()是一个 Sizzle 扩展,不是 CSS3 的一部分,因此没有被浏览器优化。它会很慢。

考虑到这一点,我会做这样的事情;

var matches = ['AA', 'AB', 'AC'];

$('#cont > h2').filter(function () {
    var text = $(this).text();

    for (var i=0;i<matches.length;i++) {
        if (text.indexOf(matches[i]) !== -1) {
            return true;
        }
    }

    return false;
}).each(function () {
    // do something with each of these
});

您可以这样做的另一件事是对matches数组进行排序,然后使用比线性搜索更智能的搜索算法(例如二进制搜索)来检测文本是否匹配。

于 2012-07-05T13:22:16.270 回答