Wonder if there is a nice way to make this shorter...

# codereview

[elect]

Wonder if there is a nice way to make this shorter

protected fun prepareStandardSemaphores(device: VulkanDevice): ConcurrentHashMap<StandardSemaphores, VkSemaphore_Array> {
        val map = ConcurrentHashMap<StandardSemaphores, VkSemaphore_Array>()
        StandardSemaphores.values().forEach {
            map[it] = VkSemaphore_Array(swapchain.images.size) {
                vkDev.createSemaphore(semaphoreCreateInfo) 
            }
        }
        return map
    }

[Dominaezzz]

protected fun prepareStandardSemaphores(device: VulkanDevice): ConcurrentHashMap<StandardSemaphores, VkSemaphore_Array> {
        return StandardSemaphores.values().associateWithTo(ConcurrentHashMap()) {
            VkSemaphore_Array(swapchain.images.size) {
                vkDev.createSemaphore(semaphoreCreateInfo) 
            }
        }
    }

[Dominaezzz]

Only 2 lines shorter but more functional I guess.

[elect]

Nice, thanks

[sitepodmatt]

You could use fold instead of foreach so it would be something like that = ss.values.fold(CHM<SS,VKSAA>(), { (map, v) -> map.also { it[v] = .. })

[elect]

problem with associateByTo is that it maps K and V the other way round as I would like it

[Dominaezzz]

What about associateWithTo?

[elect]

I see none

[Dominaezzz]

Are you on 1.3.21?

[elect]

yep, it seems to be only for CharSequence https://kotlinlang.org/api/latest/jvm/stdlib/kotlin.text/associate-with-to.html

[Dominaezzz]

https://kotlinlang.org/api/latest/jvm/stdlib/kotlin.collections/associate-with-to.html

[elect]

weird, I dont see it

[elect]

I explicitely imported, but it looks like Array<T> isnt recognized as Iterator<T>, although it has

/**
     * Creates an iterator for iterating over the elements of the array.
     */
    public operator fun iterator(): Iterator<T>

[Dominaezzz]

OH, I forgot about that. Yeah, Array doesn't implement Iterable.

[Dominaezzz]

You could use asList() on the array.