shunkominato / vue-ts-decorator-fx-api

0 stars 0 forks source link

<エラーハンドリング> ログ基盤にエラー送信 #3

Open shunkominato opened 4 years ago

shunkominato commented 4 years ago

API自体がないので、一旦コメントアウトし、ダミーでログ基盤にエラー送信する処理を書いてみました。 こんな感じでしょうか?

shunkominato commented 4 years ago

確かにそうですね。 ドメインにより、何を何のために残すかが変わりますもんね。 ドメインが必要になるような物を別途作成しようかと思います。

ログ基盤に送るのはAPI実行時だけではないですもんね。 FxService内でのAPI実行時のエラーに関しては、repositoryまでthrowし、ログ基盤repositoryに委譲する形でしょうか?

yakisuzu commented 4 years ago

改めてソース見ましたが、そもそもの話として、FxServiceがサービスっぽくなく、やっていることがrepositoryですね なので、このサービスの責務、リポジトリとの違いは何かというところを考えていかないといけません

所感としてはapiClientがfxDaoくらいの場所で、fxのデータアクセスの都合を隠蔽する場所です 使う側はrest使ってるとか意識してはいけません FxRepositoryにどの条件でデータを取得するか書きましょう、いまFxServiceに書いている内容をうつします サービスは、ドメインモデルを組み合わせて振舞うような場所なので、gettersまたは、gettersを組みあわせた場所が、サービスになってくると思います

今回の話で言うと、このサービスもまたリポジトリなので、場所はここで、移譲はおっしゃるとおり、ログ基盤リポジトリが必要です FxRepositoryがログ基盤リポジトリに依存します