Question

我有以下代码，它使用Apache HTTP Client与外部系统（Capsule CRM）联系。

override fun findPartiesByUrlFragment(urlFragment: String): PartiesSearchResult {
    val req = HttpGet("https://api.capsulecrm.com/api/v2/parties/search?q=${urlFragment}")

    req.setHeader("Authorization", "Bearer ${ApiToken}")
    req.setHeader("Accept", "application/json");

    val httpClient = HttpClients.createDefault()
    val res = httpClient.execute(req)

    val parser = Parser()
    val json:JsonObject = parser.parse(res.entity.content) as JsonObject

    [...] // What happens thereafter isn't relevant because most errors are likely to occur in the code above

    return PartiesSearchResult(false, "", emptyList())
}

这段代码必须在全天候运行的应用程序中运行，因此我需要做正确的错误处理并确保整个应用程序不会崩溃，如果出现预期的问题（例如Capsule的服务器已关闭）。 false中的PartiesSearchResult(false, "", emptyList())表示操作失败。

我可以查看HTTP响应代码并仅解析响应，如果它是200。

但我很困惑，因为我在这里看不到任何声明的异常（而不是Java）。

现在我想这样做：

override fun findPartiesByUrlFragment(urlFragment: String): PartiesSearchResult {
    var httpClient = null
    var res = null
    try {
        val req = HttpGet("https://api.capsulecrm.com/api/v2/parties/search?q=${urlFragment}")

        req.setHeader("Authorization", "Bearer ${ApiToken}")
        req.setHeader("Accept", "application/json");

        httpClient = HttpClients.createDefault()
        res = httpClient.execute(req)

        if (res == null) {
            return PartiesSearchResult(false, "null response", emptyList())
        }

        if (res.entity == null) {
            return PartiesSearchResult(false, "null entity", emptyList())
        }

        val content = res.entity.content
        if (content == null) {
            return PartiesSearchResult(false, "null content", emptyList())
        }

        if (responseCode(res) == 200) {
            val parser = Parser()
            val json:JsonObject = parser.parse(content) as JsonObject

            [...] // Convert JSON to my own objects

            return PartiesSearchResult(true, "", emptyList())
        }
        return PartiesSearchResult(false, "Invalid HTTP response code", emptyList())
    }
    catch (Throwable t) {
        return PartiesSearchResult(false, toString(t), emptyList())
    }
    finally {
        closeIfNotNull(res)
        closeIfNotNull(httpClient)
    }

    return PartiesSearchResult(false, "", emptyList())
}

是否有更好的方法在Kotlin中进行错误处理（对于使用HTTP客户端从另一个系统获取数据的特定任务），如果我的系统因为预期的问题情况而不能失败（外部服务器关闭，外部服务器过载或有内部错误，暂时失去互联网连接等）？

Answer 1

您有多种选择可以更好地维护此方法。这些选项不一定是Kotlin特有的。更像是Refactoring和Object design。最明显的一种方法是将其拆分为多种方法。由于您的方法处理多个不同的问题，您可以将其拆分为更集中的部分。然后彻底测试， errorproof 和记录他们的行为。

例如：

第一部分处理HTTP GET请求。将此解压缩为返回GET响应的方法并检查connection exceptions。
可以提取中间部分作为GET响应的验证，以确保您获得有效的数据。
最后一部分是从GET响应中解析JSON的另一个问题。处理解析器异常的地方。

在Kotlin中使用HTTP客户端时如何正确处理错误？

1 个答案: