один и тот же код в разных действиях контроллера в рельсах - как сделать СУХОЙ?

Question

У меня есть контроллер с двумя разными действиями, но оба нуждаются в одном и том же коде, который немного длинен, как я могу разрешить им доступ к тому же поведению, но сохранить его СУХИМ?

   @list = Contact.find :all,
      :select => "companies.name AS co_name, 
                  companies.id AS comp_id, 
                  COUNT(contact_emails.id) AS email_count, 
                  COUNT(contact_calls.id) AS call_count, 
                  COUNT(contact_letters.id) AS letter_count, 
                  COUNT(contact_postalcards.id) AS postalcard_count",

      :conditions => ['contact_emails.date_sent < ? and contact_emails.date_sent > ?', 
                      report_end_date, report_start_date],

      :joins => [
        "LEFT JOIN companies ON companies.id = contacts.company_id",
        "LEFT JOIN contact_emails ON contact_emails.contact_id = contacts.id",
        "LEFT JOIN contact_letters ON contact_letters.contact_id = contacts.id",
        "LEFT JOIN contact_postalcards ON contact_postalcards.contact_id = contacts.id",
        "LEFT JOIN contact_calls ON contact_calls.contact_id = contacts.id"
      ],
      #:group => "companies.id"
       :group => "companies.name"
    puts @list[0].attributes.inspect

Answer 1

Вы должны переместить этот код в модель:

# Contatct model

def self.get_list(report_start_date, report_end_date)
  self.find :all,
    :select => "companies.name AS co_name, 
              companies.id AS comp_id, 
              COUNT(contact_emails.id) AS email_count, 
              COUNT(contact_calls.id) AS call_count, 
              COUNT(contact_letters.id) AS letter_count, 
              COUNT(contact_postalcards.id) AS postalcard_count",

    :conditions => ['contact_emails.date_sent < ? and contact_emails.date_sent > ?', 
                  report_end_date, report_start_date],

    :joins => [
      "LEFT JOIN companies ON companies.id = contacts.company_id",
      "LEFT JOIN contact_emails ON contact_emails.contact_id = contacts.id",
      "LEFT JOIN contact_letters ON contact_letters.contact_id = contacts.id",
      "LEFT JOIN contact_postalcards ON contact_postalcards.contact_id = contacts.id",
      "LEFT JOIN contact_calls ON contact_calls.contact_id = contacts.id"
    ],
    #:group => "companies.id"
    :group => "companies.name"
end

Тогда вы можете использовать его в контроллерах:

@list = Contact.get_list(report_start_date, report_end_date)

Возможно, вы также можете разделить его на более мелкие части и использовать области и определенные ассоциации вместо того, чтобы писать все самостоятельно.

Answer 2

Я бы добавил функцию для генерации счета и присоединения sql:

class Contact < ActiveRecord::Base
  def  self.get_list(report_start_date,    report_end_date)
    all(:select      => "companies.name AS co_name,
                        companies.id AS comp_id,
                        #{table_count_col(
                          :contact_emails, 
                          :contact_calls, 
                          :contact_letters, 
                          :contact_postalcards
                         )}",
        :conditions => ['contact_emails.date_sent < ?  AND
                         contact_emails.date_sent > ?',
                         report_end_date, report_start_date],
        :joins      => join_table(
                    :companies, 
                    :contact_emails, 
                    :contact_letters, 
                    :contact_postalcards, 
                    :contact_calls
                    ),
       )
  end
end

Где table_count_col и table_join - статические методы внутри Contact, класс:

  def self.table_count_col(*args)    
    args.collect do |table| 
      count_col = "#{table.to_s.gsub(/^contact_/,  '').singularize}_count"
      "COUNT(#{table}.id) AS #{count_col}"    
    end.join(",")
  end

  def self.table_join(*args)    
    args.collect do |table|
     "LEFT JOIN #{table} ON #{table}.id = contacts.company_id"
    end.join(",")
  end