Rails 3 视图重构：条件和查找

Question

你会推荐什么作为重构这两个视图代码的最佳方法？

<%if Supplydetail.find_all_by_isbn_id(@isbn).first.nil? %>
<%else%>
<%if Productavailability.find_by_code(Supplydetail.find_all_by_isbn_id(@isbn).first.productsupply_supplydetail_productavailability).nil? %>
<%else%>
<li><%= Productavailability.find_by_code(Supplydetail.find_all_by_isbn_id(@isbn).first.productsupply_supplydetail_productavailability).value %></li>
<%end%>
<%end%>

和（使用格式）

%li.tip 
  = tooltip(:test, :hover)
= f.input :relatedmaterial_relatedproduct_idvalue, :label => "Related ISBN", :as => :select, :collection => Isbn.all, :label_method => :descriptivedetail_titledetail_titleelement_titlewithoutprefix, :value_method => :productidentifier_idvalue  
%li.list
  = link_to "Edit list", isbns_path 

我在我的应用程序中有无数次的这些示例，并且想知道在我投入到这项相当庞大的工作之前，我正在以最好的方式进行重构。

Answer 1

首先，if/else 中的空 if 分支通常（但并非总是如此！）闻起来很糟糕，所以不要这样做，它只会让你的代码更难阅读和理解。

另外，您计算 Supplydetail.find_all_by_isbn_id(@isbn) 和 Productavailability.find_by_code(...) 两次只是为了得到您的 <li> 输出。也不要那样做。

您可能希望将大部分逻辑推送到您的控制器中（或者可能是一个助手，具体取决于它的使用位置和频率）以减少 ERB 噪声。

也许这样的事情会更好地为您（以及维护您的代码的人）服务；首先是一些控制器的东西：

@avail  = nil
by_isbn = Supplydetail.find_all_by_isbn_id(@isbn).first
if by_isbn
    @avail = Productavailability.find_by_code(by_isbn.productsupply_supplydetail_productavailability)
end

然后在你的 ERB 中：

<% if @avail %>
    <li><%= @avail.value %></li> 
<% end %>

如果您正在做很多此类事情，那么您可以在模型中添加一个Productavailability.for_isbn 便利类方法。那么你的控制器只需要：

@avail = Productavailability.for_isbn(@isbn)

但除非你开始重复自己，否则我不会担心。

我不熟悉formtastic，所以我无法帮助您。