ruby-on-rails - 未定义方法 `count' 为 nil :NilClass rails

Question

我想在我的房间列表中添加照片上传部分。当我尝试点击照片时出现此错误

undefined method 'count' for nil:NilClass rails <% if @photos.count > 0 %> .

我添加了一个 photo_upload.html.erb页面和a _room_menu部分但我仍然得到错误。

这是我的代码:

photos_controller.rb

       class PhotosController < ApplicationController
        def create
          @room = Room.find(params[:room_id])
          if params[:images]
            params[:images].each do |img|
              @room.photos.create(image:img)
            end
            @photos = @room.photos
            redirect_back(fallback_location:request.referer, notice: "Saved...")
        end
      end


    end

**views/rooms/photo_upload.html.erb**

<div class="row">
  <div class="col-md-3">
    <%= render 'room_menu' %>
  </div>
  <div class="col-md-9">
    <div class="panel panel-default">

      <div class="panel-heading">
        Photos
      </div>

      <div class="panel-body">
        <div class="container">
          <div class="row">
            <div class="col-md-offset-3 col-md-6">
              <!-- PHOTOS UPLOAD GOES HERE -->

              <%= form_for @room, url: room_photos_path(@room), method: 'post', html: {multipart: true} do |f| %>
                <div class="row">
                  <div class="form-group">
                    <span class="btn btn-default btn-file text-babu">
                      <i class="fa fa-cloud-upload" aria-hidden="true"></i> Select Photos
                      <%= file_field_tag "images[]", type: :file, multiple: true %>
                    </span>
                  </div>
                </div>

                <div class="text-center">
                  <%= f.submit "Add Photos", class: "btn btn-form" %>
                </div>

              <% end %>
            </div>
          </div>

          <div id="photos"><%= render 'photos/photos_list' %></div>
        </div>
      </div>

    </div>
  </div>
</div>

views/rooms/_room_menu.html.erb

<ul class="sidebar-list">
  <li class="sidebar-item">
    <%= link_to "Listing", listing_room_path, class: "sidebar-link active" %>
    <span class="pull-right text-babu"><i class="fa fa-check"></i></span>
  </li>
  <li class="sidebar-item">
    <%= link_to "Pricing", pricing_room_path, class: "sidebar-link active" %>
    <% if !@room.price.blank? %>
      <span class="pull-right text-babu"><i class="fa fa-check"></i></span>
    <% end %>
  </li>
  <li class="sidebar-item">
    <%= link_to "Description", description_room_path, class: "sidebar-link active" %>
    <% if !@room.listing_nam.blank? %>
      <span class="pull-right text-babu"><i class="fa fa-check"></i></span>
    <% end %>
  </li>

    <li class="sidebar-item">
    <%= link_to "Photos", photo_upload_room_path, class: "sidebar-link active" %>
    <% if !@room.photos.blank? %>
      <span id="photo_check" class="pull-right text-babu"><i class="fa fa-check"></i></span>
    <% end %>
  </li>

  <li class="sidebar-item">
    <%= link_to "Amenities", amenities_room_path, class: "sidebar-link active" %>
    <span class="pull-right text-babu"><i class="fa fa-check"></i></span>
  </li>
  <li class="sidebar-item">
    <%= link_to "Location", location_room_path, class: "sidebar-link active" %>
    <% if !@room.address.blank? %>
      <span class="pull-right text-babu"><i class="fa fa-check"></i></span>
    <% end %>
  </li>
</ul>
<hr/>

Answer 1

你不能在 nil 上调用 .count。 @photos 必须先实例化。您的 Controller 似乎根本没有实例化 @photos。我看不到你在代码中的其他地方调用了 @photos 但在调用它的地方，实例变量尚未定义。您的 Controller 创建方法仅显示它在存在 params[:images] 时发生，否则将为零。 尝试在 if block 之外实例化 @photos。

def create
  @room = Room.find(params[:room_id])
  @photos = @room.photos
  if params[:images]
    params[:images].each do |img|
      @room.photos.create(image: img)
    end
    redirect_back(fallback_location:request.referer, notice: "Saved...")
  end
end

更新:Ruby 2.3+ 你可以使用 & 安全导航:

def create
  @room = Room.find(params[:room_id])
  params[:images]&.each{|img| @room.photos.create(image: img)}
  redirect_back(fallback_location: request.referer, notice: "Saved...")
end

或者当您应该能够在它的位置调用 @room.photos 时，为什么还要在 View 中使用单独的 @photos。

不要调用 .count 作为 View 中的条件，而是使用 @room.photos.present?

另一个建议是更喜欢使用肯定的 if 大小写。所以改变

if !@room.photos.blank?
# better to use this below
if @room.photos.present?

此外，人们会假设在创建操作之前，您需要在 Controller 中创建一个新操作，如果它确实需要与 @room 隔离，那么人们会期望定义 @photos。照片。这是 Rails 中的标准 MVC，但不确定您是否已正确发布所有代码，所以我在这里猜测。