Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ソング:ノート・テンポ・拍子の持ち方を変更する #1800

Closed
sigprogramming opened this issue Jan 30, 2024 · 4 comments · Fixed by #1821
Closed

ソング:ノート・テンポ・拍子の持ち方を変更する #1800

sigprogramming opened this issue Jan 30, 2024 · 4 comments · Fixed by #1821
Assignees

Comments

@sigprogramming
Copy link
Contributor

sigprogramming commented Jan 30, 2024

内容

(ソングのissueです)
現在、ノート・テンポ・拍子(とTPQN)はScore型で持っていますが、
将来のマルチトラック対応のことを考えると、ノートはTrack型(まだ存在しません)で持ち、テンポ・拍子は全トラックに適用されるデータとして持つ形に変更した方が良さそうです。
以下のような感じです。

type Track = {
  notes: Note[];
};

type SongEditorStoreState = {
  tpqn: number;
  timeSignatures: TimeSignature[];
  tempos: Tempo[];
  tracks: Track[]; // マルチトラックに対応するまではトラック1つ
  ...
};

Score型は楽譜情報のみを保持するものとして、楽譜のインポート・エクスポート時に使用するのが良いかなと思います。

Pros 良くなる点

  • マルチトラック対応しやすくなる

Cons 悪くなる点

  • 特にないと思います

その他

早めにやっておくと良さそう(後々楽かも)と思ったのでissueを作成しました。

@y-chan
Copy link
Member

y-chan commented Feb 1, 2024

@sigprogramming こちら取り組まれていますでしょうか 👀

#1781 に取り組む前にこちらが出来た方が後々楽かなと思っているので、サクッと出来そうでしたらお願いしたいですし、お時間かかりそうでしたら私の方でできればと....!

@sigprogramming
Copy link
Contributor Author

sigprogramming commented Feb 1, 2024

@y-chan
確かにこちら先に行ったほうが良さそうですね、取り組みます!

@sigprogramming sigprogramming self-assigned this Feb 1, 2024
@sigprogramming sigprogramming changed the title ノート・テンポ・拍子の持ち方を変更する ソング:ノート・テンポ・拍子の持ち方を変更する Feb 2, 2024
@Hiroshiba
Copy link
Member

プルリクマージさせていただきました!
こちら解決でしょうか 👀

@sigprogramming
Copy link
Contributor Author

tracks[0]からgetter(CURRENT_TRACKorSELECTED_TRACK)に変更しておいたほうが良い気がしてきたので、それも行っておこうと思います…!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants