多个 save() 调用如何在 grails/hibernate 事务中交互？

Question

我正在尝试创建一个新的数据库对象，保存它，然后检查保存是否成功。这样做的正确方法是什么：

class Document {
    String externalId;

    static constraints {
        externalId(blank: false, unique: true);
    }        
}

def createDocuments(List<String> ids) {

    Document.withTransaction() {
        ids.each { String id ->
            new Document(externalId: id).save();
        }
    }

    // want to test here if transaction succeeded.
}

此事务可能会失败，因为其他用户可能同时创建了其中一个文档。因此，虽然一个实例是有效的（即它的 externalId 不为空），但它可能不是唯一的。但是不运行事务就无法判断。

怎么办？

更新

根据目前提供的答案，这是我的问题的症结所在：

如果我运行一个调用多个保存的事务，保存的对象何时可用于其他休眠会话？一些可能性：

1. save 调用返回时
2. 事务提交时
3. 其他一些不确定的时间（大概在 #2 之前）

如果一个对象上的save 由于唯一性约束而失败，并且我回滚了事务，那么所有其他保存是否也会回滚，即使它们没有冲突？如果不是，那么将所有这些都包含在事务中的意义何在？

Answer 1

好的。在第 n 次阅读 my favorite article 之后，它确实有意义。

externalId 是 Document 的主键吗？如果是，那么这里使用的优化方法是在保存文档之前检查文档的existence：

//Avoid duplication in the submission as well by using Set<String>
def createDocuments(Set<String> ids) { 
    Document.withTransaction {
        ids.each { String id ->
            //If `id` is not the primary key then use
            //if(!Document.findByExternalId(id)){}
            //This way you maintain the integrity of the Document.
            if(!Document.exists(id)){
                if(!new Document(externalId: id).save()){
                    //Validation failed on constraints. Handle them here
                }

                //You may not need to call save on each Document, 
                //unless you need to validate the constraints
                //as all of the Documents created will be automatically saved to 
                //cache and ultimately flushed to db by end of the session, which
                //is in this case exiting the action method.
            }
        }
    }

    // You do not need to test here if transaction succeeded.
}

Answer 2

您可以在实际保存到数据库之前验证对象。
您也可以使用异常处理来回滚事务以防万一。
我在这类场景中遵循的一般模式是，

DomainClass.withTransaction{ tx ->
   try{
       def ref = new DomainClass(...)
       if(ref.validate()){
           ref.save()
       }
        else{
           //throw some invalid instance 

        }

    }
    catch(e){
      tx.setRollbackOnly()
      log.error e
    }
}