Wouldn’t this work? ```extras .getParcelableAr...

# android

[arekolek]

Wouldn’t this work?

extras
    .getParcelableArray(LocationPicker.INPUT_POLYGON)
    .filterIsInstance<LatLng>()
    .forEach(inputPolygon1::add)

[nickk]

This would work (I guess), but still not much better.

[arekolek]

what do you hate about it?

[arekolek]

You could define an extension function like inline fun <reified T> Bundle.getTypedArray(key: String) = getParcelableArray(key).filterIsInstance<T>()

[arekolek]

If there’s addAll in your polygon, you could then do: inputPolygon1.addAll(extras.getTypedArray<LatLng>(LocationPicker.INPUT_POLYGON))

[arekolek]

Define one, if there’s not.

[arekolek]

Not sure if it can be less horrible than that, but I’m curious

[nickk]

Of course your code is a lot better.
But doesn’t it still feel like a workaround?

[arekolek]

I guess that depends on your intention. If you are sure those are all LatLng (you control the code that creates the parcelable array) you’d be better off with suppressing the unchecked warning (like here https://kotlinlang.slack.com/archives/C0B8M7BUY/p1503070876000405 ). If you want to make sure all items are LatLngs, it might be better to do something like

inline fun <reified R> Iterable<*>.mapAsInstance() = map { it.apply { check(this is R) } as R }

instead of dropping those items silently. (see also https://stackoverflow.com/a/36570969 )