DRY, Short и обычный помощник, который возвращает несколько строк HTML

Question

Я ищу СУЩЕСТВЕННЫЙ, самый обычный и самый короткий способ написания этого помощника.

У меня есть что-то вроде этого в верхней части каждого файла представления индекса:

<%= page_header "Blog", "My thoughts on things" %>

И соответствующий помощник:

def page_header(title=nil, subtitle=nil)
  header = ''
  header += content_tag(:h1, title) if title.present?
  header += content_tag(:h3, subtitle) if subtitle.present?
  return header.html_safe
end

Это работает, но это уродливо, и я знаю, что есть лучший способ, я просто не могу его найти.

Answer 1

Я бы пошел на частичку.

app/views/shared/_page_header.html.erb * * 1004

<% if title.present? %>
  <h1><%= title %></h1>
<% end %>

<% if subtitle.present? %>
  <h3><%= subtitle %></h3>
<% end %>

И помощник будет:

def page_header(title = nil, subtitle = nil)
  render "shared/page_header", :title => title, :subtitle => subtitle
end

Вы также можете посмотреть на клеток .

Answer 2

Я бы попробовал что-то вроде:

def page_header(title=nil, subtitle=nil)
  [
    title && content_tag(:h1, title),
    subtitle && content_tag(:h3, subtitle)
  ].join.html_safe
end

Хотя это технически не эквивалентно - в него будут включены теги, даже если заголовок не нулевой, а пустой - он, вероятно, будет делать то, что вы хотите вэто дело.

Если вы хотите получить фантазию:

MAPPINGS = [[:title, "h1"], [:subtitle, "h3"]]
def page_header(options = {})
  MAPPINGS.map {|e| options[e[0]] && content_tag(e[1], options[e[0]]) }.join.html_safe
end

Затем вы можете позвонить:

page_header :title => "Foobar", :subtitle => "Woohoo"

И это будет работать.Я бы сказал, что второй помощник уродливее.