プロフィール画像を表示したい

Question

前提・実現したいこと

新規登録でプロフィール画像を保存した場合は登録した画像を表示し、
何も保存しなかった場合はデフォルトの画像を表示させたいです。

発生している問題・エラーメッセージ

該当のソースコード

users/show.html.erb

    <div class="col-md-4 text-center">
      <% if @user.profile_photo.present? %>
        <%= image_tag @user.profile_photo, class: "round-img" %>
      <% else %>
        <%= image_tag avatar_url(@user), class: "round-img" %>
      <% end %>
    </div>

application.helper.rb

module ApplicationHelper
  def avatar_url(user)
    return user.profile_photo unless user.profile_photo.nil?

    Digest::MD5.hexdigest(user.email).downcase
    "https://~~~.jpg"
  end
end

試したこと

if文を記述しなかった時は、デフォルト画像のみ又は保存した画像のみの表示はできるので、場合分けがうまくいっていないのではないかと思っております。
またエラーコードで調べたところ、

<%= image_tag avatar_url(@user), class: "round-img" if @user.profile_photo? %>

上記コードのように”if @user.profile_photo?”を追加すると解決したという回答をよく見たのですがこちらを追加した場合、エラーは出なくなるのですが画像が表示されませんでした。何かご助言よろしくお願い致します。

補足情報（FW/ツールのバージョンなど）

Rails 6.1.4.1
Mac OS
docker

Answer 1

carrierwave を使っているのなら、
carrierwave のカラムに nil? とすると画像が空の場合でも false を返すようです。
なのでヘルパーの nil? を blank? にしてください。

User.new.profile_photo.file #=> nil
User.new.profile_photo.present? #=> false
User.new.profile_photo.nil? #=> false
User.new.profile_photo.blank? #=> true

判定処理がビューとヘルパーで重複しているので、そこも整理すると以下のようになります

<div class="col-md-4 text-center">
  <!-- profile_photo の有無による分岐はヘルパーで行う -->
  <%= image_tag avatar_url(@user), class: "round-img" %>
</div>

module ApplicationHelper
  # profile_photo が空だった場合の画像
  BLANK_IMAGE = "https://blank.jpg"

  def avatar_url(user)
    return BLANK_IMAGE if user.profile_photo.blank?

    user.profile_photo
  end
end

Comments

[hinata_hakumai]

無事表示できました！ありがとうございます！