为什么 Grails 在返回后将我的对象保存到数据库

Why Grails is saving my object to database after returning

我使用的是 Grails 2.4.5 版本。我在保存时正在检查条件。如果我的条件匹配,我会尝试 return 发送消息而不是保存。但它向我显示了我想要的消息,但同时将对象属性保存到数据库。

这是我的代码:

@Transactional
def save(AccountHead accountHeadInstance) {
    if (!request.method.equals('POST')) {
        redirect(action: 'index')
        return
    }
    LinkedHashMap result = new LinkedHashMap()
    String outPut

    if (accountHeadInstance.id) {
        if (accountHeadInstance.id == accountHeadInstance.subParentId) {
            result.put(CommonUtils.IS_ERROR, Boolean.TRUE)
            result.put(CommonUtils.MESSAGE, "Sorry, Same account can not be parent of it's  own")
            outPut = result as JSON
            render outPut
            return
            // here should it return, is shows the message ok but saving the object. if I print something after this block it does not execute
        }
    }

    if (accountHeadInstance.hasErrors()) {
        def errorList = accountHeadInstance?.errors?.allErrors?.collect{messageSource.getMessage(it,null)}
        result.put(CommonUtils.IS_ERROR, Boolean.TRUE)
        result.put(CommonUtils.MESSAGE, errorList?.join('\n'))
        outPut = result as JSON
        render outPut
        return
    }

    accountHeadInstance.hasChild = params.position == CommonUtils.POSITION_FIRST
    accountHeadInstance.save flush:true
    result.put(CommonUtils.MESSAGE, "Account head saved successfully.")
    outPut = result as JSON
    render outPut
    return
}

好吧,你正在使用@Transactional,所以对域对象的任何更改都会将该对象设置为脏对象,并且当事务结束时,它们将被隐式保存。为防止这种情况,您可以使用 accountHeadInstance.discard().

但是我不得不告诉你,你的代码是几个 Grails 反模式。

任何业务逻辑,尤其是使用@Transactional 的业务逻辑,都应移至服务中,而不是在控制器中完成。控制器应该只用于绑定传入参数、路由到服务和渲染结果。

虽然您可以将域用作命令对象绑定参数,但从安全的角度来看,这通常不是一个好主意。

您可以选择 'Groovy' 像这样的事情:

LinkedHashMap result = [:]
result[CommonUtils.IS_ERROR] = Boolean.TRUE)
render result as JSON