qingmei2 / MVVM-Architecture

The practice of MVVM + Jetpack architecture in Android.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

无网络情况下请求接口闪退

SnowyTomorrow opened this issue · comments

无网络情况下请求接口闪退
commented

这是因为协程方式请求没有捕获OkHttp抛出的异常,改造processApiResponse方法,将函数作为参数传入,如下:
suspend fun processResponse(request: suspend () -> Response): Results {
return try {
val response = request()
val responseCode = response.code()
val responseMessage = response.message()
if (response.isSuccessful) {
Results.success(response.body()!!)
} else {
loge { "else:" + responseMessage }
Results.failure(Errors.NetworkError(responseCode, responseMessage))
}
} catch (e: IOException) {
loge { "catch:" + e.toString() }
Results.failure(Errors.NetworkError())
}
}

这个具体应该怎么改动,上面的代码放在哪里,方便加你联系方式咨询你吗 @DaQinShgy

commented

这个具体应该怎么改动,上面的代码放在哪里,方便加你联系方式咨询你吗 @DaQinShgy

用项目中的登录举例:
val userInfoResponse = serviceManager.userService.fetchUserOwner()
processApiResponse(userInfoResponse)

processApiResponse方法中虽然有try catch,但如果serviceManager.userService.fetchUserOwner异常,比如timeout,这个时候是无法捕获的,导致app崩溃。改造processApiResponse方法,将函数作为参数传入,就可以避免这个问题,很多时候使用这个项目崩溃就是因为这个原因引起的
processApiResponse{
serviceManager.userService.fetchUserOwner()
}

@DaQinShgy

谢谢你指出了项目中的问题,正确的方式的确是「将函数作为参数交给processApiResponse」,这样网络请求的行为导致的异常就能够被 processApiResponse 函数中 try catch 捕获到。

很抱歉问题拖了很久,目前最新的代码已经修复,再次感谢。