1

我的红宝石环境是：Ruby 2.3.1和Rails 5.0.0.1.

我正在尝试导入一个文本文件以导入大量购买项目。

采购文件示例：

数据.txt

客户\t描述\t单价\t数量\t地址\t供应公司\n Athos Matthew\tChocolate\t10\t5\t一些地址。\t巧克力公司\n

列由制表符 (\t) 分隔，最后有一个输入 (\n)。

我有所有属性不能为空的购买类。属性是：

custumer_name:string
product_id:integer        # It has relationship with the Product Resource
product_quantity:integer
supply_company_id:integer # It has relationship with the SupplyCompany Resource

为了导入文件，我决定创建一个 PurchaseImporter 类来完成这项工作并保持代码更简洁。

我的问题是交易部分：

  begin
    ActiveRecord::Base.transaction do
      purchase = Purchase.new
      data = line.force_encoding('UTF-8').split(/\t/)

      purchase.customer_name = data[0]
      product = Product.find_or_create_by!(description: data[1], price: data[2])
      purchase.product_quantity = data[3]
      purchase.product = product
      supply_company = SupplyCompany.find_or_create_by!(name: data[5], address: data[4])
      purchase.supply_company = supply_company

      purchase.save!
    end
  rescue Exception => e
    @errors[:import][index] = e.message
  end

我的问题是我想从 Product、SupplyCompany 和 Purchase 中捕获所有可能在此事务中发生的错误。

这是发生的顺序，没有不必要的代码来解释它。

product = Product.find_or_create_by!(description: data[1], price: data[2])
supply_company = SupplyCompany.find_or_create_by!(name: data[5], address: data[4])
purchase.save!

我需要将此错误信息打印到屏幕中的这 3 个类，但是使用我的代码，我只能捕获产品生成的第一个异常错误。如果 SupplyCompany 或 Purchase 发生错误，我会丢失这些错误消息。

导入文件时是否有其他方法可以导入和记录错误消息？

4

2 回答 2

6

您可以进行更具体的异常处理...为您要捕获的每个部分进行救援，最后如果遇到任何先前的错误（以使您退出事务块）并在最后一个错误中测试你正在拯救你自己的raise，否则这是其他问题，你需要停下来。

begin    
  ActiveRecord::Base.transaction do
    error_encountered = false
    purchase = Purchase.new
    data = line.force_encoding('UTF-8').split(/\t/)
    purchase.customer_name = data[0]
    begin    
      product = Product.find_or_create_by!(description: data[1], price: data[2])
      purchase.product_quantity = data[3]
      purchase.product = product
    rescue Exception => e
      @errors[:import][index] = e.message
      error_encountered = true
    end
    begin
      supply_company = SupplyCompany.find_or_create_by!(name: data[5], address: data[4])
      purchase.supply_company = supply_company
    rescue Exception => e
      @errors[:import][index] = e.message
      error_encountered = true
    end
    begin
      purchase.save!
    rescue Exception => e
      @errors[:import][index] = e.message
      error_encountered = true
    end
    raise 'break out of transaction' if error_encountered
  end
rescue Exception => e
  raise unless e.message == 'break out of transaction'
end
于 2016-11-07T15:48:18.730 回答
0

由于您正在救援Exception，因此很难知道实际出现了什么错误。救援时，您应该尽可能尝试使用更具体的类。

您也可能根本不需要使用救援。您正在使用的活动方法：find_or_create_by!并且save!可以在没有感叹号的情况下编写，这样它们就不会引发错误。

在活动记录中，如果您尝试保存带有验证错误的内容，<record>.errors.full_messages则会填充数组。如果您不使用感叹号，它不一定会引发错误（尽管无论如何都可能会引发错误）。

因此，例如，您可以尝试保存记录并检查以下错误：

  product = Product.find_or_initialize_by(description: data[1], price: data[2])
  product.save
  errors[:import][index] ||= []
  errors[:import][index].concat product.errors_full_messages

实际上，在这种情况下，我认为您的方法是有道理的。您正在按顺序保存几条记录。如果第一个失败，那么其他可能会失败 - 那么是否值得尝试保存这些后续记录？我让你决定。

于 2016-11-07T15:27:20.953 回答