Create a function to convert unindexed iteratee to indexed about lo HOT 4 OPEN

I would say the same for predicates— that is, with methods like Filter(slice []T, func(T, int) bool), I am far more likely to be using them in situations where I have a func(T) bool; the index is almost always irrelevant, and it's extremely annoying to have to keep writing inline anonymous wrapper funcs.

from lo.

Is this feature currently being considered?

from lo.

this would be surprisingly useful

from lo.

First of all, I'm very pleased with the functionality provided by lo package. It's great, many kudos to the author and contributors.

Despite the fact, that having an index argument increases the flexibility of functions and makes them truly general-purpose functions, I can share that based on my personal experience of lo package usage I have never ever used index argument. Literally, never. It's always func(item T, _ int) R.

Since the package is on v1.* and needs to maintain backward compatibility, it's quite hard to change it. What I suggest is releasing v2 and changing the default iterate signature from func(item T, index int) R to func(item T) R and creating additional functions like MapI, ForEachI` which will accept functions with iterate. I'm not sure if it needs a separate issue, @samber what do you think? Can you please share your thoughts on that?

UPDATE:
Also, an important example. This signature stops me from using tons of already existing functions like strings.ToUpper, and instead of this simple line of code lo.Map(strs, strings.ToUpper) I need to write this:

lo.Map(strs, func(s string, _ int) string {
    return strings.ToUpper(s)
}

At least it makes sense for me to implement some methods like lo.MapNoI in the current v1.* branch which will handle such signatures.

from lo.