0

I am trying to sort an array alphabetically in JavaScript. I reduced the size of the array for testing and readability (the original is not that much larger but still getting same problem). Problem is that output has a few values at the top of the list that are not in order. I tried running the sort in a couple different loops many times over and still doesn't seem to sort correctly. I have looked around and most answers are about basic syntax. I have sorted numbers and other lists like this so I am puzzled.

Thanks in advance.

Here is my code:

var miniArr = ['leatherwood', 'sacramento', 'ackerman', 'alma', 'anderson', 'ben wheeler', 'atlanta', 'bakersfield', 'albuquerque', 'baker', 'aspen', 'anchorage', 'antioch', 'benton', 'cedar rapids', 'el cajon', 'boca raton', 'falcon', 'crestview', 'ceres', 'deep run', 'fresno', 'eugene', 'fayetteville', 'fairfield', 'fort myers', 'butler', 'eunice', 'cary', 'cincinnati', 'buffalo', 'freedom (watsonville)', 'francis creek', 'boulder', 'fort worth', 'edgartown', 'bonita', 'glen spey', 'grants pass', 'cleveland', 'clinton', 'colonia', 'chula vista', 'gulf breeze', 'lakeland', 'la mesa', 'jasper', 'haverhill', 'ingleside (canada)', 'joplin', 'hancock', 'pasadena', 'loveland', 'london (canada)', 'lompoc', 'lemon grove', 'park city']

var sorted = [];
var sorted2 = []
for (var x = 0; x < 100; x++) {
  sorted = miniArr.sort((a, b) => a > b);
}
for (var x = 0; x < 100; x++) {
  sorted2 = sorted.sort((a, b) => a > b);
}

console.log(sorted);
console.log(sorted2);
Jack Bashford
  • 43,180
  • 11
  • 50
  • 79
Carlos
  • 3
  • 4
  • Comparator functions should return a positive (in the case a > b), negative (b > a), or zero (a == b) number rather than a boolean. Try `miniArr.sort((a, b) => a > b ? 1 : a === b ? 0 : -1);` – CollinD Jan 15 '19 at 21:24
  • 2
    for strings use `localeCompare()` – guijob Jan 15 '19 at 21:25

1 Answers1

1

For strings use localeCompare():

var miniArr = ['leatherwood', 'sacramento', 'ackerman', 'alma', 'anderson', 'ben wheeler', 'atlanta', 'bakersfield', 'albuquerque', 'baker', 'aspen', 'anchorage', 'antioch', 'benton', 'cedar rapids', 'el cajon', 'boca raton', 'falcon', 'crestview', 'ceres', 'deep run', 'fresno', 'eugene', 'fayetteville', 'fairfield', 'fort myers', 'butler', 'eunice', 'cary', 'cincinnati', 'buffalo', 'freedom (watsonville)', 'francis creek', 'boulder', 'fort worth', 'edgartown', 'bonita', 'glen spey', 'grants pass', 'cleveland', 'clinton', 'colonia', 'chula vista', 'gulf breeze', 'lakeland', 'la mesa', 'jasper', 'haverhill', 'ingleside (canada)', 'joplin', 'hancock', 'pasadena', 'loveland', 'london (canada)', 'lompoc', 'lemon grove', 'park city']

var sorted = [];
var sorted2 = []
for (var x = 0; x < 100; x++) {
  sorted = miniArr.sort((a, b) => a.localeCompare(b));
}

console.log(sorted);
guijob
  • 4,413
  • 3
  • 20
  • 39