两点建议望采纳 - Githubissues

zxyaust commented 4 years ago

1.SimpleResponse.<S, F>newBuilder() .code(httpEntity.code) .headers(response.headers()) .fromCache(fromCache) .succeed(succeedData) .failed(failedData) .build(); 这个SimpleResponse 希望加入一个msg信息,因为请求成功的情况下很多时候也需要把msg信息toast显示一下,当前请求成功时传不出msg信息,只有请求失败时才能传出msg 2.我发现没有地方打印出请求url和参数,是否考虑SimpleResponse 中能获取到请求对象,再此处打印一下日志,毕竟调试的时候想要打印请求参数,看看参数是否传入正确

另外,这个库真的是非常简单方便,不向别的,一大堆配置,学习成本高,二次封装麻烦,果然是大神,希望以后越来越完美

liangjingkanji commented 4 years ago

你说的msg是你后端返回的内容吧, 这个应该自己根据后端定义的错误码去取字段

你说的SimpleResponse中获取请求信息, https://github.com/liangjingkanji/Net 我实现了, 你可以参考或者直接使用, 并且我的错误处理方案比Kalle要简单和强大很多, 可以追踪到所有的网络错误

请求参数日志推荐在拦截器中做, 响应参数信息推荐在转换器中做

liangjingkanji commented 4 years ago

不过我目前支持的是最新的Kotlin/协程作用域的用法, 不会协程也可以使用的

liangjingkanji commented 4 years ago

Convert转换器中属于异步线程, 吐司应该考虑到这个问题否则会崩溃. 或者使用我的这个吐司库, 支持异步和全局配置以及解决覆盖问, https://github.com/liangjingkanji/Tooltip

yanzhenjie / Kalle

两点建议望采纳 #100