What's the best way to memoize the result of a del...

# getting-started

[Rob Elliot]

What's the best way to memoize the result of a delegated property when using val x by delegate?

I could stick it in a cache in the Delegate class, but wondering if there's a simpler way... I suppose I could wrap it in a lazy, but I'd rather it was eagerly evaluated. I'm really only using it for syntax sugar to reduce boilerplate.

[Sam]

If it’s both eagerly evaluated and memoized, is that different from just initializing it normally, without a delegate?

[Rob Elliot]

Context - creating classes representing database tables. They currently look like this:

class UserTable(dialect: SQLDialect) : DbTable(dialect, "USERS") {
  val USER_ID = fieldFor("USER_ID", Long::class.java)
}

and I'm using a delegate to reduce that to:

class UserTable(dialect: SQLDialect) : DbTable(dialect, "USERS") {
  val USER_ID by field<Long>()
}

[Sam]

Gotcha, so really the property delegate is just there to grab the property name 👍

[Rob Elliot]

Exactly

[Rob Elliot]

The fieldFor method actually mutates the state of the class, so I'm getting a lot of fields called USER_ID at the moment because everytime USER_ID is called a new field is created.

[Sam]

🤔 the field<Long>() method should only be called once, when the property is created. What’s your implementation of that method?

[Rob Elliot]

Untitled.kt

Untitled.kt

[Sam]

I think the problem you’re going to have is that you want the property name in order to create the value, and you want to create the value eagerly when the property is declared, but the property delegate only gives you the property name at the point when the property is accessed, not when it’s initialized.

[Sam]

As in, you ideally want the property name in the initial call to field, but you actually only get it inside the call to getValue

[Sam]

I think you can solve it with a property delegate provider, please hold

[Vampire]

but the property delegate only gives you the property name at the point when the property is accessed, not when it’s initialized.

You can probably use provideDelegate to change that

[Vampire]

That should only be called once when initializing the delegated property

[Sam]

val USER_ID by field<Long>()

private inline fun <reified T> field() =
    PropertyDelegateProvider<Any, ReadOnlyProperty<Any, T>> { _, property ->
        val value = fieldFor(property.name, T::class.java)
        ReadOnlyProperty { _,_ -> value }
    }

[Sam]

Something like that 🤷

[Rob Elliot]

Awesome, thanks both - will have a play

[Rob Elliot]

For anyone following along at home

[Rob Elliot]

Actually the compiler didn't catch it but needed an @PublishedApi internal fun

[Rob Elliot]

Untitled.kt

Untitled.kt