php - 什么是更好的设计？

Question

Closed. This question is opinion-based。它当前不接受答案。












想要改善这个问题吗？更新问题，以便 editing this post用事实和引用来回答。

2天前关闭。



Improve this question




我有以下类(class):

class User {

  public function setName($value) { ... }
  public function setEmailAddress($value) { ... }
  public function setUsername($value) { ... }
  public function getName() { ... }
  public function getEmailAddress() { ... }
  public function getUsername() { ... }

  public function isGroupAdministrator($groupId) { ... }
  public function isMemberOfGroup($groupId) { ... }
  public function isSiteAdministrator() { ... }
  public function isRoot() { ... }
  public function hasModulePermission($moduleId, $recordId, $permissionCode) { ... }
  public function hasGroupPermission($groupId, $permissionCode) { ... }
  public function hasContentPermission($recordId, $permissionCode) { ... }
  public function hasModulePermission($moduleId, $recordId, $permissionCode) { ... }
  public function canLogIn() { ... }
  public function isLoggedIn() { ... }
  public function setCanLogIn($canLogIn) { ... }

}

这会成为“上帝的类(Class)”吗？

我不确定我是否应该把这一个类分开。这样做的问题是该类的方法被其域使用，以确定是否在网页上显示给定的UI元素，因此该类中没有任何行为。

我想我可以将与权限相关的方法放在某些Permission类中，使这些方法成为静态的(例如::: userIsGroupAdministrator(...)，:: userIsMemberOfGroup(...):: userHasGroupPermission(...)，:: userHasContentPermission(...))

关于如何提高本类水平的任何建议？

最佳答案

如果您已经有正在运行的代码，则可以一小段进行重构。如果没有，我将看以下内容:

class User {
  String username
  String name
  String emailAddress
  Boolean active
  Integer permission # A bitmask of the statics in the Permission class
}

class Group {
  String name
}

class UserGroupMapping {
  User user
  Group group
  Boolean administrator
}

class Permission {
  static IS_ROOT = 1
  static IS_SITE_ADMIN = 2
}

class Session {
  User user
  Boolean logged_in
}

其余的内容实际上属于服务类:

class SecurityService {
  static public function hasModulePermission($user, $module, $record, $permission) { ... }
  static public function hasGroupPermission($user, $group, $permission) { ... }
  static public function hasContentPermission($user, $record, $permission) { ... }
  static public function hasModulePermission($user, $module, $record, $permission) { ... }
}

关于php - 什么是更好的设计？ ，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/730856/

上一篇：PHP - 删除 XML 元素

下一篇：PHP session_start 失败

相关文章：

javascript - 从 Google 图片获取背景图片

php - 第一次身份验证以获得 token

php - 使用PHP选择mysql数据库中的下一行

javascript - 在javascript中扩展函数

iOS:具有外部参数的对象类中的 setter

php - Cakephp 事件已注册但未触发

c# - Data Transfer Object 中的属性应该扩展外键还是简单地公开它们的主键

java - 打破继承层次结构的最佳方法

python - Pandas 灵活确定指标

php - 关于 PHP 中的 OO 和类的几个问题

Answer 1

如果您已经有正在运行的代码，则可以一小段进行重构。如果没有，我将看以下内容:

class User {
  String username
  String name
  String emailAddress
  Boolean active
  Integer permission # A bitmask of the statics in the Permission class
}

class Group {
  String name
}

class UserGroupMapping {
  User user
  Group group
  Boolean administrator
}

class Permission {
  static IS_ROOT = 1
  static IS_SITE_ADMIN = 2
}

class Session {
  User user
  Boolean logged_in
}

其余的内容实际上属于服务类:

class SecurityService {
  static public function hasModulePermission($user, $module, $record, $permission) { ... }
  static public function hasGroupPermission($user, $group, $permission) { ... }
  static public function hasContentPermission($user, $record, $permission) { ... }
  static public function hasModulePermission($user, $module, $record, $permission) { ... }
}