The JSON sorting functionality seems to be functioning improperly

Question

The JSON sorting functionality seems to be functioning improperly

Here is a sample of my JSON data:

[
  {
    "cash": 100,
    "uid": "LHy2qRGaf3nkWQgU4axO",
    "name": "test2"
  },
  {
    "cash": 1000000,
    "uid": "01wFhCSlnd9vSDY4NIkx",
    "name": "test"
  },
  {
    "cash": 500,
    "uid": "PBOhla0jPwI4PIeNmmPg",
    "name": "test3"
  }
]

I am attempting to sort the JSON objects by the user's cash amount. Here is the code snippet I used:

    var objArr = []; // array containing JSON objects

    function compare(a, b) {
        console.log("a" + a.cash);
        console.log("b" + b.cash);
        if (a.cash > b.cash)
            return -1;
        if (a.cash < b.cash)
            return 1;
        return 0;
    }

   var obj = objArr.sort(compare);
   response.send(obj);

However, the response returned is not ordered as expected. Can anyone help me understand how to correct this issue? Thank you.

javascript json google-cloud-functions

Answer 1

Answer №1

Here are a couple of important points to consider:

Your code currently sorts in descending order rather than being unordered.
In your code, you are utilizing the return value of the sort function. Keep in mind that this can give the incorrect impression that the returned array is a different array from the original. In reality, it is the same array, so it's generally advisable not to rely on the return value.

A shorter version of your sorting logic could be achieved with:

// For ascending order:
return a.cash - b.cash;

// For descending order:
return b.cash - a.cash;

Support for Point #1 above:

var objArr = [
  {
    "cash": 100,
    "uid": "LHy2qRGaf3nkWQgU4axO",
    "name": "test2"
  },
  {
    "cash": 1000000,
    "uid": "01wFhCSlnd9vSDY4NIkx",
    "name": "test"
  },
  {
    "cash": 500,
    "uid": "PBOhla0jPwI4PIeNmmPg",
    "name": "test3"
  }
];

function compare(a, b) {
  console.log("a" + a.cash);
  console.log("b" + b.cash);
  if (a.cash > b.cash)
    return -1;
  if (a.cash < b.cash)
    return 1;
  return 0;
}

var obj = objArr.sort(compare);
console.log(obj);

.as-console-wrapper {
  max-height: 100% !important;
}

Demonstration of Points #2 and #3:

var objArr = [
  {
    "cash": 100,
    "uid": "LHy2qRGaf3nkWQgU4axO",
    "name": "test2"
  },
  {
    "cash": 1000000,
    "uid": "01wFhCSlnd9vSDY4NIkx",
    "name": "test"
  },
  {
    "cash": 500,
    "uid": "PBOhla0jPwI4PIeNmmPg",
    "name": "test3"
  }
];

console.log("ascending", objArr.sort((a, b) => a.cash - b.cash));
console.log("descending", objArr.sort((a, b) => b.cash - a.cash))

.as-console-wrapper {
  max-height: 100% !important;
}

Answer 2

Here are a couple of important points to consider:

Your code currently sorts in descending order rather than being unordered.
In your code, you are utilizing the return value of the sort function. Keep in mind that this can give the incorrect impression that the returned array is a different array from the original. In reality, it is the same array, so it's generally advisable not to rely on the return value.

A shorter version of your sorting logic could be achieved with:

// For ascending order:
return a.cash - b.cash;

// For descending order:
return b.cash - a.cash;

Support for Point #1 above:

var objArr = [
  {
    "cash": 100,
    "uid": "LHy2qRGaf3nkWQgU4axO",
    "name": "test2"
  },
  {
    "cash": 1000000,
    "uid": "01wFhCSlnd9vSDY4NIkx",
    "name": "test"
  },
  {
    "cash": 500,
    "uid": "PBOhla0jPwI4PIeNmmPg",
    "name": "test3"
  }
];

function compare(a, b) {
  console.log("a" + a.cash);
  console.log("b" + b.cash);
  if (a.cash > b.cash)
    return -1;
  if (a.cash < b.cash)
    return 1;
  return 0;
}

var obj = objArr.sort(compare);
console.log(obj);

.as-console-wrapper {
  max-height: 100% !important;
}

Demonstration of Points #2 and #3:

var objArr = [
  {
    "cash": 100,
    "uid": "LHy2qRGaf3nkWQgU4axO",
    "name": "test2"
  },
  {
    "cash": 1000000,
    "uid": "01wFhCSlnd9vSDY4NIkx",
    "name": "test"
  },
  {
    "cash": 500,
    "uid": "PBOhla0jPwI4PIeNmmPg",
    "name": "test3"
  }
];

console.log("ascending", objArr.sort((a, b) => a.cash - b.cash));
console.log("descending", objArr.sort((a, b) => b.cash - a.cash))

.as-console-wrapper {
  max-height: 100% !important;
}

The JSON sorting functionality seems to be functioning improperly

Answer №1

Similar questions

What is the best way to clear an input value in Vuejs after it has been submitted?

After the initial rendering, JQuery can dynamically search through the DOM elements and seamlessly replace keys with their respective values

Obtaining Function Call Results by Querying Node.js MySQL Connection

Managing multiple carousels simultaneously using the middle mouse button in Swiper.js

Reset input fields upon jQuery removal

Approach for fetching these JSON records

Error message: "Asynchronous task"

What is the best way to interact with a checkbox that has a label using Selenium in Node.js?

I am facing an issue with effectively passing properties from a parent state to its child component

Could offering a Promise as a module's export be considered a legitimate approach for asynchronous initialization in a Node.js environment?

Dealing with JSON Codable in Swift 4 can be tricky, as the value returned can vary between being an object

Utilize the --stream function to break down a sizable object into smaller, more manageable components

Click event to reset the variable

Issue with DEFAULT_PATH_LEAF_TO_NULL functionality in jsonPath is not functioning as expected

Looking to tweak the date format in a Vue.js template?

Toggle the visibility of multiple divs depending on a specific attribute value

Struggling with integrating API response formatting in React and updating state with the data

Send information using AJAX within a .NET integrated browser

Enhancing a React modal to enable user input for updating state variables

The Power of Json, Ajax, and Javascript