Rubyでリファクタリングする方法

Question

Rubyを学習し始めてまだ日が浅い初心者です。

Railsチュートリアルをとりあえず１周終えたので
、今度は自分なりにカスタマイズしながら２周目を始めました。

RailsチュートリアルではユーザーのアイコンはGravatarによって表示されるようになっていたので
これをユーザーが自分で編集できるようにしたいと考えました。

仕様としては
・carrierwaveを使って画像の投稿、保存を行う
・ユーザー登録時（ユーザーモデルのimageカラムがnilの時）はこちらが用意した
デフォルト画像を表示
・プロフィール編集で画像を選択すると、それ以降その画像が表示されるようになる

<% if @user.image? %>
    <%= image_tag @user.image %>
<% else %>
    <%= image_tag("/user_images/default_image.png")%>
<% end %>

上記の式で実装すること自体はできました。
しかし、この長ったらしいコードがいたるとこに出てくるのでリファクタリング
したいのですが、@userの箇所がuserだったりcurrent_userだったり
するので単純なリファクタリングではうまく動かなくて困っています。

こういった場合、どのようにするとうまくリファクタリングできるのでしょうか？
初歩的な質問かと思いますが、ご回答お願いいたします。

Answer 1

モデルクラスに

def avatar_path
  if image?
    image
  else
    "/user_images/default_image.png"
  end
end

を追加してビューは

<%= image_tag @user.avatar_path %>

みたいな感じかな

Comments

[HirotoKawakami]

回答ありがとうございます！
おかげさまで問題を解決することができました！
画像を扱うメソッドはこのように書けばいいんだなと、非常に勉強になりました！