Trying to figure out if I need a `remember` arg I have ```itemsIndexed(items) { index, item -> va...

# compose

[Colton Idle]

Trying to figure out if I need a remember arg

I have

itemsIndexed(items) { index, item ->
 val thing = remember { MyThing(item) }
}

teammate says this should be

val thing = remember(item) { MyThing(item) }

instead of my approach

val thing = remember() { MyThing(item) }

[Colton Idle]

@agrosner "No" as in I'm right or my teammate is right?

[agrosner]

No you’re right

[Colton Idle]

I think I'm correct here because if item changes then compose should redo the memoization

[agrosner]

Remember is more expensive on simple classes

[agrosner]

It really depends on recomposing and keeping state around

[Colton Idle]

just updated my question. i think maybe i didn't convey it correctly.

[agrosner]

Ohhh. Yeah if it’s immutable then it doesn’t matter as much.

[Colton Idle]

MyThing isn't immutable. It's (unfortunately) an expensive service class that does things in the background with item that's passed in.

[agrosner]

If items list changes without a key I believe all is lost iirc

[agrosner]

So either way doesn’t quite matter

[agrosner]

Passing item into remember will recreate it if you added key to items indexed call and sent a new list with same item key and the item itself has something that changes like another field

[eygraber]

I think the answer is that it depends on what you want to do. By passing the item to remember, then MyThing will be recomputed if the item or the item's position changes. If not, then the same MyThing will be returned regardless of the item at the current index.

[Colton Idle]

So

val thing = remember(item) { MyThing(item) } == MyThing will be recomputed if the item or the item's position changes
val thing = remember() { MyThing(item) } == the same MyThing will be returned regardless of the item at the current index

?

[Albert Chang]

If items can be added to or removed from the list, you do need the key (or you can also specify the key in itemIndexed), otherwise the remembered MyThing can be different from the actual item after list items change.

[Colton Idle]

The list is a snapshot state list and yes they can be added or removed. Damn okay. I guess I still had a misunderstanding of how this was going to work. i assumed that if the item changed, then essentially the entire block was going to be re-run.

[Albert Chang]

Specifying a key in itemIndexed is the better way as it prevents the need to recreate MyThing.

[Colton Idle]

okay. my items do have an id, so I guess I will use that in the args for itemsIndexed

[Colton Idle]

lol. for what its worth (im coming into this project pretty cold) and it looks like somewhow this MyThing "service" actually extends like 5 times, and the root class it extends from is an AAC ViewModel.

[Colton Idle]

so I guess the original question should have been

"is this right?"

itemsIndexed(items) { index, item ->
 val thing = remember { MyViewModel(item) }
}

To which we learned it should instead be

itemsIndexed(items, { index, item -> item.id }) { index, item ->
 val thing = remember { MyViewModel(item) }
}

[Colton Idle]

Okay, my next thing I'm going to do is I think pull all of the code out of MyViewModel and into MyActualScreensViewModel and therefore I won't have this issue of creating this thing on the fly. But overall I think I learned something here... so thank you all!

[shikasd]

if itemsIndexed is the lazy list extension, then your thing works only when the key is specified, because compose reset composition completely when reusing items

[shikasd]

if the key is not specified, it is equivalent to using item index as a key, so if item index changes you'll get stale remember value, as commenter above mentioned

[shikasd]

fwiw, if you were to pull this into a separate composable, i would suggest to use remember(item). Using remember without keys is only technically correct in lazy since you can guarantee that only one item is going to be present

[Colton Idle]

if itemsIndexed is the lazy list extension

yes this is in a lazyColum

if the key is not specified, it is equivalent to using item index as a key

🤯

Using remember without keys is only technically correct in lazy since you can guarantee that only one item is going to be present

🤯

Why do I feel like I've been using remember all wrong for the past 5 years. lol

[Colton Idle]

really seems like https://developer.android.com/develop/ui/compose/state could be updated with some more clarity regarding this

[Colton Idle]

i think my misconception was...

in a lazy list once my composable is rendered, that the remember'd value is calculated... and once it goes off the screen and comes back it gets re-calculated.

so in this case, thats why i thought this would be fine.

itemsIndexed(items) { index, item ->
 val thing = remember { MyViewModel(item) }
 Column {
  Text(item.toString())
  Text(this.currentStopWatchValue())
 }
}

[Colton Idle]

FWIW. I was able to come up with a 50 line example showing this for anyone thats following along
https://gist.github.com/ColtonIdle/71ca77d54f84617ac897f101582df96c (much easier to see the issue in this example vs my complicated codebase)

indeed the only thing that "fixes" this is putting a key on itemsIndexed(items, KEY).

If I put a key on val thing = remember(KEY) { MyViewModel(item) } it did not work.

[shikasd]

I am surprised it didn't work when you put the key in remember, it should recreate the instance

[Colton Idle]

Feel free to look at my code and try it out there if it piques your curiosity!