Managers
- サーバAPIへアクセスする
- レスポンスをModelへパースする
- パースしたModelの一部を、インスタンス変数で管理
- シングルトン
ViewController
- 別ViewControllerに遷移
- Userのアクションへのリアクション
View
- SnapKitによるレイアウト実装
ViewModel
- Managerにデータを要求、受け取ったデータを管理
- UIViewControllerにアクセスしてUIの更新等を行う
Model
- データ

[アプリ設計]

[構成]

比較

Before

After

やったこと

APIClient / JSON デコーダーともに複数あったので、新しいAPIClientを作成し、Codableを使うように変更 -> 古いものはまとめて削除。Objective-Cの時のような、json["key"] as? Intのような実装をなくしました
ソースコードに対するコメント / BTS なぜその実装になったのか等記載がない -> 現状を知り着手しやすいように、複雑な処理になる部分は、シーケンス図 / コメントで動作を記載
一行直すだけで、関係ないと思ったところでエラーが起こる -> Component間で扱うデータを減らし、非結合にしてComponent間の影響を減らしました
Manager / Utility クラスは、シングルトンで実装 -> シングルトン実装のクラスを極力減らす。シングルトンにするとpropertyをつけたくなる人がいるので避けます
UIはSnapKitのみで実装 -> Storyboard / Xib で実装、IBInspectable, IBDesignableを使い、GUIで状況を把握しつつ実装　デザイナーが作ってくれているレイアウトに沿った物を作れる ( SnapKitだけの時、cornerRadiusなど漏れが発生していた )

コードの削除をかなりできた着手前と今の差分:
```
2467 files changed, 142326 insertions(+), 271177 deletions(-)
```
すべての画面に、MVVMをあてようとして無理している部分が多々あり結果、読みにくいコードになっていたのを直せた

レビューする際に、コードよりも、Storyboard / Xib で見た方がわかりやすかった Before:

override func viewDidLoad() {
super.viewDidLoad()
.... 40行ほど
}

After:

override func viewDidLoad() {
super.viewDidLoad()
imageView.image = UIImage(named: ... ) // 1行のみ
}

ドキュメント作成した事で効率改善が行えた
- デザイナーとの画像受け渡しは、Xcodeから直接やってもらえることになった
- ドキュメント自体もレビューされるので、その際に共有できた
- レビューする側も、どういう事をしたいロジックなのか理解できた

e.g. - ホームが重いので改善する作業中に、haptic feedbackの導入や、お気に入りのハートを他の画面と同期するように仕組みを入れました - 検索画面を改修する際に、保存した検索条件に起因する部分もまとめて、書き直し整理を行ったりしました