整理: replace_mora_pitch

[tarepan]

内容

replace_mora_pitch の変数リネーム・コメント簡略化によるリファクタリング

関連 Issue

無し

[github-actions]

Coverage Result

Resultを開く

Name	Stmts	Miss	Cover
run.py	516	330
voicevox_engine/init.py	1	0
voicevox_engine/cancellable_engine.py	94	72
voicevox_engine/core_adapter.py	51	17
voicevox_engine/core_initializer.py	59	30
voicevox_engine/core_wrapper.py	224	159
voicevox_engine/dev/core/init.py	2	0
voicevox_engine/dev/core/mock.py	36	8
voicevox_engine/dev/tts_engine/init.py	2	0
voicevox_engine/dev/tts_engine/mock.py	27	0
voicevox_engine/engine_manifest/EngineManifest.py	34	0
voicevox_engine/engine_manifest/EngineManifestLoader.py	12	0
voicevox_engine/engine_manifest/init.py	3	0
voicevox_engine/library_manager.py	92	5
voicevox_engine/metas/Metas.py	34	0
voicevox_engine/metas/MetasStore.py	18	6
voicevox_engine/metas/init.py	2	0
voicevox_engine/model.py	162	9
voicevox_engine/morphing.py	71	46
voicevox_engine/part_of_speech_data.py	5	0
voicevox_engine/preset/Preset.py	13	0
voicevox_engine/preset/PresetError.py	2	0
voicevox_engine/preset/PresetManager.py	80	2
voicevox_engine/preset/init.py	4	0
voicevox_engine/setting/Setting.py	11	0
voicevox_engine/setting/SettingLoader.py	17	0
voicevox_engine/setting/init.py	3	0
voicevox_engine/tts_pipeline/init.py	3	0
voicevox_engine/tts_pipeline/acoustic_feature_extractor.py	27	0
voicevox_engine/tts_pipeline/kana_converter.py	88	1
voicevox_engine/tts_pipeline/mora_list.py	7	0
voicevox_engine/tts_pipeline/text_analyzer.py	146	6
voicevox_engine/tts_pipeline/tts_engine.py	190	8
voicevox_engine/user_dict.py	145	12
voicevox_engine/utility/init.py	5	0
voicevox_engine/utility/connect_base64_waves.py	37	0
voicevox_engine/utility/core_version_utility.py	8	1
voicevox_engine/utility/mutex_utility.py	13	0
voicevox_engine/utility/path_utility.py	35	9
voicevox_engine/utility/run_utility.py	10	7
TOTAL	2289	728

[Hiroshiba]

プルリクありがとうございます！
ちょっと先に気になったとこだけ…！

[Hiroshiba]

ほぼLGTMです！！

コメントに関してよく考えてレビューしてみました。
コードに書いてあることでもコメントがあったほうが嬉しいと思う人もいると思うので、本当に難しいなと思いました。

[tarepan]

@Hiroshiba
全指摘箇所の反映・テストパスを確認しました。Re-Re-review よろしくお願いします。

[Hiroshiba]

LGTM！！！

コメントの方針を少し考えてみました。こんな感じですかねぇ･･･。

1. コメントはコードを読みやすくするために書かれる
2. 短くわかりやすいコメントの方が良い
3. コードから読み取れない開発者の大事な意図は書いた方が良い
4. あったほうが読む時間の短縮になる説明は書いても良い
5. リーダブルコードの考え方はよく引用します