JavascriptをjQueryの.each機能

Question

における文の構文（JavaScriptのjQueryのに非常に新しい）

はAN if文、具体的.each機能、その他の機能のコレクションを含めることができる場合は？はいの場合、実装に何か問題がありますか？

var string2 = "domain"; 
var mainpath = (window.location.href); 

if (mainpath.indexOf(string2)) { 

    $("#topnav a").each(function(){       
     var thishref = $(this).attr('href');  
     $(this).attr({href: thishref + '?lnkid=tnav'}); 
    }); 

    $("#nav a").each(function(){        
     var thishref = $(this).attr('href');  
     $(this).attr({href: thishref + '?lnkid=pnav'}); 
    }); 

} 

私は何をしたいページが（むしろCMSの開発スペース内部よりも）自分のドメインにある場合にのみ、if文の内部でコードを実行されます。 -お問い合わせ

Answer 1

ことができることを確認：それは簡単に以下のURLでだまさすることができます。あなたは基本的に正しいことをしています。提案：

が条件ならば、あなたと明示して、その条件付きでコードを実行するためにifを使用して完全に有効

if (mainPath.indexOf(string2) !== -1) {... 

} 

Answer 2

はい、何も問題ありません。

http://www.other.tld/?domain

Answer 3

ような何かを行います。あなたの状態に問題があるようですけど：

if (mainpath.indexOf(string2)) 

メインパス内の文字列2のインデックスは、実行からあなたのコードを妨げる0することができ、それはあなたが望むものは確かではありません。

このようなむしろテスト：提案として

if (mainpath.indexOf > -1) ... 

Answer 4

に、あなたはこのような各リンクにクエリを追加することができます。

$("#topnav a").each(function(){       
    this.search = 'lnkid=tnav';  
}); 

のhrefを取得関わる遠回しの必要はありません属性を追加し、それを元に戻します。

デモ：あなたの質問にhttp://jsfiddle.net/3ZpMz/1/