-
Notifications
You must be signed in to change notification settings - Fork 20
[SW2] 武器攻撃の追加オプションにおいて、オプション同士を組み合わせる記法を追加 #343
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
Open
ViVi-shark
wants to merge
12
commits into
yutorize:develop
Choose a base branch
from
ViVi-shark:feature/sw2-add_notation_to_compose_palette_attacks
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
[SW2] 武器攻撃の追加オプションにおいて、オプション同士を組み合わせる記法を追加 #343
ViVi-shark
wants to merge
12
commits into
yutorize:develop
from
ViVi-shark:feature/sw2-add_notation_to_compose_palette_attacks
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…_for_ytchat' into HEAD
…mmand_for_ytchat' into HEAD
93cf97d to
99541ff
Compare
…sw2-add_notation_to_compose_palette_attacks # Conflicts: # _core/lib/sw2/edit-chara-palette-option.pl
…' into feature/sw2-add_notation_to_compose_palette_attacks
Contributor
Author
eaa25e7 to
c0b1b9d
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
前提PR
#286, #338, #341, #377
(※2025/11/23 #377 を追加)
変更内容
武器攻撃の追加オプションにおいて、オプション同士を組み合わせる記法を追加
(例:別途《魔力撃》と《全力攻撃》を定義しておいたうえで、《魔力撃》と《全力攻撃》を組み合わせたオプションを定義する)
背景
「武器攻撃の追加オプション」機能の典型的な利用例は宣言特技であると思われる。
ここで、《バトルマスター》や《変幻自在Ⅰ~Ⅱ》の存在をかんがみると、複数の宣言特技を併用するケースは、ごく自然に発生する。
また一方で、《バトルマスター》や《変幻自在Ⅰ~Ⅱ》があったとしても、リスクの累積を嫌う場合や、《マルチアクション》等の「武器攻撃の追加オプション」に影響しない宣言特技に宣言の権利を割く場合、あるいは《両手利き》や《ファストアクション》等による行動回数の増加を前提として一回一回の攻撃にはひとつの宣言特技しか適用しない場合など、依然としてひとつのみの宣言特技を適用した攻撃をおこなう可能性は、現実的に存在する。
上記のような、「個別にも運用したいし複数を組み合わせても運用したい」というオプション群がある場合、組み合わせての運用のほうでは、個別用に定義したオプションを参照することで、記述と管理を省力化したい。
仕様
|が含まれる」という条件を満たすとき、組み合わせオプションであるとみなす。
組み合わせオプションにおいては、名称を
|で区切った部分文字列を、個別のオプションの名称と仮定して、オプション定義をひととおり検索する。発見された個別オプションの修正値をすべて組み合わせたものとして、組み合わせ用のチャットパレットを出力する。
ユーザー向けのヘルプ
区切り文字
|または|を区切り文字とする。検討過程では
&を区切り文字とする案もあったが、『ダグニア博物誌』掲載の【不死者討滅武技バニシングデス】の秘伝(《キュア&デストロイ》《エナジー&ストライク》《センス&バニッシュ》)のように、データ名≒オプション名に&が含まれるケースがあるため、却下した。(
|がデータ名に含まれるようなケースはいまのところ存在せず、また今後において追加されるとも考えがたい)組み合わせを手動で定義することについて
組み合わせを手動で定義するのは、自動的に組み合わせることには以下のような問題があるため。
例
サンプルのキャラクターシート:
https://yutorize.2-d.jp/ytsheet/sw2.5/?id=2oPNZf
このデータをもとに当PR下でチャットパレットを生成すると、武器攻撃の部分は次のようになる。
備考
魔法の追加オプションについては別途作業する予定です