Skip to content

Unnecessary, obscure way to shuffle a list in the Advanced State and Side Effects codelab? #321

@lathanh

Description

@lathanh

At the line linked below, shuffled given a Random whose seed is generated Randomly (which is not substantially different than not providing a Random at all). Why?

https://github.com/googlecodelabs/android-compose-codelabs/blob/37b4e639d3befcfd85885ba9cefceb26f8b32f5e/AdvancedStateAndSideEffectsCodelab/app/src/main/java/androidx/compose/samples/crane/home/MainViewModel.kt#L59

And if you want a number between 1 and 100, why shuffle a collection of 100 numbers rather than use Random.nextInt (1,101)?

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type
No fields configured for issues without a type.

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions