最新消息:雨落星辰是一个专注网站SEO优化、网站SEO诊断、搜索引擎研究、网络营销推广、网站策划运营及站长类的自媒体原创博客

javascript - c.replace is not a function - Stack Overflow

programmeradmin3浏览0评论

Hi everyone,

Actually, i got "c.replace is not a function" while i was trying to delete some DOM elements and..i don't understand.

i'd like to delete some

  • tags from the DOM and so, i did it :

    var liste=document.getElementById("tabs").getElementsByTagName("li");
    
        for(i=0;i<liste.length;i++)
        {
            if(liste[i].id==2)
            {
                $("#tabs").detach(liste[i]);
            }
        }
    

    I tried .detach and .remove but it's the same. My version of jQuery is 1.7.1.min.js.

    Thanks for help.

    Hi everyone,

    Actually, i got "c.replace is not a function" while i was trying to delete some DOM elements and..i don't understand.

    i'd like to delete some

  • tags from the DOM and so, i did it :

    var liste=document.getElementById("tabs").getElementsByTagName("li");
    
        for(i=0;i<liste.length;i++)
        {
            if(liste[i].id==2)
            {
                $("#tabs").detach(liste[i]);
            }
        }
    

    I tried .detach and .remove but it's the same. My version of jQuery is 1.7.1.min.js.

    Thanks for help.

    Share Improve this question asked Apr 20, 2012 at 13:01 Thomas SaldiThomas Saldi 911 gold badge2 silver badges9 bronze badges
    Add a ment  | 

    4 Answers 4

    Reset to default 5

    order of iteration on a NodeLIst

    Doing forward iteration of a NodeList that is being modified when you remove an element can be an issue. Iterate in reverse when removing elements from the DOM.


    misuse of detach()

    Also, the arguments to .detach() do not perform a nested find, but rather act as a filter on the existing element(s) in the jQuery object, and should be passed a string. It seems that you actually want to detach the li, which would mean that you'd need to call .detach() on the li itself...

    var liste=document.getElementById("tabs").getElementsByTagName("li");
    
    var i = liste.length
    
    while(i--) {
        if(liste[i].id==2) {
            $(liste[i]).detach();
        }
    }
    

    remove() may be preferred

    Keep in mind that if you use .detach(), any jQuery data is retained. If you have no further use for the element, you should be using .remove() instead.

    // ...
        $(liste[i]).remove(); // clean up all data
    

    code reduction

    Finally, since you're using jQuery, you could just do all this in the selector...

    $('#tabs li[id=2]').remove(); // or .detach() if needed
    

    valid id attributes

    Keep these items in mind with respect to IDs...

    • It's invalid to have duplicate IDs on a page
    • It's invalid in HTML4 to have an ID that starts with a number

    In the selector above, I used the attribute-equals filter, so it'll work, but you should really be using valid HTML to avoid problems elsewhere.

    liste is not (yet) a jQuery object. use $(liste[i])

    or use

    var liste= $('#tabs li');
    

    Maybe I'm missing something, but is the id suppose to match the number 2.

    var liste=document.getElementById("tabs").getElementsByTagName("li");
    
    for(i=0;i<liste.length;i++) {
        if(liste[i].id==2) {
           $(liste[i]).detach();
        }
    }
    

    Since you are already using jQuery, why not just do:

    $("li", "#tabs").filter("#2").detach();
    
    var two = document.getElementById('2');
    two.parentNode.removeChild(two);
    
  • 发布评论

    评论列表(0)

    1. 暂无评论