根据自定义条件保护对 Java 方法的访问

Question

简而言之：我想根据某些条件允许/禁止Java方法的执行。最好的可用解决方案/机制/技术是什么？

长问题，通过一个（诚然愚蠢的）示例：

假设我在车辆控制器中有几种方法，例如 void openWindow(Window w)、void openRoof() ，void keepSpeed(double speedKmh)。如果下雨，则不应调用 openRoof，我们希望建立一种机制来确保这一点。同样，如果速度低于 60 公里/小时，则禁止 keepSpeed，就像如果下雨很多或速度高于 100 公里/小时，则禁止 openWindow 。

由于访问降雨/速度传感器需要几行代码，并且这些条件随处可见，因此我不想在方法主体中使用使用断言或条件，但我宁愿让域开发人员轻松使用它们。此外，我希望安全问题与打开窗口等的实际逻辑分开。更复杂的自定义条件也应该很容易配置。

例如，我想要这个：

@ForbidIf('default:rain') openWindow();
@ForbidIf('default:speedBelow(60)') keepSpeed();
@ForbidIf('default:speedAbove(100)', 'custom:rainsALot') openWindow();

如果有帮助的话，这个应用程序是一个 Spring 支持的客户端-服务器应用程序。

Answer 1

您可以使用简单的 Spring AOP 方面，如下所示（未经测试）：

@Aspect
public class SecurityAspect{

    @Pointcut("execution(@ForbidIf * *(*))")
    public void annotatedMethodCalled(){}

    @Before("annotatedMethodCalled() && @target(annotation) && target(bean)")
    public void beforeRestrictedMethodCall(
        final ForbidIf annotation, final Object bean){
        final Expression expression =
            new SpelExpressionParser().parseExpression(annotation.value());
        if(!Boolean.TRUE.equals(expression.getValue(bean))){
            throw new IllegalArgumentException();
        }
    }

}

Answer 2

Spring Security 中实现了类似的内容 基于表达式的访问控制，但我认为它不适合您的情况。

然而，通过创建一个可以评估 针对特定上下文的 SpEL 表达式。

Answer 3

OP，评论 Andreas_D 的答案：“如果禁止执行，我希望引发运行时异常。在这种情况下，如果条件为 false，则意味着在当前情况下不应调用此方法。”

有什么问题：

public void openWindow() {
    if (itsRaining()) {
        throw new IllegalStateException("Window should not open if it's raining");
    }
}

？我的意思是，做注释实际上是同一件事，只是更复杂（特别是如果该方法的条件变得越来越复杂，例如，如果下雨、下雪、车速 > 100 KPh、有风，则不应打开窗户速度> 6，温度低于冰点等）。

当然，我也可能没有抓住要点。只是想暗示过度思考、设计或使问题复杂化并不是唯一的途径。当然，我不想说使用 AOP 或注释是过度思考/工程化/复杂化的问题。这篇文章可能也基于简化的示例。

Answer 4

这是一个有趣的想法，尽管我看到了一些实际问题。您可以注释一个方法并参考一些环境设置。但如果禁止执行，该怎么办呢？

考虑这个例子：

 @ForbidIf('default:durationIs(0))') double getSpeed(double distance);

并使用它，

 double speed = getSpeed(distance);   // the duration is set globally in this example

如果持续时间值设置为 0 会发生什么 - 在这种情况下应该将什么分配给速度？或者你想引发运行时异常吗？

对于您的情况，我们已经有两种方法来实现条件执行：

 // check before calling
 if (isSaveToOpenWindow())
   openWindow();

和

 public void openWindow() {
    if (!isSaveToOpenWindow())
      return;

    // open window
 }

Answer 5

这里需要AOP吗？装饰器模式可以更简单，并且需要更少的魔力。只需用以下内容包装实现即可：

class ProtectedWindowOpener implements WindowOpener {

   WindowOpener delegate ...

    void openWindow(Window w) {
        if (!allowOpen(w)) {
          throw new PermissionDeniedException...
        }
        delegate.openWindow(w);
    }

    boolean allowOpen(Window w) {
      // security logic here

    }
}

这使安全逻辑保持独立，并且还具有不将代码嵌入字符串的优点，这意味着 eclipse 可以完成其业务。