在 Rails 中查看演示者

Question

我最近看到 Ryan Bates 的一集从零开始的演示者，它很好地解释了演示者的工作。这就是我所指的：

http://railscasts.com/episodes/287-presenters-from-scratch

所以我决定重构一下我的一些观点，因为里面有一些非常复杂的逻辑。

我的应用中有一个页面显示用户列表以及他们的图像等。所以我所做的就是为每个用户呈现一个部分，这就是它的工作原理。

在使用演示者进行重构后，我得到了相同的部分，但所有逻辑都移到演示者之外，并且视图只显示值，但是我注意到为每个用户呈现部分需要两倍的时间以前做过。

这是之前的（来自日志的数据）：

Rendered users/_user_thumb.html.haml (6.4ms)

重构后：

Rendered users/_user_thumb.html.haml (18.5ms)

我查看了日志是否有一些额外的查询会触发或其他什么，并且在重构代码时没有触发额外的查询。

我的问题是为什么这么慢？难道我做错了什么？

这是我的代码现在的样子：

 - @users.each do |user|
   = render_partial "user_thumb", user: user

现在在渲染部分user_thumb 中我有一个复杂的逻辑，但现在我有了这个：

- present user do |user_presenter|
  ....mycode here

我用的是haml。

问题更新

比利在他的回答中有一些好点，所以我投票赞成他的回答。

我尝试了他的建议来装饰控制器中的@users：

@users = UserDecorator.decorate(users)

然后我尝试检查它，我看到类似：

#<UserDecorator:0x0000000966a6f8 @object=[#<User id: 8112 .....some users ], @context={}>

这很好，因为我似乎立即让我的用户得到了装饰。然后我尝试了比利的建议并做了：

= render partial: "user_thumb", collection: @users

但是当我尝试在我看来：

collection.each do |u|
  puts "#{user.inspect}"
end

我的装饰器看起来像：

class UserDecorator < Draper::Decorator
  decorates :user
  ....
end

我得到的是用户而不是装饰用户，这是为什么？在装饰对象列表时这是如何工作的？如果最终您仍然获得用户对象，这样做的目的是什么？

这可行，但速度真的很慢：

- @users.each do |user|
       = render_partial "user_thumb", user: user.decorate

如何创建装饰器数组？

Answer 1

有几个地方可以改进代码。

首先，确保您的@users 在传递给视图之前已经在控制器中进行了装饰。好处：

1. 这样你根本不需要编辑原始模板，只需使用相同的@users。

2. 您的视图与您使用的任何呈现/装饰模式都是分离的，并且确实有很多选择。您也可以选择在未触及视图的情况下不使用它。

3. 理论上这应该更快，因为如果你在视图中单独装饰项目，计算机需要一次又一次地加载相关模块，与批量相比。

其次，渲染 collection 而不是迭代视图。根据我的经验，这更快。

代替

- @users.each do |user|
   = render_partial "user_thumb", user: user

做

= render partial: "user_thumb", collection: @users