я кстати хуевый разработчик, вы меня не слушайте если чо.

В них надо вытаскивать когда экшен реально не помещается или выглядит херово.

ну понятно, что не просто User.save

ну например

вот у тебя есть User.find(id)

и оно по идее должно выбрасывать raise если юзера нет

ну или какую-то ошибку вообще

да?

ActiveRecord::RecordNotFound

ну типа

и оно по идее должно в нескольких местах использоваться

нужно это выделять в сервис?

нет ) есть rescue_from в ApplicationController, есть тесты.

Это базовые вещи рельсы, тут не нужны сервисы.

ну так это будет дублирование кода

мм, почему?

тесты да, но тесты и должны дублироваться на случай херовых абстракций.

ну я не хочу твой ActiveRecord::RecordNotFound

я хочу чтобы мне писались ошибки текстом на разных языках

хочешь ты его или не хочешь, AR его сам кидает.

бля

мой пример конь в вакууме

ну есть у тебя render_not_found в ApplicationController, дальше если нужны персонализированные ошибки (зачем?) то просто переопределяешь в дочернем контроллере.

Чтобы не потерять такое поведение - есть тесты.

так давай другой пример

вот есть типа код thread = Thread.where(uuid: uuid).take unless thread raise "Thread #{uuid} not found" end

мне нужно проверять, что пришел uuid и я хочу ошибки

и аналогичный код существует и в админке и в юзерской части

thread – это тред с постами, а не тред операционной системы

thread – это тред с постами, а не тред операционной системы

да понятно, там нет where)

не очень правильно только так модели называть)

да похуй

Models::Thread

да похуй

?

class ApplicationRecord < ActiveRecord::Base def seek_by(args = {}) self.where(args*).first || raise ActiveRecord::RecordNotFound end end class Thread < ApplicationRecord end class ApplicationController < Rails::Controller rescue_from ActiveRecord::RecordNotFound do render_not_found end def render_not_found render json: { error: I18n.t('record_not_found') }, status: :not_found end end class ThreadController < ApplicationController def show render Thread.seek_by(thread_show_params) end def thread_show_params params.permit(:id, :name) end end

ну вообще не обязательно что этот код участвует только в show

Ну это пример, чтобы была ясна мысль.

Я бы в рельсе вот так наговнокодил.

твоя мысль разбивается о то, что этот код должен быть использован в нескольких местах сразу

ну а если это действительно не show?

не понимаю проблемы. Тебе не нравится что тут rescue_from который в других местах не работает?

ну так в других местах логика 404 может отличаться.

ну так в других местах логика 404 может отличаться.

схуя

схуя

почему нет то?

если нет треда с этим uuid, то оно должно прекращать дальнейшее выполнение

если она не отличается, у тебя все контроллеры наследуются от ApplicationController, и все просто работает.

мне много что не нравится так-то

мне не нравится что ты не проверяешь что это uuid

мне не нравятся permitted params

мне не нравится, что код нельзя переиспользовать

мне не нравится что ты не проверяешь что это uuid

Зачем?

мне не нравится, что код нельзя переиспользовать

Где и ради чего?

Зачем?

это пример, там может быть много разной логики над параметрами

если это ДЕЙСТВИТЕЛЬНО так критично, то uuid в роутах через регексп проверяется, лол.

Где и ради чего?

в разных экшенах

это тоже пример, его можно переписать под задачу кучей разных способов.

если это ДЕЙСТВИТЕЛЬНО так критично, то uuid в роутах через регексп проверяется, лол.

ну я хочу ошибку человекочитаемую

О хоспаде, ты просто пытаешься свой пример обставить столькими условиями, чтобы я наконец выдал точку зрения которую ты хочешь.

Я не люблю в таком контексте вести дискуссию. Такого рода задачи решаются разными способами.

действительно, уже пора заканчивать, все всем понятно)

нет

это реальная жизнь

все хорошо в меру

get '/boards/:dir' do result = GetBoard.call(board_dir: params[:dir]) if result.error json_error(result.error) else coercion = Types::Board json(coercion[result.board]) end end put '/boards/:dir/fav' do result = FavoriteBoard.call(board_dir: params[:dir], user: current_user) if result.error json_error(result.error) else coercion = Types::Board json(coercion[result.board]) end end вот как это решается через use-case-driven development

По слухам, российское B2B API такое веселое, что обычное рельсовое приложение будет просто замечательной сказкой.

> вот как это решается через use-case-driven development Это решается через твои любимые сервис объекты.

Это не единственный подход.

class FavoriteBoard include Interactor::Organizer before do result = GetBoard.call(board_dir: context.board_dir) if result.failure? context.error = result.error context.fail! else context.board = result.board context.favorite_id = result.board.id context.favorite_type = 'Models::Board' end end organize CreateFavorite end

ERROR: S client not available

интерактор кстати классный да, но в реальных проектах не юзал еще.

вот только это не значит что его надо во все щели пихать.

ну я тебе видос кинул

2011 год чувак рассказывает про такой подход

Ты его сам не смотрел ))

посмотрел

я кстати не смотрел этот видос и пилил такую архитектуру с интеракторами

а, ну я тут неверно тебя понял тогда.

Я грю, подходы разные бывают, везде свои плюсы и минусы.

подход рельсы это большой минус

Плюсы рельсы - быстро наговнокодить прототип, пожалуй самое быстрое что только есть.

оно работает для маленьких проектов

ну да

тут я согласен

чем больше оно становится тем меньше оно работает

как парадигма

это капитанство какое то, всем кто с рельсой работал какое то время это очевидно.

оказывается нихуя не всем

прикол в том что если по быстрому не наговнокодить, то пилить божественную архитектуру будет некому, т.к денег не будет и проекта не будет.

Ты же сам любишь про реальный мир говорить.

я ж говорю чуваки экшен дергают и данные ему передают

из кода

ну это гавнокодеры прост.

ну вот как человеку сказать что так делать нельзя?

если человек не понимает нормальных слов, то может попрощаться с человеком?

ну я думаю его можно обучить

исправлять может выйти очень дорого и долго, а потом он умный - убежит куда еще.

просто нужны авторитеты

вот такое видео – это авторитет

а ну и я тебе говорил что это наш сто

ну экшен дергать из кода это гавнокод же. Это должно быть очевидно

еще там мании к god object

лучший способ отучить - подставить так, чтобы явно было видно почему так лучше не делать.