Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
−92
if html_content.product_id == 11 || html_content.product_id == 8 || html_content.product_id == 15 || html_content.product_id == 20
@terms_and_conditions = Page.find(212).contents
else
@terms_and_conditions = Page.find(213).contents
end
Запостил: 
paveldruzyak,
09 Июля 2010
Follow us!
А вот эта проверка идентификатора продукта вобще по всему коду разбросана :-)
рефакторится где то так
def terms_and_conditions(product)
case product.terms_type
when 'type1'
Page.find(212).contents
when 'type2'
Page.find(213).contents
else
'invalid terms'
end
end
<%= terms_and_conditions(product) %>
смена логики на использование terms_type вместо id.
Page.terms_and_conditions medical_details_required?
end
def Page.terms_and_conditions(medical_detail s_required?)
medical_details_required? ? Page.find(212) : Page.find(213)
end