Question

我想知道在grails应用程序中使用命令对象有任何优先模式。特别是如果我应该定义另一种保存对象的方法或使用相同的控制器方法来呈现表单和保存？

让我展示一个示例，其中包含用于呈现表单和保存对象的单独方法

def user(int id) {} // shows the edit user form - user.gsp. Submit takes us to saveUser method
def saveUser(UserCommand cmd) {} // actually saves the user, then redirects somewehre else

一切都应该有效，但是：

如果存在验证错误，saveUser方法必须执行user方法所执行的整个逻辑。如果在显示用户表单之前，我们必须从数据库加载其他对象，执行一些计算等...它必须再次完成，因为我们必须再次显示表单并包含验证错误。这导致不必要的代码重复。验证失败时，我无法重定向到user方法，因为我将丢失命令对象以及与之关联的任何错误。因此我无法显示验证错误。

使用相同方法保存和呈现表单的另一个示例

def user(UserCommand cmd, Integer id) {
   def u=User.load(id)
   if (request.method=="POST"&&cmd.validate()) {
      // populate u with command object values and save in database
      // then redirect somewhere
   }
}

此示例消除了对代码重复的需要。仅在存在POST请求时才会保存用户，在其他情况下，仅显示表单。如果出现验证错误，则可以访问这些错误，并且可以在gsp页面上显示。

主要问题是即使对用户页面有GET请求（这意味着显示了user.gsp表单），也会创建并验证空命令对象实例（因为它是在同一个控制器）。因此，每次显示表单时都会出现验证错误，提供的值为空（因为命令对象为空）

可以轻松修改这两种方案以使其正常工作（例如：在方案1中的会话中重定向之前保存命令对象，如果方案2中存在POST请求，则仅显示验证错误）但它需要额外的代码而且似乎不是很优雅。

这个问题有更简单 - 更多的“grails”解决方案吗？

Answer 1

以下是我在这种情况下通常做的一个示例：

def create(MeetingCommand cmdMtn) {
    switch (request.method) {
    case 'GET':
        cmdMtn = new MeetingCommand() 
        bindData cmdMtn, params, [include: boundProperties]
        createEditModel(cmdMtn: cmdMtn)
        break
    case 'POST':
            def meetingInstance
            Meeting.withTransaction { status ->
                try{
                    Boolean okValidated = (cmdMtn.validate() && !cmdMtn.hasErrors())
                    if (okValidated) {
                        meetingInstance = meetingService.bindInstance(cmdMtn)
                        flashMessage 'meeting.created', FLASH_CLASS_SUCCESS, [meetingInstance.id], 'Meeting creato'  
                        redirect action: ACTION_SHOW, id: meetingInstance.id
                    }
                    else {
                        log.error cmdMtn
                        log.error cmdMtn.errors
                        status.setRollbackOnly()
                        createEditModel(cmdMtn: cmdMtn)
                    }
                } catch (e){
                    status.setRollbackOnly()
                    log.error e
                    createEditModel(cmdMtn: cmdMtn)
                }
            }
        break
    }
}
protected createEditModel(mdl = [:]) {
    mdl
}

我从http://blog.freeside.co/post/41774629876/semi-restful-scaffolded-controllers获取灵感，域持久性逻辑已移至服务层。

Answer 2

问题：在控制器操作中复制模型填充逻辑。例如。在多个操作中将作者列表插入模型。

解决方案我有时会发现优雅：

def afterInterceptor = { model ->
    model.authors = authorService.list()
}

查看Grails文档了解更多信息： http://grails.org/doc/latest/ref/Controllers/afterInterceptor.html

Answer 3

我开始在命令对象中添加isBlank getter：

class UserCommand {
    String username
    String password

    boolean isBlank() { !username && !password }
}

然后在我的控制器中，我重新创建空白命令，因此不会运行验证：

def createUser(UserCommand cmd) {
    if (cmd.blank) cmd = new UserCommand()
    //...render form, there will be no errors on inital view
}

def saveUser(UserCommand cmd) {
    if (cmd.hasErrors()) {
        return createUser(cmd)
    } else {
    //...and so on
}

唯一的缺点是，如果用户提交空白表单，则不会出现验证错误，但另一方面，显而易见的是，空白表单必须填充某些内容，为什么会它会在那儿吗？

Grails命令对象 - 有一个共同的模式吗？

3 个答案: