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

ソング:SET_PITCH_EDIT_DATAがセッターになってないので名称を変える #2020

Open
Hiroshiba opened this issue Apr 26, 2024 · 0 comments

Comments

@Hiroshiba
Copy link
Member

内容

ピッチ編集した値をstoreに代入するSET_PITCH_EDIT_DATAがあるのですが、一般的なセッターではなく配列を要素数だけ上書きする形になっています。
普通のセッターだと思っていると不具合が生じそうだと気づいたので、メモがてらissueを作ってみました。

Pros 良くなる点

間違いづらくなる

Cons 悪くなる点

実現方法

名前をどうするかがちょっと難しそう。
javascriptのspliceと同じ動作をするので似たような名前にするとわかりやすいかも。
SETの部分をUPDATE_SLICEとかREPLACEにすると良さそう・・・?

その他

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

No branches or pull requests

1 participant