Изменение переменной в функции URLSession SWIFT?

Question

[uzolenta]

Здравствуйте! Помогите, спасите! Недавно изучаю свифт. Хочу запрос данных с сервера перенести в отдельный класс в виде функции. Перед тем, как задать вопрос здесь, я попробовал много разных вариантов, но userInfo так и не меняется в блоке do {
userInfo["firstName"]= responseApi.user[0].first_name
userInfo["lastName"] = responseApi.user[0].last_name
}
Пожалуйста, подскажите, как правильно сделать, если это реально вообще.

var userInfo = [String:String?]
func getInfoUser (id_users: String) -> [String: String?] {
     
    /*GET INFO USER*/
        guard let url = URL(string: Shared.apiUrl + "users/user.php") else { return userInfo}
    
    let postString = "id_users=" + id_users
    var request = URLRequest(url: url)
    request.httpMethod = "POST"
    request.httpBody = postString.data(using: .utf8)
    
    URLSession.shared.dataTask(with: request) {(data, response, error) in
    guard let data = data else {return}
    
    struct responseApi : Decodable {
    var error: errorApi?
    var response: String?
    var user: [userApi]
    }
    struct errorApi: Decodable {
    var error_code: String
    var error_msg: String
    }
    
    struct userApi: Decodable {
    //var : String
    }
    
    do {
    
    let responseApi = try JSONDecoder().decode(responseApi.self, from: data)
    
    /*CHECK API ERRORS*/
    if responseApi.error != nil{
    let error = responseApi.error?.error_code
    let error_msg = FuncClass().iserror(error_code: error!)
    }
    /*END CHECK API ERRORS*/
        
        userInfo["firstName"]= responseApi.user[0].first_name
        userInfo["lastName"] = responseApi.user[0].last_name
    }
        
    catch let error {
    print("ERROR CATCH")
        print(error)
    return
    }

    }.resume()
    /*END GET INFO USER*/
        
        
        return userInfo
        
        
    }

Answer 1

[Alexander D.]

Тут такая история... URLSession.shared.dataTask...resume() - асинхронный вызов, он выполняется после выхода из getInfoUser: (это еще мы не коснулись структур, что copy). Вам нужно сделать примерно так:

func getInfoUser (id_users: String, completion: @escaping (([String: String?])->())) {

а далее:

var userInfo: [String:String?] = [:]
userInfo["firstName"]= responseApi.user[0].first_name
userInfo["lastName"] = responseApi.user[0].last_name
comepletion(userInfo)

вызов:

getInfoUser(id_users: "", completion: {
~code here~
})

идея такая, писал без ide, так что примерно

Comments

[uzolenta]

Благодарю за ответ!
Либо я не до конца понял, что и куда, но попытался уже и так и сяк Ваш вариант использовать. Но опять же в блоке do { } пишет вообще, что Use of unresolved identifier 'comepletion'

Возможно надо поменять .resume() на что-то или не в этом дело?

[uzolenta]

Переписал вот так, правильно?

func getInfoUser (id_users: String, completion: @escaping (([String: String?]) -> ())){
 var userInfo = [String:String?]()
    guard let url = URL(string: Shared.apiUrl + "users/user.php") else { return }
    
    let postString = "id_users=" + id_users
    var request = URLRequest(url: url)
    request.httpMethod = "POST"
    request.httpBody = postString.data(using: .utf8)

    URLSession.shared.dataTask(with: request) {(data, response, error) in
    
        let data = data
        
        do {
            let response = try JSONDecoder().decode(responseApi.self, from: data!)
            
            userInfo["firstName"] = response.user[0].first_name
           
            completion(userInfo)
        } catch {
            print("didnt work")
        }

    }.resume()

    
    }

Но как вызывать функцию, чтобы она просто возвращала мне userInfo? Я чую, что победа близко, помогите :)

[uzolenta]

Дело то еще в том, что я эти данные загружаю при загрузке ViewController'a. Мне нужно в блоке do {} получить ссылку на 2 аватара пользователя (большой и маленький), соответственно, их тоже подгрузить во время загрузки VC. Как это все сделать, что-то я слабо понимаю сейчас. Пробую и так и сяк.

[Alexander D.]

Можно использовать DispatchSemaphore, но это - не очень хорошая идея. При загрузке контроллера стартуйте загрузку файла и обрабатывайте по результату загрузки в асинхронном режиме. Иначе загрузка файла будет вашать интерфейс. Да и зачем? Учитесь работать с асинхронными вызовами.

[uzolenta]

Александр, так подскажите как правильно с ними работать. Я же не просто так тут вопрос задаю, а хочу научиться :) при загрузке контроллера я получаю текстовую информацию, в том числе ссылки на картинки, которые мне тоже надо отобразить при загрузке.

[briahas]

ИМХО, понимать асинхронные операции это правильно, вот только вы выбрали сложный случай, потому что у вас тут два асинхронных сценария:
- показ UI
- подгрузка по сети

Для такого народ использует уже написанные либы, типа Kingfish, потому что самому писать конечно круто, но долго и ноги себе все переломаешь.

Советую выбрать что-то одно: подгрузку и вывод в консоль.

[Alexander D.]

uzolenta, прошу прощения - не заметил вопрос. разделяйте операции с данными, которые уже есть и с теми, которых еще нет. первые отображайте сразу, а вторые - загружайте асинхронно, по очереди или вместе. Вам стоит обратить внимание на работу с блоками(замыканиями), потом почитать про потоки и асинхронное выполнение, GCD. там на пальцах не получится, надо читать. удачи Вам и терпения :)

[Alexander D.]

Если мой ответ оказался полезным - напоминаю про функцию выбора правильного ответа :)