yukieiji / ExtremeRoles

AmongUs Mod for Advanced User(+90 roles, +1300 options, and Cosmic addon)
GNU General Public License v3.0
59 stars 29 forks source link

システム予約語の追加・導入 #341

Open yuhgao opened 9 months ago

yuhgao commented 9 months ago

追加する機能の詳細 / Feature details

翻訳テキストファイルで使用可能な予約語を追加する。予約語は現在の {0} のように使用でき、予め書式タグも記入されている。

機能を追加するメリット / Benefits of adding features

書式タグを導入した記述を中心に可読性の向上と表示方法の統一、および作業の効率化が期待できる。 現状ではすべてのワードに対して手動でタグを挿入している関係で、一箇所を変えるとなるとすべてを虱潰しに変更したり、手記入だと一部書式がぶれたりしてしまう。 導入により作業量が三割程度減るものと思われる。翻訳においてもマスター部分を変更するだけで複雑な書式が簡単に流用できるようになる。

デメリット:翻訳ファイルを触る人がドキュメントを理解していないといけなくなる。( yukieiji/ExtremeRoles.Beta#2 新書式の導入においても起こることではある)

yukieiji commented 9 months ago

翻訳テキストファイルで使用可能な予約語を追加する。予約語は現在の {0} のように使用でき、予め書式タグも記入されている。

{0}みたいなやつって文字列取ってくる時にstring.Format("{0}秒", 10)ってやってて文字列の中でも重ための処理なんですよね・・・ これを全システムに共通してやると速度がガッツリ落ちそうな気がしなくもないです

もし全体的にやるってなった場合別ファイルで用意してPythonでコンバートする時に文字列埋め込むくらいですかねー

yuhgao commented 9 months ago

なるほど了解です。そうするとナイス・イビルの判定をして表示など、ゲーム内状態の取得は負荷かかりそうな感じでしょうか。 それ以外であればコンバートで対応できそうですね。

yukieiji commented 9 months ago

そうするとナイス・イビルの判定をして表示など、ゲーム内状態の取得は負荷かかりそうな感じでしょうか。

これどういうことです?

yuhgao commented 9 months ago

これはtextrevampの話になりますが、たとえばCombination枠であるゲッサー・ムーバーなどは「陣営」の枠をクルー・インポスター併記することで役職が陣営固定でない点を解消しています。可能であればナイス・イビルにアサインされたときの判定を用いて表示できれば、この併記が必要なくなる他、他のロール表示も含めて表記が統一できるのでは、と。

yukieiji commented 9 months ago

これはtextrevampの話になりますが、たとえばCombination枠であるゲッサー・ムーバーなどは「陣営」の枠をクルー・インポスター併記することで役職が陣営固定でない点を解消しています。可能であればナイス・イビルにアサインされたときの判定を用いて表示できれば、この併記が必要なくなる他、他のロール表示も含めて表記が統一できるのでは、と。

これ確か出来るけどめんどいんですよね、以前イビルゲッサーがサーヴァントされた時にナイスゲッサー表記になってたと思うんですが内部的な陣営ってころころ変わるから一概簡単に取得できないので初期化時に強制的にやっててそれと似たようなことをやる必要が出てきますね

yukieiji commented 9 months ago

ここらへんの翻訳をこれを本当にガチ目にやりたいのであれば役職クラスを根本的に書き直す必要があるとは思いますね

yuhgao commented 9 months ago

うーむ、流石にそこまでやるものではないですね。現実的に考えますとやはりPythonのコンバート時に入れ替えられる部分を予約語とする形が妥当でしょうか。

yukieiji commented 9 months ago

いつかは書き直さないと限界が出てくるとは思うのですが現状は問題ないからね・・・ Python変換時に入れるにしても予約語を自動で検出はできないので自分自身で作る必要はありますね

github-actions[bot] commented 8 months ago

自動:このチケットは30日間更新がないため「凍結/Freeze」ラベルが付与しました。ラベルを削除するか何らかのコメントがない場合、3日後に自動クローズされます。
Automate:This issue is 凍結/Freeze because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 3 days

github-actions[bot] commented 6 months ago

自動:このチケットは30日間更新がないため「凍結/Freeze」ラベルが付与しました。ラベルを削除するか何らかのコメントがない場合、3日後に自動クローズされます。
Automate:This issue is 凍結/Freeze because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 3 days

github-actions[bot] commented 5 months ago

自動:このチケットは30日間更新がないため「凍結/Freeze」ラベルが付与しました。ラベルを削除するか何らかのコメントがない場合、3日後に自動クローズされます。
Automate:This issue is 凍結/Freeze because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 3 days

github-actions[bot] commented 4 months ago

自動:このチケットは30日間更新がないため「凍結/Freeze」ラベルが付与しました。ラベルを削除するか何らかのコメントがない場合、3日後に自動クローズされます。
Automate:This issue is 凍結/Freeze because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 3 days

github-actions[bot] commented 3 months ago

自動:このチケットは30日間更新がないため「凍結/Freeze」ラベルが付与しました。ラベルを削除するか何らかのコメントがない場合、3日後に自動クローズされます。
Automate:This issue is 凍結/Freeze because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 3 days

github-actions[bot] commented 2 months ago

自動:このチケットは30日間更新がないため「凍結/Freeze」ラベルが付与しました。ラベルを削除するか何らかのコメントがない場合、3日後に自動クローズされます。
Automate:This issue is 凍結/Freeze because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 3 days

github-actions[bot] commented 1 month ago

自動:このチケットは30日間更新がないため「凍結/Freeze」ラベルが付与しました。ラベルを削除するか何らかのコメントがない場合、3日後に自動クローズされます。
Automate:This issue is 凍結/Freeze because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 3 days

github-actions[bot] commented 3 days ago

自動:このチケットは30日間更新がないため「凍結/Freeze」ラベルが付与しました。ラベルを削除するか何らかのコメントがない場合、3日後に自動クローズされます。
Automate:This issue is 凍結/Freeze because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 3 days