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.

Sign up for GitHub

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

Jump to bottom

AudioItemをコピーするための関数を作る #683

Open
Hiroshiba opened this issue Jan 31, 2022 · 0 comments
Open

AudioItemをコピーするための関数を作る #683

Hiroshiba opened this issue Jan 31, 2022 · 0 comments
Labels
優先度：低 機能向上

Comments

@Hiroshiba
Copy link
Member

Hiroshiba commented Jan 31, 2022 •
edited
Loading

内容

AudioItemをコピーするべき経路はいくつもあり、それらによって何をコピーするべきかが違うのですが、今はコピー対象を足していく実装になっているので、コピーすべきものがコピーされないということがまれによく起こっています。

コピーする経路は次の４つです

  • Home.vueで新しくAudioItemを作るとき（＋ボタン）
  • PUT_TEXTSで複数行のAudioItemを作るとき
  • IMPORT_FROM_FILEで複数行のAudioItemを作るとき
  • テキスト欄複製機能

実現方法

今はどれもGENERATE_AUDIO_ITEMを使っています。
これを、空っぽのAudioItemを作るGENERATE_AUDIO_ITEMと、パラメータを継承するINHERIT_AUDIO_ITEMに分け、INHERIT_AUDIO_ITEMはデフォルト値をなくして引数指定を必須にするのが良いかなと思っています。

Pros 良くなる点

ミスが減りそう

Cons 悪くなる点

設計がちょっと大変（でもいろいろ検討した↑なら大丈夫な気がします）
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
優先度：低 機能向上
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Hiroshiba