让 Grails 控制器更加 DRY？

Question

我正在寻找清理 Grails 控制器代码的方法。在各种控制器中，我或多或少具有相同的逻辑..

• 获取对象
• 检查它是否存在
• 等..

是否有建议的方法使控制器操作重用通用代码？

--- 解决方案 ---

问题的所有答案都有助于我们实施的解决方案。

我们使用 Mixin 方法创建了一个在控制器中使用的类。 mixin 公开的方法之一是 withObject 方法。此方法从控制器获取域名并将其用作该方法的基础。当然可以覆盖此行为！

def withObject(object=this.getClass().getName()-"Controller", id="id", Closure c) {
    assert object
    def obj =  grailsApplication.classLoader.loadClass(object).get(params[id])
    if(obj) {
        c.call obj
    } else {
        flash.message = "The object was not found"
        redirect action: "list"
    }
}

所以所有的答案都有助于解决问题！多谢！

Answer 1

当这个问题出现时，我总是拿出这篇博客文章：

http://mrpaulwoods.wordpress.com/2011/01/23/a-pattern-to-simplify-grails-controllers/

基本上你在控制器中为各个域提供一个私人助手。

private def withPerson(id="id", Closure c) {
    def person = Person.get(params[id])
    if(person) {
        c.call person
    } else {
        flash.message = "The person was not found."
        redirect action:"list"
    }
}

对getter 进行编码的方式非常灵活，对我来说（博客中未介绍）的典型用途是编辑等。

这种方式进行编码（我喜欢这种模式，因为它具有清晰的划分和可读性）：

 def editIssue() {
    withIssue { Issue issue ->
        def issueTypes = IssueTypeEnum.values().collect {it.text }
        [issueTypes:issueTypes,activePage:"issue", issue: issue]
    }
}

 def doEditIssue(IssueCommand cmd) {
    if(cmd.validate()) {
        withIssue { Issue issue ->
            issue.updateIssue(cmd)
            redirect(action: "show", id: issue.id)
        }
    }
    else {
        def issueTypes = IssueTypeEnum.values().collect {it.text }
        render(view: "edit", model:[issueTypes:issueTypes,issue:cmd,activePage:"issue"])
    }
}

我通常以 helper 是：

private def withIssue( Closure c) {
    def issue = Issue.get(params.id)
    if(issue) {
        c.call issue
    }
    else {
        response.sendError(404)
    }
}

我确实认为 mixin 方法（与“扩展通用抽象控制器”方式非常相似）也很好，但这种方式有两个优点：

1. 您可以输入 helper，就像您在闭包中看到的那样您可以访问方法等STS/IDEA（未测试 Netbeans）
2. 重复率不是很高，并且能够更改 getter（例如使用 BarDomain.findByFoo(params.id) 等）

在我绑定到 edit() 的视图中，我只是放置了一个 中的 id="${issue.id}" 并且可以无缝运行。

Answer 2

我不建议为此使用继承，因为您不能在多个超类中传播通用方法。如果您有很多控制器，您的抽象类很快就会变得混乱。您无法使用组合（例如使用服务），因为您无法直接从以下位置访问 response、render 或 params那里。

我使用的方法是通过 Mixins 注入通用方法。

@Mixin(ControllerGenericActions)
@Mixin(ControllerUtil)
class BookController {
  def show = &genericShow.curry(Book)

  def exists = {
    render(idExists(Book))
  }
}

第一个操作 show 使用 ControllerGenericActions.groovy 中的通用方法，并绑定了一个参数。 mixin idExists 方法的第二次使用是在控制器操作内部。

下面是 src/groovy/ControllerGenericActions.groovy

class ControllerGeneric {
  def genericShow(Class clazz) {
    render clazz.get(params.id) as XML
  }
}

和 src/groovy/ControllerUtil.groovy 中

class ControllerUtil {
  def idExists (Class clazz) {
    return clazz.get(params.id) != null
  }

的示例代码，在这种情况下不是很有用，但您明白了。

Answer 3

使用通用方法实现抽象控制器（使用“受保护”指令）并从中扩展您的真实控制器。不要在此方法名称的开头使用“get”和“set”单词。不好，但它有效。