[Revulture] 『光砕のリヴァルチャー』のダイスボットを作成

[ViVi-shark]

対象ゲームシステム

　『光砕のリヴァルチャー』

• どらこにあん特設ページ
• 新紀元社
• Amazon

用意した機能

• 【アタック判定】（原則的な処理はルールブック p140 ～ 141 ）
  • ならびに、それに干渉する諸要素

[codecov]

Codecov Report

Merging #427 (c31787e) into master (2df3312) will increase coverage by 0.05%.
The diff coverage is 100.00%.

❗ Current head c31787e differs from pull request most recent head f613a81. Consider uploading reports for the commit f613a81 to get more accurate results

@@            Coverage Diff             @@
##           master     #427      +/-   ##
==========================================
+ Coverage   95.14%   95.20%   +0.05%     
==========================================
  Files         286      288       +2     
  Lines       18471    18648     +177     
==========================================
+ Hits        17575    17753     +178     
+ Misses        896      895       -1     

Impacted Files	Coverage Δ
lib/bcdice/game_system.rb	100.00% <100.00%> (ø)
lib/bcdice/game_system/MonotoneMuseum.rb	98.30% <100.00%> (+1.75%)	⬆️
lib/bcdice/game_system/Revulture.rb	100.00% <100.00%> (ø)
lib/bcdice/game_system/Skynauts.rb	99.40% <100.00%> (ø)
lib/bcdice/game_system/SkynautsBouken.rb	100.00% <100.00%> (ø)
lib/bcdice/game_system/StellarKnights.rb	98.73% <100.00%> (+0.10%)	⬆️
lib/bcdice/game_system/SwordWorld.rb	97.35% <100.00%> (+0.09%)	⬆️
...cdice/game_system/sword_world/transcendent_test.rb	100.00% <100.00%> (ø)

---

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 2df3312...f613a81. Read the comment docs.

[ysakasin]

@ViVi-shark
超速寄稿ありがとうございます。
現在のコードだと、目標値は未指定だが追加ダメージは指定されている状況があり得ますが、その際はどのように扱いますか？

[ViVi-shark]

@ysakasin

目標値は未指定だが追加ダメージは指定されている状況

　ゲームルール上、目標値がないことには追加ダメージを導出できないので、メッセージでそれを明示するようにしました。 a51b0ad

　（ダイスを振らないようにしたのは、現実的なゲームプレイにおいてこれに該当した場合、ユーザーにとって「ヒット数や追加ダメージを人力で算出する」より「コマンドに目標値指定を追加のうえであらためて実行する」ほうが負担がすくないという判断です）

[ViVi-shark]

　あと、ダイス数が 0 となる場合にもメッセージで示すようにしました。 9613d18

　（ゲームルールに「ダイス数を半分（端数切り捨て）にする」効果が存在し、元が 1 個の判定にそれが適用されると発生するケース）

[ysakasin]

@ViVi-shark ナイスです:+1: どちらとも現状でベストな挙動だと思います。
コードの細かいところで気になる部分があるので、残りをこちらで修正してから取り込もうと思います。

[ViVi-shark]

　承知しました、よろしくお願いします。

[ysakasin]

LGTM!