这是检查用户对数据层的访问权限的良好设计吗？

Question

我有一个数据访问 API，其内容类似于：

public interface IDataAccess {
   void save(Doc doc);
   Doc retrieve(DocId id);
   void delete(Doc doc);
}

我需要限制用户可以根据其权限对文档执行的操作。我必须这样做的想法是创建另一个 API，它镜像数据访问操作，但也在执行操作之前接收用户凭据来验证它们：

public interface IAuthDataAccess {

   // User is a previously authenticated user
   void save(Doc doc, User user) throws SecurityException;
   ...
}

public class AuthDataAccess implements IAuthDataAccess {
   IAuth auth = ...
   IDataAccess dataAccess = ...

   public void save(Doc doc, User user) throws SecurityException {
      if (auth.saveAllowed(doc, user)) {
         dataAccess.save(doc);
      } else {
         throw new SecurityException("access denied");
      }
   }
   ...
}

IAuthDataAccess API 将用于访问数据层。有没有更好的方法将授权/身份验证与数据访问层结合在一起？

Answer 1

更好的设计是将方面编程合并到您的代码中。这可以通过使用注解来实现。

示例：

public interface IAuthDataAccess {

   @Restricted
   void save(Doc doc, User user) throws SecurityException;
   ...
}

显然，您必须创建其他代码来进行真正的检查。

Answer 2

您的方法使接口毫无意义，因为您实际上并未实现这些方法 - 它所做的只是混淆。让授权访问者实际实现该接口怎么样？例如类似 -

public class AuthorizedDataAccessor implements IAuthDataAccess { 

 public AuthorizedDataAccessor(User user){ .. }

  void save(Doc doc) { ... save if authorized ...}
}

所以你可以写

IAuthDataAccess a=new AuthorizedDataAccessor(user);
 a.save(doc);