Неопределенный метод «уничтожить» при попытке разрушить / отклонить дружеские отношения

Question

Я установил модель отношений с самообращением, чтобы пользователи дружили друг с другом. Я могу создать запрос на добавление в друзья (две записи в моей БД, одна для дружбы и одна для обратной дружбы) без каких-либо проблем.

У меня проблемы с отклонением запроса на добавление в друзья. Я продолжаю получать эту ошибку, когда нажимаю отклонение, которое приводит меня к действию уничтожения:

NoMethodError in FriendshipsController#destroy
undefined method `destroy' for nil:NilClass

Одна из записей в базе данных, ожидающая дружбы, уничтожается, хотя я получаю сообщение об ошибке выше, и если я закомментирую @friendship2 = @user.friendships.find_by_friend_id(params[:id]).destroy, другая запись также уничтожается. Таким образом, похоже, что обе строки работают по отдельности, но возвращает вышеуказанную ошибку.

РЕШЕНИЕ: Похоже, мои два оператора уничтожения ссылались на одну и ту же запись. Я закомментировал неправильный код в моем контроллере и добавил те, которые работают для меня.

Вот мой контроллер дружбы:

class FriendshipsController < ApplicationController
  before_filter :authenticate, :only => [:update, :create, :destroy]

def create
  @user = User.find(current_user)
  @friend = User.find(params[:friend_id])
  params[:friendship1] = {:user_id => @user.id, :friend_id => @friend.id, :status => 'requested'}
  params[:friendship2] = {:user_id => @friend.id, :friend_id => @user.id, :status => 'pending'}
  @friendship1 = Friendship.create(params[:friendship1])
  @friendship2 = Friendship.create(params[:friendship2])
  redirect_to @friend
 end  
end

  def destroy
    @user = User.find(params[:user_id])
    @friend = User.find(params[:id])
    #@friendship2 = @user.friendships.find_by_friend_id(params[:id]).destroy
    #@friendship1 = @friend.friendships.find_by_id(params[:user_id]).destroy  
    @friendship1 = @user.friendships.find_by_friend_id(@friend.id).destroy #removes the requested friendship
    @friendship2 = @friend.friendships.find_by_friend_id(@user.id).destroy #removes the pending friendship

    flash[:success] = "Removed."
    redirect_to @user   
  end 
end

Вот моя модель пользователя:

class User < ActiveRecord::Base
  has_many :friendships, :dependent => :destroy

  has_many :friends, 
     :through => :friendships, 
     :conditions => "status = 'accepted'", 
     :source => :friend 

  has_many :pending_friends, 
     :through => :friendships, 
     :conditions => "status = 'pending'", 
     :foreign_key => "user_id", 
     :source => :friend

  has_many :requested_friends, 
     :through => :friendships,
     :source => :friend, 
     :conditions => "status = 'requested'"    
end

Вот мой взгляд (users / show.html.erb):

<% if signed_in? && @user == current_user %>
  <% unless current_user.pending_friends.empty? %>
    <h2>Pending</h2>
    <% current_user.pending_friends.each do |pending| %>
       <%= pending.name %>
       <%= link_to '[Accept]', friendship_path(:user_id => current_user, :id => pending), :method => :put, :confirm => "Accept?" %>
       <%= link_to '[Decline]', friendship_path(:user_id => current_user, :id => pending), :method => :delete, :confirm => "Decline?" %>
    <% end %>
  <% end %>
<% end %>

У меня два вопроса.

1. Я видел из некоторых уроков / дискуссий о использовании inverse_friendships , но действительно ли это необходимо? До сих пор я не видел необходимости в этом, но опять же, я только дошел до того, что закодировал мои действия по созданию и уничтожению.

2. Что случилось с неопределенной ошибкой метода 'destroy'?

Спасибо, что прошли через эту стену текста! : D

Answer 1

Поставь чек перед судьбой.

friendship = @friend.friendships.find_by_id(params[:user_id])
@friendship1 = friendship.destroy if !friendship.nil?

А также использование может превратить эту модель в модель с помощью обратных вызовов .

Answer 2

Что касается первого, это зависит от вас и вашей системы. Создание другой переменной для хранения каждого типа дружбы может быть немного сложным, но дает вам полную информацию, больше контроля и в базе данных нет дублирования данных.

Что касается второго, вы должны проверить, возвращает ли ваш запрос результат. Так как ваше исключение:

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

Говорит, что в NilClass нет метода "уничтожить" (не дружба). Просто проверьте параметр: id и убедитесь, что find_by_friend_id вообще что-либо возвращает.