1

我正在尝试为 Rails 中的表设置逻辑。基本上它是一个时间表应用程序，我想做的是设置它，如果你达到目标，它会显示为绿色。如果您快到了，则为黄色。如果你还没有接近，那就是红色。我在下面的代码有效。但是看看我的日志，它似乎效率低下。我怎样才能写得更好？

这是视图。用户/hours.html.erb

    <div class="page-header"><h1>Weekly Timesheets</h1></div>

<table class="nicetable table-condensed">
  <thead>
    <tr>
      <th>Name</th>
      <th>Email</th>
      <th>Total Hours</th>
      <th>Role</th>
      <th>Change Role</th>
      <th>Timesheet</th>
    </tr>
  </thead>
  <tbody>
  <% @users.each do |user| %>
    <tr class="<%= 'success' if user.has_role? :staff and user.add_hours >= 10 %><%= 'warning' if user.has_role? :staff and user.add_hours < 10 and user.add_hours > 6 %><%= 'error' if user.has_role? :staff and user.add_hours >= 0 and user.add_hours <= 6 %><%= 'success' if user.has_role? :new_staff and user.add_hours >= 15 %><%= 'warning' if user.has_role? :new_staff and user.add_hours < 15 and user.add_hours >= 12 %><%= 'error' if user.has_role? :new_staff and user.add_hours < 12 and user.add_hours >= 0 %>">
      <td><%= user.name%></td>
      <td><%= user.email %></td>
      <td><%= user.add_hours %></td>
      <td><%= user.roles.first.name.titleize unless user.roles.first.nil? %></td>
      <td>
        <a data-toggle="modal" href="#role-options-<%= user.id %>" class="btn btn-mini" type="button">Change role</a>
        <%= render user %>
      </td>
      <td><%= link_to 'Timesheet', user_timesheets_path(user, @timesheets), class: "btn btn-mini" %>
    </tr>

这是我的用户模型。

    def add_hours
  self.timesheets.where('day BETWEEN ? AND ?', Date.today.beginning_of_week, Date.today.end_of_week).order('created_at DESC').sum{|p| p.teacher + p.conversation + p.study}

 end
4

3 回答 3

1

dax 说的是正确的，但是请不要将该代码放入您的模型中。您可以使用演示者模式或装饰者模式。我发现draper gem 非常有用。在您的情况下，您将创建一个用户装饰器，其中包含专门为此视图返回数据的所有方法。就像是：

class UserDecorator < Draper::Decorator
  delegate_all

  def role_name
    object.roles.first.name.titleize if object.roles.first.present?
  end

  def hours_stauts
    if object.add_hours >= hours_success
      'success'
    elsif object.add_hours >= hours_warning
      'warning'
    else
      'error'
    end
  end

  def hours_success
    object.has_role? :staff ? 10 : 15
  end

  def hours_warning
    object.has_role? :staff ? 6 : 12
  end
end

您可以添加此视图或其他视图所需的任何方法。阅读有关如何使用 gem 的 draper 文档。

于 2013-08-21T10:50:08.597 回答
1

这是一个重构版本。我进行了以下更改：

  • 将状态逻辑提取到 user#progress_status 并对其进行重构以使其更可重用并清理视图
  • 在视图中使用 for 循环，因为它更具可读性
  • 将 user#add_hours 重命名为 total_hours 所以它并不意味着添加（命令），而只是一个虚拟属性（这是一个查询）
  • 删除了方法中的订单范围，因为您不需要这种情况
  • 使 total_hours 将其结果缓存在用户身上，因此多个后续调用不会多次命中数据库（就像我们在进度状态中所做的那样）

首先是观点。我只包含了 tbody 中的内容：

<% for user in @users %>                                                         
    <tr class="<%= user.progress_status.to_s %>">                                  
      <td><%= user.name %></td>                                                    
      <td><%= user.email %></td>                                                   
      <td><%= user.total_hours %></td>                                               
      <td><%= user.roles.first and user.roles.first.name.titleize %></td>          
      <td>                                                                         
        <a data-toggle="modal" href="#role-options-<%= user.id %>" class="btn btn-mini" type="button">Change role</a>
        <%= render user %>                                                         
      </td>                                                                        
      <td><%= link_to 'Timesheet', user_timesheets_path(user, @timesheets), class: "btn btn-mini" %></td>
    </tr>                                                                          
  <% end %>

以及重构的用户模型方法：

def total_hours                                                                    
  @total_hours ||= timesheets                                                   
    .where('day BETWEEN ? AND ?', Date.today.beginning_of_week, Date.today.end_of_week)
    .sum {|p| p.teacher + p.conversation + p.study}                             
end                                                                             

def progress_status                                                             
  if has_role? :staff                                                           
    if total_hours >= 10                                                           
      :success                                                                     
    elsif (7..9).include? total_hours                                           
      :warning                                                                     
    else                                                                           
      :error                                                                       
    end                                                                                                                                         
  elsif has_role? :new_staff                                                       
    if total_hours >= 15                                                           
      :success                                                                     
    elsif (12..14).include? total_hours                                            
      :warning                                                                     
    else                                                                           
      :error
    end
  end
end

您可以使用最后一种方法走得更远（当提供更多上下文时），但现在应该没问题。

于 2013-08-21T11:28:32.160 回答
0

我认为这应该工作！

在user.rb：

def user_css
  if self.has_role? :staff do |staff|
    if staff.add_hours >= 10
      'success'
    elsif staff.add_hours >= 6
      'warning'
    else
      'error'
  end
  if self.has_role? :new_staff do |new_staff|
    if new_staff.add_hours >= 15
      'success'
    elsif new_staff.add_hours >= 12
      'warning'
    else
      'error'
  end
end

在你的view

<% @users.each do |user| %>
  <tr class="<%= user.user_css %>">
    ...
  </tr>
<% end %>
于 2013-08-21T09:30:58.990 回答