javascript sort array

Question

My array isn't being sorted properly. Can someone let me know what I am doing wrong?

...
 sortArray = new Array ("hello", "Link to Google", "zFile", "aFile");

//sort array
        if (dir == "asc") { 
            sortArray.sort(function(a,b){return a - b}); 
        } else { 
            sortArray.sort(function(a,b){return b - a});
        }

        for(var i=0; i<sortArray.length; i++) { 
            console.log(sortArray[i]);
        }

the log is showing them in the same order as they were entered.

of course you can also do this handy little shortcut .......................................................... (sortArray.sort())[(dir === 'asc' ? 'reverse' : 'slice' )]() — qwertymk
– qwertymk, Commented Mar 29, 2011 at 18:27

Community · Accepted Answer · 2017-05-23 10:32:41Z

12

You want to make a comparison in your sort, not a subtraction:

if (dir == "asc") {
    sortArray.sort(function(a, b) {
        a = a.toLowerCase();
        b = b.toLowerCase();
        return a === b ? 0 : a > b : 1 : -1;  
    });
} else {
    sortArray.sort(function(a, b) {
        a = a.toLowerCase();
        b = b.toLowerCase();
        return b === a ? 0 : b > a : 1 : -1;  
    });
}

I also used toLowerCase() so that 'Link to Google' is placed appropriately.

EDIT: Updated to fix comparison issue according to comment.

See example →

edited May 23, 2017 at 10:32

CommunityBot

11 silver badge

answered Mar 29, 2011 at 17:58

mVChr

50.3k11 gold badges111 silver badges105 bronze badges

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

4 Comments

PruitIgoe Over a year ago

I accepted yours for the toLowerCase() add on...though everyone deserves a +1.

Mike Over a year ago

This answer does not seem to work in Chrome. The comparator function is supposed to return 0 if a and b are equal, a positive integer if a should be sorted to a higher index than b, and a negative integer if a should be sorted to a lower index than b.

Digant C Kasundra Over a year ago

@Mike is right. This should be: if (a.toLowerCase() === b.toLowerCase()) { return 0 } else { return a.toLowerCase() < b.toLowerCase() ? -1 : 1 }

mVChr Over a year ago

@Mike && digant-c-kasundra I updated to fix the issue. Thanks!

user142162 · Accepted Answer · 2011-03-29 17:52:59Z

7

You're trying to sort by subtracting strings, to which you'll get NaN.

answered Mar 29, 2011 at 17:52

user142162

1 Comment

PruitIgoe Over a year ago

Thanks everyone, obviously I misread the docs on that one - : D

DocMax · Accepted Answer · 2011-03-29 17:56:53Z

7

The trouble is that "a - b" is treating the strings like numbers, which returns NaN. You will get the behavior you are looking for (assuming you are looking for case-sensitive sorts) if you replace your sorts with:

    if (dir == "asc") { 
        sortArray.sort(function(a,b){return a < b ? -1 : 1}); 
    } else { 
        sortArray.sort(function(a,b){return b < a ? -1 : 1});
    }

answered Mar 29, 2011 at 17:56

DocMax

12.2k7 gold badges48 silver badges45 bronze badges

Comments

davin · Accepted Answer · 2011-03-29 18:03:01Z

6

Your comparator functions returns NaN, since it receives two strings, and performs subtraction, an operation that isn't well-defined on strings.

What you should have is something more like:

function(a,b){
   return a>b? 1 : (a<b ? -1 : 0);
}

or you can use localeCompare:

function(a,b){
   return a.localeCompare(b);
}

Remember to treat case appropriately, e.g. "L" < "a" whilst "l" > "a"

edited Mar 29, 2011 at 18:03

answered Mar 29, 2011 at 17:57

davin

45.6k9 gold badges81 silver badges78 bronze badges

Collectives™ on Stack Overflow

javascript sort array

4 Answers 4

4 Comments

1 Comment

Comments

Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

4 Comments

1 Comment

Comments

Comments

Linked

Related