Rails 3 views refactoring：conditionals and finds

Question

これらの2ビットのビューコードをリファクタリングするにはどうすればよいでしょうか？

<%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%> 

と（formtasticを使用して）

%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 

私は私のアプリでbazillion時間に関するこれらのそれぞれの例を持っている、と私は飛び込む前に、私は最善の方法でリファクタリングよ知っていただきたいと思いますこのかなり巨大な仕事に。

Answer 1

まず最初に、if/elseのブランチ（通常はそうではありませんが）が悪いので、コードを読みにくくするだけです。

また、<li>の出力を得るには、Supplydetail.find_all_by_isbn_id(@isbn)とProductavailability.find_by_code(...)を2回計算するだけです。どちらもしないでください。

そして、ERBノイズを減らすために、そのロジックの大部分をコントローラに（あるいはどこでどのくらい頻繁に使用されているかに応じてヘルパーに）プッシュしたいと思うかもしれません。

多分このような何かがあなた（あなたのコードを維持する者）に役立つでしょう。最初のコントローラのもののビット：あなたの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 

そして：あなたはこの種のものをたくさんやっている場合は

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

あなたはあなたの中Productavailability.for_isbn便利なクラスメソッドを追加することができますモデル。あなたのコントローラは、ただ必要とするでしょう：

@avail = Productavailability.for_isbn(@isbn) 

しかし、私はあなた自身を繰り返すまで心配しません。

私はformtasticに精通していないので、私はそれを手伝ってはいけません。