How to sort array of list of objects in javascript?

Question

I have an array which has list of objects.

each value of array looks like :

I want to sort name : elements[ele].options.name.

this is what I am trying to do ,but not working for me

           var sortedElements = elements.sort(function (a, b) {
                return a.options.name - b.options.name
            });
            for (var ele in sortedElements) {
                if (sortedElements.hasOwnProperty(ele)) {
                    if (!sortedElements[ele].options.outOfNetworkFlag) {
                        if (sortedElements[ele].options.category === "person") {
                            assignCount++
                        }
                        if (sortedElements[ele].options.category === "unknown") {
                            unassignCount++
                        }
                    }
                    else {
                        oonCount++;
                    }

                    occupancylist += '<a href="#" class="elementLink"><li style="list-style: none" data-elementid="' + sortedElements[ele].options.elementId + '">' + sortedElements[ele].options.icon.options.html + ' ' + sortedElements[ele].options.name + '</li></a>';
                }
            }

Please suggest.

You can't subtract a string from another string. To compare strings, use `.localeCompare()`: `return a.options.name.localeCompare(b.options.name);` — Pointy, Oct 17 '18 at 13:00
@Pointy Yes you can in most cases sort on ascii. You need localeCompare if there are chars other than `[a-zA-Z0-9]` — mplungjan, Oct 17 '18 at 13:19
@mplungjan OK yes, I agree that that's often fine, but it remains true that the comparison needs to account for equal sort keys. — Pointy, Oct 17 '18 at 13:21
@mplungjan indeed, in my personal experience I don't have to deal with anything other than Latin 1, but `.localeCompare()` is just convenient and I'm pretty sure it won't do anything that seems wrong. — Pointy, Oct 17 '18 at 13:22

Saurabh Yadav · Answer 1 · 2019-11-12T09:01:00.757

You can use sort method

  var items = [
      { name: 'saurabh'},
      { name: 'hello'},
      { name: 'jsbee'},
      { name: 'abc'}
    ];

    // sort by name
    items.sort(function(a, b) {
      var nameA = a.name.toUpperCase(); // ignore upper and lowercase
      var nameB = b.name.toUpperCase(); // ignore upper and lowercase
      if (nameA < nameB) {
        return -1;
      }
      if (nameA > nameB) {
        return 1;
      }

      // names must be equal
      return 0;
    });

score 1 · Answer 2 · edited Oct 17 '18 at 13:30

1

You should just use Array.prototype.sort

let src = [{options: {name: 'bbb'}}, {options: {name: 'aaa'}}]
 
src.sort((a, b) => a.options.name > b.options.name ? 1 : -1)

console.log(src);

edited Oct 17 '18 at 13:30

mplungjan

169,008
28
173
236

answered Oct 17 '18 at 13:00

Nurbol Alpysbayev

19,522
3
54
89

3

No, this is not correct. The sort comparator has to return a **number**, not a boolean. – Pointy Oct 17 '18 at 13:00
Besides fixing the answer so it isn't wrong, this answer also needs an explanation. – Oct 17 '18 at 13:01
@Pointy sorting using ascii order does actually work – mplungjan Oct 17 '18 at 13:02
Guys please stop downvoting, fixed it already – Nurbol Alpysbayev Oct 17 '18 at 13:03
2

@mplungjan please be cautious with editing: in your (first) edit you destroyed an improvement from the author. – trincot Oct 17 '18 at 13:03
@mplungjan no it does not, not with a simple `>` comparison in general. It might work for some particular lists, but it simply cannot work in all cases. – Pointy Oct 17 '18 at 13:04
1

I agree with Pointy: don't destroy the author's improvement. – trincot Oct 17 '18 at 13:04
1

If the comparator does not return 0 for equal strings, it will confuse the sort algorithm. – Pointy Oct 17 '18 at 13:04
Not returning 0 is ok, because equal strings would be next to each other which is ok I guess – Nurbol Alpysbayev Oct 17 '18 at 13:06
And thanks to those who defended my improvements :-) – Nurbol Alpysbayev Oct 17 '18 at 13:07
@NurbolAlpysbayev I did not willingly DESTROY anything. We were just editing at the same time. Please add the SNIPPET I made for you and do with it what you want – mplungjan Oct 17 '18 at 13:16
Sir, just let's go to some next question leaving this one as it is – Nurbol Alpysbayev Oct 17 '18 at 13:19
You can add snippet if you want *now*, no one is against it :-) it's just that your first two editions overwrote my ones. – Nurbol Alpysbayev Oct 17 '18 at 13:21

score 1 · Answer 3 · answered Oct 17 '18 at 13:03

Use sort function like this:

objs.sort((a,b) => (a.options.name > b.options.name) ? 1 : ((b.options.name > a.options.name) ? -1 : 0));

Of course you can use toUpperCase() (or lower) in sort function.

objs.sort((a,b) => (a.options.name.toUpperCase() > b.options.name.toUpperCase()) ? 1 : ((b.options.name.toUpperCase() > a.options.name.toUpperCase()) ? -1 : 0));

How to sort array of list of objects in javascript?

3 Answers3