Moving-Methode-Logik mit SQL zu modellieren in Schienen Projekt

Question

Verschieben einer Methode in meinem Controller (Invoices) in das entsprechende Modell und ich vermisse etwas. Ich habe versucht, folgende this und this und sogar this und es gibt kleine Probleme, die ich habe, dass ich zusätzliche Augen brauchen.

Meine Arbeits-Controller-Methode ist dies.

def array_of_disbursable_invoices 
    sql = "SELECT MIN(departure_date), ch_invoice.invoice_id 
    FROM ch_invoice 
    INNER JOIN ch_trip 
    ON ch_invoice.invoice_id = ch_trip.invoice_id 
    WHERE departure_date <= (SELECT SYS_EXTRACT_UTC(SYSTIMESTAMP)FROM DUAL) 
    AND service_rendered = 0 
    AND paid = 1 
    Group By ch_invoice.invoice_id" 

    report = ActiveRecord::Base.connection.exec_query(sql) 
    render json: report 
    end 

Und ich versuche, es in das zu verwandeln.

def array_of_disbursable_invoices 
    report = report.array_of_disbursable_invoices 
    render json: report 
    end 

Mit der Logik in meinem Modell hier.

def array_of_disbursable_invoices 
    sql = "SELECT MIN(departure_date), ch_invoice.invoice_id 
    FROM ch_invoice 
    INNER JOIN ch_trip 
    ON ch_invoice.invoice_id = ch_trip.invoice_id 
    WHERE departure_date <= (SELECT SYS_EXTRACT_UTC(SYSTIMESTAMP)FROM DUAL) 
    AND service_rendered = 0 
    AND paid = 1 
    Group By ch_invoice.invoice_id" 

    ActiveRecord::Base.connection.exec_query(sql) 
    end 

Aktuelle Fehlermeldung

nicht definierte Methode `array_of_disbursable_invoices' für nil: NilClass

Answer 1

Sie schlagen mich zu ihm (und gingen im Wesentlichen mit meiner zweiten Option). Aber ich poste das trotzdem.

Wenn Sie dies tun:

def array_of_disbursable_invoices 
    report = report.array_of_disbursable_invoices 
    render json: report 
end 

Du nennst array_of_disbursable_invoices auf einer Instanz. Aber Sie instanziieren Report - also den undefined method 'array_of_disbursable_invoices' for nil:NilClass Fehler nicht.

Also, ich denke, haben Sie zwei Möglichkeiten:

(1) Man könnte die Methode auf einer Instanz aufrufen, so etwas wie:

report = Invoice.new.array_of_disbursable_invoices 

(2) Man könnte das Verfahren eine Klassenmethode machen , etwas wie:

class Invoice < ActiveModel::Base 
    class << self 
    def array_of_disbursable_invoices 
     sql = "SELECT MIN(departure_date), ch_invoice.invoice_id 
     FROM ch_invoice 
     INNER JOIN ch_trip 
     ON ch_invoice.invoice_id = ch_trip.invoice_id 
     WHERE departure_date <= (SELECT SYS_EXTRACT_UTC(SYSTIMESTAMP)FROM DUAL) 
     AND service_rendered = 0 
     AND paid = 1 
     Group By ch_invoice.invoice_id" 

     connection.exec_query(sql) 
    end 
    end 
end 

Ich denke, ich würde empfehlen (1). Außerdem würde ich persönlich die ActiveRecord-Abfrage-Schnittstelle anstelle von SQL verwenden (vorausgesetzt, Sie haben alle Assoziationen in Ihren Modellen eingerichtet). Aber das ist eine persönliche Vorliebe.

Answer 2

Verstanden mit dem folgenden Code in meinem Controller zu arbeiten.

def array_of_disbursable_invoices 
    report = Invoice.array_of_disbursable_invoices 
    render json: report 
    end 

Und das im Modell.

def self.array_of_disbursable_invoices 
    sql = "SELECT MIN(departure_date), ch_invoice.invoice_id 
    FROM ch_invoice 
    INNER JOIN ch_trip 
    ON ch_invoice.invoice_id = ch_trip.invoice_id 
    WHERE departure_date <= (SELECT SYS_EXTRACT_UTC(SYSTIMESTAMP)FROM DUAL) 
    AND service_rendered = 0 
    AND paid = 1 
    Group By ch_invoice.invoice_id" 

    ActiveRecord::Base.connection.exec_query(sql) 
    end