将html解析到Rails中，每次都没有新记录？

Question

我有以下代码，它尽可能简单地解析 HTML 表。

# Timestamp (Column 1 of the table)
page = agent.page.search("tbody td:nth-child(1)").each do |item|
  Call.create!(:time => item.text.strip)
end

# Source (Column 2 of the table)
page = agent.page.search("tbody td:nth-child(2)").each do |item|
  Call.create!(:source => item.text.strip)
end

# Destination (Column 3 of the table)
page = agent.page.search("tbody td:nth-child(3)").each do |item|
  Call.create!(:destination => item.text.strip)
end

# Duration (Column 4 of the table)
page = agent.page.search("tbody td:nth-child(4)").each do |item|
  Call.create!(:duration => item.text.strip)
end

尽管上面的代码运行良好，但它将每个“项目”视为一条新记录。因此，它为每个时间行添加一条记录，为每个源列添加另一条记录，等等。

使其循环上述代码的最简单方法是什么，但将四个项目添加到一个记录中，然后继续到下一个记录记录？

有关其他信息，这里是显示我的数据库结构的迁移文件：

class CreateCalls < ActiveRecord::Migration
  def change
    create_table :calls do |t|
      t.datetime :time
      t.string :source
      t.string :destination
      t.string :duration

      t.timestamps
    end
  end
end

任何帮助都是值得赞赏的。

Answer 1

考虑迭代每一行而不是每一列。

page = agent.page.search("table tbody tr").each do |row|
  time        = row.at("td:nth-child(1)").text.strip
  source      = row.at("td:nth-child(2)").text.strip
  destination = row.at("td:nth-child(3)").text.strip
  duration    = row.at("td:nth-child(4)").text.strip
  Call.create!(:time => time, :source => source, :destination => destination, :duration => duration)
end

Answer 2

不必每次都调用 call.create，只需将所有源附加到字符串中，然后在最后保存记录。