ruby-on-rails - 如何干燥两个不同类中使用的范围方法？

Question

我正在使用 Ruby on Rails 3.2.2，我想通过“指定”/“过滤”这些关联对象的属性值来检索/范围关联对象。也就是此时我使用的是如下代码:

class Article < ActiveRecord::Base
  def self.search_by_title(search)
    where('articles.title LIKE ?', "%#{search}%")
  end
end

class ArticleAssociation < ActiveRecord::Base
  def self.search_by_article_title(search)
    joins(:article).where('articles.title LIKE ?', "%#{search}%")
  end
end

在上面的代码中，where('articles.title LIKE ?', "%#{search}%") 子句重复了两次，所以我认为可以通过 DRY 改进它原理:是否可以直接在 ArticleAssociation.search_by_article_title 方法中使用 Article.search_by_title 方法？

---

典型用例是:

• ArticleAssociation.search_by_article_title("示例字符串")
• Article.search_by_title("示例字符串")

Answer 1

除非你完全改变代码结构，否则不会。

您可以使用 lambda 进行一些黑客攻击，但那会比您正在 DRYing 的代码更多。有好的重构之类的东西，也有坏的重构之类的东西。除非在 2 个或更多地方使用一段非常复杂或长的代码，否则您可以担心重构。代码约定很重要，但对于像这样的微小的单一方法调用的事情来说，这是一种浪费，并且可能会使您的代码更加神秘。

不过，我知道人们不回答您的问题会很烦人，所以在这里:

class Article < ActiveRecord::Base
  SEARCH_BY_TITLE=lambda {|obj, search| obj.where('articles.title LIKE ?', "%#{search}%")}
  def self.search_by_title(search)
    SEARCH_BY_TITLE.call(self, search)
  end
end

class ArticleAssociation < ActiveRecord::Base
  def self.search_by_article_title(search)
    Article::SEARCH_BY_TITLE.call(joins(:article),search)
  end
end

这只是将 lambda 作为常量对指定对象执行 where 调用。这两种方法都只是包装那个 lambda。

注意:虽然这可能被认为更优雅，但它会大大降低性能，因为 lambda、闭包和额外调用在像 Ruby 这样的动态语言中是昂贵的。但我认为这对您来说不是问题。