I have such method:
function sort(arr) {
for (let i = 0; i < arr.length; i++) {
if (arr[i].title !== 'Test') {
arr[i] = {
...arr[i],
data: sortByType(arr[i].data),
full: sortByType(arr[i].full)
};
}
}
return arr;
}
Can it be simplified to use filter or foreach?
I have done it with map:
I havent tested it, but should work like you original code
function sort(arr) {
return arr.map(el => {
if(el.title === "Test") return el;
return { ...el, data: sortByType(el.data), full: sortByType(el.full) }
})
}
I think that your loop is the simplest right now.
Image you want to switch it to using a map, your code will become:
function sort(arr) {
return arr.map(elem => {
// you now need to handle this case too
if (elem.title === "Test")
return elem;
return {
...elem,
data: sortByType(elem.data),
full: sortByType(elem.full)
}
});
}
Which is essentially more code, and I don't think you gain anything from it imho.
A filter won't work in your case though.
this should help
function sort(arr) {
return arr.map((item) =>{
if(item.title != "Test") {
item = {
...item,
data: sortByType(item.data),
full: sortByType(item.full)
}
}
return item;
})
}
