#118 ホスト作成、更新画面の作成 #184

RikitoNoto · 2023-09-06T12:19:48Z

Issue

ホスト情報入力画面（ホストデータ作成・更新画面）の実装 #118

close #118

説明

ホスト作成、更新画面を作成しました。

仕様

作成画面での位置情報初期値は東京駅としています。
ホスト情報作成時にホストロケーションも作成、更新を行います。
ホストロケーションは将来的にホストと1対多の関係になりますが、現状は1つしか更新ができません。(ホストに紐づく最初のホストロケーションを更新対象としています。)

デバッグ用開発ページ

ホストとして登録ページ(hosts/create)
ホスト情報更新ページ(/hosts/:hostId/update)(hosts/updateでもいいかもと思いました。)

UI

■バリデーション実行

■作成

■更新

その他

その他に言及したいことがあれば書いてください。

チェックリスト

PR の冒頭に関連する Issue 番号を記載しましたか？
本 PR の変更に関して、エディタや IDE で意図しない警告は増えていませんか？（lint 警告やタイポなど）
Issue の完了の定義は満たせていますか？
当該 Issue のスレッドで、レビュワーにレビュー依頼をしましたか？

…to 118_create_update_host

packages/firebase_common/lib/src/firestore_documents/host.dart

packages/firebase_common/lib/src/firestore_repositories/host_location.dart

kosukesaigusa · 2023-09-07T02:48:39Z

packages/mottai_flutter_app/lib/host/ui/host_controller.dart

+    // Hostの作成
+    final imageUrl = await _uploadImage(imageFile);
+    await _hostService.create(
+      workerId: workerId,
+      displayName: displayName,
+      introduction: introduction,
+      hostTypes: hostTypes,
+      urls: urls,
+      imageUrl: imageUrl,
+    );
+
+    // Locationの作成
+    await _locationService.create(
+      hostId: workerId,
+      address: address,
+      geo: geo,
+    );


後で対応メモ：画面全体の OverlayLoading を有効にして使うと良さそう

こちらは何か修正しますか？
ついでに直せる内容でしたら修正します！

kosukesaigusa · 2023-09-07T02:56:53Z

packages/mottai_flutter_app/lib/host/ui/host_controller.dart

+    await _hostService.create(
+      workerId: workerId,
+      displayName: displayName,
+      introduction: introduction,
+      hostTypes: hostTypes,
+      urls: urls,
+      imageUrl: imageUrl,
+    );
+
+    // Locationの作成
+    await _locationService.create(
+      hostId: workerId,
+      address: address,
+      geo: geo,
+    );


この 2 つは実際にはセットでコールするものなので、Controller が順に呼ぶというよりは、host と hostLocation を一緒に更新するサービスクラスのメソッドを定義するべきかも？と思いました！（そのサービスクラスの名前は HostService で良いと思います。今回する必要はないですが、本来はバッチ書き込みしても良いような内容なので）

修正しました！

kosukesaigusa · 2023-09-07T02:59:59Z

packages/mottai_flutter_app/lib/host/ui/host_controller.dart

+    await _hostService.update(
+      hostId: hostId,
+      displayName: displayName,
+      introduction: introduction,
+      hostTypes: hostTypes,
+      urls: urls,
+      imageUrl: imageUrl,
+    );
+
+    // Locationの更新
+    final locations =
+        await _locationService.fetchHostLocationsFromHost(hostId: hostId);
+    if (locations != null && locations.isNotEmpty) {
+      await _locationService.update(
+        hostLocationId: locations.first.hostLocationId,
+        address: address,
+        geo: geo,
+      );
+    }


ここもまるっとひとつのモデル（サービス）クラスのメソッドで書くべきかもなと思いました！

また前にちょっと話しましたが、host vs. hostLocation は常に 1 対 1 の前提で良いと思っていました！（ホストに「あなたが登録している hostLocation 一覧」みたいな画面を作るつもりもないです）分かりづらくてすみません🙏

https://flutteruniv.slack.com/archives/C05HHFHGJSC/p1690290571400399?thread_ts=1690284627.798819&cid=C05HHFHGJSC

それならそもそも hostLocation の id は hostId と一致でも良さそうですね...！

※ 上記は対応したかったら & できそうだったらでいいです！

修正しました！

kosukesaigusa · 2023-09-07T03:01:38Z

packages/mottai_flutter_app/lib/host/ui/host_form.dart

+  final formKey = GlobalKey<FormState>();
+
+  /// 選択中のホストタイプ
+  final List<HostType> _selectedHosyTypes = [];


タイポ：_selectedHostTypes

ご指摘ありがとうございます。
修正しました！

kosukesaigusa · 2023-09-07T03:04:20Z

packages/mottai_flutter_app/lib/host/ui/host_form.dart

+  static LatLng convertGeoToLatLng(Geo? geo) {
+    var latitude = 0.0;
+
+    var longitude = 0.0;
+
+    if (geo != null) {
+      latitude = geo.geopoint.latitude;
+      longitude = geo.geopoint.longitude;
+    }
+
+    return LatLng(latitude, longitude);
+  }


static メソッドにしている必要はなさそう？単に private メソッドで OK？

[確認]

null が入ってくると LatLng(0, 0) が返されるのは意図通りですか？

すみません、開発過程で別のクラスに置いていたので、修正が漏れていました。
nullが入ってくることがないので、修正しました！

kosukesaigusa · 2023-09-07T03:06:18Z

packages/mottai_flutter_app/lib/host/ui/host_form.dart

+
+/// 入力が必須か任意かを表すバッジ
+/// [isRequired] の値によって、必須か任意の文字を選択して返す。
+class _OptionalBadge extends StatelessWidget {


host_form と job_form とで Duplicated なので共通のものとして lib/widgets/ にでも置いてほしいです！

新しくOpstionalBadgeクラスを定義しました！

kosukesaigusa · 2023-09-07T03:06:56Z

packages/mottai_flutter_app/lib/host/ui/host_form.dart

+/// [TextField]をGridで表示するウィジェット
+/// GridViewの場合は、aspect比を指定する必要があり、
+/// [TextField]のような可変長aspect比が必要なウィジェットでは使用できないため作成
+class _VariableHeightGrid extends StatelessWidget {


なるほど... 意外と難しいんですね

kosukesaigusa · 2023-09-07T03:08:49Z

packages/mottai_flutter_app/lib/user/host.dart

@@ -51,4 +51,40 @@ class HostService {
  /// 指定した [Host] を取得する。
  Future<ReadHost?> fetchHost({required String hostId}) =>
      _hostRepository.fetchHost(hostId: hostId);
+
+  /// [Host] の情報を作成する。
+  Future<void> create({


上で書いたとおりですが、Controller で色々書いているのをモデルに移動してほしいです。現状モデルは単に Repository のメソッドを呼ぶだけの感じになっています。Controller が肥大化するのではなく、モデルにロジックを書きます。Controller はユーザー操作をモデルに伝えるもの、UI を操作するものであることを意識してみてください！

修正しました！

- urlMaxCountの定義場所変更 - HostLocation作成、更新処理の定義場所移動 - タイポ修正: _selectedHostTypes - 座標変換関数のprivate化 - OptionalBadge の作成

…to 118_create_update_host

RikitoNoto and others added 10 commits August 29, 2023 22:20

#118 ホスト情報入力画面のたたき台作成

a82eb98

Merge branch 'main' of github.com:KosukeSaigusa/mottai-flutter-app in…

2f43232

…to 118_create_update_host

#118 URL入力フォームの作成

60d64aa

#118 Host作成時にIDを指定できる処理の作成

2e056a3

#118 HostLocationのリポジトリ作成

8e67ccf

#118 位置情報フィールドの作成

d325038

#118 位置情報選択ダイアログの作成

ab12194

#118 HostLocationを作成、更新する機能の追加

fab1bf3

#118 位置情報選択時のダイアログ表示、ホストタイプのバリデーション追加

a0d7616

Merge branch 'main' of github.com:KosukeSaigusa/mottai-flutter-app in…

8f19c20

…to 118_create_update_host

RikitoNoto added the 1. 機能開発機能開発 label Sep 6, 2023

RikitoNoto added 2 commits September 6, 2023 21:21

#118 更新画面でホストタイプが表示されないバグ修正

6b56f00

#118 lint対応

2e8cdc1

RikitoNoto requested a review from kosukesaigusa September 6, 2023 12:26

#118 lint対応

3a9f608

kosukesaigusa reviewed Sep 7, 2023

View reviewed changes

RikitoNoto added 2 commits September 7, 2023 20:39

#118 レビュー内容の反映

4a1c595

- urlMaxCountの定義場所変更 - HostLocation作成、更新処理の定義場所移動 - タイポ修正: _selectedHostTypes - 座標変換関数のprivate化 - OptionalBadge の作成

#118 hostLocationのIDをhostIdと同じになるように修正

f05a9c8

kosukesaigusa approved these changes Sep 8, 2023

View reviewed changes

Merge branch 'main' of github.com:KosukeSaigusa/mottai-flutter-app in…

0652d1d

…to 118_create_update_host

RikitoNoto merged commit 7885639 into main Sep 8, 2023
5 checks passed

RikitoNoto deleted the 118_create_update_host branch September 8, 2023 10:10

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

#118 ホスト作成、更新画面の作成 #184

#118 ホスト作成、更新画面の作成 #184

RikitoNoto commented Sep 6, 2023 •

edited

Loading

kosukesaigusa Sep 7, 2023

RikitoNoto Sep 7, 2023

kosukesaigusa Sep 7, 2023

RikitoNoto Sep 7, 2023

kosukesaigusa Sep 7, 2023

RikitoNoto Sep 7, 2023

kosukesaigusa Sep 7, 2023

RikitoNoto Sep 7, 2023

kosukesaigusa Sep 7, 2023

RikitoNoto Sep 7, 2023

kosukesaigusa Sep 7, 2023

RikitoNoto Sep 7, 2023

kosukesaigusa Sep 7, 2023

kosukesaigusa Sep 7, 2023

RikitoNoto Sep 7, 2023

#118 ホスト作成、更新画面の作成 #184

#118 ホスト作成、更新画面の作成 #184

Conversation

RikitoNoto commented Sep 6, 2023 • edited Loading

Issue

説明

仕様

デバッグ用開発ページ

UI

その他

チェックリスト

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

RikitoNoto commented Sep 6, 2023 •

edited

Loading