Any better way than this to transform a string to a map? ```private fun cookieStringToMap(string: St...

# codereview

[jean]

Any better way than this to transform a string to a map?

private fun cookieStringToMap(string: String): Map<String, String> {
    val map = mutableMapOf<String, String>()
    string.split(";").forEach { entry ->
        val parts = entry.split("=")
        map[parts[0]] = parts[1]
    }
        
    return map
}

[bezrukov]

how about this:

private fun cookieStringToMap(string: String): Map<String, String> {
    return string.split(";").map { entry ->
        val (key, value) = entry.split("=")
        key to value
    }.toMap()
}

[Liudvikas Sablauskas]

private fun cookieStringToMap(string: String) =
    string.split(";").associate { entry ->
        val (key, value) = entry.split("=");
        key to value
    }

associate does the toMap part for you basically

[jean]

nice 🙂 thanks guys!

[aleksey.tomin]

or

private fun cookieStringToMap(string: String) =
    string.split(";")
        .map { it.split("=") }
        .associate { it[0] to it[1] }

[gildor]

it’s the same as Liubvikas code, but less efficient, it requires additional intermediate list

[christophsturm]

"a=b;c=d".splitToSequence(";")
        .map { it.split("=") }
        .associate { it[0] to it[1] }

with a sequence you have both readability and performance