UITextField de liaison à ViewModel avec RxSwift

@XFreire est juste que orEmpty a été l'absence de la magie, mais il pourrait être intéressant pour vous de voir ce que votre code devrait ressembler à la dernière mise à jour de syntaxe et les erreurs corrigées:

D'abord le modèle de vue...

• Variable types doivent toujours être définies avec let. Vous ne voulez pas jamais remplacer une Variable, vous voulez juste de pousser de nouvelles données dans un.
• La manière que vous avez de votre isValid défini, un nouveau sera créé à chaque fois que vous liez/abonner. Dans ce cas simple, qui n'a pas d'importance parce que vous ne lient que pour une fois, mais en général, ce n'est pas une bonne pratique. Le mieux est de faire le isValid observables juste une fois dans le constructeur.

Lors de l'utilisation de Rx entièrement, vous trouverez généralement que les modèles de vue se composent d'une série de let's et un seul constructeur. Il est inhabituel d'avoir d'autres méthodes ou même les propriétés calculées.

struct LoginViewModel {

    let username = Variable<String>("")
    let password = Variable<String>("")

    let isValid: Observable<Bool>

    init() {
        isValid = Observable.combineLatest(self.username.asObservable(), self.password.asObservable())
        { (username, password) in
            return username.characters.count > 0
                && password.characters.count > 0
        }
    }
}

Et la vue du contrôleur. Encore une fois, utilisez let lors de la définition de Rx éléments.

• addDisposableTo() a été désapprouvée de préférence à l'aide de disposed(by:)
• bindTo() a été désapprouvée de préférence à l'aide de bind(to:)
• Vous n'avez pas besoin de la map dans votre viewModel.isValid de la chaîne.
• Vous avez été absent de la disposed(by:) de la chaîne.

Dans ce cas, vous pourriez voulez vraiment votre viewModel être une var si elle est affectée par quelque chose d'extérieur à la vue-contrôleur avant que celui-ci est chargé.

class LoginViewController: UIViewController {
    var usernameTextField: UITextField!
    var passwordTextField: UITextField!
    var confirmButton: UIButton!

    let viewModel = LoginViewModel()
    let disposeBag = DisposeBag()

    override func viewDidLoad() {
        usernameTextField.rx.text
            .orEmpty
            .bind(to: viewModel.username)
            .disposed(by: disposeBag)

        passwordTextField.rx.text
            .orEmpty
            .bind(to: viewModel.password)
            .disposed(by: disposeBag)

        //from the viewModel
        viewModel.isValid.map { $0 }
            .bind(to: confirmButton.rx.isEnabled)
            .disposed(by: disposeBag)
    }
}

Enfin, votre modèle d'affichage peut être remplacé par une seule fonction, au lieu d'une structure (struct):

func confirmButtonValid(username: Observable<String>, password: Observable<String>) -> Observable<Bool> {
    return Observable.combineLatest(username, password)
    { (username, password) in
        return username.characters.count > 0
            && password.characters.count > 0
    }
}

Alors votre viewDidLoad devrait ressembler à ceci:

override func viewDidLoad() {
    super.viewDidLoad()

    let username = usernameTextField.rx.text.orEmpty.asObservable()
    let password = passwordTextField.rx.text.orEmpty.asObservable()

    confirmButtonValid(username: username, password: password)
        .bind(to: confirmButton.rx.isEnabled)
        .disposed(by: disposeBag)
}

L'utilisation de ce style, la règle générale est de considérer chaque sortie de virage. Trouver toutes les entrées de l'influence de sortie et d'écrire une fonction qui prend toutes les entrées Observables et produit de la sortie comme un fait Observable.

OriginalL'auteur Daniel T.