How to sort array of object by key?

Question

I tried this method:

if (field == 'age') {

      if (this.sortedAge) {
        this.fltUsers.sort(function (a, b) {
          if (b.totalHours > a.totalHours) {
            return 1;
          }
        });

        this.sortedAge = false;

      } else {

        this.sortedAge = true;
        this.fltUsers.sort(function (a, b) {
          if (b.totalHours < a.totalHours) {
            return 1;
          }
        });

      }
    }

So I have array of objects. Each object has property: totalHours.

I need to order this array by desc/asc this field totalHours after click.

My code does not work.

This is hard to tell without actually seeing the array, it's also most likely a duplicate. Anyway, you're only returning if something happens, you should be returning 1, -1 or 0 depending on the comparison. — adeneo
– adeneo, Commented Jan 13, 2018 at 14:38

Hoang Duc Nguyen · Accepted Answer · 2018-01-13 14:42:17Z

4

I usually follow 1-line rule for the kind of simple comparator like yours, so I would modify it like this

this.fltUsers.sort((a,b) => b.totalHours < a.totalHours ? 1 : -1);

You need at least 1 and -1 returned so that the comparator logic can work.

answered Jan 13, 2018 at 14:42

Hoang Duc Nguyen

4194 silver badges14 bronze badges

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

Comments

Nikola Andreev · Accepted Answer · 2018-01-13 14:42:05Z

2

This code must sort the array.

  function compare(a,b) {
      if (b.totalHours < a.totalHours)
        return -1;
      if (b.totalHours > a.totalHours)
        return 1;
      return 0;
  }

  objects.sort(compare);

answered Jan 13, 2018 at 14:42

Nikola Andreev

6345 silver badges18 bronze badges

Collectives™ on Stack Overflow

How to sort array of object by key?

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related