[StellarKnights] アタック判定のダイス数指定において加算・除算をあつかえるように

[ViVi-shark]

内容

• アタック判定のダイス数指定において加算 + をあつかえるように
• アタック判定のダイス数指定において除算 / をあつかえるように
  • なお、現行ルールのかぎりではこれが必要なケースは端数切り捨てのはずなので、そのようにあつかっている
• 上記にともない、括弧による計算順序の指定ができるように

動機（必要性）

• 加算については、「ブーケ」ルールによってきわめて頻繁に発生する（セッションによっては、ＰＣ側のすべてのアタック判定に臨時の加算が発生する場合すらある）
• 除算についても、それなりにメジャーな（複数のサンプルキャラクターが取得している）スキルの効果として存在し、わりと頻繁に発生する

[codecov]

Codecov Report

Merging #465 (0b2a84a) into master (7c0b4e7) will increase coverage by 0.08%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #465      +/-   ##
==========================================
+ Coverage   95.26%   95.34%   +0.08%     
==========================================
  Files         290      290              
  Lines       18783    18790       +7     
==========================================
+ Hits        17893    17915      +22     
+ Misses        890      875      -15     

Impacted Files	Coverage Δ
lib/bcdice/game_system/StellarKnights_Korean.rb	100.00% <ø> (ø)
lib/bcdice/game_system/StellarKnights.rb	98.86% <100.00%> (+0.09%)	⬆️
lib/bcdice/common_command/upper_dice/parser.rb	86.02% <0.00%> (+3.67%)	⬆️
lib/bcdice/common_command/reroll_dice/parser.rb	84.32% <0.00%> (+3.73%)	⬆️
lib/bcdice/common_command/barabara_dice/parser.rb	85.84% <0.00%> (+4.71%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7c0b4e7...0b2a84a. Read the comment docs.

[ysakasin]

@ViVi-shark ありがとうございます。現状でも (4+2)SK のように記述できますが、これだとやはり不便ということなのでしょうか？ 4+2X のようなスタイルは (4)+(2X) の意味に捉えられやすいと思うので、結構悩ましいですね。といいつつ光彩のリヴァルチャーで採用してしまいましたが。

[ViVi-shark]

現状でも (4+2)SK のように記述できますが、これだとやはり不便ということなのでしょうか？

　計算順序の指定のようなやむを得ないケース（ (a+b)/c ）はともかくとして、そうでない場合に括弧を入力するのは不自由を感じます。

4+2X のようなスタイルは (4)+(2X) の意味に捉えられやすいと思うので、結構悩ましいですね

　仰有ることは理解できます。
　BCDice の標準機能における、 xDy を含む計算式の場合（ ex: 3+2D6 ）との混同ですよね。（一般的な数学ライクな書式との混同は考慮しなくてよいとおもっています。それを考慮するなら標準機能の xDy や xBy がすでにおかしいので）

　ただ、現行のいろいろダイスボットをみるかぎり、 BCDice::Command::Parser （※ステラナイツはまぁ使ってないのですが、ニュアンスの話です）ライクな書式の command の左側はだいたい本質的に数値をとるはずで、そして結構な割合でそこの数値には計算を適用したくなるはずなので（わざわざ数値指定できるようにしてある以上は増減が頻繁な傾向にある）、許容してしまうほうが全体にわたって利益がおおきそうな気がしています。

　（本件の観点だと、いまさら言ってもしょうがないのですが、 GardenOrder や Amadeus や SteamPunkers みたいな（左側に値をとらない）構文のほうが丸い場合が多そうですね……）