3

我的搜索方法又臭又肿，我需要一些帮助来重构它。我是 Ruby 新手，我还没有弄清楚如何有效地利用它，这导致了像这样的臃肿方法：

  # discussion.rb
  def self.search(params)
    # If there is a search query, use Tire gem for fulltext search
    if params[:query].present?
      tire.search(load: true) do
        query { string params[:query] }
      end

    # Otherwise grab all discussions based on category and/or filter
    else

      # Grab all discussions and include the author
      discussions = self.includes(:author)

      # Filter by category if there is one specified
      discussions = discussions.where(category: params[:category]) if params[:category]

      # If params[:filter] is provided, user it
      if params[:filter]
        case params[:filter]
        when 'hot'
          discussions = discussions.open.order_by_hot
        when 'new'
          discussions = discussions.open.order_by_new
        when 'top'
          discussions = discussions.open.order_by_top
        else
          # If params[:filter] does not match the above three states, it's probably a status
          discussions = discussions.order_by_new.where(status: params[:filter])
        end
      else

        # If no filter is passed, just grab discussions by hot
        discussions = discussions.open.order_by_hot
      end
    end
  end

  STATUSES   = {
    question:   %w[answered],
    suggestion: %w[started completed declined],
    problem:    %w[solved]
  }

  scope :order_by_hot,  order('...') DESC, created_at DESC")
  scope :order_by_new,  order('created_at DESC')
  scope :order_by_top,  order('votes_count DESC, created_at DESC')

这是一个可以按类别过滤（或不过滤）的讨论模型：question、problem、suggestion。

所有讨论或单个类别可以通过hot、new、votes或进一步过滤status。状态是模型中的一个哈希值，它有几个取决于类别的值（状态过滤器仅在 params[:category] ​​存在时出现）。

使事情复杂化的是使用轮胎的全文搜索功能

但我的控制器看起来又漂亮又整洁：

  def index
    @discussions = Discussion.search(params)
  end

我可以把它干掉/重构一下，也许使用元编程或块？我设法将其从控制器中提取出来，但随后就没有想法了。我对 Ruby 的了解还不够深入，无法更进一步。

4

1 回答 1

3

对于初学者，“基于类别和/或过滤器获取所有讨论”可以是一种单独的方法。

params[:filter]重复了很多次，所以把它放在顶部：

filter = params[:filter]

您可以使用

if [:hot, :new, :top].incude? filter
  discussions = discussions.open.send "order_by_#{filter}"
...

此外，考虑 if then else if case else 语句。我更喜欢分解成单独的方法并尽早返回：

def do_something
  return 'foo' if ...
  return 'bar' if ...
  'baz'
end

discussions = discussions...出现了很多次，但看起来很奇怪。可以return discussions...改用吗？

为什么常数STATUSES出现在最后？通常常量出现在模型的顶部。

请务必在重构之前编写所有测试。

要回复有关以下内容的评论return 'foo' if ...：

考虑：

def evaluate_something
  if a==1
    return 'foo'
  elsif b==2
    return 'bar'
  else
    return 'baz'
  end
end

我建议将其重构为：

def evaluate_something
  return 'foo' if a==1
  return 'bar' if b==2
  'baz'
end

也许您可以重构一些 if..then..else..if 语句。

推荐书籍：清洁代码

于 2012-11-01T14:29:32.987 回答