loop_4

Author: aleksey.ryabchikov

case-study.md

@@ -69,14 +69,13 @@
 - При прогоне теста, среднее значение метрики упало с 0.25 сек. до 0.18 сек.
 
 ### Ваша находка №4
-- По какой-то причине показания stackprof и ruby-prof стали расходится.
-- stackprof указал на главную точку роста в строке `{ 'dates' => user.sessions.map{|s| s['date']}.map {|d| Date.parse(d)}.sort.reverse.map { |d| d.iso8601 } }`
-- ruby-prof указал на строку `{ 'totalTime' => user.sessions.map {|s| s['time']}.map {|t| t.to_i}.sum.to_s + ' min.' }`
+- Попробовал увеличить кол-во данных, чтобы проще было увидеть проблематику до 100_000 строк.
+- По отчетам нашел новую точку роста, это метод `collect_stats_from_users`, он аффектит на два проблемных места сразу
 ```
   %Total	%Self	Total	Self	Wait	Child	Calls	Name
-  90.29%	21.60%	0.37	0.09	0.00	0.28	10	Array#each
+  89.28%	23.89%	1.43	0.38	0.00	1.05	10	Array#each
 ```
-- Попробовал увеличить кол-во данных, чтобы проще было увидеть проблематику до 100_000 строк.
-- Изучив код более подробно, понял, что обе точки роста связаны между собой.
+- Оптимизировал, а именно избавился от collect_stats_from_users и начал подготавливать данные за один проход.
+- При прогоне теста, среднее значение метрики упало с 0.18 сек. до 0.14 сек. на 20_000 строк.
 
 

spec/task-1_spec.rb

@@ -30,14 +30,14 @@
 
     describe 'with partial file' do
       it 'works under 0.4 sec' do
-        expect { work(file) }.to perform_under(0.18).sec.warmup(2).times.sample(10).times
+        expect { work(file) }.to perform_under(0.14).sec.warmup(2).times.sample(10).times
       end
     end
 
-    describe 'with full file' do
-      it 'works under 30 sec' do
-        expect { work('./data_large.txt') }.to perform_under(30).sec
-      end
-    end
+    # describe 'with full file' do
+    #   it 'works under 30 sec' do
+    #     expect { work('./data_large.txt') }.to perform_under(30).sec
+    #   end
+    # end
   end
 end

task-1.rb

@@ -101,39 +101,31 @@ def work(file_path, result_path = 'spec/fixtures/files/result.json')
 
   report['usersStats'] = {}
 
-  # Собираем количество сессий по пользователям
-  collect_stats_from_users(report, users_objects) do |user|
-    { 'sessionsCount' => user.sessions.count }
-  end
-
-  # Собираем количество времени по пользователям
-  collect_stats_from_users(report, users_objects) do |user|
-    { 'totalTime' => user.sessions.map {|s| s['time']}.map {|t| t.to_i}.sum.to_s + ' min.' }
-  end
-
-  # Выбираем самую длинную сессию пользователя
-  collect_stats_from_users(report, users_objects) do |user|
-    { 'longestSession' => user.sessions.map {|s| s['time']}.map {|t| t.to_i}.max.to_s + ' min.' }
-  end
-
-  # Браузеры пользователя через запятую
-  collect_stats_from_users(report, users_objects) do |user|
-    { 'browsers' => user.sessions.map {|s| s['browser']}.map {|b| b.upcase}.sort.join(', ') }
-  end
-
-  # Хоть раз использовал IE?
-  collect_stats_from_users(report, users_objects) do |user|
-    { 'usedIE' => user.sessions.map{|s| s['browser']}.any? { |b| b.upcase =~ /INTERNET EXPLORER/ } }
-  end
-
-  # Всегда использовал только Chrome?
-  collect_stats_from_users(report, users_objects) do |user|
-    { 'alwaysUsedChrome' => user.sessions.map{|s| s['browser']}.all? { |b| b.upcase =~ /CHROME/ } }
-  end
-
-  # Даты сессий через запятую в обратном порядке в формате iso8601
-  collect_stats_from_users(report, users_objects) do |user|
-    { 'dates' => user.sessions.map{|s| s['date']}.map {|d| Date.parse(d)}.sort.reverse.map { |d| d.iso8601 } }
+  users_objects.each do |user|
+    user_key = "#{user.attributes['first_name']} #{user.attributes['last_name']}"
+
+    # Подготовим данные для сессий
+    sessions = user.sessions
+    times = sessions.map { |s| s['time'].to_i }
+    browsers = sessions.map { |s| s['browser'].upcase }
+    dates = sessions.map { |s| Date.parse(s['date']) }
+
+    report['usersStats'][user_key] = {
+      # Количество сессий
+      'sessionsCount' => sessions.count,
+      # Общее время
+      'totalTime' => "#{times.sum} min.",
+      # Самая длинная сессия
+      'longestSession' => "#{times.max} min.",
+      # Браузеры через запятую
+      'browsers' => browsers.sort.join(', '),
+      # Хоть раз использовал IE?
+      'usedIE' => browsers.any? { |b| b.include?('INTERNET EXPLORER') },
+      # Всегда использовал только Chrome?
+      'alwaysUsedChrome' => browsers.all? { |b| b.include?('CHROME') },
+      # Даты сессий через запятую в обратном порядке в формате iso8601
+      'dates' => dates.sort.reverse.map(&:iso8601)
+    }
   end
 
   File.write(result_path, "#{report.to_json}\n")