Как сделать так, чтобы функция возвращала значения типа Array?

Question

[nastya_zholudeva]

Хочу сделать универсальную функцию, которая бы делала запрос к API. Собственно вот она.

func request(uri: String, params: Dictionary<String, Any>) -> Array<Dictionary<String, String>> {
        Alamofire.request(self.url + uri, method: .get, parameters: params as! Parameters).responseJSON { response in
            if let json: [AnyObject] = response.result.value as? [AnyObject] {
                var items = [Dictionary<String, String>]()
                let data = JSON(json)
                for item in data {
                    items.append(["id": item.1["id"].stringValue, "title": item.1["title"].stringValue])
                }
                return items
            }
        }
    }

Но такой код выдает ошибку Unexpected non-void return value in void function. Как это можно сделать?

Comments

[NSA-bot]

А если зажать клавишу Alt и левой кнопкой мыши кликнуть на слово item в строке "return items", то что там напишет? Какой у item тип данных?

[nastya_zholudeva]

NSA-bot, вот

Answer 1

[doublench21]

Ну вы наверное просто забыли или не понимаете, что функция responseJson выполняется асинхронно. Это просто коллбэк, который выполнится тогд, когда сервер на ваш запрос вернёт Json. Но тк функция работает асинхронно в другом потоке, на текущем потоке после запроса к серверу ничего нет. И функция завершает свою работу. Но она ничего не возвращает, а вы указали что она должна что-то возвращать. Отсюда и ошибка про non Void.

То что вы возращаете item это хорошо. Ток он возвращается уже не в этой функции.

Если вы хотите что бы эта функция возвращала ответ сервера, то вам нужно каким то образом ждать ответа от асинхронной функции. Может воспользоваться DispatchGroup и wait.

А вообще так делать не нужно. Весь смысл асинхронности пропадёт. Эта функция не должна ничего возвращать. А сам факт того что пришли данные должен обрабатываться в responseJson или в наблюдателе переменной куда вы присвоите то что получите.

Comments

[nastya_zholudeva]

doublench21, а как тогда мне эту функцию можно поправить, чтобы заработало?

[doublench21]

nastya_zholudeva, дополнил ответ. Если Вам это вот хоть убей нужно, то я дополню ответ до того что Вы хотите. Нл я вас уверяю, вам это не нужно)

[nastya_zholudeva]

doublench21, цель моей функции - это чтобы я с любого места кода могла делать запрос к API. Сейчас у меня была идея сделать данную функцию классом, потом в переменную кладу экземпляр класса и делаю запрос. Примерно так

let api = JsonPlaceholder()
        let items = api.request(uri: <#T##String#>, params: <#T##Dictionary<String, Any>#>)

Подскажите, как тогда иначе можно реализовать мою идею?

[doublench21]

Какова суть апи? Сделать запрос с нужными параметрами и получить ответ. Ваша функция с этим справляется вполне. Только нужно учесть, что данные от сервера возвращаются уже после того, как это функция завершит свою работу. Поэтому у Вас есть тут следующий выход. В функции responceJson обработать ответ и записать ответ скажем в некое св-во класса, скажем в
var data: [[String : String]]?

Плюс Вам нужно добавить наблюдателя к этому св-ву, что бы быть в курсе, когда туда запишутся новые данные
var data: [[String : String]]? {
didSet { // что-то делаем с полученными данными }
}

[doublench21]

class SomeClass {

private _data: [[String: String]]? {
  didSet {
   /* Тут я получил данные и что-то с ними уже делаю. Этот код выполнится так скоро, как запрос запишет ответ в данную переменную */
 }
}

...

private func request(uri: String, params: [String : Any])  {
        Alamofire.request(self.url + uri, method: .get, parameters: params as! Parameters).responseJSON { response in
            if let json: [AnyObject] = response.result.value as? [AnyObject] {
                var items = [[String : String]]()
                let data = JSON(json)
                for item in data {
                    items.append(["id": item.1["id"].stringValue, "title": item.1["title"].stringValue])
                }
                _data = items
            }
        }
    }

...

}

Answer 2

[briahas]

Либо сделать так как советует doublench21 :

private _data: [[String: String]]? {
  didSet {
   /* Тут я получил данные и что-то с ними уже делаю. Этот код выполнится так скоро,
 как запрос запишет ответ в данную переменную */
 }
}

Либо, переписать функцию чтоб был completion блок:

func request(uri: String, params: Dictionary<String, Any>, completion: ( Array<Dictionary<String, String>>)->())  {
        Alamofire.request(self.url + uri, method: .get, parameters: params as! Parameters).responseJSON { response in
            if let json: [AnyObject] = response.result.value as? [AnyObject] {
                var items = [Dictionary<String, String>]()
                let data = JSON(json)
                for item in data {
                    items.append(["id": item.1["id"].stringValue, "title": item.1["title"].stringValue])
                }
                completion(items)
            }
        }
    }

, в последнем случае, применение функции будет следующим:

classElement.request(uri: myPath, params: requestParams) { items in
    print(items)
}