捕获 ActiveRecord::RecordNotUnique 和之后的所有其他异常

Question

我的 LineItem 模型中有一个唯一性约束，如下所示：

class LineItem < ApplicationRecord
    # this add uniqueness validation to [cart_id + product_id] together
    validates :cart_id, uniqueness: { scope: [:product_id] }
end

为了提高安全性，我还为数据库中的这些列添加了索引 + unique:true

在我的LineItemsController#create 我有以下内容

class LineItemsController < ApplicationController

    def create 

        @cart = Cart.find(session[:cart_id]

        product = Product.find(params[:product_id]

        @line_item = @cart.add_product(product, params[:licence])

        respond_to do |format|

            @line_item.save!

            format.html { redirect_to products_url }
            format.js 

         rescue ActiveRecord::RecordNotUnique

          @cart.line_items
             .find_by(product_id: params[:product_id])
             .update(licence_type: params[:licence], price: product.price)

          format.js

        end

    end

end

我想要做的是：如果用户添加一个已经具有相同 product_id 和 cart_id 的行项目，则使用 params[:licence] 更新 licence_type 列

我为此使用rescue ActiveRecord::RecordNotUnique：

1 - 这是一个好方法吗（这样我可以为自己保存一个额外的请求，每次检查数据库中是否存在记录）？

2 - 我如何捕捉除 ActiveRecord::RecordNotUnique 之外的任何其他异常/错误？我想在底部添加另一个rescue Exception => e，这样我就可以捕获所有其他异常，但我想我在某处读到这样的一般异常并不好，我应该改用rescue => e 之类的东西？

感谢任何代码 sn-p，谢谢！

Answer 1

Never rely on uniqueness validation 让您免于重复数据。它对竞争条件有一个公认的弱点。

在这种情况下，您可能想要的是find_or_initialize_by。

line_item = @cart.line_items.find_or_initialize_by(product_id: params[:product_id])
line_item.license_type = params[:license]
line_item.save

除此之外，您应该将此数据的唯一性约束移动到数据库中。具体的做法取决于你的数据库是什么。

保留验证器是可以的，但把它当作前端验证；有作为一种方便，创造更好的用户体验。但不要依赖它来保持数据的一致性，这是数据库的工作。

---

对代码示例的另一种评论：

rescue 必须放置在 begin..end 块内或作为方法体的一部分。但在您的情况下，您将它放在 do..end 块内，而周围没有 begin..end，这是行不通的。