スイフト3可変自己消化

Question

私は非常に奇妙なことがあります。 私の簡単な関数では、設定パラメータの辞書を含む変数を作成します。それは 'let'として設定されているので、内部ループはただそれを読み込みます。 ループタイムのランダムな瞬間に、 "未解決の設定"でクラッシュします。 スムースのように見えないようです。それは誰ですか？

private static func preferencesFilter(userIDs: [Int], access: String) -> [User] { 
    self.sharedInstance.delegate?.updateActionLabel(label: "Filter") 
    var result = [VKUser]() 
    let settings = self.parseSettings() 
    let progressFraction = 1.00/Float(userIDs.count) 
    var n = 0 
    for userID in userIDs { 
     if sharedInstance.stopped { 
      return [] 
     } 
     n += 1 
     let user = VKUser.getUser(id: userID, access_token: access_token) 
     if settings["gender"] != nil { 
      if user.sex == settings["gender"] as! String { 
       if (user.born?.isBetweeen(date1: settings["minAge"] as! Date, date2: settings["maxAge"] as! Date))! { 
        if settings["country"] != nil { 
         if user.country == settings["country"] as! String { 
          result.append(user) 
         } 
        } 
        else { 
         result.append(user) 
        } 
       } 
      } 
     } 
     else { 
      if (user.born?.isBetweeen(date1: settings["minAge"] as! Date, date2: settings["maxAge"] as! Date))! { 
       if settings["country"] != nil { 
        if user.country == settings["country"] as! String { 
         result.append(user) 
        } 
       } 
       else { 
        result.append(user) 
       } 
      } 
     } 
     self.sharedInstance.delegate?.updateProgress(value: Float(n) * progressFraction) 
    } 
    return result 
} 

Answer 1

私はのようなものより迅速にコードをリファクタリング：

private static func preferencesFilter(userIDs: [Int], access_token: String) -> [User]? { 
    guard userIDs.count > 0 else { 
     return [User]() // no input, return empty list 
    } 
    let settings = self.parseSettings() 
    guard let minAge = settings["minAge"] as? Date, 
     let maxAge = settings["maxAge"] as? Date 
     else { 
      return nil 
    } 
    let country = settings["country"] as? String // specified or nil 
    let gender = settings["gender"] as? String  // specified or nil 

    sharedInstance.delegate?.updateActionLabel(label: "Filter") 
    var result = [VKUser]() 
    let progressFraction = 1.00/Float(userIDs.count) 
    var n = 0 
    for userID in userIDs { 
     if !sharedInstance.stopped { 
      n += 1 
      let user = VKUser.getUser(id: userID, access_token: access_token) 
      var shouldInclude = true 

      if user.sex != gender { // wrong sex or no required gender specified 
       shouldInclude = false 
      } 
      if user.country != country { // wrong country or no required country specified 
       shouldInclude = false 
      } 
      if let born = user.born { 
       if !born.isBetweeen(date1: minAge, date2: maxAge) { 
        shouldInclude = false 
       } 
      } else { // no user.born date, cant check if in range 
       shouldInclude = false 
      } 

      if shouldInclude { 
       result.append(user) 
      } 
      sharedInstance.delegate?.updateProgress(value: Float(n) * progressFraction) 
     } 
    } 
    return result 
} 

はあなたが書くために意図したものということですか？どのようにあなたのために実行されていますか？ これを非静的メソッドに変更できますか？私にはもっと意味がある。 このメソッドは、nilで失敗する可能性があるので、これはオプションのnowを返します。あなたの呼び出しコードはそれを正しく処理する必要があります。