Create a jQuery object collection from separate jQuery objects

Question

$.fn.sortByDepth = function() {
    var ar = [];
    var result = $([]);

    $(this).each(function() {
        ar.push({length: $(this).parents().length, elmt: $(this)});
    });
    ar.sort(function(a,b) {
        return b.length - a.length;
    });
    for (var i=0; i<ar.length; i++) {
        result.add(ar[i].elmt);
    };
    alert(result.length);
    return result;
};

In this function I try to create a jQuery collection from separate jQuery object. How can i do that ?

The code below doesn't work:

result.add(ar[i].elmt);

The jsfiddle: http://jsfiddle.net/hze3M/14/

Matt Ball · Accepted Answer · 2011-04-22 16:08:26Z

18

.add() returns a new jQuery object. Change this line:

result.add(ar[i].elmt);

to this:

result = result.add(ar[i].elmt);

This still won't work, though

As of jQuery 1.4 the results from .add() will always be returned in document order (rather than a simple concatenation).

So you just use a vanilla JS array, push() the sorted elements into it, and then $() the whole thing.

Other code cleanup:

$.fn.sortByDepth = function() {
    var ar = this.map(function() {
            return {length: $(this).parents().length, elt: this}
        }).get(),
        result = [],
        i = ar.length;


    ar.sort(function(a, b) {
        return a.length - b.length;
    });

    while (i--) {
        result.push(ar[i].elt);
    }
    return $(result);
};


var x = $('input').sortByDepth().map(function() {
    return this.id;
}).get().join(' - ');

$('#selection').text(x);

http://jsfiddle.net/mattball/AqUQH/.

edited Apr 22, 2011 at 16:08

answered Apr 22, 2011 at 15:52

Matt Ball

361k102 gold badges655 silver badges725 bronze badges

Sign up to request clarification or add additional context in comments.

4 Comments

alexl Over a year ago

thanks for your code cleanup i apreciate. Rocket was faster for the answer that's why i accept that one.

Matt Ball Over a year ago

@alexl: yes, he was faster, but the code still won't work (see my latest edit).

alexl Over a year ago

arg, so all the code it's pointless if the add method reorder all the objects ...

alexl Over a year ago

nevermind didn't seen the last update. I didn't known you can create a jquery collection directly from the array good to see that.

gen_Eric · Accepted Answer · 2011-04-22 15:51:59Z

4

.add returns a new object, and does not modify the old one.

Try changing:

result.add(ar[i].elmt);

To:

result = result.add(ar[i].elmt);

Also, $([]) is unneeded, you can just do var result = $();

answered Apr 22, 2011 at 15:51

gen_Eric

228k42 gold badges304 silver badges343 bronze badges

Collectives™ on Stack Overflow

Create a jQuery object collection from separate jQuery objects

2 Answers 2

This still won't work, though

4 Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

This still won't work, though

4 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related