Sort array by Criticality

Question

I have a criticality array and I want to sort by criticality, something like this:

let criticalityTypes = ['CRITICALITY_LOW', 'CRITICALITY_HIGH', 'CRITICALITY_MEDIUM'];

I get this order randomly, sometimes ** CRITICALITY_LOW ** comes in position 1 of the matrix ie either position 2, or 'CRITICALITY_MEDIUM' in 0 position,

What I want to do is order in the following order, regardless of the order that comes to me, sometimes I have just one criticality, or two:

['CRITICALITY_HIGH', 'CRITICALITY_MEDIUM', 'CRITICALITY_LOW'];

I tried to use sort function to order what I've done so far is this:

return criticalityTypes.sort((a, b) => {
    if (a < b) return -1;
    if (a > b) return 1;
});

But without success, any help?

Are those strings (like `'CRITICALITY_HIGH'`) or constant (like `CRITICALITY_MEDIUM`)? — Blackhole, Jul 30 '19 at 19:31
If you know the values and their order already, why do you have to retrieve them in a random order & sort them at all? — Jonas Wilms, Jul 30 '19 at 19:33
I receive the data from API, and the API I don't have access to change — Jefferson Bruchado, Jul 30 '19 at 19:33
Does the array have duplicate `CRITICALITY` values? If it always has 3 items, sorting seems redundant — adiga, Jul 30 '19 at 19:34
I use this array to render a component, and I need it to follow the order described — Jefferson Bruchado, Jul 30 '19 at 19:36
So, why not just use `['CRITICALITY_HIGH', 'CRITICALITY_MEDIUM', 'CRITICALITY_LOW']` directly? What difference does it make if server is sending in random order and it always has 3 items — adiga, Jul 30 '19 at 19:36
The items come dinamically based in component, sometimes I have just one item or two. — Jefferson Bruchado, Jul 30 '19 at 19:39

score 4 · Accepted Answer · answered Jul 30 '19 at 19:36

You could take an object with the wanted order and sort by the delta of the values.

var criticalityTypes = ['CRITICALITY_LOW', 'CRITICALITY_HIGH', 'CRITICALITY_MEDIUM'],
    order = { CRITICALITY_HIGH: 1, CRITICALITY_MEDIUM: 2, CRITICALITY_LOW: 3 };

criticalityTypes.sort((a, b) => order[a] - order[b]);

console.log(criticalityTypes);

score 1 · Answer 2 · answered Jul 30 '19 at 19:49

Just another way:

let criticalityTypes = ['CRITICALITY_LOW', 'CRITICALITY_HIGH', 'CRITICALITY_MEDIUM'];
let orderedItems = [];
let desiredOrder = ['CRITICALITY_HIGH', 'CRITICALITY_MEDIUM', 'CRITICALITY_LOW'];

 for (var i = 0; i < desiredOrder.length; i++) {
    if (criticalityTypes.indexOf(desiredOrder[i]) > -1) {
        orderedItems.push(desiredOrder[i]);
    }
 }

 console.log(orderedItems);

MKougiouris · Answer 3 · 2019-07-30T19:49:30.043

-1

The problem here is that you as the consumer of the api, do not have any clue what an "order" means for this items, as they are not "quantized", they are ideas, not an integer that you could call 'order' so you know what is what, the string is just for the display anyway.

Since you can not change the api, BUT they are always only these 3 and you always want them in the same order, you can mock your current call, put this as a 'placeholder' comparer so that you mock the calls to this function, until someones gives you something that you can use numericaly to determine an order, 'ascending' or 'descending' requires something quantized to have a meaning.

function criticalityTypes(a,b){
    return ['CRITICALITY_HIGH', 'CRITICALITY_MEDIUM', 'CRITICALITY_LOW'];
}

edited Jul 30 '19 at 19:49

answered Jul 30 '19 at 19:37

MKougiouris

2,821
1
16
19

1

The parameters aren't needed. – Blackhole Jul 30 '19 at 19:37
Either the api provider will return to you keys with an id and a string value, or you will just return them directly if indeed "there is always only theese 3 and i need them always returned in this way". – MKougiouris Jul 30 '19 at 19:40
This doesn't answer the question, and it's based on incorrect assumptions. There are *not* "always only these 3". – user229044 Aug 23 '19 at 16:24

Sort array by Criticality

3 Answers3