处理闭包-使代码更通用

Question

有两个功能，如下所示。 两者的大多数功能都相同。 它的想法是从getResponse（）[Helper Callback]获取Web服务的输出，解析并通过getResult（）将信息传递给包装器回调。

 static func getAllDealers(dealerSearchServiceDomain: ARSDealerSearchServiceDomain, wrapperCallback:(getResult: () throws -> Void) -> Void) throws
    {
        try ARSInputValidator.validateZipCode(dealerSearchServiceDomain.zip)

        try ARSDealerConnection.getAllDealers(dealerSearchServiceDomain, helperCallback: { (getResponse) -> Void in

            do
            {
                let result = try getResponse()

                try ARSDealerParser.parseDealerSearchResponse(dealerSearchServiceDomain)

                wrapperCallback(getResult: { return })
            }
            catch
            {
                wrapperCallback(getResult: { throw error })
            }
        })
    }

    static func getDealerDetails(dealerDetailsServiceDomain: ARSDealerDetailsServiceDomain, wrapperCallback:(getResult: () throws -> Void) -> Void) throws
    {
        try ARSDealerConnection.getDealerDetails(dealerDetailsServiceDomain, helperCallback: { (getResponse) -> Void in

            do
            {
                let result = try getResponse()

                try ARSDealerParser.parseDealerDetailsResponse(dealerDetailsServiceDomain)

                wrapperCallback(getResult: { return })
            }
            catch
            {
                wrapperCallback(getResult: { throw error })
            }

        })
    }

我正在尝试为常见功能添加单独的功能，例如，

static func parser(serviceCallDomain: ARSServiceCallDomain ,wrapperCallback:(getResult:() throws -> String) -> Void,  helperCallback:(getResponse:() throws -> String) -> Void) throws
    {
        helperCallback { (getResponse) -> Void in

但是有一个编译错误，我无法完成。 有15个以上的Web服务调用，因此我正在尝试显示的一个通用名称将非常有帮助。

下一步，我还需要将函数parseDealerSearchResponse（）和parseDealerDetailsResponse（）传递给通用函数。

我是封闭的新手。 请帮助。

//编辑-添加样本

我在Git中有一个问题的示例-请参阅类Layer1.swift https://github.com/vivinjeganathan/ErrorHandling/tree/Closures-Refactor

Answer 1

我认为重构代码的最佳方法是定义一个函数，该函数处理一些常见功能，例如解析和验证，并最终将完成闭包回调给控制器，如下所示：

static func handleResponse(parser: Parser, validator: Validator, getResult: () throws -> AnyObject, completion: (getParsedResult: () throws -> AnyObject) -> Void)  {
    do
    {
        let result = try getResult()
        let parsedObject = try parser.parse(result)
        try validator.validate(parsedObject)
        completion(getParsedResult: { return parsedObject })
    }
    catch
    {
        completion(getParsedResult: { throw error })
    }
}

请注意，它接收到解析器，验证器，从下面的层捕获结果的闭包以及属于最终用户（通常是View Controller）的完成闭包，然后可以按以下方式使用此函数：

static func getAllDealers(dealerSearchServiceDomain: AnyObject, wrapperCallback:(getResult: () throws -> AnyObject) -> Void) throws {
    let validator = DealersValidator() // create real validator
    let parser = DealersParser() // create real parser

    try validator.validate(dealerSearchServiceDomain)

    try ARSDealerConnection.getAllDealers(dealerSearchServiceDomain, helperCallback: { (getResponse) -> Void in
        self.handleResponse(parser, validator: validator, getResult: getResponse, completion: wrapperCallback)
    })
}

在这种情况下， handleResponse与getAllDealers处于同一类中，但实际上它可以是每个服务都可以调用的全局函数。

我认为可能可以使用泛型编写更好的实现，但是不会比这短得多，最后，您将无法避免创建验证器和解析器并调用下一层。