関数（..）throws 'eval is evil'メッセージ

Question

私はevalを使用していないので、Crockfordが次の点でどのような問題があるのか​​分かりません。次の問題を解決するためのより良いアプローチがありますか、これは私が無視する必要があるものです（改善のための領域があれば私のソリューションを完璧に/改善することが望ましい）。

私はいくつかのピクセルトラッキングを使用しています。この場合、クライアントはJS関数を、サイトからリダイレクトされるHTMLイメージタグのonclickプロパティにバインドしています。画像のイベントリスナーの倍数で競合状態に陥ることなく、確実にクリックを追跡する必要があります。戦略は、実行時にイベントをオーバーライドし、自分の関数でイベントをコピーして実行することです。これは私が制御しないサイトに適用されており、変更することはできません。だから、解決策のようなものになります。

... 
func = Function(img.attr('onclick')); 
... 
img.attr('onclick', ''); 
... //some custom tracking code 
func.call(this); 

をしてJSLintチェッカーがeval is evilエラーがスローされます。

hrefアクションの周りの複数のイベントの競合状態を回避する良い方法はありますか？

Answer 1

それは文字列し、それをFunctionを構築するなど、HTMLの属性として指定されたとして、あなたは、コールバック関数を求めているので、あなたは、暗黙的にevalを使用しています。

だけではなく、img.onclickプロパティを使用して、あなたが直接、その後.callをブラウザがそのあなたができる属性から構築された機能を取得します：まだ

var func = img.onclick; // access already compiled function 
img.onclick = null;  // property change updates the attribute too 

... // some custom tracking code 

func.call(img, ev);  // call the original function 

以上：

(function(el) { 
    var old = el.onclick; 
    el.onclick = function() { 
     // do my stuff 
     .. 
     // invoke the old handler with the same parameters 
     old.apply(this, arguments); 
    } 
})(img); 

この後者の方法の利点は、2倍である：

1. は、それが新しいグローバル変数を作成していない - すべてがそれはあなたの置換機能

Answer 2

var oldClick = myImg.onclick; 
myImg.onclick = function(evt){ 
    // Put you own code here 
    return oldClick.call(this, evt); 
};