投稿から読み上げまでの文章処理をリファクタリング

[book000]

https://github.com/jaoafa/JDA-VCSpeaker/blob/main/src/main/java/com/jaoafa/jdavcspeaker/Event/Event_SpeakVCText.java

読みにくい + MsgFormatter とかに混在してて処理順番がよくわからん

[yuuahp]

ごめん...

[yuuahp]

数日間で何とかします

[book000]

いやまあ私がいじくってわからなくなったところもあってエイリアスがどこでかかるべきかも考えたほうがいいのでアレです

[yuuahp]

フォーマットとかエイリアスパースとかいろいろ小分けした方がいいんかな

[book000]

どういう形が見やすいのかねというところよねえ

[yuuahp]

メソッドチェーンとか

LibAlias.parse(string);
LibMsgFormatter.format(string);

的なのを並べてくとか？

[book000]

うーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーん

まあ onMessageReceived(MessageReceivedEvent event) に処理をガンガン書いてる今のアレだと読みにくいのは確かなので各ファイルに処理渡してもいいかもしれん

[yuuahp]

全部直接書いたままでも見直せば多少短くなるとは思うけど、あの量を見やすく収めるってのは厳しいと思った

[book000]

むむむ

[yuuahp]

まあでもやってみないとわかんないか...

[book000]

1. 対象外サーバ除外
2. VCテキストチャンネル以外からのメッセージ除外
3. Botによるメッセージ除外
4. . 単体、/ や ; から始まるテキストの場合は読み上げ除外
5. 必要に応じて自動参加
6. ignoreの適用
7. リンクの置き換え適用
8. 強調の適用 replacerEmphasizeMessage
9. VoiceTextによるテキストパース
10. 読み上げ

その後画像処理

この10処理(最初4つは違うから6処理？)を一つのライブラリ(それこそMessageFormatter的な)にまとめて、投稿のパース以外にも動作検証用にparseコマンドとか作ってそっちでも使えるようにするといいかも
個々の処理も関数分割して扱いやすくとか

[book000]

つか画像処理もネストしすぎてたりメソッドチェーンしすぎてたりで読みづらいんだよなああの辺
難しいわ

[book000]

ちなみにあやまっとったけど普通にあの辺のコードメインでいじってるのは私なので私が謝るべきアレです…
https://github.com/jaoafa/JDA-VCSpeaker/commits/main/src/main/java/com/jaoafa/jdavcspeaker/Event/Event_SpeakVCText.java

[yuuahp]

とりあえずまあどうにかしてみるわ…

[yuuahp]

メソッドチェーンはあんまりやらない方がいい感じ？

[book000]

メソッドチェーンは見づらさの割にstreamとか使いやすいし正直完全悪とも言えない…
とりあえず先行は関数分けかなあ

[yuuahp]

帰ったらやってみる👍

[book000]

適度にjavadocとか書いてコード追いやすくしとくといいかも💪