startActivityForResult 已经标为废弃了,今天整理代码时,考虑把它换成 Activity Result API。所以去了解了一下。
关于 Activity Result API,郭神这篇文章很好地介绍过了:Activity Result API详解,是时候放弃startActivityForResult了
新的 API 不再需要我们维护 request code 了,我觉得这是一个大的提升。
但这个新 API 要求必须在 STARTED 生命周期前注册,也就是只能声明为全局饿汉式初始化,或者在 onCreate 中初始化,否则就会遇到这个 crash:
vbnet
java.lang.IllegalStateException: LifecycleOwner XXXActivity@85b22d0 is attempting to register while current state is RESUMED. LifecycleOwners must call register before they are STARTED.
at androidx.activity.result.ActivityResultRegistry.register(ActivityResultRegistry.java:121)
at androidx.activity.ComponentActivity.registerForActivityResult(ComponentActivity.java:885)
at androidx.activity.ComponentActivity.registerForActivityResult(ComponentActivity.java:894)
但有时候页面上只有一个小的功能需要用到这个回调,大部分用户都不会用到这个功能。所以这里最适合的初始化应该是懒汉式。
也就是说,用 Result API 多初始化了一个全局变量,用以前的 startActivityForResult 就没有这个问题。
虽然 Activity Result API 的注册本身非常轻量,不会带来太多性能开销,但却增加了代码复杂度。因为以前我读代码时,读到这个小功能才会看到 startActivityForResult,可以很自然地了解到它的作用。现在 Result API 是全局声明的,我会认为它是一个全局需要的东西,但实际上并不是,容易造成困惑。我觉得这是 Result API 的一个缺点。
所以要不要换成这个新 API 呢,我目前的选择是不换,jym 怎么看?