-
Notifications
You must be signed in to change notification settings - Fork 300
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
ソング:audioRendering.tsのリファクタリング #1811
Labels
Comments
sigprogramming
changed the title
audioRendering.tsのリファクタリング
ソング:audioRendering.tsのリファクタリング
Feb 2, 2024
用語や設計は @sigprogramming さんがまとめてくださったこちらが参考になるかもです! |
This was referenced Feb 19, 2024
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
内容
audioRendering.ts
のスケジューリング関連の処理・クラスが整理しきれていないので、ループ機能実装のためにも一旦リファクタリングを行いたいと思います。何を行うかは検討中ですが、
Transport
が「再生・停止の制御」と「イベントのスケジューリング」の2つを行っていたりするので(単一責任の原則に反しているので)、その辺りをまず改善できればと思います。Pros 良くなる点
Cons 悪くなる点
その他
The text was updated successfully, but these errors were encountered: