Forkwell
2015/12/02 13:46:18 投稿
0

【Refactor Me】viewに条件式

<p>リファクタリングのネタが思いつかない方のために、beforeコードを用意しました。<br><br> このbeforeコードをリファクタリングしてくれた方にも <img alt="sushi" src="/assets/emoji/unicode/1f363-831097849a370c02369283b17255c6e4f0ffe9e0a1269610119e49b26401f621.png" style="vertical-align:middle" width="20" height="20" /> のチャンスがあります!</p> <p>beforeコードをコピーして新規投稿を作成してね!</p>

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

みんなのコメント