googolmo / OkVolley

A volley library used okhttp
Apache License 2.0
71 stars 27 forks source link

增加不同请求对比 #4

Closed cxyxlxdm closed 8 years ago

cxyxlxdm commented 8 years ago
  1. 增加了 Gson 对 解析 Json 进行解耦;
  2. 增加了 Demo 里 Retrofit 请求与 OKVolley 的对比;
  3. Retrofit 请求的实现用了两种方式:Callback 和 RxJava 。
googolmo commented 8 years ago

okvolley 是一个比较纯洁的基础framework 我想不太应该将 gson 引入, 因为有可能使用jackson而不是json

我个人比较建议增加一个subproject 是继承 okvolley 是专门使用 gson 的。。。而且可以增加jackson什么的

在我记忆中 以前好象是有这个一个subproject 支持gson的 但是我忘记后来为什么我给去掉了。。。

/*** *

On Sat, Oct 24, 2015 at 6:17 PM, Ailurus notifications@github.com wrote:

  1. 增加了 Gson 对 解析 Json 进行解耦;
    1. 增加了 Demo 里 Retrofit 请求与 OKVolley 的对比;
    2. Retrofit 请求的实现用了两种方式:Callback 和 RxJava 。

You can view, comment on, or merge this pull request online at:

https://github.com/googolmo/OkVolley/pull/4 Commit Summary

  • 1.增加了 Gson 对 解析 Json 进行解耦;

File Changes

Patch Links:

— Reply to this email directly or view it on GitHub https://github.com/googolmo/OkVolley/pull/4.

cxyxlxdm commented 8 years ago

嗯,理解的。一开始我也是把 Gson 那一部分写在了 Demo 里,不过我以为 fork 过来的项目不会 pull 到原作者的项目信息里,就放在了 library 里,主要还是侧重于 Demo 里不同请求方式的对比。谢谢你的项目。