Created
October 15, 2019 08:54
-
-
Save gzigzigzeo/6c1b9d71c7c544d5cade25f250ce4c7f to your computer and use it in GitHub Desktop.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
class DealsController < ApplicationController | |
def show | |
Deals::ShowAction.new(self).call | |
end | |
end | |
class Deals::ShowAction < SimpleDelegator | |
def call | |
render ... | |
end | |
end |
Я не обертываю. Я хочу разделить контроллер на составные части, не
используя консерны. Потому что в данный конкретный момент не могу применить
другие идиомы.
…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>
.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
А зачем ты обертываешь экшен вокруг контроллера? Кмк, экшен -- это объект твоей доменной модели с какой-то там бизнес-логикой, а контроллер -- чисто транспортный слой. Контроллеру нужно
то, что он при этом что-то рендерит -- это особенность транспорта, я бы эту логику оставил в контроллере