AP
да... потихоньку въезжаю уже)
Size: a a a
2019 March 06
RV
ну, декомпозия сложных запросов на кусочки это раз. второе - я бы за такую простыню по ручкам давал. ибо сложночитаемо
нормальный запрос вполне
RV
две таблицы и два действия, что ты не можешь прочитать?
RV
def bulk_destroy
current_company.trucks.where(id: params[:ids]).left_joins(:crews).where(crews: { truck_id: nil }).destroy_all
trucks_with_crew = current_company.trucks.where(id: params[:ids])
return head :no_content if trucks_with_crew.empty?
render json: { message: "Trucks: '#{trucks_with_crew.pluck(:name).join(', ')}' can't be deleted because used by crews." }, status: :unprocessable_entity
end
Вот такое работает :)
current_company.trucks.where(id: params[:ids]).left_joins(:crews).where(crews: { truck_id: nil }).destroy_all
trucks_with_crew = current_company.trucks.where(id: params[:ids])
return head :no_content if trucks_with_crew.empty?
render json: { message: "Trucks: '#{trucks_with_crew.pluck(:name).join(', ')}' can't be deleted because used by crews." }, status: :unprocessable_entity
end
Вот такое работает :)
а вот это, если bulk_destroy - экшн контроллера, билет в крематорий
AP
Почему? Потому что не бекграунд воркер ?
RV
а ну он и есть, не дочитал до рендера
RV
нууу
RV
это уже сорт оф размахивание руками, но операции с AR внутри контроллера - хуевая практика, контроллер это просто триггер определенного поведения внутри приложения, веревочка за которую запрос дергает
RV
в идеальном случае все что там есть - вызов какого-то функционала в приложении и решение о том какой ответ отправить
AP
Я понял, ты имеешь ввиду об сервисе с бекграунд воркером. а на контроллере просто .call вызывать сервиса. Это я знаю, и почему так и какие могут быть последствия тоже знаю)
NT
Я понял, ты имеешь ввиду об сервисе с бекграунд воркером. а на контроллере просто .call вызывать сервиса. Это я знаю, и почему так и какие могут быть последствия тоже знаю)
контроллер - передаван модели чо ты с ней хош сделать.
RV
ну бекграунд воркер это опционально
NT
а вот это вся простыня проверок - в код модели
AP
Вы опять загоняете не в ту степь
RV
> в код модели
в соседнюю печку быстро, решительно
в соседнюю печку быстро, решительно
NT
> в код модели
в соседнюю печку быстро, решительно
в соседнюю печку быстро, решительно
меня всегда учили что контроллер должен быть максимально облегчен. все эти проверки - в код модели
RP
за речью следи
A
лучше вообще без кода
A
нет кода - нет проблем)
RV
лучше вообще без кода
этот все понял