android / android/codelab-android-compose

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

未关闭
#321 0 条评论 0 个 reaction 已指派 1 人 已被 @dturner 认领 在 GitHub 查看
主要语言
Kotlin
星标
1.7k
派生
799
PR 合并指标
30 天内没有已合并 PR

描述

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)`?

贡献指南

打开贡献指南

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。