Неопределенный метод Ruby on Rails `destroy 'для nil: NilClass

Question

У меня возникла проблема при удалении сообщений пользователя:

неопределенный метод `destroy 'для nil: NilClass

# routes.rb
Rails.application.routes.draw do
  get 'sessions/new'
  get 'users/new'
  get 'user/new'
  root to:'pages#home'
  get '/home', to:'pages#home'
  get '/help', to:'pages#help'
  get '/about', to:'pages#about'
  get '/contact',  to:'pages#contact'
  get '/signup', to: 'users#new'
  post '/signup', to: 'users#create'
  get '/login', to: 'sessions#new'
  post '/login', to: 'sessions#create'
  delete '/logout', to: 'sessions#destroy'
  post '/micro_posts', to: 'microposts#create'
  delete '/micro_posts', to: 'microposts#destroy'

  resources :users
  resources :account_activations, only: [:edit]
  resources :microposts, only: [:create, :destroy]
end


# microposts_controller.rb 

class MicropostsController < ApplicationController
  before_action :logged_in_user, only: [:create, :destroy]
  before_action :correct_user, only: [:destroy]

  def create
    @micropost = current_user.micro_posts.build(req_params)

    if @micropost.save
        flash[:success] = "Post created"
        redirect_to root_url    
    else
        @feed_items = []
        render 'pages/home'

    end     
  end

  def destroy   
    @micropost.destroy
    flash[:success] = "Post deleted"
    redirect_to request.referrer || root_url
  end    

  private

  def req_params
    params.require(:micro_post).permit(:content)
  end

  def correct_user
    @micropost = current_user.micro_posts.find_by_id(params[:id])
    redirect_to root_url if @micropost.nil?
  end    
end

# _micro_post.html.erb file

<li id="micropost" ><%= micro_post.id %>
  <%= link_to gravatar_for(micro_post.user, size: 50), micro_post.user %>
  <span class="user" ><%= link_to micro_post.user.name, micro_post.user %></span><br/>
  <span class="content" ><%= micro_post.content %></span>
  <br/> 
  <span  class="timestamp">
  posted <%= time_ago_in_words(micro_post.created_at) %> ago    

  <% if current_user?(micro_post.user) %>
    <%= link_to '  | Delete', micro_posts_path, method: :delete , data: { confirm: "Are you sure" } %>
  <% end %>    
  </span>
  <br/>    
</li>

Теперь я понял, чточто моя ошибка в файле microposts_controller.rb в ссылочном методе "correct_user", потому что он не может найти идентификатор микросообщения и возвращает ноль.И поэтому мы не можем вызвать метод destroy для объекта nil.Подскажите, пожалуйста, почему он не находит идентификатор микросообщений?

Started DELETE "/micro_posts" for 127.0.0.1 at 2019-02-21 21:43:17 +0500
   (0.7ms)  SELECT "schema_migrations"."version" FROM "schema_migrations" ORDER BY "schema_migrations"."version" ASC
  ↳ C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/activerecord-5.2.2/lib/active_record/log_subscriber.rb:98
Processing by MicropostsController#destroy as HTML
  Parameters: {"authenticity_token"=>"aNeBnIy/JOrWz+sQdzU2w06faILW2dM+f8SPwmXVmJs1XfJ8OqbkgPVwkpoY/HBjh8YJhVPWZ4BC1Onxg0AtWw=="}
  User Load (0.2ms)  SELECT  "users".* FROM "users" WHERE "users"."id" = ? LIMIT ?  [["id", 1], ["LIMIT", 1]]
  ↳ app/helpers/sessions_helper.rb:16
  MicroPost Load (0.3ms)  SELECT  "micro_posts".* FROM "micro_posts" WHERE "micro_posts"."id" IS NULL LIMIT ?  [["LIMIT", 1]]
  ↳ app/controllers/microposts_controller.rb:36
Completed 500 Internal Server Error in 56ms (ActiveRecord: 1.9ms)



NoMethodError (undefined method `destroy' for nil:NilClass):

app/controllers/microposts_controller.rb:22:in `destroy

Answer 1

Я не заметил, что @zeitnot ответил передо мной с той же рекомендацией.

---

Я думаю, что ваша проблема - это ваша ссылка для удаления, не ссылающаяся на реальный объект.Вы не передаете идентификатор объекта, поэтому params[:id] всегда будет нулевым.

Попробуйте, чтобы увидеть, работает ли он:

<% if current_user?(micro_post.user) %>
  <%= link_to '  | Delete', micro_post_path(micro_post), method: :delete , data: { confirm: "Are you sure" } %>
<% end %>

Answer 2

Попробуйте:

<%= link_to '  | Delete', micro_post_path(micro_post), method: :delete , data: { confirm: "Are you sure" } %>

Проблема в том, что вы используете маршрут сбора.micro_posts_path - это маршрут сбора, и вам нужно использовать маршрут участника, который принимает идентификатор ресурса.Таким образом, в этом сценарии мы удаляем s из маршрута, и он становится micro_post_path, который является участником маршрута.

Answer 3

Если не возражаете, я преобразую свои комментарии в ответ, чтобы не оставить вопрос без ответа.

У вас есть проблема с именами.Модель MicroPost, но контроллер MicropostsController.Вот почему вы пытаетесь создавать собственные маршруты.

Вам необходимо переименовать контроллер в MicroPostsController, а его файл - в micro_posts_controller.rb.Удалите из rout.rb эти 2 строки:

post '/micro_posts', to: 'microposts#create'
delete '/micro_posts', to: 'microposts#destroy'

и измените microposts на micro_posts (обратите внимание на подчеркивание) здесь:

resources :microposts, only: [:create, :destroy]

Используйте micro_post_path(micro_post) для удаления ссылкив представлении

Answer 4

Выглядит проблема в вашем correct_user методе.Попробуйте удалить его и использовать этот способ:

class MicropostsController < ApplicationController
  before_action :logged_in_user, only: [:create, :destroy]
  before_action :set_micropost, only: :destroy

  ...

  def set_micropost
    @micropost ||= Micropost.find params[:id]
  end
end

А на ваш взгляд:

<%= link_to '  | Delete', @micropost, method: :delete , data: { confirm: "Are you sure" } %>

Answer 5

Я думаю, что вы должны передавать id-идентификатор вашему маршруту удаления в вашем представлении.

Вместо этого:

<%= link_to ' | Delete', micro_posts_path, method: :delete , data: { confirm: "Are you sure" } %>

Попробуйте это: <%= link_to ' | Delete', micro_posts_path(micro_post.id), method: :delete , data: { confirm: "Are you sure" } %>

Answer 6

Да, вы должны установить значение @micropost, прежде чем пытаться уничтожить его.Обычно это то, что вы хотите сделать с хэшем params, чтобы в начале вашего метода destroy вы могли иметь что-то вроде @micropost = Micropost.find(params[:id]).Это строка, которую вы, возможно, захотите использовать в других методах вашего контроллера, чтобы вы могли реорганизовать ее следующим образом

class MicropostsController < ApplicationController
  before_action :set_micropost, only: [:destroy]

 ...

  private

  def set_micropost
    @micropost = Micropost.find(params[:id])
  end
end

Надеюсь, это поможет:)