android / android/codelab-android-compose
Unnecessary, obscure way to shuffle a list in the Advanced State and Side Effects codelab?
Đang mở
- Ngôn ngữ chính
- Kotlin
- Star
- 1.7k
- Fork
- 799
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Mô tả
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)`?
Hướng dẫn đóng góp
Đánh giá
Issue này chưa được đánh giá.