keeeeSB / Yonda

0 stars 0 forks source link

ER図の作成 #15

Closed keeeeSB closed 8 months ago

keeeeSB commented 8 months ago

リンク https://app.diagrams.net/#G1oU1t0dwHZVyBXSCezUmb5FSRisYwlfd2

確認お願いします。

busitora commented 8 months ago

ER図はスクリーンショットでお願いしますm

keeeeSB commented 8 months ago

ER図はスクリーンショットでお願いしますm

Image from Gyazo

確認お願いします。

kerochelo commented 8 months ago

それぞれどんなテーブル&カラム情報なのか、それぞれ説明もこのPullRequestに記載していただいてもいいでしょうか?

現状把握できた部分に関してレビュー致しますね!

keeeeSB commented 8 months ago
  • familyとuserの関係性に関して、usersにfamily_idがあったのでどのようにデータが作られるのか確認です。 このアプリの初回登録時はユーザーからですかね?

初回登録はユーザーからなので、user_idをfamiliesテーブルに追加する形に変更しました。

画面遷移図を見る限り、初回ユーザー登録時にもfamily情報も付随して作成される形でしょうか?

  • picture_booksテーブルですかね?piture_booksになっていたので修正しましょう。

タイポです。picture_booksに修正しました。申し訳ありません。

修正後のER図です。 image

確認お願いします。

tmtkzk0823 commented 8 months ago

説明ありがとうございます。 以下、追加のコメントになります。

全体

テーブル感の紐付けを表している線(鳥の足)ですが、重なっている部分が多々あり、正確な紐付けが読み取れないので、各線が独立するように修正をお願いします。 (現状ですと、invitesテーブルとfollowsテーブルも紐づいているように見えてしまうため)

keeeeSB commented 8 months ago

全体

テーブル感の紐付けを表している線(鳥の足)ですが、重なっている部分が多々あり、正確な紐付けが読み取れないので、各線が独立するように修正をお願いします。 (現状ですと、invitesテーブルとfollowsテーブルも紐づいているように見えてしまうため)

修正しました。 確認お願いします。 image

Tsuchiya2 commented 8 months ago

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


■共通 ・各カラムの型も記載してください。

■familiesテーブル ・user_id(外部キー)がありますが、リレーションはこのテーブルが親でusersテーブルは子になっていますが。。。どちらが適切なのでしょうか?

■picture_booksテーブル ・isbn_13:isbnで良いのではないでしょうか?

keeeeSB commented 8 months ago

■共通 ・各カラムの型も記載してください。

記入しました。 ■familiesテーブル ・user_id(外部キー)がありますが、リレーションはこのテーブルが親でusersテーブルは子になっていますが。。。どちらが適切なのでしょうか?

リレーションを追加しました。 users→familiesの流れにするつもりです。 ■picture_booksテーブル ・isbn_13:isbnで良いのではないでしょうか? isbnに変更しました。

image

確認お願いします。

tmtkzk0823 commented 8 months ago

修正ありがとうございます。 私からもコメントさせていただきます。

read_recordsテーブル, read_recordsテーブル

紐づいているこれらテーブルの両方にuser_idが見受けられるので、どちらかのテーブルのuser_idを参照するように設計するといいかとおもいます。

usersテーブル, familiesテーブル

修正していただいたER図ですと、「userが複数のfamilyに所属している」というような情報がアソシエーションが読み取れるのですが、こちらで正しいでしょうか?

keeeeSB commented 8 months ago

修正ありがとうございます。 私からもコメントさせていただきます。

read_recordsテーブル, read_recordsテーブル

紐づいているこれらテーブルの両方にuser_idが見受けられるので、どちらかのテーブルのuser_idを参照するように設計するといいかとおもいます。

read_recordsとpicture_booksテーブルでuser_id, family_idが重複していたので、read_recordsテーブルのみにuser_id, family_idを設計するようにしました。

usersテーブル, familiesテーブル

修正していただいたER図ですと、「userが複数のfamilyに所属している」というような情報がアソシエーションが読み取れるのですが、こちらで正しいでしょうか?

以下のようなリレーションにしたいのですが、、 Image from Gyazo

大丈夫でしょうか?

最終的に以下のようになりました。 確認お願いします。

image

shota-hope commented 8 months ago

修正ありがとうございます。

以下のようなリレーションにしたいのですが、、

一つのfamilyは複数のuserに紐づいているという形にしたいのであれば、usersテーブルにfamily_idが必要でfamiliesテーブルにはuser_idが不要かと思いますがいかがでしょうか? ご確認よろしくお願いします。

keeeeSB commented 8 months ago

一つのfamilyは複数のuserに紐づいているという形にしたいのであれば、usersテーブルにfamily_idが必要でfamiliesテーブルにはuser_idが不要かと思いますがいかがでしょうか? ご確認よろしくお願いします。

usersテーブルにfamily_idを追加、familiesテーブルのuser_idを削除しました。 流れとしましては、user登録時にはfamily_idはnilのままで、ログイン後にfamilyを登録し、usersテーブルのfamily_idと 紐付けるようにしたいです。

以下のようになりました。 確認お願いします。 image

Tsuchiya2 commented 8 months ago

概ね良いかと思います。LGTMです