zastosowanie zamknięcia braku ucieczki parametru może pozwolić uciec

Question

mam protokołu:

enum DataFetchResult { 
    case success(data: Data) 
    case failure 
} 

protocol DataServiceType { 
    func fetchData(location: String, completion: (DataFetchResult) -> (Void)) 
    func cachedData(location: String) -> Data? 
} 

Z przykładowej implementacji:

/// An implementation of DataServiceType protocol returning predefined results using arbitrary queue for asynchronyous mechanisms. 
    /// Dedicated to be used in various tests (Unit Tests). 
    class DataMockService: DataServiceType { 

     var result  : DataFetchResult 
     var async  : Bool = true 
     var queue  : DispatchQueue = DispatchQueue.global(qos: .background) 
     var cachedData : Data? = nil 

     init(result : DataFetchResult) { 
      self.result = result 
     } 

     func cachedData(location: String) -> Data? { 
      switch self.result { 
      case .success(let data): 
       return data 
      default: 
       return nil 
      } 
     } 

     func fetchData(location: String, completion: (DataFetchResult) -> (Void)) { 

      // Returning result on arbitrary queue should be tested, 
      // so we can check if client can work with any (even worse) implementation: 

      if async == true { 
       queue.async { [weak self ] in 
        guard let weakSelf = self else { return } 

        // This line produces compiler error: 
        // "Closure use of non-escaping parameter 'completion' may allow it to escape" 
        completion(weakSelf.result) 
       } 
      } else { 
       completion(self.result) 
      } 
     } 
    } 

Kod powyżej zestawiane i pracuje w Swift3 (Xcode8- beta5), ale już nie działa z wersją beta 6. Czy możesz wskazać mi na przyczynę?

Answer 1

Jest to spowodowane zmianą domyślnego zachowania parametrów funkcji. Przed wersją Swift 3 (konkretnie kompilacja dostarczona z beta-beta 8 beta 6) domyślnie uniknęliby - musielibyśmy oznaczyć je jako @noescape, aby zapobiec ich przechowaniu lub przechwyceniu, gwarantując, że nie będą się nazywać po wyjściu funkcji.

Jednak teraz domyślnie jest @noescape - należy teraz oznaczyć parametry funkcji jako @escaping, aby poinformować kompilator, że mogą być przechowywane lub przechwytywane.

protocol DataServiceType { 
    func fetchData(location: String, completion: @escaping (DataFetchResult) -> Void) 
    func cachedData(location: String) -> Data? 
} 

func fetchData(location: String, completion: @escaping (DataFetchResult) -> Void) { 
    // ... 
} 

Zobacz Swift Evolution proposal uzyskać więcej informacji na temat tej zmiany.

Answer 2

Od @noescape jest domyślnym, tam 2 opcje, aby naprawić ten błąd: tylko zaznaczyć

1) @Hamish wskazał w swojej odpowiedzi na zakończenie jak @escaping jeśli dbam o wyniku i naprawdę chcesz go do ucieczki (to prawdopodobnie jest w przypadku @ pytanie Łukasza z testami urządzenia jako przykład i możliwość zakończenia asynchroniczny)

func fetchData(location: String, completion: @escaping (DataFetchResult) -> Void) 

LUB

2) zachować domyślne zachowanie @noescape poprzez ukończenie opcjonalne odrzucając wyniki altoget jej w przypadkach, gdy nie dbasz o wynik. Na przykład, gdy użytkownik już "odszedł", a kontroler widoku wywoływania nie musi zawieszać się w pamięci tylko z powodu nieostrożnego połączenia sieciowego. Tak jak było w moim przypadku, kiedy przyjechałem tutaj szukając odpowiedzi, a przykładowy kod nie był dla mnie zbyt trafny, więc oznaczenie @noescape nie było najlepszą opcją, chociaż brzmiało ono jako jedyne od pierwszego spojrzenia.

func fetchData(location: String, completion: ((DataFetchResult) -> Void)?) { 
    ... 
    completion?(self.result) 
}