chore: 3rd & 4th steps case b

Author: potashin

app/controllers/trips_controller.rb

@@ -2,6 +2,7 @@ class TripsController < ApplicationController
   def index
     @from = City.find_by_name!(params[:from])
     @to = City.find_by_name!(params[:to])
-    @trips = Trip.where(from: @from, to: @to).preload(bus: :services).order(:start_time)
+    @services = Service.pluck(:id, :name).to_h
+    @trips = Trip.where(from: @from, to: @to).preload(bus: :bus_services).order(:start_time)
   end
 end

app/models/bus.rb

@@ -21,7 +21,8 @@ class Bus < ApplicationRecord
   ].freeze
 
   has_many :trips
-  has_and_belongs_to_many :services, join_table: :buses_services
+  has_many :bus_services, class_name: 'BusService'
+  has_many :services, through: :bus_services
 
   validates :number, presence: true, uniqueness: true
   validates :model, inclusion: { in: MODELS }

app/models/bus_service.rb

@@ -0,0 +1,14 @@
+# == Schema Information
+#
+# Table name: buses_services
+#
+#  id         :bigint           not null, primary key
+#  bus_id     :integer
+#  service_id :integer
+#
+class BusService < ApplicationRecord
+  self.table_name = :buses_services
+
+  belongs_to :bus
+  belongs_to :service
+end

app/models/service.rb

@@ -19,7 +19,8 @@ class Service < ApplicationRecord
     'Можно не печатать билет',
   ].freeze
 
-  has_and_belongs_to_many :buses, join_table: :buses_services
+  has_many :service_buses
+  has_many :buses, through: :service_buses
 
   validates :name, presence: true
   validates :name, inclusion: { in: SERVICES }

app/views/trips/_service.html.erb

@@ -1 +1 @@
-<li><%= "#{service.name}" %></li>
+<li><%= "#{@services[bus_service.service_id]}" %></li>

app/views/trips/_services.html.erb

@@ -1,4 +1,4 @@
 <li>Сервисы в автобусе:</li>
 <ul>
-  <%= render partial: "service", collection: services %>
+  <%= render partial: "service", collection: bus_services, as: :bus_service %>
 </ul>

app/views/trips/_trip.html.erb

@@ -4,8 +4,8 @@
     <li><%= "В пути: #{trip.duration_minutes / 60}ч. #{trip.duration_minutes % 60}мин." %></li>
     <li><%= "Цена: #{trip.price_cents / 100}р. #{trip.price_cents % 100}коп." %></li>
     <li><%= "Автобус: #{trip.bus.model} №#{trip.bus.number}" %></li>
-    <% if trip.bus.services.present? %>
-        <%= render "services", services: trip.bus.services %>
+    <% if trip.bus.bus_services.present? %>
+        <%= render "services", bus_services: trip.bus.bus_services %>
     <% end %>
 </ul>
 

app/views/trips/index.html.erb

@@ -2,11 +2,11 @@
   <%= "Автобусы #{@from.name} – #{@to.name}" %>
 </h1>
 <h2>
-  <%= "В расписании #{@trips.count} рейсов" %>
+  <%= "В расписании #{@trips.length} рейсов" %>
 </h2>
 
 <%= render partial: "trip", collection: @trips, spacer_template: 'delimiter' %>
 
-<% if @trips.size > 0 %>
+<% if @trips.length > 0 %>
     <%= render "delimiter" %>
 <% end %>

case-study-b.md

@@ -34,14 +34,26 @@
 - количество sql запросов для `trips/index.html.erb` сократилось до 12
 
 ### Ваша находка 2
-- rack mini profiler (и логи веб сервера) показал, что основное время тратится на рендеринг шаблонов (`trips/index.html.erb	2091.1
-4917.9`).
+- rack mini profiler (и логи веб сервера) показал, что основное время тратится на рендеринг шаблонов `trips/index.html.erb	2091.1
+4917.9`.
 - рендереринг всех коллекций через `render partial:`
 - метрика снизилась до 1 секунды
 - `Rendering: trips/index.html.erb	716.2 975.7`
 
-### Ваша находка №2
-- rack mini profiler показал, что дольше всего (412.1 ms) выполняется запрос `SELECT "buses_services".* FROM "buses_services" WHERE "buses_services"."bus_id" IN ($1, <...>`. делаю explain, он показывает `Seq Scan on buses_services` как основную точку роста.
+### Ваша находка 3
+- rack mini profiler показал, что основное время занимает запрос в сервисы (около 400мс).
+- вместо habtm делаю промежуточную таблицу, для нее и делаю eager loading вмсто services. в контроллере достаю весь и собираю хэш-справочник (id=>name) сервисов и использую его во вью.
+- метрика особенно не снизилась
+- запрос пропал из логов профилировщика (вместо него показывается запрос в сервисы, около 25мс, что 8 раз меньше)
+
+### Ваша находка 4
+- rack mini profiler показал, что производится и запрос по count, и идентичный по выборке данных.
+- меняю count на length.
+- метрика особенно не снизилась
+- запрос count пропал из логов профилировщика
+
+### Ваша находка № 5
+- rack mini profiler показал, что долго выполняются запросы `SELECT "buses_services".* FROM "buses_services" WHERE "buses_services"."bus_id" IN ($1, <...>` (делаю explain, он показывает `Seq Scan on buses_services` как основную точку роста),``
 - Добавляю индекс на `bus_id`
 - как изменилась метрика
 - как изменился отчёт профилировщика - исправленная проблема перестала быть главной точкой роста?