CBA_fnc_sortNestedArray - Preserve order of elements with same sorting value

[mharis001]

When merged this pull request will:

• title

Example (before changing to default behaviour):

_a = [
    ["a", 100],
    ["b", 200],
    ["e", 100],
    ["d", 150],
    ["c", 100],
    ["abc", 200]
];

[_a, 1, true, true] call CBA_fnc_sortNestedArray;
// [["a",100],["e",100],["c",100],["d",150],["b",200],["abc",200]]

_a pushBack ["x", 100];
// [["a",100],["e",100],["c",100],["d",150],["b",200],["abc",200],["x",100]]

[_a, 1, true, true] call CBA_fnc_sortNestedArray;
// [["a",100],["e",100],["c",100],["x",100],["d",150],["b",200],["abc",200]]

// above, ["x", 100] is sorted after all the elements that appeared before it
// however, if preserve is false, the ordering of elements relative to each other is not preserved
[_a, 1, true, false] call CBA_fnc_sortNestedArray;
// [["e",100],["c",100],["x",100],["a",100],["d",150],["abc",200],["b",200]]

[commy2]

I don't get it.

[mharis001]

Updated the example to be more clear. My use case for this is an array of actions (each with a priority). This addition will allow actions to be sorted in terms of decreasing priority (like current), but also keep actions with the same priority in the order they were added - instead of performing additional unwanted sorting on the full "action sub-arrays".

[commy2]

keep actions with the same priority in the order they were added

Shouldn't that always be the case?

instead of performing additional unwanted sorting on the full "action sub-arrays"

Seems like a bug/oversight to me.

I think this does not need additional syntax, but just always sort the way your alt syntax is now.

This is the original: https://github.com/CBATeam/CBA_A3/blob/79d52a8338fc9457e6830530794ab6ed03eefd4b/addons/arrays/fnc_sortNestedArray.sqf

[commy2]

Please keep the apply to save N pushBack commands.

private _helperArray = _array apply {
    [_x select _index, _coefficient * _forEachIndex, _x]
};

[PabstMirror]

no _forEachIndex in apply

[commy2]

Dang.