SuperWeb icon indicating copy to clipboard operation
SuperWeb copied to clipboard

Pull Request申请

Open chuckiefan opened this issue 6 years ago • 4 comments

鲁宇峰: 你好! 很感谢您开源的SuperWeb正好解决我的文档访问问题! 今天在使用时做了两处调整,请审核:

  1. 迁移至AndroidX 2.新增DocumentHelper类,从网络获取文档并缓存

chuckiefan avatar Jan 08 '19 12:01 chuckiefan

你好! 感谢你对本项目的贡献! 开源项目为很多人服务,所以应该考虑使用者的简洁轻便, 不宜在项目中使用retrofit2、gson等第三方框架,而使使用者强制引入了该框架。 (新增DocumentHelper类,从网络获取文档并缓存)之前也有想法做此工具类, 迫于时间关系还未完成,感谢你也有同样的想法,并实现。 但最优的解决方式应使用Android原生网络请求以及json的解析来做这件事事情。 如果你有多余的时间可以尝试修改,如果没有的话,我会根据你提供的DocumentHelper 做适当的调整。 辛苦,谢谢!

Victory-Over avatar Jan 09 '19 01:01 Victory-Over

你好! 感谢您的回复和建议! 由于工作的紧急要求,当时只能采用Retrofit2+RxJava的方式快速实现,没有考虑到开源项目的性质,这是我的疏忽。后面有多余的时间我会尝试修改一版,如果来不及,期待您增加网络获取文件并缓存的功能 :) 此外请问您对使用RxJava和Kotlin持怎样的态度?如果觉得不妥我在修改时会根据您的意见调整。 再次感谢您的回复!

chuckiefan avatar Jan 09 '19 02:01 chuckiefan

我公司目前的网络库也是采用Retrofit2+RxJava实现的,这种方式也是当前热门。 但在开源项目中,除非该项目跟第三方框架有直接关系,比如说在做二次封装网络库时, 你有需要或者必要使用Retrofit2+RxJava此类框架,此时使用完全没问题。 但是本项目SuperWeb只是跟网络有稍许关系,并且可以使用其他方案替代,所以不建议此类方式实现。

Kotlin确实是很好的语言,但目前的普及性还是不够,自己的Demo中是可以使用学习的, 如果公司项目对语言没有做限制,可以推广使用的,但在开源项目中还是不建议使用, 原因有2: 1、你无法保证使用者项目是否有做Kotlin配置 2、对于未接触过Kotlin的使用者来说,可能阅读你的源码有一定的难度

Victory-Over avatar Jan 09 '19 03:01 Victory-Over

我会根据你的意见进行调整,感谢回复!

chuckiefan avatar Jan 09 '19 03:01 chuckiefan