Question

This is for Rails 3, almost always I think a content_for?(:foo) is followed by content_for(:foo) (in haml):

%title= content_for?(:title_for_page) ? "#{content_for(:title_for_page)} - Our great website" : 'Our great website'

So instead of doing 2 lookups, isn’t it better to just do 1 lookup and use longer code:

- title_for_page = content_for(:title_for_page)    # is "" when not previously set
%title= title_for_page.blank? ? 'Our great website' : "#{title_for_page} - Our great website"

? But if content_for? is implemented as a hash, then maybe it is super quick anyway, comparable to the blank? anyways?

Answer 1

A one-liner to solve the problem:

- title_for_page = (c = content_for(:title_for_page)).blank? ? 'Our great website' : "#{c} - Our great website"