Simple array of objects sort is not working properly

Question

Array is not sorting properly. I am sorting the objects based on id , I have written in correct way only

but it is not sorting at all.

 list = list.sort((item1, item2) => {
        return item1.id > item2.id ? 1 : -1;
      });

the sample array of objects are like this

0: {baseCalendar: {…}, id: "0"}
1: {baseCalendar: {…}, id: "1"}
2: {baseCalendar: {…}, id: "10"}
3: {baseCalendar: {…}, id: "11"}
4: {baseCalendar: {…}, id: "12"}
5: {baseCalendar: {…}, id: "2"}
6: {baseCalendar: {…}, id: "4"}
7: {baseCalendar: {…}, id: "5"}

use parseInt(). Also comparisons should return -1, 0, or 1. — John
– John, Commented Apr 19, 2021 at 13:03
In case the first comment is not clear enough: you're sorting strings instead of integers — Nir Alfasi
– Nir Alfasi, Commented Apr 19, 2021 at 13:04

Andrew Ymaz · Accepted Answer · 2021-04-19 13:09:17Z

3

The issue is that your ids are Strings, not Numbers, but you are comparing them as Numbers.

You have to convert them to Numbers, while you are comparing them, for examples as:

list.sort((item1, item2) => { return +item1.id > +item2.id ? 1 : -1; });

Or using parseInt(item1.id, 10) and parseInt(item2.id, 10) as proposed above.

edited Apr 19, 2021 at 13:09

answered Apr 19, 2021 at 13:06

Andrew Ymaz

2,2331 gold badge10 silver badges12 bronze badges

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

1 Comment

georg Over a year ago

could be just list.sort((item1, item2) => item1.id - item2.id)

Collectives™ on Stack Overflow

Simple array of objects sort is not working properly

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related