3

我正在寻找一个必须是常见场景的最佳实践:将从 Rails(ActiveRecord,SQL)数据库表中提取的稀疏记录按摩到一个结构中,以便在 HTML 中呈现表。

出于性能原因,我做了一个查询,它返回这样的数据(为了清楚起见,我简化了示例):

Lineitem.all
=> [#<Lineitem id: 1, username: "Bob", category: "A", amount: 10>, 
    #<Lineitem id: 2, username: "Bob", category: "C", amount: 20>, 
    #<Lineitem id: 3, username: "Jen", category: "A", amount: 30>, 
    #<Lineitem id: 4, username: "Ken", category: "B", amount: 40>,
    #<Lineitem id: 5, username: "Ken", category: "E", amount: 50>]

我的目标是这样的 HTML 表格:

             A     B     C     D     E
           ---   ---   ---   ---   ---
  Bob       10          20
  Jen       30   
  Ken             40                50
  Sam

如果每个类别都作为单独的列存储在数据库中(或者如果我使用的是 NoSQL ......?!)或者如果我不关心数据库性能,这将是微不足道的。

为了解决这个问题,我一直在编写这样的臭帮助代码:

# create hash lookup, index first by username then by category, eg:
#   ["Bob"]["A"] = #<Lineitem id: 1, ...>
#   ["Bob"]["C"] = #<Lineitem id: 2, ...>
#   ["Jen"]["A"] = #<Lineitem id: 3, ...>  ...
def index_lineitems(lineitems)
  h = {}
  lineitems.each do |li|
    h[li.username] = {} unless h.key? li.username
    h[li.username][li.category] = li
  end
  h
end

# look up value from indexed hash
def get_lineitem_amount(indexed_lineitems, username, category)
  if indexed_lineitems.key?(username) && indexed_lineitems[username].key?(category)
    indexed_lineitems[username][category].amount
  else
    ""
  end
end

或对此的一些变化。然后我确定最终的行和列列表是什么(注意“Sam”行......)并通过get_lineitem_amount每次循环和调用来呈现 HTML 表。这是如此糟糕的代码,我很尴尬地分享它。

对于这个常见问题,肯定有一种更简洁、更 OO 和 Rails 友好的方法。

有什么建议吗?

4

1 回答 1

1

我正在做一些非常相似的稍微清洁的事情:

假设这是在控制器中:

@data = LineItem.all

这就是观点

columns = @data.map(&:category).uniq

%table
  %thead
    %tr
      %th &nbsp;
      - columns.each do |column|
        %th= column
  %tbody
    - @data.group_by(&:username).each do |username, rows|
      %tr
        %td= username
        - cursor = 0
        - rows.group_by(&:category).sort_by{|cat,rows| columns.index(cat)}.each do |category, rows|
          - until cursor == columns.index(category) do
            - cursor += 1
            %td &nbsp;
          %td= rows.sum(&:amount)

如果您将列存储在单独的数据库表中并将它们包含到当前模型中,那么它会变得更干净. 一个额外的查询并不会真正破坏应用程序的性能。

于 2012-07-17T19:51:33.847 回答