search

kanyun-inc / Kace

Kace: Kotlin Android Compatible Extensions, a framework for assisting in the seamless migration from kotlin-android-extensions
Apache License 2.0
261 stars 11 forks source link

小建议：关于部分包名中含kotlin关键字所导致的异常 #29

Closed Petterpx closed 1 year ago

Petterpx commented 1 year ago

生成扩展属性时，有些View对应的包名是 in.com.xx、此时生成的扩展属性 findViewByIdCached 就会报错，可以考虑对于包名是kotlin关键字的增加 ``

RicardoJiang commented 1 year ago

Kotlin 的关键字还挺多的，如果每个 xml 的 tag 都需要判断一下包名是否包含这些关键字，对生成代码的性能还是有一定影响。最好还是避免使用这些关键字作为包名吧，或者将这些生成有问题的 layout 加入 blackList，禁止生成代码，手动迁移。

Petterpx commented 1 year ago

Kotlin 的关键字还挺多的，如果每个 xml 的 tag 都需要判断一下包名是否包含这些关键字，对生成代码的性能还是有一定影响。最好还是避免使用这些关键字作为包名吧，或者将这些生成有问题的 layout 加入 blackList，禁止生成代码，手动迁移。

项目本身的问题还好说，主要是某些三方库使用了，这就比较痛苦:(...

RicardoJiang commented 1 year ago

可以用一个项目中的自定义类继承三方库的类，然后在 xml 里使用自定义的？

Petterpx commented 1 year ago

可以用一个项目中的自定义类继承三方库的类，然后在 xml 里使用自定义的？

虽然可以这样，但仍希望这种事情是由工具来处理。