ios - 如何在 RAC MVVM 中正确分离 ViewModel 和 ViewController

Question

我刚刚开始更新我的 ReactiveCocoa 应用程序以使用 MVVM 模式，并且有几个关于 ViewController 和 ViewModel 之间的边界以及 ViewController 应该有多笨的问题。

我要更新的应用程序的第一部分是登录流程，其行为如下。

• 用户输入电子邮件地址、密码并点击登录按钮
• 一个成功的响应包含一个或多个User模型
• 这些 User 模型与注销按钮一起显示
• 必须在登录 View 关闭并显示主视图之前为 session 选择一个用户模型。

MVVM 之前

• LoginViewController 直接处理LoginButton 命令
• LoginButton 命令直接与 SessionManager
对话
• LoginViewController 显示用于选择 User 模型或注销的 UIActionSheet
• LoginViewController 的用户选择和注销功能直接与 SessionManager 对话

MVVM 之后

• LoginViewModel 公开登录命令以及用户选择和注销方法
• LoginViewModel 用户选择和注销方法直接与 SessionManager
对话
• LoginViewController 响应 LoginViewModel
的登录命令
• LoginViewController 显示用于选择 User 模型或注销的 UIActionSheet
• LoginViewController 的用户选择和注销功能与 LoginViewModel 对话

LoginViewModel.h

@interface LoginViewModel : RVMViewModel

@property (strong, nonatomic, readonly) RACCommand *loginCommand;
@property (strong, nonatomic, readonly) RACSignal *checkingSessionSignal;
@property (strong, nonatomic, readonly) NSArray *users;
@property (strong, nonatomic) NSString *email;
@property (strong, nonatomic) NSString *password;

- (void)logout;
- (void)switchToUserAtIndex:(NSUInteger)index;

@end

LoginViewModel.m

@implementation LoginViewModel

- (instancetype)init {
    self = [super init];
    if (self) {
        @weakify(self);

        // Set up the login command
        self.loginCommand = [[RACCommand alloc] initWithEnabled:[self loginEnabled]
                                                    signalBlock:^RACSignal *(id input) {
            @strongify(self);
            [[[SessionManager sharedInstance] loginWithEmail:self.email
                                                    password:self.password]
             subscribeNext:^(NSArray *users) {
                 self.users = users;
             }];

            return [RACSignal empty];
        }];

        // Observe the execution state of the login command
        self.loggingIn = [[self.loginCommand.executing first] boolValue];
    }
    return self;
}

- (void)logout {
    [[SessionManager sharedInstance] logout];
}

- (void)switchToUserAtIndex:(NSUInteger)index {
    if (index < [self.users count]) {
        [[SessionManager sharedInstance] switchToUser:self.users[index]];
    }
}

- (RACSignal *)loginEnabled {
    return [RACSignal
            combineLatest:@[
                RACObserve(self, email),
                RACObserve(self, password),
                RACObserve(self, loggingIn)
            ]
            reduce:^(NSString *email, NSString *password, NSNumber *loggingIn) {
                return @([email length] > 0 &&
                         [password length] > 0 &&
                         ![loggingIn boolValue]);
            }];
}

@end

LoginViewController.m

- (void)viewDidLoad {
    [super viewDidLoad];

    @weakify(self);

    // Bind to the view model
    RAC(self.controlsContainerView, hidden) = self.viewModel.checkingSessionSignal;
    RAC(self.viewModel, email) = self.emailField.rac_textSignal;
    RAC(self.viewModel, password) = self.passwordField.rac_textSignal;
    self.loginButton.rac_command = self.viewModel.loginCommand;
    self.forgotPasswordButton.rac_command = self.viewModel.forgotPasswordCommand;

    // Respond to the login command execution
    [[RACObserve(self.viewModel, users)
     skip:1]
     subscribeNext:^(NSArray *users) {
         @strongify(self);

         if ([users count] == 0) {
             [Utils presentMessage:@"Sorry, there appears to be a problem with your account."
                         withTitle:@"Login Error"
                             level:MessageLevelError];
         } else if ([users count] == 1) {
             [self.viewModel switchToUserAtIndex:0];
         } else {
             [self showUsersList:users];
         }
     }];

    // Respond to errors from the login command
    [self.viewModel.loginCommand.errors
     subscribeNext:^(id x) {
         [Utils presentMessage:@"Sorry, your login credentials are incorrect."
                     withTitle:@"Login Error"
                         level:MessageLevelError];
     }];
}

- (void)showUsersList:(NSArray *)users {
    CCActionSheet *sheet = [[CCActionSheet alloc] initWithTitle:@"Select Organization"];

    // Add buttons for each of the users
    [users eachWithIndex:^(User *user, NSUInteger index) {
        [sheet addButtonWithTitle:user.organisationName block:^{
            [self.viewModel switchToUserAtIndex:index];
        }];
    }];

    // Add a button for cancelling/logging out
    [sheet addCancelButtonWithTitle:@"Logout" block:^{
        [self.viewModel logout];
    }];

    // Display the action sheet
    [sheet showInView:self.view];
}

@end

问题

1. 创建额外的 ViewModel 层意味着我需要代理 SessionManager 调用。我想从 SessionManager 中解耦 LoginViewController 的好处超过了 ViewModel 层的额外代码和函数调用？
2. LoginViewController 了解User 模型，以便显示可供选择的用户列表。这打破了 MVVM 模式，当然感觉不对。 LoginViewModel 是否应该只提取 LoginViewController 所需的 User 模型的必要属性，并将它们添加到字典中，并返回一个数组到 LoginViewController？或者在 LoginViewModel 上有一个方法会更好，该方法返回给定索引的用户的名称，允许 LoginViewController 显示此名称？我知道 ViewModel 负责弥合模型和 View 之间的差距，但这确实感觉像是双重处理。根据我对第一个问题的直觉，我猜想将这些关注点分开的好处远远超过感觉起来有点费力的映射过程。
3. 如果 LoginViewModel 调用包含在 SessionManager 中的所有功能，是否仅针对 LoginViewModel 编写测试就足够了，还是应该也编写测试专门针对SessionManager?

Answer 1

已经很晚了，我相信你已经继续前进了。

1) 将程序逻辑移出 View /控件始终值得您将额外的几行锥形代码写入代理。 MVVM 的要点是鼓励关注点分离，并通过 ViewModel 在 View/Controller 和 Model 之间提供清晰的数据通道。

从 View / Controller 的角度来看，您的 View 模型应该执行以下功能:

充当数据黑盒，您的 View / Controller 可以在不执行任何业务规则的情况下利用它，并始终假定数据是正确的。

充当用户输入处理的管道，无需执行任何业务规则即可获取用户输入。

2) 在我的 MVVM 实现中，我尝试遵循以下范例:包含 CollectionView/TableView 的 View / Controller 是父 View ，单元格是 subview 。因此，您应该有一个父 ViewModel，其工作是初始化和管理子 ViewModel。

在您的情况下，您没有使用集合/ TableView ，但概念是相同的。您应该向您的父 View Model 询问一个子 ViewModel 列表，您可以将其传递给另一个 View 以进行利用。根据答案 #1 中的要点，父 View 模型应确保正确初始化 subview 模型，以便 subview 无需担心任何数据验证。

3) 在测试 View 模型的数据验证/规则时，您可以完全关闭 session 管理器，只测试 View 模型。我所做的是创建断言，在我的单元测试中正确调用 stub /模拟 session 管理器函数。