Skip to content

Pull Request申请#12

Open
chuckiefan wants to merge 5 commits into
Victory-Over:masterfrom
chuckiefan:master
Open

Pull Request申请#12
chuckiefan wants to merge 5 commits into
Victory-Over:masterfrom
chuckiefan:master

Conversation

@chuckiefan

Copy link
Copy Markdown

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

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

2. use retrofit2 & RxJava
* cache:
  download cache
  1. support Kotlin 2. use retrofit2 & RxJava
@Victory-Over

Copy link
Copy Markdown
Owner

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

@chuckiefan

Copy link
Copy Markdown
Author

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

@Victory-Over

Copy link
Copy Markdown
Owner

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

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

@chuckiefan

Copy link
Copy Markdown
Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants