Sorting an Array of Objects by multiple criteria

Question

I have an Array of Objects (Properties) that I need to sort by multiple criteria, 4 in total. I have managed two of them, but the second and third are giving me issues.

The sorting order has to be:

1st - if a property has a status of "Active", that goes before a property with a status of "Escrow", this one is easy because I can just so alphabetically since "active" goes before "escrow".

2nd - If a property has a property-type of "Multifamily" that goes before the rest (not working)

3rd - If a property has a status of "Land" that goes before the rest (but after Multifamily, not working)

4th - Sort by number of Units, again, this one is easy.

Here is my code so far, I'm using a microlibrary to sort Arrays but the usuall Array.sort logic functions will work here, the library is called thenBy.js:

Not only the "Multifamily" and "Land" sorting is not working, it's also duplicating entries...I'm really not sure how to approach this one.

const filtered_properties = []

filtered_properties.sort(
  firstBy(function(v1, v2) {
    if (v1.status > v2.status) {
      return 1;
    } else if (v1.status < v2.status) {
      return 0;
    }
  })
  // Not working
  .thenBy(function(v1) {
    return (
      v1["property-type"] &&
      v1["property-type"][0].name === "Multifamily"
    );
  })
  // Not working
  .thenBy(function(v1) {
    return (
      v1["property-type"] &&
      v1["property-type"][0].name === "Land"
    );
  })
  .thenBy(function(v1, v2) {
    return v2.units - v1.units;
  })
);

<script src="https://cdn.jsdelivr.net/npm/[email protected]/thenBy.min.js"></script>

Very relevant: Sorting in JavaScript: Shouldn't returning a boolean be enough for a comparison function? — VLAZ
– VLAZ, Commented Jan 10, 2022 at 14:04
I made you a snippet. Please add your object and expected output — mplungjan
– mplungjan, Commented Jan 10, 2022 at 14:12

Nina Scholz · Accepted Answer · 2022-01-10 14:12:02Z

1

Depending on your properties, you could compare with the wanted top string and take the delta of b - a of the comparison and if you need the wanted string at bottom, switch a and b.

The same applies for all other comparisons where an item has to be sorted to top.

array.sort((a, b) =>
    (b.status === 'Active') - (a.status === 'Active') ||
    (b.type === 'Multifamily') - (a.type === 'Multifamily') ||
    (b.status === 'Land') - (a.status === 'Land') ||
    a.units - b.units
);

answered Jan 10, 2022 at 14:12

Nina Scholz

388k26 gold badges367 silver badges417 bronze badges

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

4 Comments

Sergi Over a year ago

That's close but not 100% there. The properties that have a different type that is not "Multifamily" or "Land" just appear in the middle. How can I place those after "Multifamily" and "Land"?

Nina Scholz Over a year ago

please add a small amount of data to the question.

Sergi Over a year ago

The above works, it's just that for example, a property has a type of "Office", then it just appears at the same spot instead of at the end, every other type should be after "Multifamily" and "Land".

Nina Scholz Over a year ago

do you want to sort the same properties by some values (like this)?

Collectives™ on Stack Overflow

Sorting an Array of Objects by multiple criteria

1 Answer 1

4 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

4 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related