controller.rb

class DealsController < ApplicationController
  def show
    Deals::ShowAction.new(self).call
  end
end
 
class Deals::ShowAction < SimpleDelegator
  def call
    render ...
  end
end

А зачем ты обертываешь экшен вокруг контроллера? Кмк, экшен -- это объект твоей доменной модели с какой-то там бизнес-логикой, а контроллер -- чисто транспортный слой. Контроллеру нужно

• скомандовать домену что-то сделать (выполнить сервис)
• получить от домена какие-то данные (page object)

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

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

…

On Tue, Oct 15, 2019 at 3:54 PM Andrew Kozin ***@***.***> wrote: А зачем ты обертываешь экшен вокруг контроллера? Кмк, экшен -- это объект твоей доменной модели с какой-то там бизнес-логикой, а контроллер -- чисто транспортный слой. Контроллеру нужно - скомандовать домену что-то сделать (выполнить сервис) - получить от домена какие-то данные (page object) то, что он при этом что-то рендерит -- это особенность транспорта, я бы эту логику оставил в контроллере — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <https://gist.github.com/6c1b9d71c7c544d5cade25f250ce4c7f?email_source=notifications&email_token=AAAGZNV7K7HX45ICL3UFZ63QOWVOBA5CNFSM4JA3PD5KYY3PNVWWK3TUL52HS4DFVNDWS43UINXW23LFNZ2KUY3PNVWWK3TUL5UWJTQAF2QFC#gistcomment-3055697>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAAGZNRFFHQPPDMDEVG3GWLQOWVOBANCNFSM4JA3PD5A> .