Swift中嵌套数据任务实现方案求点评：先获JSON再取图片

这个方案完全可行，逻辑上能按预期完成「先获取包含图片URL的JSON数据，再下载对应图片」的需求，但在错误处理、代码可读性和健壮性上还有不少可以优化的地方。

一、方案核心可行性说明 #

嵌套的URLSession.dataTask确实能保证请求的顺序执行：第一个dataTask完成并解析出图片URL后，才会启动第二个dataTask下载图片，最终回到主线程更新UI，这个执行流程是通顺的，能实现你想要的功能。

二、现存问题 #

1. 错误处理过于激进：多处使用fatalError()，一旦出现URL构造失败、JSON解码失败、图片转换失败等情况，会直接导致App崩溃。实际开发中这是不可接受的，应该用更友好的方式处理错误（比如通过回调传递错误、记录错误日志）。
2. 回调嵌套导致可读性下降：两层dataTask嵌套已经开始出现「回调地狱」的雏形，如果后续需要增加更多请求步骤，代码会变得臃肿难维护。
3. 缺少HTTP响应校验：没有检查响应的状态码（比如仅200-299区间才视为请求成功），可能拿到错误的响应数据却继续后续处理。
4. URL构造存在风险：直接用字符串拼接API Key，如果参数包含特殊字符（如&、=）会导致URL无效，安全性和兼容性不足。

三、优化后的实现方案 #

推荐使用Swift 5.5+引入的Async/Await语法替代嵌套回调，同时完善错误处理和URL构造逻辑：

// 定义自定义错误类型，统一处理各类请求错误
enum FetchError: Error {
    case invalidURL
    case invalidHTTPResponse
    case dataMissing
    case jsonDecodeFailed
    case imageConversionFailed
}

func fetchImageOfTheDay() async throws {
    // 用URLComponents构造安全的请求URL，避免字符串拼接的风险
    guard var urlComponents = URLComponents(string: pictureOfTheDayUrl) else {
        throw FetchError.invalidURL
    }
    urlComponents.queryItems = [URLQueryItem(name: "api_key", value: api_key)]
    guard let requestUrl = urlComponents.url else {
        throw FetchError.invalidURL
    }
    
    // 第一步：请求图片信息JSON
    let (jsonData, jsonResponse) = try await URLSession.shared.data(from: requestUrl)
    guard let httpResponse = jsonResponse as? HTTPURLResponse, (200...299).contains(httpResponse.statusCode) else {
        throw FetchError.invalidHTTPResponse
    }
    let imageText = try JSONDecoder().decode(ImageOfTheDay.self, from: jsonData)
    
    // 第二步：下载图片
    guard let imageUrl = URL(string: imageText.url) else {
        throw FetchError.invalidURL
    }
    let (imageData, imageResponse) = try await URLSession.shared.data(from: imageUrl)
    guard let imageHttpResponse = imageResponse as? HTTPURLResponse, (200...299).contains(imageHttpResponse.statusCode) else {
        throw FetchError.invalidHTTPResponse
    }
    guard let image = UIImage(data: imageData) else {
        throw FetchError.imageConversionFailed
    }
    
    // 回到主线程更新UI
    await MainActor.run {
        self.imageOfTheDay = image
        self.imageText = imageText
    }
}

如果需要兼容Swift 5.5以下版本，可以把每个请求封装成独立函数，用闭包链式调用替代嵌套，同样能提升代码可读性。

内容的提问来源于stack exchange，提问作者Kuba Lhotský