hookQueryに関わる関数のリファクタリング - Githubissues

Lorenzras / kokoas-client

Internal System

MIT License

0 stars 0 forks source link

hookQueryに関わる関数のリファクタリング #140

Closed Lorenzras closed 2 years ago

Lorenzras commented 2 years ago

早川さんのリファクタリング (ファイル構成の改善) 作業の続きです。

変更

react-queryの独自フックに処理を別のファイル・関数にする。
同じような関数をGET, PUT, POSTなどで整理していましたが、１関数＝１ファイルという構成に変更

変更理由

react-queryの独自フックに処理を直接組むと、単体テストが難しいと気づきました。
コード移動やTESTがより簡単になるし、コンフリクトの対策にもなります。

備考

hooksQueryに実装してある関数だけカーバします。
一括リファクタリングが難しいので、既存のものをdeprecatedにして、リファクタリング作業をします。もし、将来的タスクに関わるところにdeprecatedがあれば、その時修正を行います。

参考

https://levelup.gitconnected.com/advanced-code-organization-patterns-the-case-for-one-function-per-file-da68c15826e8