AD
Как раньше было:
orders_count = Order.last.id - Order.previous_day.last.idSize: a a a
2020 September 07
T
Вабсарвар девелопер
О, вы тоже знаток Дургасофта
AP
Мы все тут немного знатоки
АШ
Как раньше было:
orders_count = Order.last.id - Order.previous_day.last.idА вот так нельзя было?
Order.where(created_at: 1.day.ago..Time.current).count
VA
А вот так нельзя было?
Order.where(created_at: 1.day.ago..Time.current).count
Правильнее наверно так всё же:
size = Order.count
Order.order(:created_at).each do |order|
size -=1 if order.created_at < 1.day.ago || if order.created_at > Time.current
end
size
АШ
Правильнее наверно так всё же:
size = Order.count
Order.order(:created_at).each do |order|
size -=1 if order.created_at < 1.day.ago || if order.created_at > Time.current
end
size
это что троллинг такой, да? =))
АШ
вы уж извините, но если бы мне такое человек на интервью написал, я бы его не взял себе в команду
VA
это что троллинг такой, да? =))
Можно ежедневную таску написать, которая в 23:59 выполняется
И тогда просто выбирать:
namespace :daily do
task :refresh_count do
previous_day = Date.today
Order.all.each do |order|
if order.created_at.to_date == previous_day
order.update(tomorrow: true)
else
order.update(tomorrow: false)
end
end
end
end
И тогда просто выбирать:
Order.where(tomorrow: true).count
AG
Голосую за последний вариант. Только надо всё же не previous day, а
yesterday = Date.todayME
Тогда уж yesterday хотя б ☝🏻
AG
ME
Голосую за последний вариант. Только надо всё же не previous day, а
yesterday = Date.todayВас посетила полиция времени, вы почто такие присваивания употребляете?
ME
Вы небось и true = false балуетесь?
AG
Хотел предложить убрать лишнюю переменную
Но кажется их тут наоборот не хватает
order.update(tomorrow: order.created_at.today?)Но кажется их тут наоборот не хватает
ME
Где то был твит про
even? на JS, там красивые варианты былиAG
Думаю, неплохо было бы на микро сервисы разбить. Плюс чтение/изменение тоже разнести
ME
Думаю, неплохо было бы на микро сервисы разбить. Плюс чтение/изменение тоже разнести
Точно. Пару архитекторов туда ещё на эту задачу выделить
VA
Ну всё-таки с
id лучший вариант только надо его дописать - учитывать те, которые были удалены по какой-то причине:last_id = Order.yesterday.last.id
first_id = Order.yesterday.first.id
count = 0
(first_id..last_id).each do |id|
count = count + (Order.where(id: id)[0].nil? ? 0 : 1)
end
count
AG
scope yesterday тут все портит) слишком красиво
VA
scope yesterday тут все портит) слишком красиво
сек