RailsのControllerにApplicationService相当のロジックを書くのはありなしや？ - 猫型の蓄音機は 1 分間に 45 回にゃあと鳴く

誤解を産んでいそうだったので追記します。ここでいうApplicationServiceというのは、いわゆるレイヤードアーキテクチャのアプリケーション層のApplicationServiceレイヤの話です。別の言葉だと、「Usecase層」とか言う言葉で呼ばれたりするアレのことです。追記おしまい。 Railsにおいては、ApplicationService相当のロジックをコントローラーに書いても良いものとする— 猫でもわかるしんぺい入門 (@shinpei0213) June 4, 2019 これについてです。 結論から先にいうと、ぼくは「正しい」と思っています（ただ、自分ではあまりやらず、ApplicationServiceに切り出しちゃいますが、これは好みの問題だと思っています）。 なぜ正しいと思うのか。それを考える際に、まずは「一般的に、なぜControllerとApplication

[D_first]

Railsに限らず正しいと思う。初手で分ける必要はなくて、必要になったらあとで分けるので十分。ちゃんと設計してればアプリケーションサービスにドメインロジックは無いはずだし。

[daichirata]

良い。大抵はユーザー登録用のserviceを作るのでテストでもそれを使うっていうのはよくやる。一方でテストが重くなりがちなので部分的にFactoryBotを使うケースもある。

[gfx]

FactoryBotの話はここ発か。ふーむ…。「適当にフィールドが生成されたarticle model instance」をアプリコードのために用意するイメージが全然ないし、FactoryBotの話はちょっと違うんじゃないかと。

[takahashim]

Railsはレイヤードアーキテクチャじゃないのでそっち方面の用語を使う時点で話が噛み合いづらい気がしますね…

[el-condor]

なしかな。セッションの受け渡しその他とビジネスロジックの密結合はあり得んでしょ。

[asonas]

全体的に同意できる内容だった。「「正しい」を選ぶ、けど自分はあまりやらない」となるところまでわかる

[luccafort]

“なぜここでみんな（たとえば）「UserSignupApplicationService」を作ってそれを呼び出すだけにしないんだろう”このケースではそうかなって思うけどちょっとこれは趣旨がズレている気がする。

[tsugitta]

これについては純粋にテスト対象以外の実装に依存すべきでないという話ではないかな > なぜ「テスト済みの、前提データを作れる部品」があるのに、FactoryBotを使う必要があるのでしょうか？

[ledsun]

「Controllerで複数Modelを扱うロジックがあって、テストを書かないと不安なぐらいには複雑」て、文脈ならわけるかなあ。「複雑」ってことが伝わるので。

[akatakun]

PDSの文脈において、ControllerがPresentationのレイヤーに属すものであり、ApplicationServiceはDomainに属すものである、という理解がまずあるから

[benridane111]

サービスごとにロジックを分けてもいいけど、まさか ApplicationService って名前で分けたりしないよね。曖昧なサービス名はトラブルの元ってことはわかる。

[jmatsu]

1つのendpointで書かなきゃいけないテストが増えちゃうので、切り出して依存サービスごとの実装を検証したい派 “RailsってControllerのテストめっちゃふつうにしやすい”

[tkmkg8m]

″メイクがセンスするし、パルスのファルシのルシがパージでコクーン″。テストだけ考えてもいくらRailsがコントローラーのテストしやすいとはいえやっぱり考えることは増えるので切り出したい派