TOSHI-54a / Let-s-Travel

0 stars 0 forks source link

Add:ER図作成 #5

Closed TOSHI-54a closed 2 months ago

TOSHI-54a commented 2 months ago

ER図を作成しました。

Image from Gyazo

tmtkzk0823 commented 2 months ago

ご提出ありがとうございます。 以下、コメントになります。


TOSHI-54a commented 2 months ago

コメントありがとうございます。 以下2点の修正を行いました。 ・リレーションを表す線を独立 ・created_at,update_atカラムは省略する形で統一

Image from Gyazo

Mariko222 commented 2 months ago

修正頂きありがとうございます。 リレーション問題なさそうです。 追加で、BoardsテーブルとSavingsテーブルの各カラムの説明を、簡単で構いませんので記載をお願いします。

TOSHI-54a commented 2 months ago

コメントありがとうございます。 コメントいただいたテーブル各カラムの説明は以下のとおりです。 ・Boardsテーブル(旅行体験記)  - place_name:旅行先の都市名  - start_duration:旅行のスタート日  - end_duration:旅行の最終日  - budget:予算額  - body:旅行体験記の本文  - top_image:旅行体験記一覧に表示するサムネイル画像1枚  - other_image:旅行体験記詳細に表示する画像(複数枚可) ・Savings(節約機能)  - date_and_time:節約額を記録した日時  - value:節約額

ER図再掲 Image from Gyazo

Mariko222 commented 2 months ago

カラムの説明ありがとうございます。 other_imageカラムで複数枚可とありますが、画像ファイルパスをカラムに保存する認識で合ってますでしょうか? であれば別にテーブルを作成して、Boardsテーブルと紐づける必要があるかなと思いました。

TOSHI-54a commented 2 months ago

コメントありがとうございます。 その認識であっております。 私の方でも複数の画像ファイルパス保存について調べたところ、データ管理の理由から別テーブルを作成をする方が良いことが分かりましたので、Boardsテーブルに紐付いた別テーブル(Images)を作成することとしました。

修正点 ・Boardsテーブルからother_imageカラムを削除 ・Imagesテーブルを作成  カラム board_id(FK) , other_image

Image from Gyazo

TOSHI-54a commented 2 months ago

コメントありがとうございます。 以下のとおり命名を変更しております。

・Savings  - date_and_timeカラムのデータ型をdatetime型へ変更    'ymd'形式で表示させるためdate型を選択していましたが、表示形式は設定で好きなようでできることが    分かりましたので時間管理の観点からdatetime型へ変更しました。  - valueカラムの名前をamountへ変更 ・Images  - テーブル名をOther_imagesへ変更  - other_imagesカラムの名前をimagesへ変更

Image from Gyazo

Tsuchiya2 commented 2 months ago

概ね良いかと思います。LGTMです。 修正が必要になった際は適宜修正して開発を進めましょう。