1d5707f13f1f9840f3c7c05adfcd8fa0
2015/12/02 13:46:18 投稿
0

【Refactor Me】viewに条件式

リファクタリングのネタが思いつかない方のために、beforeコードを用意しました。

このbeforeコードをリファクタリングしてくれた方にも sushi のチャンスがあります!

beforeコードをコピーして新規投稿を作成してね!

Before

# view
<% if current_user.admin? %>
  <%= current_user.message_for_admin(@message) %>
<% else %>
  <%= current_user.message_for_general(@message) %>
<% end %>

# app/models/user.rb
def message_for_admin message
  img_tag = '<img src="/assets/message/admin.png" />'
  span_tag = "<span>#{message.body}</span>"
  "<div>#{img_tag}#{span_tag}</div>"
end

def messsage_for_general message
  img_tag = '<img src="/assets/message/general.png" />'
  span_tag = "<span>#{message.body}</span>"
  "<div>#{img_tag}#{span_tag}</div>"
end

After

# your cool code

みんなのコメント