Question

我正在将.csv文件导入Ruby on Rails应用程序。导入程序将从文件的每一行创建一个新的数据库记录。

class Invoice < ApplicationRecord

  def self.import(file)
    output_log = []
    CSV.foreach(file.path) do |row|
      output_log << some_method_name(row)
    end
    return output_log
  end

end

我希望将数据验证，记录创建和错误报告的所有复杂性都用另一种方法来解决，而不是使我的import方法变得混乱。我以some_method_name为例。我到底应该打什么电话？

我想到了两种可能性。实例方法：

output_log << Invoice.new.populate_from_row(row)

或者，一个类方法：

output_log << Invoice.create_from_row(row)

（任何一个都会返回记录成功或失败的字符串。）

两者都可以，但是哪个更有意义？是否有一些设计原则或模式可以让我了解如何选择？

Answer 1

我建议您在import方法中使用最合适的方法名称，并将所有逻辑封装在私有方法（或服务对象）中。在我的应用程序中，我通常会执行以下操作：

class Invoice < ApplicationRecord

  def self.import(file)
    output_log = []
    CSV.foreach(file.path) do |row|
      output_log << create_invoice_from_csv_row(row)
    end
    return output_log
  end

  private

  def create_invoice_from_csv_row(row)
    Invoice.find_or_create_by(
      order_number: row["Order Number"],
      customer_name: row["Customer Name"],
      # ...
    )
    return ""
  rescue => e
    return e.message
  end
end

我应该在创建对象的循环中引用实例还是类方法？

1 个答案: