Breadcrumbs in Ruby on Rails

Question

I\'m slightly insecure about my breadcrumb solution. Names and links are defined in each controller action:

Home         


        

Answer 1

This is mostly a matter of opinion, but anyway:

1. I would not want that much logic in a view. We've probably all done it, but it gets messy quickly.
2. The code is not safe against future changes that affect the depth of the tree.
3. Instead of linked variables *_name and *_link, I'd suggest using proper objects anyway, with some link_to functionality.

You might find Episode 162 of Railscasts of interest for a nice solution that gets by with

<% for page in @page.ancestors.reverse %>
  <%= link_to h(page.name), page %> >
<% end %>