Пожалуйста, помогите мне упростить эту простую функцию Ruby on Rails

Question

Может ли кто-нибудь сказать мне, как упростить следующую функцию в моей модели Invoice? Он должен вернуть все Invoices, связанные с конкретным Client (оба подключены через таблицу Projects).

def self.search_by_client_id(client_id) 
    if client_id 
    projects = Project.where(:client_id => client_id) 
    Invoice.where(:project_id => projects) 
    else 
    scoped 
    end 
end 

Я действительно не могу обдумать это. Спасибо за любой вклад!

Answer 1

Это похоже на прекрасную возможность для прицела!

scope :client, lambda{|id| includes(:projects).where('projects.client_id = ?', id)} 

Invoice.client(4).all # returns all invoices for the client with the specified ID. 

Answer 2

Если я правильно понимаю ваш вопрос, вы должны быть в состоянии сделать это с помощью ActiveRecord ассоциации:

class Client < ActiveRecord::Base 
    has_many :projects 
    has_many :invoices, :through => :project 
end 

class Project < ActiveRecord::Base 
    has_many :invoices 
    belongs_to :client 
end 

class Invoice < ActiveRecord::Base 
    belongs_to :project 
    has_one :client, :through => :project 
end 

Тогда все Invoices, связанные с конкретной Client:

@client.invoices 

Для получите Client, связанный с конкретным счетом:

@invoice.client