Rails：保留中のすべてのフレンドリクエストを表示するには？

Question

私は最初のRailsプロジェクトに取り組んでおり、小さな問題があります。私は本当に助けに感謝します。 各イテレータを使用して、現在のユーザーに対して保留中のフレンドリクエストをすべて表示します。私のコントローラ：

class FriendRequestsController < ApplicationController 
before_action :set_friend_request, except: [:index, :new, :create] 

def index 
    @incoming = FriendRequest.where(friend: current_user) 
    @outgoing = current_user.friend_requests 
end 

def new 
    @friend_request = FriendRequest.new 
end 

def create 
    friend = User.find(params[:friend_id]) 
    @friend_request = current_user.friend_requests.new(friend: friend) 
    if @friend_request.save 
    redirect_back(fallback_location: root_path), status: :created, location: @friend_request 
    else 
    render json: @friend_request.errors, status: :unprocessable_entity 
    end 
end 

私は以下のコードのようなものをしようとすると、それはちょっと働き、条件文は、それが必要として動作しますが、私はそれはそれを動作させるために恐ろしい方法です知っているので、私が使用したいのですが@incomingは定義されているので

<% if FriendRequest.where(friend: current_user).present? %> 
    <% ?.each do |request| %> 
     <li><%= ? %></li> 
    <% end %> 
<% else %> 
    You don't have any friend requests 
<% end %> 

しかし、私のような何かをしようとすると：

<% if @incoming.present? %> 

条件文が正常に動作し、現在のユーザが保留中であっても、「あなたはどんな友達リクエストはありません」がありますしません。友達リクエスト。 私はRoRでのすべての動作を正確に把握していないので、説明に感謝します。

Answer 1

受信友人要求の特定の関連付けを作成することから始めましょう。

class User < ActiveRecord::Base 
    # ... 
    has_many :incoming_friend_requests, 
    class_name: 'FriendRequest', 
    source: :friend 
end 

Railsは、我々はclass_nameを指定アソシエーションの名前から適切な列を導き出すことができないので。 sourceは、FriendRequestの関連が逆であることをRailsに伝えます。

これは、熱心な負荷とパフォーマンスを見始めたときに非常に重要です。

たとえば、それはあなたが行うことができます：スコープまたはコレクション使用.any?のいずれかの項目があるかどうかをテストするには

def index 
    @incoming = current_user.incoming_friend_requests 
    @outgoing = current_user.friend_requests 
end 

：

@user = User.joins(:friend_requests, :incoming_friend_requests) 
      .find(params[:id]) 

だから新しい関係を使用することができます。 .any?は、関連付けが既にロードされている場合にクエリを発行しないという点でかなりスマートです。

<% if @incoming.any? %> 
    <ul> 
    <% @incoming.each do |fr| %> 
    <li><%= fr.name %></li> 
    <% end %> 
    </ul> 
<% else %> 
    <p>You don't have any friend requests</p> 
<% end %> 

Answer 2

<% if (frs = FriendRequest.where(friend: current_user)).present? %> 
    <% frs.each do |fr| %> 
     <li><%= fr.name %></li> 
    <% end %> 
<% else %> 
    You don't have any friend requests 
<% end %>