Sort an Array using sort(): wrong solution provided

Sort an Array using sort(): wrong solution provided
0
#1

Functional Programming: Sort an Array Alphabetically using the sort Method

The solution provided doesn’t work:

function alphabeticalOrder(arr) {
  // Add your code below this line
  return arr.sort(function(a,b) {
    return a > b;
  });
  // Add your code above this line
}
alphabeticalOrder(["a", "d", "c", "a", "z", "g"]);

This solution works:

function alphabeticalOrder(arr) {
  // Add your code below this line
  return arr.sort(function(a,b) {
    if (a > b) return 1;
    if (a == b) return 0;
    if (a < b) return -1;
  });
  // Add your code above this line
}
alphabeticalOrder(["a", "d", "c", "a", "z", "g"]);

Reason (from MDN web docs):
If compareFunction is supplied, all non- undefined array elements are sorted according to the return value of the compare function (all undefined elements are sorted to the end of the array, with no call to compareFunction ). If a and b are two elements being compared, then:

  • If compareFunction(a, b) is less than 0, sort a to an index lower than b (i.e. a comes first).
  • If compareFunction(a, b) returns 0, leave a and b unchanged with respect to each other, but sorted with respect to all different elements. Note: the ECMAscript standard does not guarantee this behaviour, and thus not all browsers (e.g. Mozilla versions dating back to at least 2003) respect this.
  • If compareFunction(a, b) is greater than 0, sort b to an index lower than a (i.e. b comes first).
  • compareFunction(a, b) must always return the same value when given a specific pair of elements a and b as its two arguments. If inconsistent results are returned then the sort order is undefined.
#2

Thank you for helping make FCC better. Bugs can be reported as GitHub Issues. Whenever reporting a bug, please check first that there isn’t already an issue for it and provide as much detail as possible.

Updating guide articles (which is what the hints and suggested solutions are) is actually extremely easy and requires almost no programming knowledge! Check it out and get that first PR under your belt!

#3

Actually, this challenge has been updated to include similar information in the challenge.

I had added this information to the Guide on a PR, but then we decided it would be better placed in the challenge itself, so @Manish-Giri did that with PR #35756. Once master is deployed to production, the challenge will look like below:

1 Like