DOMの重複IDをチェックするためのjQuery


106

ASP.NET MVCでアプリケーションを作成しています。従来のASP.NETとは対照的に、生成されたページですべてのIDを作成する責任ははるかに大きくなります。ASP.NETは厄介ですが、一意のIDを提供します。

重複するIDがないかドキュメントをチェックするための簡単なjQueryスクリプトを追加したい それらはDIVS、画像、チェックボックス、ボタンなどのIDです。

<div id="pnlMain"> My main panel </div>
<div id="pnlMain"> Oops we accidentally used the same ID </div> 

不注意なことをしたときに警告するだけの、セットアンドフォーゲットタイプユーティリティを探しています。

はい、私はこれをテスト中にのみ使用し、代替手段(firebugプラグインなど)も歓迎します。

回答:


214

以下は、コンソールに警告を記録します。

// Warning Duplicate IDs
$('[id]').each(function(){
  var ids = $('[id="'+this.id+'"]');
  if(ids.length>1 && ids[0]==this)
    console.warn('Multiple IDs #'+this.id);
});

完璧!ありがとう!IDが重複している3つの場所がすでに見つかりました。この問題に対するほとんどの人々の解決策は「firebug」または「htmlバリデーター」を使用することであることに少し不満を感じます。それは十分ではありません!奇妙な状況で予期しない重複をキャッチしたい。
Simon_Weaver

4
heheと私はconsole.warnをalert(...)に切り替えたので、修正する必要があります:)
Simon_Weaver

これは非常に有用で貴重であることがわかりました。特に、デバッグ中に-私はそれがフレームワーク内の標準であるべきだと思う
Simon_Weaver

6
これが機能するために必要なDOMトラバーサルの量はかなり驚くべきものです
Josh Stodola

8
非常に素晴らしい解決策ですがvar ids = $('[id=\''+this.id+'\']');、ID内のドットやその他の奇妙なもので機能するように、追加の引用符が必要です。
zidarsk8

33

このバージョンは多少高速で、ブックマークボタンにコピーしてブックマークレットにすることができます。

javascript:(function () {
  var ids = {};
  var found = false;
  $('[id]').each(function() {
    if (this.id && ids[this.id]) {
      found = true;
      console.warn('Duplicate ID #'+this.id);
    }
    ids[this.id] = 1;
  });
  if (!found) console.log('No duplicate IDs found');
})();

3
このアルゴリズムはより優れており、一致する要素ごとに1つではなく、1つのdomトラバーサルのみが必要です。受け入れられる答えでなければなりません。
m_x 2015年

1
これは、name = idの入力を持つフォームに対して誤検知を与えます。 javascript:(function () { var ids = {}; var found = false; $('[id]').each(function() { var id = this.getAttribute('id'); if (id && ids[id]) { found = true; console.warn('Duplicate ID #'+id); } ids[id] = 1; }); if (!found) console.log('No duplicate IDs found'); })(); 方が良いだろう。
alpo 2017

14

大きなページがあるので、スクリプトの実行が遅すぎて完了しません(複数の「スクリプトの続行」メッセージ)。これは正常に動作します。

(function () {
    var elms = document.getElementsByTagName("*"), i, len, ids = {}, id;
    for (i = 0, len = elms.length; i < len; i += 1) {
        id = elms[i].id || null;
        if (id) {
            ids[id] =  ids.hasOwnProperty(id) ? ids[id] +=1 : 0;
        }
    }
    for (id in ids) {
        if (ids.hasOwnProperty(id)) {
            if (ids[id]) {
                console.warn("Multiple IDs #" + id);
            }
        }
    }
}());

すごい!ありがとう。私はこれを本番環境で実行していることをしばしば忘れており、本当に今はそれを最適化する必要があります-またはデバッグ設定を追加してオン/オフを切り替えます!
Simon_Weaver

私は絶えず異なる構成でスクリプトを組み合わせることに取り組んでおり、これは確かに多くの助けになります。ありがとう:)
Andy Gee

プレーンなJavaScriptソリューションの場合は+1。重複したIDを見つけた後$x("//*[@id='duplicated-id']")、コンソールでXPath式()を使用して、重複したIDを持つ要素をクエリしました。
カシオモリン


8

なぜあなたはあなたのhtmlを検証しないのですか?

二重IDは許可されておらず、通常は解析エラーが発生します。


2
これにはどのようなオプションがありますか?
Simon_Weaver 2009年

また、FFでは、バリデーターがあるツールの下でWeb開発者ツールバーを使用します
IEnumerator

4
jquery uiからのダイアログのようなウィジェットを操作する場合、ダイアログの作成後にクリーンアップしないと、DOMに重複が発生することがよくあります。
guido

4

重複を見つけるもう1つの方法ですが、これによりエラーのクラスが追加されるため、赤いテキストが表示されます。

// waits for document load then highlights any duplicate element id's
$(function(){ highlight_duplicates();});

function highlight_duplicates() {
  // add errors when duplicate element id's exist
  $('[id]').each(function(){ // iterate all id's on the page
    var elements_with_specified_id = $('[id='+this.id+']');
    if(elements_with_specified_id.length>1){
      elements_with_specified_id.addClass('error');
    }
  });


  // update flash area when warning or errors are present
  var number_of_errors = $('.error').length;
  if(number_of_errors > 0)
    $('#notice').append('<p class="error">The '+number_of_errors+
      ' items below in Red have identical ids.  Please remove one of the items from its associated report!</p>');
}

それはちょっとクール!ありがとう。私は実際に元の受け入れられた答えが計り知れないと思いました。たくさんのものを捕まえて、おそらく何時間も節約できました!
Simon_Weaver 2009年

かっこいいですが、コンソール機能だけを使用して残りの作業を行わないのはなぜですか。ロジックとプレゼンテーションの分離など
ウィルモーガン、

3

ES6で書き直された上位のjQuery回答:

  [...document.querySelectorAll('[id]')].forEach(el => {
    const dups = document.querySelectorAll(`[id="${el.id}"]`);

    if (dups.length > 1 && dups[0] === el) {
      console.error(`Duplicate IDs #${el.id}`, ...dups);
    }
  });

2

これはトリックを行うかもしれませんそれは重複を持つ要素のすべてのIDを警告します。

<script src="https://cdnjs.cloudflare.com/ajax/libs/jquery/3.3.1/jquery.min.js"></script>
<html>
    	<head>
    		<script type="text/javascript" src="jquery-1.3.1.min.js"></script>
    		<script type="text/javascript">
    			function findDupes()
    			{
    			  var all = $("*");
    			  for(var i = 0; i < all.length; i++)
    			  {
    			      if (all[i].id.length > 0 && $("[id='" + all[i].id + "']").length > 1) alert(all[i].id);
    			  }
    			}
    		</script>
    	</head>
    	<body onload="findDupes()">
    		<div id="s"></div>
    		<div id="f"></div>
    		<div id="g"></div>
    		<div id="h"></div>
    		<div id="d"></div>
    		<div id="j"></div>
    		<div id="k"></div>
    		<div id="l"></div>
    		<div id="d"></div>
    		<div id="e"></div>
    	</body>
    </html>


1

コンソールに実際の要素を吐き出すので、私はこれが好きです。何が起こっているのかを簡単に調査できます。

function CheckForDuplicateIds() {
var ids = {};
var duplicates = [];

$("[id]").each(function() {
    var thisId = $(this).attr("id");
    if (ids[thisId] == null) {
        ids[thisId] = true;
    } else {
        if (ids[thisId] == true) {
            duplicates.push(thisId);
            ids[thisId] = false;
        }
    }
});
if (duplicates.length > 0) {
    console.log("=======================================================");
    console.log("The following " + duplicates.length + " ids are used by multiple DOM elements:");
    console.log("=======================================================");
    $(duplicates).each(function() {
        console.warn("Elements with an id of " + this + ":");
        $("[id='" + this + "']").each(function() {
            console.log(this);
        });
        console.log("");
    });
} else {
    console.log("No duplicate ids were found.");
}
return "Duplicate ID check complete.";

}


新しいHTMLがページに追加されたときにレプリケートされたIDを検出できたため、提案されたChrome拡張HTMLバリデーターが機能しない場合、この関数は非常に役立ちました。
Giselle

1

存在する場合は、重複するIDのリストをコンソールに出力するこのソリューションを使用できます。

DOMが読み込まれた後、コンソールで直接コードを実行(コピー/貼り付け)でき、jQueryのような追加の依存関係は必要ありません。

これを使用して、HTMLマークアップのエラーの可能性をすばやく見つけることができます。

    (function (document) {
        var elms = document.body.querySelectorAll('*[id]'),
            ids = [];
        for (var i = 0, len = elms.length; i < len; i++) {
            if (ids.indexOf(elms[i].id) === -1) {
                ids.push(elms[i].id);
            } else {
                console.log('Multiple IDs #' + elms[i].id);
            }
        }
    })(document);

例:

https://jsbin.com/cigusegube/edit?html,console,output

(ここでは、bodyタグを閉じる前にコードが追加されています)


0

ページ内またはページ全体で重複するIDを検索する特定の要素を検査できる関数を作成しました。

function duplicatedIDs(container) {

    var $container  = container ? $(container) : $('body'),
        elements = {},
        duplicatedIDs = 0;
        totalIDs = 0;

    $container.find('[ID]').each(function(){
        var element = this;

        if(elements[element.id]){
            elements[element.id].push(element);
        } else  {
            elements[element.id] = [element];
        }
        totalIDs += 1;

    });

    for( var k in elements ){
        if(elements[k].length > 1){
            console.warn('######################################')
            console.warn('        ' + k )
            console.warn('######################################')
            console.log(elements[k]);
            console.log('---------------------------------------');
            duplicatedIDs += elements[k].length
        }
    }
    console.info('totalIDs', totalIDs);
    console.error('duplicatedIDs', duplicatedIDs);
}

duplicatedIDs('#element'); //find duplicated ids under that element
duplicatedIDs(); // entire page
弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.