Как получить сумму значений по группе из таблицы?

Question

[Александр Вульф]

Привет! У меня в базе хранятся записи о онлайне

id | user_id | count | created_at
1  | 1       | 12    | сегодня
2  | 4       | 26    | вчера

И я вывожу пользователям график, где показываю когда и какой был онлайн.
А сейчас мне надо админам вывести общий график. Для этого надо сгруппировать по дате (всех юзеров) и получить сумму онлайна.
Однако, дата у всех разная. Надо округлить до часа в меньшую сторону.

Помогите, совсем уж запарился.

Answer 1

[Александр Вульф]

Пересмотрел архитектуру приложения. Теперь подобного вопроса нет. Всем спасибо.

Answer 2

[_ _]

Как насчет .group('FLOOR(UNIX_TIMESTAMP(created_at) / 3600)')

Answer 3

[tpepost]

@grouped_entities = Entity.order('created_at DESC').group_by { |group| group.created_at.to_date }

Entity – наша модель модель (таблица).
.order('created_at DESC') – сортируем по убыванию.
.group_by { |group| group.created_at.to_date } – группируем по дате. Обратите внимание, что приходится created_at превратить в дату (to_date). Так как мы группируем по дате, а не по секундам.

Теперь во View можно сделать следующее:

<% @grouped_entities.each do |date, record_group| %>
  <h1>date</h1>
  <% record_group.each do |record| %>
    <%= record.some_method_to_display %>
  <% end %>
<% end %>

Comments

[thepry]

Такой вариант не очень потому, что группировка происходит на уровне приложения, а не СУБД. Если там миллион записей, то рельсы создадут миллион active record объектов, чтобы потом их агригировать.

[tpepost]

thepry: Считаю, что тут есть два больших плюса: независимость от БД и человекопонятность. Конечно на 1 000 000 записей уже другие решения, но думаю в случае автора все ок. Так что если памяти хватает, то я бы делал так. У меня сейчас платежи за последний месяц так агрегируется. Около 20 000 записей за запрос. К этому на одной странице происходит еще агрегация по филиалам и другие запросы на статистику. Это конечно не мульон, но на DigitalOcean(512Mb), рельса кушает совсем не много. prntscr.com/6wtr39