Ошибка CLGeocoder EXC_BAD_INSTRUCTION

Question

Я использую CLGeocoder reverseGeocodeLocation. Я получаю крах после запуска в течение 5-10 минут (без заметной картины) и случайных сбоев. Вот мой код:

if CLLocationManager.authorizationStatus() == .AuthorizedWhenInUse { 

     let currentLatCoord = manager.location?.coordinate.latitude 
     let currentLongCoord = manager.location?.coordinate.longitude 

     CLGeocoder().reverseGeocodeLocation(CLLocation(latitude: currentLatCoord!, longitude: currentLongCoord!)) { (placemarks, error) -> Void in 

      if error != nil { 
       print(error) 
       return 
      } 

      let placeArray = placemarks as [CLPlacemark]! 
      var placeMark: CLPlacemark 

      placeMark = placeArray![0] 

      self.locationLabel.text = String(placeMark.addressDictionary?["Thoroughfare"]!) 
     } 
    } 

А также, только, чтобы помочь, вот картина линии и ошибки:

Answer 1

Я думаю, что вам нужно некоторые дополнительные привязки:

if let thoroughfare = placeMark.addressDictionary?["Thoroughfare"] as? String { 
    self.locationLabel.text = thoroughfare 
} 

Я предполагаю, что в адресном словаре не может быть ключ "Thoroughfare", и вы предоставляете значение nil назначенным инициализатор для String.

Есть ли вероятность, что представление, обновляемое в фрагменте кода, не находится на экране (удалено), когда CLGeocoder закончил свое обратное геокодирование? Если розетка определяется как неявно развернутый дополнительно:

@IBOutlet var locationLabel : UILabel!

мне интересно, если он уже установлен на nil, но из-за взрыва (!) компилятор не делает вас проверить ,

Но, конечно, если ваше представление по-прежнему находится на экране при крахе, это, вероятно, не проблема.

Answer 2

Вы дали нам пример кода:

let currentLatCoord = manager.location?.coordinate.latitude 
let currentLongCoord = manager.location?.coordinate.longitude 

CLGeocoder().reverseGeocodeLocation(CLLocation(latitude: currentLatCoord!, longitude: currentLongCoord!)) { (placemarks, error) -> Void in 

    if error != nil { 
     print(error) 
     return 
    } 

    let placeArray = placemarks as [CLPlacemark]! 
    var placeMark: CLPlacemark 

    placeMark = placeArray![0] 

    self.locationLabel.text = String(placeMark.addressDictionary?["Thoroughfare"]!) 
} 

Вы можете более корректно обрабатывать nil значения, если вы используете if let конструкцию:

CLGeocoder().reverseGeocodeLocation(manager.location!) { placemarks, error in 
    guard error == nil else { 
     print(error) 
     return 
    } 

    if let placemark = placemarks?.first { 
     self.locationLabel.text = placemark.thoroughfare 
    } 
} 

И, конечно, если вы звоните в этом неоднократно, я бы не стал повторно создавать новый CLGeocoder каждый раз, но, надеюсь, это иллюстрирует шаблон.

Но, как вы можете видеть, вы можете избежать выделения широты и долготы из свойства location, чтобы создать новый объект CLLocation, просто используя manager.location. Аналогично, вы можете использовать свойство thoroughfare, что избавит вас от необходимости использовать значение addressDictionary.

Ключевое наблюдение, упомянутое выше Крейгом, заключается в том, чтобы неукоснительно использовать оператор принудительной разворачивания !, если вы не уверены, что переменная никогда не может быть nil. Аналогично, не используйте синтаксис [0], если вы не знаете, что в массиве есть хотя бы один элемент (поэтому я использую first, который является необязательным, для которого я могу легко протестировать).

Честно говоря, я бы даже убедиться, что location был действительным (не nil и с неотрицательным horizontalAccuracy, а отрицательное значение указывает на то, что координаты являются недопустимыми):

if let location = manager.location where location.horizontalAccuracy >= 0 { 
    CLGeocoder().reverseGeocodeLocation(location) { placemarks, error in 
     guard error == nil else { 
      print(error) 
      return 
     } 

     if let placemark = placemarks?.first { 
      self.locationLabel.text = placemark.thoroughfare 
     } 
    } 
}