bind

?

How the heck did we end up with `minOf()`/`maxOf()`?? I quote @elizarov :

Please, consider them naming top-level functions simply min/max like in all(?) other languages and in mathematics.

https://youtrack.jetbrains.com/issue/KT-7417#comment=27-1721394

It just reads better when using in a context not related to math, check these usages out https://github.com/JetBrains/kotlin/commit/844c68286a369bec87f1e6dacf17c68f60ad2fe4

On the other hand, it hurts discoverability

Did we have a KEEP for this?

Why does it hurt? You type "min", you get "minOf" in completion.

Did we have a KEEP for this?

No, but you can comment in that issue

`min`/`max` in my mind have always implicitly associated with taking only two arguments, whereas

minOf

and

maxOf

make me think they take multiple. /shrug

Or a collection of items.

Yes I agree with @fellshard

Min/max is available across almost all language libs too. MinOf would make more sense on multiple inputs.

but I think it’s ok we can always import Math.min so minOf targets both tastes

The new `Map`'s function

.getValue(key)

having the same name as the delegate operator concerns me a bit, especially given that the delegate is implemented in stdlib, so that the two functions are shown together in IDE completion. Do you think it might be a source of confusion? For me, it was when I first tried to use it.

Yes it was one of concerns regarding

getValue

name. If it would be a source of confusion, we could tune the completion somehow.

Tuning the IDEs/tools to work around a language/library issue sounds a bit suspect to me

What can be tuned here is to hide from completion those members that are only used to satisfy operator conventions, such as getValue/setValue/provideDelegate, plus/minus, etc.

If you have an idea, how to name this function the other way, it's not too late to share it here https://youtrack.jetbrains.com/issue/KT-11851

Assume I have a list with objects that all have a

month

property, which the list is already sorted by. I want a new version of that list where there is an additional element between every two neighbouring items that have different

month

properties. Can this be accomplished in an easy, functional way with the stdlib functions or do I have to manually iterate through the list?

@robin I just curious, would you actually like to have such a freaking complex narrow-specialised function in stdlib?

^ Choosing from the naming options listed (https://youtrack.jetbrains.com/issue/KT-11851), I'd rather prefer

Map.getOrDefault(K): V

. Though this naming correlates with

Map.getOrDefault(K, V): V

, the latter has much closer semantics to the subject (and this correlation might even hint that the default value should be provided from somewhere else). Otherwise, an option I like even more is to expose the internal function

getOrImplicitDefault

or to choose its naming.

What I also find confusing in

getValue(key)

is that the name itself says nothing about the implicit default being used. That is, nothing in it hints that it might return a value that a simple

get

would not.

By the way, the same applies to

Map.getOrDefault(K): V

, because of the

Map.getOrDefault(K, V): V

not using the implicit default.

Though for most of maps you'll get an exception, if there is no mapping to that key

@robin There's no such single function, but you can copy a list with

toMutableList

, then find an insertion point with

binarySearch

and then just insert it with

add(index, element)

Didn't get that you need to insert an item between every two items with different

month

@voddan Of course not, but I thought maybe there is some combination of more broadly applicable functions that would work for this, as I'm not familiar with all the stuff that's in there yet 🙂

A combination of groupBy and join with an element as separator should do.

@mg6maciej That sounds promising, I'll look into it. Thanks!

But I don't know if there is a join function for such case.

There doesn't seem to be, but I'm trying a flatmap right now, adding an additional item for each element of the map from groupBy