Er

[okjm3324]

ER図を追加しました。

以下このアプリ固有のカラム・テーブルの説明です。

• Tracksテーブル
• start : 曲の指定範囲のスタート地点を数値で持つ
• end : 曲の指定範囲のエンド地点を数値で持つ
• section : 曲の指定範囲がどのセクションに該当するかをSectionsテーブルから取得（サビ、Aメロ、ソロなど）Enumで持つ
• bpm : ユーザーが今演奏可能なbpmを数値で持つ
• instrument : 演奏する楽器を指定する Enumで持つ

以上 ご確認の程よろしくお願いします。

[busitora]

ER図のスクリーンショットを概要欄に添付して下さい！

[okjm3324]

失礼しました。添付しましたのでよろしくお願いします。

[busitora]

• FK が instrument になっていますね！
• また各カラムの説明も(一般的なもの以外)お願いします！

[okjm3324]

概要欄にカラム等の説明を追記し、修正後のER図を添付しましたのでよろしくお願いします。

[Kei5665]

ご提出ありがとうございます！

• artist_nameをalbumsテーブルに持たせれば、artistテーブルは不要そうです！
• profileもusersテーブルにまとめられそうです！
• お聞きしたいのですが、sections や instruments のレコードって増えそうですか？もし固定になりそうならtracksテーブルにまとめて enum とかで対応しても良いかもと思いました。もしかしたらtext型やstring型でも問題なさそうと思いました。

[okjm3324]

ご提案ありがとうございます。

• artist_nameをalbumsに持たせました。
• profileをusersにまとめました。
• sections、instrumentsは固定の予定ですのでEnumで扱うよう変更しました。 ご確認のほどよろしくお願いします。

[murata0705]

更新したER図がこのプルリク上で見えないので、画像をどこかにホスティングするか（Gyazoなど）などして見えるようにしていただけますか 🙏🏻

[okjm3324]

ER.mdにgyazoのurlを追記しました。 概要欄の画像は変更済みのものです。 いただいたご指示と違っていたらすみません。

[murata0705]

instrument_id と artist_id は消し忘れですかね？ こちら確認お願いします 🙏🏻

[okjm3324]

失礼いたしました。消し忘れです。 修正いたしましたのでご確認お願いします。

[tarakish]

修正確認しました！ current_bpmはbpmの方が良い気がしますね。 currentとあると、過去の状態のbpmも存在していてそれが取得できそうに思えてしまうので。（過去のbpmを別で持つ、とかがあればそのままでも大丈夫です！）

[okjm3324]

承知しました。過去の状態は持たないので、 current_bpm -> bpm に変更修正しました。 ご確認よろしくお願いします。

[tarakish]

修正確認しました。LGTMです！！！

[okjm3324]

ありがとうございました！