https://kotlinlang.org
Join Slack
If I have ```val data = hashMapOf( "firstName" to firstName, "lastName"...
# getting-started
c
If I have
Copy code
val data =
    hashMapOf(
            "firstName" to firstName,
            "lastName" to lastName)
but I wanted to not add firstName or lastName to the hashmap if they are null. What would be the best way to do that?
e
must it be a HashMap or are other maps allowed?
j
Do you really need an explicit 
java.util.HashMap
as output? In general it's more common to stick with 
mapOf
or 
mutableMapOf
. When building a read-only map, you can use 
buildMap
with conditions inside:
Copy code
val map = buildMap {
    firstName?.let { put("firstName", it) }
    lastName?.let { put("lastName", it) }
}
If what you need is a mutable map, you can also add stuff conditionally with `apply`:
Copy code
val map = mutableMapOf<String, String>().apply {
    firstName?.let { put("firstName", it) }
    lastName?.let { put("lastName", it) }
}
c
I'm following firebases docs which have a hashMapOf, but I suppose I can try other maps? Shouldn't be an issue.
e
also options,
Copy code
buildMap {
    firstName?.let { put("firstName", it) }
    lastName?.let { put("lastName", it) }
}

@Suppress("UNCHECKED_CAST")
fun <K, V> Map<K, V?>.filterNotNullValues(): Map<K, V> = filterValues { it != null } as Map<K, V>

mapOf("firstName" to firstName, "lastName" to lastName).filterNotNullValues()
https://firebase.google.com/docs/reference/android/com/google/firebase/functions/HttpsCallableReference#call(java.lang.Object)
The data passed into the trigger can be any of the following types:
• Any primitive type, including null, int, long, float, and boolean.
• 
String
• 
List<?>
, where the contained objects are also one of these types.
• 
Map<String, ?>>
, where the values are also one of these types.
• 
JSONArray
• 
JSONObject
• 
NULL
you do not need a HashMap, that Kotlin example was simply translated from Java
c
Thanks. I like the conciseness of just having the ext func. But TIL about buildMap!
r
You can always make a helper function:
Copy code
fun <K: Any, V: Any> nonNullHashMapOf(
    vararg entries: Pair<K?, V?>
): HashMap<K, V> {
    val map = HashMap<K, V>()
    for ((key, value) in entries) {
        map[key ?: continue] = value ?: continue
    }
    return map
}
And use it
Copy code
val data = nonNullHashMapOf(
    "firstName" to firstName,
    "lastName" to lastName,
)
e
I would name it as 
mapOfNotNull
just to match 
listOfNotNull
better, but yeah that can also be done
c
Oooh boy! So many options!
e
r
@ephemient That's probably a good idea. I'm not great at naming (like most programmers...)
e
I'm not great either, I'm just copying stdlib here :)
😁 1
c
If I use
Copy code
fun <K: Any, V: Any> nonNullHashMapOf(vararg entries: Pair<K?, V?>) {
    val map = HashMap<K, V>()
    for ((key, value) in entries) {
        map[key ?: continue] = value ?: continue
    }
}
should I try to use a Regular map internally, or is HashMap alright?
r
Use a hash map if you need a hash map for some reason, otherwise just use 
mutableMapOf<K, V>()
e
if your function is named 
hashMapOf
it should probably return a 
HashMap
to reduce confusion 😛
r
(By the way, I update my comment to include the return value, which I forgot the first time.)
e
in practice 
MutableMap
is backed by a 
LinkedHashMap
, so it is a 
HashMap
in a sense
(but that's not a public API contract so don't rely on it)
➕ 2
c
Okay, so I'm using @Ruckus method you outlined, but @ephemient naming (stdlib naming). Anything else I should do to improve? (the only reason I asked about converting this to a regular Map instead of HashMap was because that was in the opening comment.
Copy code
fun <K: Any, V: Any> mapOfNotNull(
    vararg entries: Pair<K?, V?>
): HashMap<K, V> {
    val map = HashMap<K, V>()
    for ((key, value) in entries) {
        map[key ?: continue] = value ?: continue
    }
    return map
}
r
If you don't need a hash map:
Copy code
fun <K: Any, V: Any> mapOfNotNull(
    vararg entries: Pair<K?, V?>
): Map<K, V> = buildMap {
    for ((key, value) in entries) {
        this[key ?: continue] = value ?: continue
    }
}
c
@ephemient I'll file a bug fore firebase docs to just be mapOf instead of hashMapOf
r
With the new definitely non null types, this could also be
Copy code
fun <K, V> mapOfNotNull(
    vararg entries: Pair<K, V>
): Map<K & Any, V & Any> = buildMap {
    for ((key, value) in entries) {
        this[key ?: continue] = value ?: continue
    }
}
Though I don't know if that's any better... https://blog.jetbrains.com/kotlin/2022/04/kotlin-1-6-20-released/#definitely-non-nullable-types
c
TIL "Definitely non-nullable types"
I'm going to go with this for now. Learned a ton in the past 15 minutes. Filed a bug. Thanks everyone for teaching!
👍 1
l
Just in case it’s helpful, GitLive has a library that lets you read and write kotlinx.serialization objects to/from firebase instead of using the HashMap based API. https://firebaseopensource.com/projects/gitliveapp/firebase-kotlin-sdk/
👍 1
7 Views
Open in Slack
PreviousNext
Powered by

kotlinlang

A modern programming language that makes developers happier.

Powered by