![function getItemsWithNext() {
return items
.filter(item => {
if (isDeleted(item)) {
return false
}
return true
})
.map((item, i) => {
const next = items[i + 1]
return {
item,
next,
}
})
}](https://cdn.bsky.app/img/feed_thumbnail/plain/did:plc:p2cp5gopk7mgjegy6wadk3ep/bafkreifjayqiaf2nznful46gpc5kf2y5gjkktlmophvgefr2nvp7uorxtm "function getItemsWithNext() {
return items
.filter(item => {
if (isDeleted(item)) {
return false
}
return true
})
.map((item, i) => {
const next = items[i + 1]
return {
item,
next,
}
})
}")
I guess the real bug is that the code to get the next accesses the original items array whereas the map function runs on a filtered array from which items may be removed.
Just don't have deleted items and all will be fine...
1 3Samuel
@samuel.fm
correct! solution is to use the 3rd arg of map, which will be the filtered array
2 5
🦖 Plankie 🏳️🌈 Life, Uh, Finds a way 🦕These kind of posts gives me butterflies in my tummy 😳
3Michel BestaatI'd split it in two parts.
1. const filtered = items.filter ...
2. return filtered .map ....
2 1Samuelthat’d work
1Michel BestaatNot sure if there would be performance and/or additional memory costs though.
I probably cannot count on the compiler to optimize the code. :-)
1 1
David BuchananThis was also my thought, although I'd call it something cool like nondeletedItems
1 2Michel BestaatWe really have to spell out everything for those bsky devs.
2