miura-taiga / dust-hunters

0 stars 0 forks source link

ER図作成 #8

Closed miura-taiga closed 2 months ago

miura-taiga commented 2 months ago

概要

ER図を作成しました。

変更内容

READMEにER図のリンクおよび画像を追加しました。

確認ポイント

関連issue

5

具体的な変更内容

ER図

ER図リンク

ER図

ER図画像

Image from Gyazo

miura-taiga commented 2 months ago

こちらの方で回答させていただきました。内容を変更し、pushし直しました。 ご確認のほどよろしくお願いいたします。

miura-taiga commented 2 months ago

以下コメントに対しての返答です。ご確認よろしくお願いします。


コメント回答

こちらのみ主キーをuuidにしている理由があれば説明いただけますか?

特に理由なくuuidを採用していたので、吟味した結果IDを採用することにしました。

こちらは確認ですが、defeat_countはそのモンスターの総討伐回数を全ユーザー共通で保存するという認識であっていますか?

いいえ、私の思い描いていた設計はユーザー1人当たりのモンスターの討伐数をカウントしたかったのでGuildCardsへ移動しました。

変更内容

Image from Gyazo

Tsuchiya2 commented 2 months ago

私からもコメントします。


■Users ・ユーザー、例えばAさんと分かるための要素(メールアドレスとパスワードが一致しているetc)が無さそうですが大丈夫そうでしょうか。

miura-taiga commented 2 months ago

ご指摘ありがとうございます。以下回答になりますので、ご確認お願いいたします。


ユーザーごとの識別要素はUsersテーブルのuidで行う予定でした。このuidはGoogleの認証システムを通じて取得される一意の識別子で、ユーザーを特定するための要素です。本アプリはログインシステムを外部認証システムのみ(Google)としているため、メールアドレスやパスワードは不要だと判断しました。

変更内容

uidだけだとわかりにくい、今後LINE認証の追加予定がありますので、以下の内容を変更しました。

Image from Gyazo

kenchasonakai commented 2 months ago

LGTMです! SorceryやDeviseを使用すると外部認証用のテーブルを作ってくれるので実装を進めながら調べて適宜調整していくとよいかと思います

miura-taiga commented 2 months ago

レビューいただきありがとうございました。 講師の方からのLGTMがいただけましたので、マージさせていただきます。