`store/audio.ts`周りのリファクタリング

[thiramisu]

内容

リファクタリング案です。 適当に弄っていたらdiffが1000行を軽く超えたので以下に小分けにしてPRを出そうかと思っています。 一人で完走できるか分からないので、とりあえずまとめておきます。

• [x] テキストファイル出力処理をwriteTextFile()としてファイル先頭に移動

  1448

• [x] 連番取得処理をchangeTailToNonExistent()としてファイル先頭に移動

  1478

• [x] キャラ名+スタイル名から${speakerName}(${styleName || DEFAULT_STYLE_NAME})を返す関数formatCharacterStyleNameを定義しutility.tsに移動

  1510

  • [x] "ノーマル"を定数DEFAULT_STYLE_NAMEとして定義

    1520

• [x] デフォルトファイル名系の処理をgetterに移動

  1490

  • buildFileName()をgetters.DEFAULT_AUDIO_FILE_NAME()に変更
  • buildProjectName()をstore/utilility.tsからstore/audio.tsに移しgetters.DEFAULT_PROJECT_FILE_NAME()に変更
• [ ] 保存先選択ダイアログ表示の処理をcomponents/Dialog.tsに移動し、filePathをオプション引数から必須にする
• [x] getCacheOrFetchFromEngine？するGENERATE_AUDIO(audioItem) returns { audioKey, audioBlob }を根幹とする関連実装をstore/audioGenerator.tsへ分離
  • [x] モーフィングエラーのエラーハンドリングもその中に関数としてまとめる
  • [x] generateUniqueIdAndQuery()もその中に移動、また一部をutility.ts内generateUniqueId(serializable)に分離
  • [x] GENERATE_LABもstoreを参照する必要がないのでgenerateLabFromAudioQuery(audioQuery)としてその中に移動？
• [ ] (PLAY|STOP)_AUDIO(audioKey)系のHTMLAudioElementに対する操作をstore/audioPlayer.tsへ分離

  1492

  →さらに分解をする方針に

  • [x] audioElementsがRecordである意味がないので単体にする

    1525

  • [x] state.audioStatesからnowPlayingを分離し単体にする

    1530

  • [x] audioElement系の処理を別のactionへ分離

    1553

  • [x] audioPlayerへ移動

    1579

  • [ ] その他雑多処理

    1548

• [ ] ACTIVE_AUDIO_KEY前提のものの引数をなくす？
• [ ] DirPath/FilePath/FileName/FileBaseName/FileExtensionを型から区別し、見通しを良くする
  • [x] DEFAULT_FILE_NAME_TEMPLATEをDEFAULT_AUDIO_FILE_NAME_TEMPLATE = ${DEFAULT_AUDIO_FILE_BASE_NAME_TEMPLATE}.wavとする

    1483

• [ ] getCharacterInfo()をgetters.CHARACTER_INFOに変更 関連処理のmutationにデフォルト動作が定義されていてかつアプリ初期化付近で呼び出されているので一見して影響が不明だな、とか思った記憶があります。
• [ ] state.startPointが一見意味不明なので名前をselectedAccentPhraseIndexに変更
• [x] accentPh"r"aseの脱字を直す

  1482

• [x] HAVE_AUDIO_QUERYとIS_ACTIVEが未使用っぽいので削除

  1580

• [ ] GET_AUDIO_QUERY_OR_ERRORを作る
• [ ] バグ修正: テキスト欄追加→元に戻す→連続再生で「エンジンの再起動を～」という不正確なエラーが出るのを修正
• [ ] 妥当な順番にソートしなおす(store/type.ts含め)

Pros 良くなる点

• 254 をはじめとした、非同期の生成リクエストがしやすくなりそう。

• 1474 が達成できそう。

Cons 悪くなる点

• 変更が多岐にわたるため、コンフリクトしやすそう。

その他

細かい挙動について気づいた点を箇条書きにしておきます

• 上書きの防止時のファイル存在確認は.wavでしか走査されていない
• 再生/停止キー（連続埼栄/停止キーも）を長押しすると開発者ツール上でエラーが出る

[Hiroshiba]

まとめありがとうございます！！！

コードって結構なんか未知の理由があってリファクタリングできていないという時もたまにあると思うので、こうやって先に列挙していただけるとその手戻りが防げるからとても良いなと思いました！！！

全部見ていったのですが、問題がありそうなものはありませんでした！！

ちょっと 妥当な順番にソートしなおす(store/type.ts含め) に関しては、別でそもそもVuexをやめるというのを今画策中↓なので、これはもしかしたら大々的にコンフリクトが発生してしまう可能性があるのでちょっと気をつけた方がいいかもと思いました･･･！！

• https://github.com/VOICEVOX/voicevox/issues/1004

[thiramisu]

確認ありがとうございます。pathモジュール周りは地雷っぽいのでできる限り回避しようかなと思っています！ 実際にコードを書いたらまずかった、ということもあるかもしれないので、その時はまた言っていただけると助かります。

[cm-ayf]

getCacheOrFetchFromEngine？するGENERATE_AUDIO(audioItem) returns { audioKey, audioBlob }を根幹とする関連実装をstore/audioGenerator.tsへ分離

を試みます