-
-
Notifications
You must be signed in to change notification settings - Fork 69
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Bug]:开启混淆后,Kotlin的data class 内部类解析失败。 #43
Comments
小伙子,找不到类这个问题跟 Gson 解析没有任何关系吧?类找不到是因为 ClassLoader 没有加载所在类的 dex 文件到 dex 数组中,这个问题大概率是在编译的过程中,出现问题的类没有编译进去导致的,你可以用 jadx 反编译核查一下。 |
使用Gson直接解析和8.0版本都是好的。主要还是混淆的问题,有些地方用的反射 |
@rejigtian 小伙子,Kotlin data class 使用 Gson 和 GsonFactory 8.0 版本去解析都是有 Bug 的,看似能解析,但是存在较大的问题,具体有什么问题你可以回看一下版本更新记录,就算 8.0 的版本能解析,我不推荐你用。 |
demo代码在这里,在demo基础上,新增了kt类和混淆规则 |
你这个是类找不到的异常,属于运行环境的问题,GsonFactory 是 Json 框架,不存在任何类加载的逻辑,除非你有直接的证据证明这个就是 GsonFactory 的问题,否则我将不会介入处理。 |
这边看8.0的代码KotlinDataClassDefaultValueConstructor这个类中没有使用Kt的反射去构造,所以是正常的。 |
@rejigtian 8.0 那种反射创建是存在缺陷的,并且无法解决,一旦 kotlin data class 在字段超过 32 个的情况下默认值不生效的问题,还有一些其他别的缺陷,也是没有办法解决的,框架是不可能回退回到之前那种方法去反射类的构造函数。 |
这里直接这样调用混淆的类就有问题,也就是这个方案必须要要求数据类不被混淆,加入keep规则。 |
小伙子,这个问题不是 Kotlin 官方提供的反射库有问题?你试一下 |
找了相关资料,确实用KClass反射,不能去混淆。 |
不过使用GsonFactory的开发者可能并不知道其内部实现是KClass的反射机制,因为Gson库本身的解析方案也没用这个,使用GsonFactory主要还是希望对Gson库有一些优化,既然有差异,可能踩坑。感觉可以在文档中写一下。 避免构造方法被混淆。 Kt的封装确实很好用,但是如果有更好的方案去实例化对象也能解决该问题 |
框架版本【必填】
9.6
问题描述【必填】
开启混淆后,Kotlin的data class 内部类解析失败。
复现步骤【必填】
创建一个kotlin 文件
在其中定义多个data class ,并且相互有所引用,使用@SerializedName方法进行注解。
开启混淆,解析对应代码。
是否必现【必填】
是
项目 targetSdkVersion【必填】
33
出现问题的手机信息【必填】
全部
出现问题的安卓版本【必填】
全部
问题信息的来源渠道【必填】
自己遇到的
是部分机型还是所有机型都会出现【必答】
全部
框架最新的版本是否存在这个问题【必答】
是
框架文档是否提及了该问题【必答】
否
是否已经查阅框架文档但还未能解决的【必答】
是
issue 列表中是否有人曾提过类似的问题【必答】
否
是否已经搜索过了 issue 列表但还未能解决的【必答】
是
是否可以通过 Demo 来复现该问题【必答】
否
提供报错堆栈
提供截图或视频
No response
提供解决方案
堆栈中的com.wepie.wespy.module.party.home.PartyShareData是原类名,但是被混淆了。通过反编译kotlin的字节码发现里面确实有一些反射代码,所以理论上keep一下就可以。
可以通过添加混淆规则解决,但是看堆栈还是报在了sdk这里,正常java类直接这样用没什么问题,还是想知道与根本原因.
目前使用8.0版本也无该问题。
根据demo差异,发现我们并未使用kotlin的kotlin-reflect库,有空后排查一下这里。
The text was updated successfully, but these errors were encountered: