Nokogiri XML导入提要组织？

Question

我已经建立了一个网站，该网站依赖于我目前使用Nokogiri解析的XML feed。 尽管我现在在Admin控制器中拥有所有代码，但一切正常且繁琐，因此我实际上可以通过URL调用导入，即/admin/import/ 。

我忍不住认为这不属于控制器。 有没有更好的方法可以做到这一点， import.rb代码移到独立的import.rb文件中，以便只能从控制台访问它？ 如果是这样，我需要将该文件放在/lib/目录中的什么位置？

这是一个代码片段：

class AdminController < ApplicationController

    def import
      f = File.open("#{Rails.root}/public/feed.xml")
      @doc = Nokogiri::XML(f)
      f.close

      ignore_list = [] # ignore list

      @doc.xpath("/*/product[not(name = following-sibling::product/name)]").each do |node|
        if !ignore_list.include? node.xpath("./programName").inner_text.strip
          Product.create(:name => clean_field(node.xpath("./name").inner_text).downcase, 
          :description => clean_field(node.xpath("./description").inner_text),
          :brand => Brand.find_or_create_by_name(clean_field_key(node.xpath("./brand").inner_text).downcase),         
          :merchant => Merchant.find_or_create_by_name(clean_field_key(node.xpath("./programName").inner_text).downcase),     
          :image => node.xpath("./imageUrl").inner_text.strip,
          :link => node.xpath("./productUrl").inner_text.strip,
          :category => Category.find_or_create_by_name(clean_field_key(node.xpath("./CategoryName").inner_text).downcase),
          :price => "£" + node.xpath("./price").inner_text.strip)
          print clean_field(node.xpath("./name").inner_text).downcase + "\n"       
        end
      end
    end
end

Answer 1

您的代码听起来像作为Rails Runner脚本运行一样可以正常工作。 它们是在您网站的常规Rails流程之外运行的脚本，但是具有对ActiveRecord设置的完全访问权限，因此您可以轻松访问数据库。

我认为Rails对文件位置的要求并不像对其他所有文件一样严格，但是我会在“ app”下创建一个名为“ scripts”的子目录，并将其放在该目录中。 保持整洁的目录结构对维护而言是一件好事。

您没有说是在运行Rails 3还是以前的版本。 如果您正在运行Rails 3，请在Rails应用程序的命令行中输入rails runner -h以获取更多信息。

有些人认为脚本应该使用rake来运行， 如果它们正在处理文件和文件夹并对应用程序运行所在的rails-space进行常规维护，则我同意这些说法。如果您要执行的定期任务是我整理数据库的内容，或者在您的情况下，检索用于支持您的应用程序的内容，我认为这应该是“运行者”任务。

您可以构建功能，以便仍然可以触发代码通过URL运行，但是我认为有可能滥用它，特别是如果您可以覆盖所需的数据或用重复/冗余数据填充数据库。 我认为最好是通过操作系统启动的cron来使任务定期运行，以使事情保持一定间隔，或者仅手动运行。 如果您可以通过URL访问使其保持可用状态，建议您使用密码来避免滥用。

最后，作为一个长期从事此工作的人，我建议您在代码中进行一些结构化和对齐：

Product.create(
  :name        => clean_field(node.xpath("./name").inner_text).downcase,
  :description => clean_field(node.xpath("./description").inner_text),
  :brand       => Brand.find_or_create_by_name(clean_field_key(node.xpath("./brand").inner_text).downcase),
  :merchant    => Merchant.find_or_create_by_name(clean_field_key(node.xpath("./programName").inner_text).downcase),
  :image       => node.xpath("./imageUrl").inner_text.strip,
  :link        => node.xpath("./productUrl").inner_text.strip,
  :category    => Category.find_or_create_by_name(clean_field_key(node.xpath("./CategoryName").inner_text).downcase),
  :price       => "£" + node.xpath("./price").inner_text.strip
)

简单的对齐可以大大帮助您维护代码，或帮助最终保持代码完整性的人保持理智。 我可能会看起来像这样：

Product.create(
  :name        => clean_field( node.xpath( "./name"        ).inner_text ).downcase,
  :description => clean_field( node.xpath( "./description" ).inner_text ),

  :brand       => Brand.find_or_create_by_name(    clean_field_key( node.xpath( "./brand"       ).inner_text ).downcase ),
  :merchant    => Merchant.find_or_create_by_name( clean_field_key( node.xpath( "./programName" ).inner_text ).downcase ),

  :image       => node.xpath( "./imageUrl"   ).inner_text.strip,
  :link        => node.xpath( "./productUrl" ).inner_text.strip,

  :category    => Category.find_or_create_by_name( clean_field_key( node.xpath( "./CategoryName" ).inner_text ).downcase ),

  :price       => "£" + node.xpath( "./price" ).inner_text.strip
)

就是我 我喜欢有更多的空格，尤其是在有嵌套方法时，并且我喜欢在通用/相似函数之间进行垂直对齐。 我发现它使扫描代码和查看任何差异变得更加容易，这在调试或查找特定内容时会有所帮助。 同样，这只是我的偏爱，但这是我多年来用许多不同语言编写代码的中学到的东西。

Answer 2

我的某些应用程序具有类似的功能。 我通常将此逻辑放在一个类中，例如Importer 。 这样，我既可以从控制台使用它，也可以进行访问控制的控制器操作，以允许其他人从Web使用它。 只要在应用程序的加载路径中，放置类的位置就并不重要。 我倾向于将我的应用程序放置在app / models中，而不是放在/ lib中，以便在进行更改时不必在开发中重新加载应用程序。