Home/ Questions/Q 8670385
In Process

The Archive Base Latest Questions

Editorial Team
  • 0
Asked: 2026-06-12T18:41:43+00:00

I have the feeling I’m doing dirty code, although it works fine… How could

  • 0

I have the feeling I’m doing dirty code, although it works fine…

How could I replace those IF statement by regular JQuery selectors in such case:

see code + fiddler

<html>
<body>
    <a href="#b" class="cssPauseAll" historyID="12">Pause All</a><br/>

        <a href="#b" itemHistoryID="12" ContentID="45">AA</a><br/>
        <a href="#b" itemHistoryID="12" ContentID="14">BB</a><br/>
        <a href="#b" itemHistoryID="12" ContentID="78">AA</a><br/>  

    <a href="#b" class="cssPauseAll" historyID="13">Pause All</a><br/>

        <a href="#b" itemHistoryID="13" ContentID="45">BB</a><br/>
        <a href="#b" itemHistoryID="13" ContentID="14">BB</a><br/>
        <a href="#b" itemHistoryID="13" ContentID="78">CC</a><br/> 
</body>    
</html>​

$(document).ready(function () {

    $(document).on("click","a.cssPauseAll[historyID]", function () {
        var historyID = $(this).attr("historyID");
        //alert("historyID from PauseAll  " + $(this).attr("historyID"));
        $(document).find("[itemHistoryID]").each(function() {

            var itemHistoryID = $(this).attr("itemHistoryID");

            if(historyID == itemHistoryID)
            {
                alert($(this).attr("ContentID"));
            }
        });

    });

});​

Fiddler

Share

Leave an answer

You must login to add an answer.

Need An Account,

1 Answer

  1. Editorial Team

    You can concatenate the strings:

    $("a[itemHistoryID='" + historyID + "']");

    http://jsfiddle.net/FBCBP/

    Please note that there is no need to select the document object and use the find method. You can select the element using $ or jQuery directly and also using attribute selector alone is very slow, you can use element selector before attribute selector(in this casea).

    • 0