重构 Rails 视图

Question

我是 Rails 开发的新手，所以请耐心等待。我正在创建一个视图，其中有几个看起来非常相似的字段。它只是请求以某种方式重构，但我一直无法弄清楚。请参阅下面的代码示例：

<%= form_for(@tapelog) do |f| %>
  <div class="container">
    <div class="field span-16">
      <div class="span-8 labelRight">
        <%= f.label :client %>
      </div>
      <div class="span-8 last">
        <%= f.collection_select :client_id, Client.find(:all), :id, :name,
                                { :prompt => "Select a Client..." },
                                { :class => "automatixSelect" } %>
      </div>
    </div>
    <div class="field span-16">
      <div class="span-8 labelRight">
        <%= f.label :employer %>
      </div>
      <div class="span-8 last">
        <%= f.collection_select :employer_id, Employer.find(:all), :id, :name,
                                { :prompt => "Select an Employer..." },
                                { :class  => "automatixSelect" } %>
      </div>
    </div>
  ....
<% end %>

大约有 7 个这样的字段。我试图将它们全部放入部分中，这样我就可以减少此页面上的混乱，但由于未定义“f”，所以出现错误。关于如何减少这里的一些混乱有什么想法吗？任何其他有关 Ruby 重构的一般提示也将受到欢迎。

谢谢， ——A.

Answer 1

如果您想在后续部分中使用“f”，请将其作为参数传递

<%= render :partial => :some_partial, :locals => { :f => f } %>

Answer 2

为了清楚起见，我要做的第一件事是将“f”局部变量重命名为“form”。然后，将代码提取到部分是合理的：

<div class="field span-16">
  <div class="span-8 labelRight">
    <%= form.label :employer %>
  </div>
  <div class="span-8 last">
    <%= form.collection_select :employer_id, Employer.find(:all), :id, :name,
                               { :prompt => "Select an Employer..." },
                               { :class  => "automatixSelect" } %>
  </div>
</div>

在我看来，这是一个belongs_to关系，所以我可以创建一个名为“belongs_to”的部分并将其呈现为：

<%= render :belongs_to, :parent => :employer, :form => form %>

我们的想法是我们将有一个名为“parent”的本​​地我们可以部分改变。请注意，我使用了速记部分语法。它等同于：

<%= render :partial => :belongs_to, :locals => { :parent => :employer, :form => form } %>

<div class="field span-16">
  <div class="span-8 labelRight">
    <%= form.label parent %>
  </div>
  <div class="span-8 last">
    <%= form.collection_select :"#{parent}_id", parent.to_s.capitalize.constantize.find(:all), :id, :name,
                               { :prompt => "Select an #{parent.to_s.capitalize}..." },
                               { :class  => "automatixSelect" } %>
  </div>
</div>