Grails：在控制器的基类中重定向或转发后防止进一步执行代码

Question

我有以下控制器结构：

abstract class AbstractController {
  // ...
}

abstract class AbstractDiagramController extends AbstractController {
  // ...
}

class PopulationController extends AbstractDiagramController {
  // ...
}

大多数控制器操作调用抽象基类的各种方法。如果现在这些基本方法之一需要向客户端发送重定向（或转发），Grails 无论如何都不会阻止应用程序处理控制器操作的剩余操作代码。

从我的角度来看，这是一种不受欢迎的行为，因为基本方法会进行某种验证（例如验证参数、用户、会话等），并且控制器假设验证成功（因此会产生后续错误）。

我怎样才能防止这种不足的行为？

亲切的问候， 克里斯托弗

PS：我已经发现 这个问题，但答案并没有满足我的需求，因为它们都没有涉及基本控制器：

PPS: I am using Grails in version 1.3.7

EDIT

This is a的反应维克多·塞尔吉延科的评论。 在这里我给出了我的问题的更详细的代码示例。

// The very base controller
abstract class AbstractController { 

    // The interceptor gets called before any action of inheritors get processed
    def beforeInterceptor = [action:this.&initialize]

    // Method validates various stuff
    protected initialize() {
        if( someThingIsWrong() ) {
            // This redirect should stop any other code of inheritors
            redirect( controller: "foo", action: "bar" )
            return false
        }
    } 
}

// The second base controller
abstract class AbstractDiagramController extends AbstractController { 

    // This object must get initialized. If not (eg any errors or exceptions occured) 
    // all inheritors actions are not allowed to do anything
    MyObject myGreatObject = null

    // Overriden, because of additional individual diagram validation
    @Override
    protected initialize() {
        // Do parents stuff first
        super.auth()

        // If parent failed, this code should not get executed anymore.
        // Yes, here I could check if parent returned false and return false as well before
        // continuing the next validation. Anyway I have to do this because grails, comprehendibly,
        // will throw an exception if two redirects were executed in a row.
        // (With this I just want to visualize the behaviour I'd expect)
        if( someThingElseIsWrong() ) { 
            redirect( controller: "hello", action: "world")
            return false
        }

        // After validation I can initialize the object
        myGreatObject = new MyObject()
    }
}


// A controller implementation
class MyDiagramController extends AbstractDiagramController {

    // Overriden because of inidividual validation
    @Override 
    protected initialize() {

        // First do parent stuff
        boolean succeeded = super.auth()

        // Again, annoying double check
        if( !succeeded ) {
            return false
        }
    }

    def myAction = { 
        myGreatObject.SpinAroundAndBeHappy()
    } 
}

看起来将用例减少到最少的代码行是个好主意。现在看来，Victor 的建议（canContinue 或 hasErrors）可以以某种方式解决这种不愉快的情况，尽管这是某种解决方法。

但不知怎的，我不喜欢那些双重检查。我仍然在与这样一个事实作斗争：抽象基本控制器之上的所有层都必须对之前已经发生的无效验证做出反应（并且也应该由基本控制器自己管理）。从我的角度来看，这些检查不应该是控制器实现的业务。

PS：我希望代码中没有出现严重错误。

Answer 1

作为解决方法，您可以从祖先操作返回 boolean canContinue 或引发异常，或者根据您的情况检查 instance.hasErrors() 。

编辑：事实上 initialize() 在操作之前被调用，看起来像访问控制或操作语义的另一个完全覆盖（在操作的任何部分之前）被执行）。请告诉我的假设是否错误。

当我们对不同操作进行自定义安全访问时，我们使用自己的标签（例如 @SecuredDoodah）对操作进行注释（请参阅@Secured），并添加了一个完全覆盖操作的 Filter （对于我们来说，Filter响应 403，但这不是必需的）。

Filter 可能比 beforeInterceptor 更好。如果您需要从 Filter 传递某些状态，例如示例中的 myGreatObject，您可以将 Service 注入 Filter 并将状态保存在 Service 中。

我确信有比我的想法更好的方法，但这对于控制器来说应该是透明的。

Answer 2

您受到以下事实的限制：这是 Java/Groovy，方法调用无法立即触发方法（或闭包）退出。我看到另一个框架作弊，当你调用渲染、重定向等时，它会抛出一个异常，该异常会捕获框架基类。这就像 Goto，但实际上并不存在。

但这是一种昂贵的方法 - 不必要地填充所有这些堆栈帧是浪费的，因为这不是异常情况，并且堆栈将始终被忽略。

不幸的是，您需要像 Victor 那样使用布尔返回值的方法。