1

从现在开始，我对这个问题感到很困惑。

比如说，我必须设计一个 OO 系统，其中管理员将是唯一可以注册大学新生的用户。现在，我（显然）有系统的其他用户。我应该如何控制这些权限仅在用户是管理员时才适用？

我有两种可能性：将这些角色作为函数嵌入到与Adminclass 通信的Student类中，或者只是控制新学生在程序执行中的注册，例如：

if(user_type() == 'a') register_student(); // a type means an admin user

else cout << "Access denied";

...
4

1 回答 1

3

有几种 OO 方法可以做到这一点：

if( user->has_permission( REGISTER_STUDENT ) ) {
  register_student();
}

这是询问用户实例是否有权注册学生。其他方式：

 if( user->in_role( ROLE_ADMIN ) ) {
   register_student();
 }

我对代码的问题是它公开了有关如何确定管理员角色的内部细节。这封信a最终将失去上下文，对于新维护者的地位变得毫无意义；然而，常量ROLE_ADMIN/REGISTER_STUDENT阐明了代码的意图，而不需要额外的注释。

执行注册并不是User类或类的真正责任。a注册自己Admin确实有意义：Student

 if( user->in_role( ROLE_ADMIN ) ) {
   student->register();
 }

我希望在Controller课堂上看到这段代码。该类Controller将能够检查用户界面对象以确定：

  • 当前选择了哪个学生
  • 用户从界面中选择了什么选项
  • 用户是什么角色

您可以实现以下内容：

if( user->is_admin() ) {
  student->register();
}

虽然代码非常清晰，但灵活性较低，因为您可能希望保留以下可能性：

if( controller->can_execute( user, action ) ) {
  action->execute();
}
else {
  controller->execute_error( user, action );
}

这提供了一个更加灵活的系统，可以将角色动态分配给操作。例如，您可以将方法映射student->register()到ROLE_ADMIN. 这将整个应用程序中的所有安全约束保持在一个位置，从而大大简化了维护。

还可以进一步简化：

controller->execute( user, action );

然后执行方法将类似于：

void execute( User user, Action action ) {
  if( can_execute( user, action ) ) {
    action->execute();
  }
  else {
    error( user, action );
  }
}

bool can_execute( User user, Action action ) {
  return user->is_in_role( action->get_role() );
}

cout现在抽象了如何显示权限错误（对话框与 ）的实现细节。此外，错误消息可以更全面，例如：

"User registration is restricted to the administrator role."

编码为以下字符串：

"%s is restricted to the %s role."

控制器类可以自动替换%s。

于 2013-01-09T17:17:09.630 回答