投稿のタイトルや説明文のバリデーション時にActiveStorageでエラー発生

Question

お疲れさまです。
ActiveStorageで画像投稿(複数可)機能を実装しています。
投稿の空白や文字数に対するバリデーション実装で詰まっている部分があるので質問させてください。

やりたいこと

以下のエラーを解消したい

詰まっていること

更新時にtitleやdescription(テキスト)、座標の空欄や文字数オーバーのバリデーション機能を付けました。
その状態でeditページにて、「更新するボタン」を押すと以下のエラーが発生してしまいます。

NoMethodError in Roads#update
Showing /var/www/app/app/views/roads/edit.html.erb where line #23 raised:

undefined method `road_images' for nil:NilClass

Extracted source (around line #23):        
    </div>
      <% if @road.road_images.present? %>
        <p class="text-center">現在投稿されている画像(削除するものはチェックしてください)</p>
        <% @road.road_images.each do |image| %>
          <p class="d-flex justify-content-center">

試してみたこと

<%= raise @road.road_images.inspect %>で指摘の中身を表示すると、以下のように表示されました。
@name="road_images" となっているので問題ないように思えるのですが、何が違うのかわかりません。

#<ActiveStorage::Attached::Many:0x00007f4ba4804cf0 @name="road_images", @record=#<Road id: 10, user_id: 1, title: "test", description: "test", created_at: "2021-09-06 18:26:54.419859000 +0900", updated_at: "2021-09-07 15:37:52.720666000 +0900", latitude: nil, longitude: nil, likes_count: 0>>

ソースコード

# road_controller.rb

class RoadsController < ApplicationController
  PER_PAGE = 15

  def index
    @q = Road.ransack(params[:q])
    @roads = @q.result.includes(:user, :likes).page(params[:page]).per(PER_PAGE).order(:created_at)
  end

  def show
    @road = Road.find(params[:id])
    @comments = @road.comments
    @comment = Comment.new
  end

  def new
    @road = Road.new
  end

  def create
    @road = Road.new(road_params)
    if @road.save
      flash[:success] = "投稿が完了しました"
      redirect_to roads_path
    else
      flash[:alert] = "投稿に失敗しました"
      render :new
    end
  end

  def edit
    @road = Road.find(params[:id])
  end

  def update
    road = Road.find(params[:id])
    if params[:road][:road_image_ids]
      params[:road][:road_image_ids].each do |road_image_id|
        image = road.road_images.find(road_image_id)
        image.purge
      end
    end
    if road.update(road_params)
      flash[:success] = "投稿を編集しました"
      redirect_to roads_path
    else
      flash[:alert] = "更新に失敗しました"
      render :edit
    end
  end

  def destroy
    Road.find(params[:id]).destroy
    flash[:success] = "投稿を削除しました"
    redirect_to roads_path
  end

  private
    def road_params
      params.require(:road).permit(:title, :description, :latitude, :longitude, :content, road_images: []).merge(user_id: current_user.id)
    end
end

# road.rb

class Road < ApplicationRecord
  has_many_attached :road_images
  has_many :comments, dependent: :destroy
  has_many :likes, dependent: :destroy
  has_many :liked_users, through: :likes, source: :user
  belongs_to :user

  validates :latitude, :longitude, presence: true
  validates :title, presence: true, length: { maximum: 50 }
  validates :description, presence: true, length: { maximum: 400 }

  def liked_by?(user)
    likes.any?{ |like| like.user_id == user.id }
  end
end

# edit.html.erb

<div class="container py-5 mx-auto">
  <h3 class="text-center">編集</h3>

  (略)

    </div>
      <%= raise @road.road_images.inspect %>
      <% if @road.road_images.present? %>
        <p class="text-center">現在投稿されている画像(削除するものはチェックしてください)</p>
        <% @road.road_images.each do |image| %>
          <p class="d-flex justify-content-center">
            <%= f.check_box :road_image_ids, {multiple: true}, image.id, false %>
            <%= image_tag image, width: '300px', height: '200px' %>
          </p>
        <% end %>
      <% end %>
    
      <div class="text-center">
        <%= f.label :title, "タイトル" %>
        <div class="text-center">
          <p><%= f.text_field :title, class: "title-field" %></p>
        </div>
        <%= f.label :description, "説明文" %>
        <p><%= f.text_area :description, class: "description-field" %></p>
        <%= f.submit '更新する', class: "btn btn-outline-primary submit-btn" %>
      </div>

  <% end %>
</div> 

Answer 1

NoMethodError in Roads#update Showing /var/www/app/app/views/roads/edit.html.erb
ということは、validationにかかってsaveできなかったときの動作だと思います。
controllerが載っていないのでエスパーモードですが、
else の方で @road を定義していないのでしょう。

raise での確認は、update ではなく edit のときのものでは？

Comments

[husqy]

ありがとうございます！
controller内のローカル変数をインスタンス変数にすることで解決できました！