shusuke0812 / KikurageApp-iOS

🚀 Dev - Kikurage Cultivation App(Recording、Monitoring of temperature and humidity)
MIT License
6 stars 1 forks source link

Storyboard replacement/Home #130

Open shusuke0812 opened 3 months ago

shusuke0812 commented 3 months ago

プルリク内容

やったこと

確認したこと

補足

shusuke0812 commented 2 months ago

マルチモジュール

レイヤードアーキテクチャ https://martinfowler.com/bliki/PresentationDomainDataLayering.html

image

考えたこと KUIDeviceStatusImageViewのPropsとしてKikruageDataモジュールに定義したKikurageState型を渡している. しかし、KikurageStateが変わったら(API、そのレスポンス型を変更したら)、UI層も変える必要が発生する. よって、KUIDeviceStatusImageViewのPropsとしては標準で定義されたプリミティブな型を渡した方が依存関係の複雑さを防ぐことができて保守しやすいかもしれない.

その場合は、下記のような構成にする. ただし、本プロジェクトではServiceはViewModelとする.

image

結論 KikurageDataモジュールではなく、KikurageDomainに変更する. 既にあるKikurageFeatureはKikurageDomainに統合する.(別PRで実施)