【rails】画像投稿機能で、#{post.id}.jpegファイルが作られません

Question

前提・実現したいこと

ruby on railsで、progateを参考に画像投稿機能を作っています。
投稿すると、public/post_imagesの中に#{post.id}.jpegという画像ファイルが作られるはずでした。
しかし、新規投稿すると、「#{post.id}.jpeg」ではなく、「.jpg」というファイルが作られてしまいます。
このため、最初の投稿の画像が、そのあとの投稿でも出続けることになってしまいます。
新規投稿でなく、編集で画像をアップロードすると、そこで初めて#{post.id}.jpegが作られます。

該当のソースコード

新規投稿を行う処理です（うまくいっていない部分）

  def create
    @post=Post.new(content: params[:content],
                    user_id: @current_user.id,
                  title: params[:title])
    if params[:image]
      @post.image_name="#{@post.id}.jpg"
      image=params[:image]
      File.binwrite("public/post_images/#{@post.image_name}",image.read)
    end
    if @post.save
      flash[:notice]="投稿しました"
      redirect_to("/users/#{@current_user.id}")
    else
      render("/posts/new")
    end
  end

投稿の編集を行う処理です（成功している部分）

  def update
    @post=Post.find_by(id: params[:id])
    @post.title=params[:title]
    @post.content=params[:content]
    if params[:image]
      @post.image_name="#{@post.id}.jpg"
      image=params[:image]
      File.binwrite("public/post_images/#{@post.image_name}",image.read)
    end
    if @post.save
      redirect_to("/users/#{@post.user_id}")
      flash[:notice]="投稿を編集しました"
    else
      render("posts/edit")
    end
  end

考えたこと

Post.newしたときに、新しいデータにidが割り振られていないのかな、と思ったのですが、そんなこと起こりますか？

Answer 1

こんにちは！
自分もrailsを勉強している身です。
間違っているかもしれませんが、
Post.newだとまだ保存されてないので、idが割り振られてないのだと思います。
Post.createを使う。かつ、if文で保存成功かどうかの分岐したいのであれば、入れ子構造にしてみたらいいのではないでしょうか？

if @post=Post.create(content: params[:content], user_id: @current_user.id, title: params[:title])

  if params[:image]
      @post.image_name="#{@post.id}.jpg"
      image=params[:image]
      File.binwrite("public/post_images/#{@post.image_name}",image.read)
  end
  flash[:notice]="投稿しました"
  redirect_to("/users/#{@current_user.id}")
else
　render("/posts/new")
end

のような感じで。

Answer 2

他の方の質問みたところ、@post.saveしないとidが割り振られないことがわかりました・・・。以下のように書き直したら、{@post.id}.jpgが無事作られました。
しかし、ファイルは作られたものの、今度は投稿に画像が乗らなくなってしまったので、また別で質問したいと思います。

  def create
    @post=Post.new(content: params[:content],
                    user_id: @current_user.id,
                  title: params[:title])

    if @post.save
      if params[:image]
        @post.image_name="#{@post.id}.jpg"
        image=params[:image]
        File.binwrite("public/post_images/#{@post.image_name}",image.read)
      end
      flash[:notice]="投稿しました"
      redirect_to("/users/#{@current_user.id}")
    else
      render("/posts/new")
    end
  end