Sorting JSON in JavaScript

Question

I know similar question is asked before and I referred to this one here: sort json object in javascript but I still cannot find answer to my question. So here I go. I have an JSON object structure as below:

 [
  {
    "toothNumber": "01",
    "name": "John"
  },
  {
    "toothNumber": "18",
    "name": "John"
  },
  {
    "toothNumber": "19",
    "name": "John"
  },
  {
    "toothNumber": "17",
    "name": "John"
  },
  {
    "toothNumber": "01,32",
    "name": "John"
  },
  {
    "toothNumber": "25,32",
    "name": "John"
  },
  {
    "toothNumber": "",
    "name": "John"
  },
  {
    "toothNumber": "15",
    "name": "John"
  }
]

When I use the below code to sort, I do not get expected results:

json.sort(function(a, b){
    return a.toothNumber - b.toothNumber;
});

Below, is the actual result which is not the one I expected. Any help will be appreciated.

Actual Result:

[
  {
    "toothNumber": "",
    "name": "John"
  },
  {
    "toothNumber": "01",
    "name": "John"
  },
  {
    "toothNumber": "15",
    "name": "John"
  },
  {
    "toothNumber": "17",
    "name": "John"
  },
  {
    "toothNumber": "18",
    "name": "John"
  },
  {
    "toothNumber": "19",
    "name": "John"
  },
  {
    "toothNumber": "01,32",
    "name": "John"
  },
  {
    "toothNumber": "25,32",
    "name": "John"
  }
]

Expected Result:

[
    {
        "toothNumber": "",
        "name": "John"
    },
    {
        "toothNumber": "01",
        "name": "John"
    },
    {
        "toothNumber": "01,32",
        "name": "John"
    },
    {
        "toothNumber": "15",
        "name": "John"
    },
    {
        "toothNumber": "17",
        "name": "John"
    },
    {
        "toothNumber": "18",
        "name": "John"
    },
    {
        "toothNumber": "19",
        "name": "John"
    },
    {
        "toothNumber": "25,32",
        "name": "John"
    }
]

You are subtracting strings not numbers. Convert to numbers first. — grandouassou
– grandouassou, Commented Oct 28, 2021 at 22:41
Not sure how you expect this to work. The toothnumber is a string first of all, and it's a multi-valued string in certain items. What should be the result of "25,32" - "18", for example? — jarmod
– jarmod, Commented Oct 28, 2021 at 22:41
@jarmod those are valid values as saved in the client database. they save multiple tooth numbers comma separated if they are all part of one claim. Do I agree, no but this is the data I have — DAK
– DAK, Commented Oct 28, 2021 at 22:52
No problem with the representation at all. I'm just highlighting the fact that an implementation that results in "25,32" - "18" makes little sense. — jarmod
– jarmod, Commented Oct 28, 2021 at 23:27

ProDec · Accepted Answer · 2021-10-28 22:41:56Z

Sort with string, instead of number comparison.

const json = [  {    "toothNumber": "01",    "name": "John"  },  {    "toothNumber": "18",    "name": "John"  },  {    "toothNumber": "19",    "name": "John"  },  {    "toothNumber": "17",    "name": "John"  },  {    "toothNumber": "01,32",    "name": "John"  },  {    "toothNumber": "25,32",    "name": "John"  },  {    "toothNumber": "",    "name": "John"  },  {    "toothNumber": "15",    "name": "John"  }];

json.sort(function(a, b){
    return a.toothNumber > b.toothNumber ? 1 : (a.toothNumber === b.toothNumber ? 0 : -1 );
});

console.log(json);

user3133 · Accepted Answer · 2021-10-28 22:49:59Z

0

You could just do this

const json = [  {    "toothNumber": "01",    "name": "John"  },  {    "toothNumber": "18",    "name": "John"  },  {    "toothNumber": "19",    "name": "John"  },  {    "toothNumber": "17",    "name": "John"  },  {    "toothNumber": "01,32",    "name": "John"  },  {    "toothNumber": "25,32",    "name": "John"  },  {    "toothNumber": "",    "name": "John"  },  {    "toothNumber": "15",    "name": "John"  }];

json.sort(function(a, b) {
   return a.toothNumber.split(",")[0] - b.toothNumber.split(",")[0];
});

console.log(json)

edited Oct 28, 2021 at 22:49

answered Oct 28, 2021 at 22:45

user3133

6125 silver badges13 bronze badges

2 Comments

Spectric Over a year ago

The parseInt is redundant.

user3133 Over a year ago

@Spectric yeah it was left when I changed it to work with a snippet. But anyway, split returns string array, so parseInt would be advisable

Collectives™ on Stack Overflow

Sorting JSON in JavaScript

2 Answers 2

Comments

2 Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

2 Comments

Linked

Related