画像の初期投稿時、Postカラムにその#{Post.id}.jpgとしたい

Question

画像をアップデートする時に、そのポストが保有するimage_name欄に、
そのポストのid.jpg（#{Post.id}.jpg）という名前を入れたいです。

=> #<Post id: 1, title: "新しい投稿", user_id: 1,image_name: "default_post.jpg",created_at: "xx", updated_at: "xx">

Post Modelは上記のようになります。

  def create
    @post = Post.new(
      title: params[:title],
      user_id: @current_user.id,
      image_name: "default_post.jpg")
    if @post.save
    if params[:image_name]
      @post.image_name = "#{@post.id}.jpg"
      image = params[:image_name]
      File.binwrite("public/#{@post.image_name}", image.read)
    end
      redirect_to("/posts/index")
    else
      render("posts/new")
    end
  end

Post Controllerは上記のようになります。
Post.newで、データを登録し、もしデータが登録された場合と続き、
if params[:image_name]では、もし画像の投稿があればというif文にしています。

上記の場合、Postのimage_nameに登録されるのが、"default_post.jpg"になります。
その部分に、"#{@post.id}.jpg"と入れると、データのセーブ前なのでかと思いますが、
undefined method `id' for nil:NilClass　というエラーが出てしまいます。

Publicディレクトリ内に保存される画像の画像名はちゃんと"#{@post.id}.jpg"で保存されています。

質問内容がややこしくて申し訳ないです。
恐れ入りますが、ご教示頂ければ幸いです。

Answer 1

Post作成前にIDは分からないのでできません
作成後に無理やり更新するならできますがIDの二重保存なので非推奨です
image_name カラムは削除してDBに保存せず、関数にするのはどうでしょう？

class Post < ApplicationRecord
  def image_name
    "#{id}.jpg"
  end
end

class PostsController < ApplicationController
  def create
    @post = Post.new(
      title: params[:title],
      user_id: @current_user.id)
    if @post.save
      if params[:image_name]
        image = params[:image_name]
        File.binwrite("public/#{@post.image_name}", image.read)
      end
      redirect_to("/posts/index")
    else
      render("posts/new")
    end
  end
end

もし画像の有無を判断したい（デフォルト名なら画像なし）等の意図があるとしても
そのような仕様はバグの元なので、専用の情報として保持するべきだと思います

if params[:image_name]
  image = params[:image_name]
  File.binwrite("public/#{@post.image_name}", image.read)

  # 画像フラグカラムにTrueを入れる等
  @post.update(imaged: true)
end

Comments

[tomtom1]

ご回答ありがとうございます。
作成後に無理やり更新するならできますがIDの二重保存なので非推奨について、承知しました。また新たな方法をお教えいただきありがとうございます。

お伺いしたいのですが、それでは各投稿が保有するimage_nameカラムにはどのような名前を取り入れるのが一般的なのでしょうか？
保存された画像名にはそのポストのidがつくので、そのカラムに入る値を気にしないというのが通常なのでしょうか。
恐れ入りますが、お教え頂ければ幸いです

[Ighrs]

1. 拡張子
　jpg以外に対応するときは拡張子を保存することがあります

2. アップロードされた実際のファイル名
　保存するときは idで管理するけど、ユーザーに見せるリストやダウンロード時のファイル名は本人がアップロードした名前であるべきということで、表向きのファイル名を保存することがあります

　なお、その場合は image_name が２つになるので、ID付きのほうは別名にすべきでしょう

[tomtom1]

ご丁寧にご説明ありがとうございます。
また、他の未解決の質問もございますので、
Ighrs様のお時間が許せば、ご協力頂ければ幸いです。
宜しくお願い致します。