Javascript Bookmarklet在单击时意外打开新页面

Javascript Bookmarklet在单击时意外打开新页面,javascript,onclick,bookmarklet,Javascript,Onclick,Bookmarklet,因此,我制作了一个bookmarklet来折叠表中的每一行(第一行除外),然后使第一行切换/显示表的其余部分: javascript:(function(){ function open(tableid){ console.log('open'); for (j=1;j<table[tableid].rows.length;j++){ if(table[tableid].rows[j].style.display == 'none' ){

因此,我制作了一个bookmarklet来折叠表中的每一行(第一行除外),然后使第一行切换/显示表的其余部分:

javascript:(function(){

function open(tableid){
    console.log('open');
    for (j=1;j<table[tableid].rows.length;j++){
        if(table[tableid].rows[j].style.display == 'none' ){
         table[tableid].rows[j].style.display = '';
         } else if(table[tableid].rows[j].style.display == ''){
         table[tableid].rows[j].style.display = 'none';
         }
    }
}
var table = document.getElementsByTagName("table");
for (i=0;i<table.length;i++){
 for (j=0;j<table[i].rows.length;j++){
  if( j != 0){
   table[i].rows[j].style.display = 'none';

   }
   if( j == 0){
    table[i].rows[j].onclick = new Function("open("+i+")");
   }  
  }
 }
})();
javascript:(函数(){
函数打开(tableid){
console.log('open');

对于(j=1;j,函数构造函数创建一个全局函数,而不是在函数中以词汇形式创建。因此,它无法访问名为open的本地函数,因此它使用窗口的open方法

根据我的测试,无论是从bookmarklet还是从页面本身运行,您的代码都应该始终引用全局打开方法

看看这个例子:

(function(){
  function giveme5(x) { return 5;}

  var func = new Function("return giveme5()");
  // should output 5, but we get an error, giveme5 is not defined
  console.log( func() );
})()
这里有一个可能解决你问题的办法

(function(){
    // Binds an argument to a method, this could be much better, but serves our 
    // purpose here
    function bindArg(fun, arg) {
        return function() {
            fun.call(this, arg);
        }
    }

    function toggleTable(table) {
        var rows = table.rows;
         for (var j=1; j<rows.length; j++){
             if(rows[j].style.display == 'none' ){
                 rows[j].style.display = '';
             } else if(rows[j].style.display == ''){
                 rows[j].style.display = 'none';
             }
         }
    }

    var tableList = document.getElementsByTagName("table");
    for ( var i=0; i<tableList.length; i++){
        var table = tableList[i];
        for ( var j=0; j < table.rows.length;j++){
            if( j != 0){
                table.rows[j].style.display = 'none';
            } else {
                table.rows[j].onclick = bindArg(toggleTable, table);
            }
        }
    }
})();
(函数(){
//将一个参数绑定到一个方法,这可能会更好,但服务于我们的
//这里的目的
函数bindArg(fun,arg){
返回函数(){
有趣。打电话(这个,arg);
}
}
功能切换表(表){
var rows=table.rows;
对于(var j=1;j