Как провести рефакторинг этого кода Ruby (контроллера)?
-
09-09-2019 - |
Вопрос
Это код моего контроллера отчетов, он выглядит так плохо, может ли кто-нибудь дать мне несколько советов, как его привести в порядок?
# app\controller\reports_controller.rb
@report_lines = []
@sum_wp, @sum_projcted_wp, @sum_il, @sum_projcted_il, @sum_li,@sum_gross_profit ,@sum_opportunities = [0,0,0,0,0,0,0]
date = @start_date
num_of_months.times do
wp,projected_wp, invoice_line,projected_il,line_item, opp = Report.data_of_invoicing_and_delivery_report(@part_or_service,date)
@sum_wp += wp
@sum_projcted_wp +=projected_wp
@sum_il=invoice_line
@sum_projcted_il +=projected_il
@sum_li += line_item
gross_profit = invoice_line - line_item
@sum_gross_profit += gross_profit
@sum_opportunities += opp
@report_lines << [date.strftime("%m/%Y"),wp,projected_wp ,invoice_line,projected_il,line_item,gross_profit,opp]
date = date.next_month
end
Я хочу использовать какой-то метод, например
@sum_a,@sum_b,@sum_c += [1,2,3]
Решение
Моя мгновенная мысль:переместите код в модель.
Целью должны быть «Тонкие контроллеры», поэтому они не должны содержать бизнес-логику.
Во-вторых, мне нравится представлять строки отчета в представлениях как объекты OpenStruct(), что мне кажется более понятным.
Поэтому я бы рассмотрел возможность перемещения этой логики накопления в (скорее всего) метод класса в Report и возврата массива OpenStructs «строки отчета» и одного итогового OpenStruct для передачи в мое представление.
Мой код контроллера будет выглядеть примерно так:
@report_lines, @report_totals = Report.summarised_data_of_inv_and_dlvry_rpt(@part_or_service, @start_date, num_of_months)
РЕДАКТИРОВАТЬ:(Через день)
Глядя на добавление накопления в массив, я придумал следующее:
require 'test/unit'
class Array
def add_corresponding(other)
each_index { |i| self[i] += other[i] }
end
end
class TestProblem < Test::Unit::TestCase
def test_add_corresponding
a = [1,2,3,4,5]
assert_equal [3,5,8,11,16], a.add_corresponding([2,3,5,7,11])
assert_equal [2,3,6,8,10], a.add_corresponding([-1,-2,-2,-3,-6])
end
end
Смотреть:тест!Кажется, работает нормально.Нет никаких проверок на разницу в размерах между двумя массивами, поэтому существует множество причин, по которым что-то может пойти не так, но концепция кажется достаточно разумной.Я подумываю попробовать что-то подобное, что позволило бы мне взять набор результатов ActiveRecord и накопить его в OpenStruct, который я обычно использую в своих отчетах...
Наш новый метод Array может привести к сокращению исходного кода примерно к следующему:
totals = [0,0,0,0,0,0,0]
date = @start_date
num_of_months.times do
wp, projected_wp, invoice_line, projected_il, line_item, opp = Report.data_of_invoicing_and_delivery_report(@part_or_service,date)
totals.add_corresponding [wp, projected_wp, invoice_line, projected_il, line_item, opp, invoice_line - line_item]
@report_lines << [date.strftime("%m/%Y"),wp,projected_wp ,invoice_line,projected_il,line_item,gross_profit,opp]
date = date.next_month
end
@sum_wp, @sum_projcted_wp, @sum_il, @sum_projcted_il, @sum_li, @sum_opportunities, @sum_gross_profit = totals
...что, если бы Report#data_of_invoicing_and_delivery_report также мог бы вычислить gross_profit
еще больше снизится до:
num_of_months.times do
totals.add_corresponding(Report.data_of_invoicing_and_delivery_report(@part_or_service,date))
end
Совершенно непроверено, но это адское сокращение для добавления однострочного метода в Array и выполнения одного дополнительного вычитания в модели.
Другие советы
Создайте объект суммирования, содержащий все эти поля, передайте весь массив в @sum.increment_sums(Report.data_of...)