2016-11-08 10 views
2

Я делаю пользовательскую панель мониторинга для школьного приложения, которое требует от меня вычисления некоторых ключевых показателей эффективности, так как сейчас я делаю это, вызывая несколько методов класса из класса Opportunity в действии панели/индекса из контроллера и сохраняя каждый метод приведет к переменной, которая будет использоваться в плитке. Таким образом, каждая переменная является другой плитой панели инструментов.Custom Rails Dashboard, как оптимизировать поиск данных для отображения в поле зрения?

методов относятся к классу возможностей, показанный ниже:

class Opportunity < ApplicationRecord 
    belongs_to :organization 
    belongs_to :opportunity_status 
    has_many :tasks, dependent: :destroy 
    has_many :opportunity_status_logs, dependent: :destroy 

    before_create :create_status_log 
    after_update :create_status_log, if: :opportunity_status_id_changed? 

    validates :name, :description, :revenue, :opportunity_status_id, :closing_date, presence: true 
    validates :name, :description, format: { with: /\A[[:alpha:]a-zA-Z0-9ñÑ#()\-.,\s]+\z/ } 
    validates :revenue, numericality: true 
    validates :closing_date, inclusion: { in: (Time.zone.today..Time.zone.today+5.years) } 


    def create_status_log 
    OpportunityStatusLog.create(opportunity_id: self.id, opportunity_status_id: self.opportunity_status_id) 
    end 

    def status_updated_by(user) 
    @status_log = self.opportunity_status_logs.last 
    @status_log.user_id = user.id 
    @status_log.save! 
    end 

    def self.actives 
    self.where.not(opportunity_status_id: [11,12]) 
    end 

    def self.won 
    self.where(opportunity_status_id: 11) 
    end 

    def self.lost 
    self.where(opportunity_status_id: 12) 
    end 

    def self.average_revenue 
    self.won.average(:revenue) 
    end 

    def self.minimum_revenue 
    self.won.minimum(:revenue) 
    end 

    def self.maximum_revenue 
    self.won.maximum(:revenue) 
    end 

    def self.filter_by_status(status_id) 
    self.where(opportunity_status: status_id) 
    end 

    def self.relative_percentage(item_amount, total) 
    item_amount * 100/total 
    end 

    def self.conversion_rate 
    self.won.count/self.all.count.to_f * 100 
    end 

    def self.potential_revenue 
    self.actives.sum(:revenue) 
    end 
end 

и это путь контроллер структурирован:

class DashboardController < ApplicationController 
    before_action :authenticate_user! 

    def index 
    @opportunities = Opportunity.includes(:opportunity_status).all 
    @actives = Opportunity.actives.count 
    @won = Opportunity.won.count 
    @lost = Opportunity.lost.count 
    @average_revenue = Opportunity.average_revenue 
    @minimum_revenue = Opportunity.minimum_revenue 
    @maximum_revenue = Opportunity.maximum_revenue 
    @in_appreciation = Opportunity.filter_by_status(6).count 
    @in_value_proposition = Opportunity.filter_by_status(7).count 
    @in_management_analysis = Opportunity.filter_by_status(8).count 
    @in_proposal = Opportunity.filter_by_status(9).count 
    @in_review = Opportunity.filter_by_status(10).count 
    @app_perc = Opportunity.relative_percentage(@in_appreciation, @opportunities.count) 
    @vp_perc = Opportunity.relative_percentage(@in_value_proposition, @opportunities.count) 
    @ma_perc = Opportunity.relative_percentage(@in_management_analysis, @opportunities.count) 
    @pp_perc = Opportunity.relative_percentage(@in_proposal, @opportunities.count) 
    @rw_perc = Opportunity.relative_percentage(@in_review, @opportunities.count) 
    @conversion_rate = '%.2f' % [Opportunity.conversion_rate] 
    @potential_revenue = Opportunity.potential_revenue 
    end 
end 

Несмотря на то, что работает, как и ожидался, это выглядит как контроллер немного тоньше, и я чувствую, что при нынешнем подходе, если приложение масштабируется, оно будет очень медленным из-за количества запросов, которые выполняются. Итак, есть ли способ реорганизовать это для оптимизации поиска данных и отображения KPI?

Заранее спасибо

ответ

1

Вы можете попробовать реализовать Facade Pattern in Rails. Это сделает ваш контроллер тощим, но в части запроса вам все равно придется делать эти запросы, нет способа пропустить это.

Вы можете попытаться оптимизировать БД путем добавления индекса и создания SQL представлений в будущем, когда вы получите отставание производительности, в это время будет как преждевременная оптимизация

+0

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

+0

Рад знать, что это помогло :) –