RAILS 3 - 控制器中的事务

Question

我在控制器中有一个示例操作。

def some_action 
product = Product.new 
product.name = "namepro" 
    if product.save 
    client.update_attribute(:product_id,product.id) 
    end 
end 

如何为此代码添加事务？我尝试用这个例子代码：

def some_action 
**transaction do** 
    product = Product.new 
    product.name = "namepro" 
    if product.save 
    client.update_attribute(:product_create,Time.now) 
    end 
**end** 
end 

但它会产生这个错误：

undefined method `transaction' 

我读到的控制器使用的交易是一个不好的做法，但我不知道为什么原因（http://markdaggett.com/blog/2011/12/01/transactions-in-rails/ ）

在此示例中，如果产品已创建并保存，并且客户端更新失败...... Rails一定不能做任何事情。

谢谢。

Answer 1

如果您确实想要，可以在控制器中使用事务。正如你所指出的，这是不好的做法，但如果你想这样做，只需拨打Product.transaction do而不是transaction do。 transaction是ActiveRecord::Base上的一个类方法，因此您需要在ActiveRecord派生类上调用它。在你的应用程序中的任何模型类都会做（无奈地提醒：如果你连接到不同模型的不同数据库，这可能不是真的......但你可能不这么做）。

这是一种不好的做法，原因是根据MVC范式它没有适当地区分问题。你的控制器不应该如此关心你的数据持久化实现。更好的方法是将方法添加到Product。也许是这样的：

def save_and_update_create_time 
    transaction do 
    if save 
     client.update_attribute(:product_create, Time.now) 
    end 
    end 
end 

然后，而不是在你的控制器调用product.save，叫product.save_and_update_client_create_time。您可能还需要将client也传递给该方法;目前还不清楚你的代码来自何处client。如果它是product上的属性，那么上面的方法应该可以工作。

Railsy还有更好的方法来做到这一点，特别是如果product知道它的client而不需要任何控制器数据。然后，你可以使用一个after_save回调，像这样（添加到Product类）：

after_save :update_client 

private 

def update_client(product) 
    product.client.update_attribute(:product_create, Time.now) 
end 

然后每一个Product保存时间，相关的客户端上的字段将被更新。您可能必须先介绍一些代码，以便首先检查是否存在client。

除了更干净的代码之外，使用回调的好处在于整个回调链随着保存一起运行在单个事务中;您不需要手动创建交易。您可以在Rails documentation中了解更多关于回调的信息。