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

refactor: 指定されたファイルパスに対応するファイルが既に存在する場合の処理が散らばってるので統合する #2440

Open
Hiroshiba opened this issue Dec 27, 2024 · 0 comments
Labels
初心者歓迎タスク 初心者にも優しい簡単めなタスク 機能向上

Comments

@Hiroshiba
Copy link
Member

内容

src/store/audio.tsのchangeFileTailToNonExistent関数と、
src/sing/fileUtils.tsのgenerateUniqueFilePath関数の処理がほとんど同じなので、統合したいです。

Pros 良くなる点

メンテナンス性が上がる

実現方法

たぶんsrc/store/utility.ts辺りに。統合した関数を置くと良さそう!

片方の関数は拡張子があることを、片方の関数は拡張子がないことを想定しているので注意。

その他

コミッター募集中です!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
初心者歓迎タスク 初心者にも優しい簡単めなタスク 機能向上
Projects
None yet
Development

No branches or pull requests

1 participant