运行方法时检查权限是否有良好的设计模式？

Question

我正在寻找一种设计模式或一种方法来整理我的代码。这是在给定主题的安全和检查权限的上下文中。 这是一个简化的例子：

public Pizza bakePizza() throws UnauthorizedException{
if (subject.isPermitted("BAKE_PIZZA")){
        return new Pizza();
    } else {
        throw new UnauthorizedException();
    }
}

有没有办法让它更干净一点，因为当我有很多不同类型的方法时，这会变得很混乱。

Answer 1

我认为使用decorator pattern 之类的东西来拆分安全约束和业务逻辑将是一个好的开始。

这样的事情怎么样：

// an interface defines the operations
public interface PizzaService {
  Pizza bakePizza();
}

// standard implementation that contains the business logic
public class PizzaServiceImpl implements PizzaService {
  @Override
  public Pizza bakePizza() {
    // implementation ..
  }
}

// implementation with security constraints that delegates to the real implementation
public class SecurePizzaService implements PizzaService {
  private PizzaService service;
  public SecurePizzaService(PizzaService service) {
    this.service = service;
  }

  @Override
  public Pizza bakePizza() {
    if (!subject.isPermitted("BAKE_PIZZA")){
      throw new UnauthorizedException();
    }
    service.bakePizza()
  }
}

// usage
PizzaService pizzaService = new SecurePizzaService(new PizzaServiceImpl());
...
Pizza pizza = pizzaService.bakePizza();

这样您就可以在不触及业务逻辑的情况下更改安全约束，反之亦然。

如果你有很多这样的情况，你应该看看 AOP 像 AspectJ 这样的框架

Answer 2

一种方法是将检查逻辑移动到一个方法中，该方法会抛出并提前结束调用方法的程序流程。

public Pizza bakePizza() throws UnauthorizedException{
    ensurePermission("BAKE_PIZZA");
    return new Pizza();
}

private void ensurePermission(String permission) throws UnauthorizedException {
    if (!subject.isPermitted(permission))
        throw new UnauthorizedException();
}