Question

我有一些代码正在为我的工作而努力。我们正在尝试遍历页面上的所有链接并自动添加onclick事件。但是，循环似乎根本不是“循环”。有人可以帮忙吗？

var ourdomainname = "ourdomain.com";


function linkallthelinks(domain) {
    var links = document.getElementsByTagName("a");
    for (i = 0; i < links.length; i++) {
        var link = links[i];
        var href = link.getAttribute("href");

        if (href.indexOf(read_today) != -1) {
            link.setAttribute('onclick', 'alert("Okay")');
        }
    }
}
//function call
linkallthelinks(ourdomainname);

Answer 1

这里缺少引号：

if(href.indexOf(read_today) != -1)

应该是：

if(href.indexOf('read_today') != -1)

总的来说，这就是你应该拥有的：

var ourdomainname = "ourdomain.com";

function linkallthelinks(domain) {
    var links = document.getElementsByTagName("a");
    for (var i = 0; i < links.length; i++) {
        var link = links[i];
        var href = link.getAttribute("href");

        if (href.indexOf('read_today') != -1) {
            link.setAttribute('onclick', 'alert("Okay")');
        }
    }
}
//function call
linkallthelinks(ourdomainname);

Answer 2

试试这个：

if(link.href.indexOf("read_today") != -1)

Javascript for循环不循环

2 个答案: