Forum: Ruby on Rails Bad Smells sing in a rails application

Announcement (2017-05-07): is now read-only since I unfortunately do not have the time to support and maintain the forum any more. Please see and for other Rails- und Ruby-related community platforms.
F09383a4b8725aa99cf892813608de76?d=identicon&s=25 Pedro Del gallego (pedro)
on 2007-06-27 19:32
(Received via mailing list)
Hi all,

   Sometime ago, i read the slides  "when V is for Vexing: Patterns to
DRY Up Your Views" (Bruce Williams  railsconf). They are not online
anymore.  I post one slide in my[1}.. But only the bad smells sign in
the view, and only problem,  not the solution.  here they are

   1. Calling find on a model directly

      <% Sales Sales.find_all_by_region(params[:region]).each .do |t|

      % end %>

   2. Calling find on an association

   3. Conditionally inserting content if/else clauses, case statements,

      <% if current_user.admin? %>Admins see this

      <% end %>

   4. Doing a complex inline ma map, sor sort, t, select, etc on a

      <% applications.sort_by{|app| [app.priority,]
}.each do |app|

      <% end %>

   5. Assigning temporar temporary variables

      <% indicator_id = "loading_indicator_#{}" %><%=
link_to_remote("Show Status",:url => item_path(item),

      :loading => "$('#{indicator_id}').show()",

      :complete => "$('#{indicator_id}').hide()",

      ) %>

      <%= image_tag('progress.gif', :id => indicator_id)%>

I would like to ask for new bad smell signs. In the view, in the
controller or in the model.  (and discuss this , specially number 3)


[1] :
Pedro Del Gallego

Email              :   pedro.delgallego@gmail.cot
This topic is locked and can not be replied to.