Обработчик завершения - Parse + Swift

Question

Я пытаюсь создать массив PFObjects, называемый «areaList». Я немного разбираюсь в этом и понимаю, что я мог бы воспользоваться обработчиком завершения для обработки асинхронного характера загруженных результатов. Мой вопрос, в частности, заключается в том, чтобы получить некоторые рекомендации о том, что я делаю неправильно, а также о потенциальных советах о том, как добиться результата «лучше».

Вот моя функция запроса с обработчиком завершения:

func loadAreasNew(completion: (result: Bool) ->()) -> [Area] { 
     var areaList = self.areaList 
     let areaQuery = PFQuery(className: "Area") 
     areaQuery.findObjectsInBackgroundWithBlock { 
      (areas: [PFObject]?, error: NSError?) -> Void in 
      if error == nil { 
       for area in areas! { 
        let areaToAdd = area as! Area 
        areaList.append(areaToAdd) 
//     print(areaList) // this prints the list each time 
//     print(areaToAdd) // this prints the converted Area in the iteration 
//     print(area) // this prints the PFObject in the iteration 
        if areaList.count == areas!.count { 
         completion(result: true) 
        } else { 
         completion(result: false) 
        } 
       } 
      } else { 
       print("There was an error") 
      } 
     } 
     return areaList 
    } 

Вот как я пытаюсь вызвать его в viewDidLoad:

loadAreasNew { (result) ->() in 
    if (result == true) { 
     print(self.areaList) 
    } else { 
     print("Didn't Work") 
    } 
} 

я присвоил эту переменную перед viewDidLoad:

var areaList = [Area]() 

В консоли я получаю следующее:

Didn't Work 
Didn't Work 
Didn't Work 
Didn't Work 
[] 

Представляя 5 пунктов, которые я знаю, есть в Parse ...

Answer 1

Это интересный вопрос. Во-первых, PFQuery в основном имеет встроенный обработчик завершения, что очень приятно! Как вы, наверное, знаете, весь код в areaQuery.findObjectsInBackgroundWithBlock {...} запускает ПОСЛЕ ответа сервера. Заполнение чаще всего служит для создания блока, с возможностью асинхронного возвращения данных и ошибок.

Лучшая практика (IMO), чтобы просто вызвать код, который вы хотите использовать, с результатами вашего PFQuery сразу после вашего цикла добавления области (который я собираюсь вынуть, потому что я придирчив к этому), например так:

 func loadAreasNew() { 
     var areaList = self.areaList 
     let areaQuery = PFQuery(className: "Area") 
     areaQuery.findObjectsInBackgroundWithBlock { 
      (areas: [PFObject]?, error: NSError?) -> Void in 
      if error == nil { 
       let areasFormatted = areas! As [Areas] 
       areasList += areasFormatted 
       //Something like this 
       self.codeINeedAreasFor(areasList) 
       } 
      } else { 
       print(error) 
      } 
     } 
    } 

ОДНАКО! Если вы, действительно, считают, что использовать некоторые завершающие обработчики, заканчивая this other answer для получения дополнительных сведений о способах их использования. Но имейте в виду, что у всех инструментов есть время и место ...

Answer 2

Здесь есть несколько проблем.

Ваш обработчик завершения не требует, чтобы определить имя для параметров ваших завершающих обработчика, так что вы можете легко использовать completion: (Bool) ->()

Далее в функции, вы возвращение areaList. Это нужно поместить через обработчик завершения, как этот onComplete(areaList), и изменить параметр обработчика завершения, чтобы ожидать, что ваш список областей.

Тогда при вызове функции, она может выглядеть следующим образом:

loadAreasNew { result in 
    if (result == true) { 
     print(self.areaList) 
    } else { 
     print("Didn't Work") 
    } 
} 

Answer 3

Вот моя забота:

1) Не передавайте в локальной переменной и сделать возврат функции это, это бессмысленно и опасно.

Возможно, вы захотите инициировать пустой массив и сделать свою выборку, а затем «вернуть» ее.

2) Запрос на выборку обрабатывается в фоновом режиме, вы не будете знать, когда он будет завершен. Если вы немедленно вернете массив, он всегда будет пустым массивом.

Положите «возврат» в ваше завершение.

3) Parse уже имеет метод проверки расстояния, вам не нужно делать это вручную. aPARSEQUERRY.where (ключ:, nearGeoPoint:, inKilometers :)

Я переписать функцию:

func loadNewAreas(completion:([Area],err?)->()){ 
    let areaQuery = PFQuery(className: "Area") 
    areaQuery.where("location",nearGeoPoint:MYCURRENTLOCATION,inKilometers:50) 
    areaQuery.findObjectInBackgroundWithBlock(){objects,err 
     if objects.count == 0 
     { 
      completion([],err) 
     } 
     let areas = Area.areasFromPFObjects(objects) 
     completion(areas,err) 
    } 
}