We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
ピッチ編集した値をstoreに代入するSET_PITCH_EDIT_DATAがあるのですが、一般的なセッターではなく配列を要素数だけ上書きする形になっています。 普通のセッターだと思っていると不具合が生じそうだと気づいたので、メモがてらissueを作ってみました。
SET_PITCH_EDIT_DATA
間違いづらくなる
名前をどうするかがちょっと難しそう。 javascriptのspliceと同じ動作をするので似たような名前にするとわかりやすいかも。 SETの部分をUPDATE_SLICEとかREPLACEにすると良さそう・・・?
splice
The text was updated successfully, but these errors were encountered:
No branches or pull requests
内容
ピッチ編集した値をstoreに代入する
SET_PITCH_EDIT_DATA
があるのですが、一般的なセッターではなく配列を要素数だけ上書きする形になっています。普通のセッターだと思っていると不具合が生じそうだと気づいたので、メモがてらissueを作ってみました。
Pros 良くなる点
間違いづらくなる
Cons 悪くなる点
実現方法
名前をどうするかがちょっと難しそう。
javascriptの
splice
と同じ動作をするので似たような名前にするとわかりやすいかも。SETの部分をUPDATE_SLICEとかREPLACEにすると良さそう・・・?
その他
The text was updated successfully, but these errors were encountered: