Лучшая практика для реализации отказоустойчивого инициализатора в Swift

С помощью следующего кода я пытаюсь определить простой класс модели и его сбойный инициализатор, который принимает словарь (json-) в качестве параметра. Инициализатор должен вернуться, nilесли имя пользователя не определено в исходном json.

1. Почему код не компилируется? В сообщении об ошибке говорится:

Все сохраненные свойства экземпляра класса должны быть инициализированы перед возвратом nil из инициализатора.

В этом нет смысла. Зачем мне инициализировать эти свойства, когда я планирую вернуться nil?

2. Правильный ли мой подход или есть другие идеи или общие шаблоны для достижения моей цели?

class User: NSObject {

    let userName: String
    let isSuperUser: Bool = false
    let someDetails: [String]?

    init?(dictionary: NSDictionary) {
        if let value: String = dictionary["user_name"] as? String {
            userName = value
        }
        else {
           return nil
        }

        if let value: Bool = dictionary["super_user"] as? Bool {
            isSuperUser = value
        }

        someDetails = dictionary["some_details"] as? Array

        super.init()
    }
}

Обновление: из журнала изменений Swift 2.2 (выпущено 21 марта 2016 г.):

Инициализаторы назначенных классов, объявленные как сбойные или вызывающие, теперь могут возвращать nil или вызывать ошибку, соответственно, до полной инициализации объекта.

Для Swift 2.1 и ранее:

Согласно документации Apple (и ошибке вашего компилятора), класс должен инициализировать все свои сохраненные свойства перед возвратом nilиз неудачного инициализатора:

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

Примечание. На самом деле он отлично работает для структур и перечислений, но не для классов.

Предлагаемый способ обработки сохраненных свойств, которые не могут быть инициализированы до сбоя инициализатора, - объявить их как неявно развернутые необязательные параметры.

Пример из документов:

class Product {
    let name: String!
    init?(name: String) {
        if name.isEmpty { return nil }
        self.name = name
    }
}

В приведенном выше примере свойство name класса Product определяется как имеющее неявно развернутый необязательный строковый тип (String!). Поскольку это необязательный тип, это означает, что свойство name имеет значение по умолчанию nil, прежде чем ему будет присвоено конкретное значение во время инициализации. Это значение по умолчанию, равное nil, в свою очередь, означает, что все свойства, представленные классом Product, имеют допустимое начальное значение. В результате сбойный инициализатор для продукта может вызвать сбой инициализации в начале инициализатора, если ему передается пустая строка, перед присвоением определенного значения свойству name в инициализаторе.

В вашем случае, однако, простое определение userNameкак a String!не устраняет ошибку компиляции, потому что вам все еще нужно беспокоиться об инициализации свойств в вашем базовом классе NSObject. К счастью, с userNameопределением как a String!, вы можете вызвать super.init()перед собой, return nilкоторый запустит ваш NSObjectбазовый класс и исправит ошибку компиляции.

class User: NSObject {

    let userName: String!
    let isSuperUser: Bool = false
    let someDetails: [String]?

    init?(dictionary: NSDictionary) {
        super.init()

        if let value = dictionary["user_name"] as? String {
            self.userName = value
        }
        else {
            return nil
        }

        if let value: Bool = dictionary["super_user"] as? Bool {
            self.isSuperUser = value
        }

        self.someDetails = dictionary["some_details"] as? Array
    }
}

В этом нет смысла. Зачем мне инициализировать эти свойства, если я планирую вернуть nil?

По словам Криса Латтнера, это ошибка. Вот что он говорит:

Это ограничение реализации в компиляторе swift 1.1, задокументированное в примечаниях к выпуску. В настоящее время компилятор не может уничтожить частично инициализированные классы во всех случаях, поэтому он не допускает формирования ситуации, в которой это было бы необходимо. Мы считаем, что это ошибка, которая будет исправлена ​​в будущих выпусках, а не функция.

Источник

РЕДАКТИРОВАТЬ:

Итак, swift теперь имеет открытый исходный код и, согласно этому списку изменений, теперь исправлен в снимках swift 2.2.

Инициализаторы назначенных классов, объявленные как сбойные или вызывающие, теперь могут возвращать nil или вызывать ошибку, соответственно, до полной инициализации объекта.

Я согласен с тем, что ответ Майка С - это рекомендация Apple, но я не думаю, что это лучшая практика. Весь смысл сильной системы типов состоит в том, чтобы переместить ошибки времени выполнения во время компиляции. Это «решение» противоречит этой цели. ИМХО, лучше было бы инициализировать имя пользователя на"" а затем проверить его после super.init (). Если пустые имена пользователей разрешены, установите флаг.

class User: NSObject {
    let userName: String = ""
    let isSuperUser: Bool = false
    let someDetails: [String]?

    init?(dictionary: [String: AnyObject]) {
        if let user_name = dictionary["user_name"] as? String {
            userName = user_name
        }

        if let value: Bool = dictionary["super_user"] as? Bool {
            isSuperUser = value
        }

        someDetails = dictionary["some_details"] as? Array

        super.init()

        if userName.isEmpty {
            return nil
        }
    }
}

Другой способ обойти это ограничение - работать с классом-функциями для инициализации. Возможно, вы даже захотите переместить эту функцию в расширение:

class User: NSObject {

    let username: String
    let isSuperUser: Bool
    let someDetails: [String]?

    init(userName: String, isSuperUser: Bool, someDetails: [String]?) {

         self.userName = userName
         self.isSuperUser = isSuperUser
         self.someDetails = someDetails

         super.init()
    }
}

extension User {

    class func fromDictionary(dictionary: NSDictionary) -> User? {

        if let username: String = dictionary["user_name"] as? String {

            let isSuperUser = (dictionary["super_user"] as? Bool) ?? false
            let someDetails = dictionary["some_details"] as? [String]

            return User(username: username, isSuperUser: isSuperUser, someDetails: someDetails)
        }

        return nil
    }
}

Использование его станет:

if let user = User.fromDictionary(someDict) {

     // Party hard
}

Хотя был выпущен Swift 2.2, и вам больше не нужно полностью инициализировать объект перед ошибкой инициализатора, вам нужно подождать, пока https://bugs.swift.org/browse/SR-704 не будет исправлен.

Я узнал, что это можно сделать в Swift 1.2.

Есть некоторые условия:

• Обязательные свойства должны быть объявлены как неявно развернутые необязательные параметры
• Присвойте значение требуемым свойствам ровно один раз. Это значение может быть нулевым.
• Затем вызовите super.init (), если ваш класс наследуется от другого класса.
• После того, как всем вашим обязательным свойствам присвоено значение, проверьте, соответствует ли их значение ожидаемому. Если нет, верните ноль.

Пример:

class ClassName: NSObject {

    let property: String!

    init?(propertyValue: String?) {

        self.property = propertyValue

        super.init()

        if self.property == nil {
            return nil
        }
    }
}

Неисправный инициализатор для типа значения (то есть структуры или перечисления) может вызвать сбой инициализации в любой точке в своей реализации инициализатора.

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

Выдержка из: Apple Inc. « Быстрый язык программирования. IBooks. https://itun.es/sg/jEUH0.l

Вы можете использовать удобный init :

class User: NSObject {
    let userName: String
    let isSuperUser: Bool = false
    let someDetails: [String]?

    init(userName: String, isSuperUser: Bool, someDetails: [String]?) {
        self.userName = userName
        self.isSuperUser = isSuperUser
        self.someDetails = someDetails
    }     

    convenience init? (dict: NSDictionary) {            
       guard let userName = dictionary["user_name"] as? String else { return nil }
       guard let isSuperUser = dictionary["super_user"] as? Bool else { return nil }
       guard let someDetails = dictionary["some_details"] as? [String] else { return nil }

       self.init(userName: userName, isSuperUser: isSuperUser, someDetails: someDetails)
    } 
}