Sorting JSON Object via JavaScript odd results

Question

Alright Odd results, not so much as they are expected. However I'm not sure how to over come it, I am having one of those days where every logical thing is the equivalent of a massive brain fart for me. Anyway. Lets say for the sake of ease. My array is numeric only nothing else in there.. My array ranges from 1-50 so my results upon sorting it are similar to 1, 10, 11, 12, 13.... 2, 20, 21, 22, 23... etc. When I need it to go like 1,2,3,4,5,6,7,8,9,10,11,12...

My simple little canned function is..

function sortJSONresultsByWidgetID(a,b)
{   
    if(parseInt(a.wigetID) == parseInt(b.wigetID))
    {
        return 0;
    }
    return parseInt(a.wigetID) > parseInt(b.wigetID) ? 1 : -1;
}

for reference I parseInt due to the JSON the way my JSON handles when I post it back and forth from the DB, I store the actual JSON in the DB and when passing it to the PHP it wraps quotes around the number turning them from INT to string (or from what I notice that may be browser based).

So here I am stuck now cause I want these things listed in a particular order and my brain wont work today.

EDIT example of me sorting results:

dashboardJSON.widgets.sort(sortJSONresultsByWidgetID);

I see you sort by wigetID instead of widgetId. Could your problem be caused by this typo? — Elian Ebbing
– Elian Ebbing, Commented Aug 21, 2011 at 22:56
I could see where you might think that, unfortunately in the original JSON string that typo is actually legitimate. But the sources creating said JSON initially are from a JAVA level infrastructure so that portion is out of my control gotta work with what I get ya know. — chris
– chris, Commented Aug 21, 2011 at 23:00

nickytonline · Accepted Answer · 2011-08-22 01:04:55Z

1

You need to parse the ints with a radix of 10 and use the === operator instead of ==. I think that should do it.

function sortJSONresultsByWidgetID(a,b)
{
    var widgetAId = parseInt(a.wigetID, 10);
    var widgetBId = parseInt(b.wigetID, 10);

    if(widgetAId === widgetBId)
    {
        return 0;
    }
    return widgetAId > widgetBId ? 1 : -1;
}

UPDATE - Here's with Ellian's optimization:

function sortJSONresultsByWidgetID(a,b)
{
    return parseInt(a.wigetID, 10) - parseInt(b.wigetID, 10);
}

edited Aug 22, 2011 at 1:04

answered Aug 21, 2011 at 23:10

nickytonline

6,9797 gold badges47 silver badges76 bronze badges

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

2 Comments

Elian Ebbing Over a year ago

You can simplify that sort function, and just return parseInt(a.wigedID, 10) - parseInt(b.wigedID, 10).

RobG Over a year ago

If the incoming values are known to be digits, there's no need for parseInt, the subtraction operator (-) will convert strings to numbers.

Collectives™ on Stack Overflow

Sorting JSON Object via JavaScript odd results

1 Answer 1

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related