ruby-on-rails – Rails

我的应用程序有一些报告,我正在尝试为所有这些集合的group_by创建一个帮助方法.

例：

def group_collection(collection,options = {})
    column = options[:column]
    group_count = collection.group_by{ |item| item.column.strftime('%b %y')}
end

这就是我打算如何使用它

@user_groups = group_collection(@users,:column => "created_at")

不幸的是,这不起作用.

undefined method `column' for... [CollectionObject]

有关如何在运行时使“列”变量成为实际列类型的任何线索,因此它将自身视为activerecord列而不是实例方法？

解决方法

忽略代码中的一些其他问题,您要对列执行的操作可以这样完成：

collection.group_by { |item| item.send(column).strftime('%b %y') }

这是有效的,因为在Ruby中,访问实例变量的方式是通过访问器方法,通常以您尝试访问的变量命名,因此@ item.foobar在@item上调用foobar方法.

现在,回到那些“其他问题”.你试图将重复的行为转移到一个地方是很棒的,并且当你把事情变得不那么明确而有利于灵活时,它表明你正在考虑可扩展性.但是,有一些事情对你来说不是很好,我觉得有必要指出.

>分组适用于许多数据类型,其中大多数数据类型不响应strftime.通过对其进行硬编码,您会引入意外行为,这意味着您无法运行group_collection(@ users,：column =>’phone_number’).相反,只有在测试后才能运行该列数据可以响应它.

collection.group_by do |item|
  data = item.send(column)
  data.respond_to?(:strftime) ? data.strftime('%b %y') : data
end

>如果确定此辅助方法的行为是对任意列进行分组,则可以放弃接受选项哈希的额外复杂性,只是为了规避它.

def group_by_column(collection,column)
  collection.group_by { ... }
end
group_by_column(@users,:column)

>如果您使用的是Ruby 1.9并且不需要执行任何其他格式化,则可以更轻松地按任意列进行分组.

@users.group_by &:created_at

ruby-on-rails – Rails – group_by

解决方法

相关文章