Skip to content
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

BattleTech: CDxコマンドを改良する #445

Merged
merged 3 commits into from
May 8, 2021
Merged

Conversation

ochaochaocha3
Copy link
Member

バトルテックのメック戦士意識維持ロール(CDx)を改良しました。

  • 2018年発行の「BattleTech: A Game of Armored Combat」p. 44に合わせて、識別子名や処理を改良しました。
  • 結果のテキストを分かりやすくしました。
    • ダメージ6はロールなしで「死亡」、ダメージ1〜5は2D6の成功判定、とやや複雑な処理だったため、最近のゲームシステムを参考にして結果のテキストを詳しくしました。
    • 例1(CD3):7[5,2]>=7 > 成功CD3 > (2D6>=7) > 7[5,2] > 7 > 成功
    • 例2(CD6):死亡CD6 > 死亡
  • 結果に成功、失敗、ファンブルを含めました。CD6 のときはファンブルかつ失敗、CD1CD5 のときは成功または失敗となります。

* 「BattleTech: A Game of Armored Combat」ルールブックp. 44に
  合わせて識別子名と処理を改良する。
* 結果のテキストを分かりやすくする。
* 結果に成功/失敗/ファンブル(死亡時)を含める。
@ochaochaocha3 ochaochaocha3 added the enhance dicebot ダイスボットへの機能追加 label Apr 30, 2021
@ochaochaocha3 ochaochaocha3 requested a review from ysakasin April 30, 2021 03:53
@ochaochaocha3 ochaochaocha3 self-assigned this Apr 30, 2021
@codecov
Copy link

codecov bot commented Apr 30, 2021

Codecov Report

Merging #445 (35579e3) into master (20b9e89) will increase coverage by 0.11%.
The diff coverage is 96.77%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #445      +/-   ##
==========================================
+ Coverage   95.21%   95.33%   +0.11%     
==========================================
  Files         288      289       +1     
  Lines       18698    19193     +495     
==========================================
+ Hits        17803    18297     +494     
- Misses        895      896       +1     
Impacted Files Coverage Δ
lib/bcdice/game_system/BattleTech.rb 99.64% <96.77%> (+0.25%) ⬆️
lib/bcdice/game_system/LiveraDoll.rb 97.56% <0.00%> (-0.97%) ⬇️
lib/bcdice/game_system.rb 100.00% <0.00%> (ø)
lib/bcdice/game_system/ZettaiReido.rb 100.00% <0.00%> (ø)
lib/bcdice/game_system/StarryDolls.rb 100.00% <0.00%> (ø)
lib/bcdice/game_system/Satasupe.rb 98.29% <0.00%> (+0.26%) ⬆️
lib/bcdice/game_system/Skynauts.rb 100.00% <0.00%> (+0.59%) ⬆️
lib/bcdice/game_system/TrinitySeven.rb 98.19% <0.00%> (+0.66%) ⬆️
lib/bcdice/command/parsed.rb 97.72% <0.00%> (+1.06%) ⬆️
... and 2 more

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 20b9e89...35579e3. Read the comment docs.

@ochaochaocha3
Copy link
Member Author

@randomizer はRepeatコマンドで同じインスタンスが使いまわされるので、 Rnadomizer#rand_results から出目の情報を取得すると意図しない状況になることがあります。Randomizer#roll_barabara でロールしてそこから合計値を出力するようにしてください。

おっと、これには気づいてませんでした。そのように変えます。

今回のように roll_sum で合計値とともに各出目も取得することは頻出なので、今後Randomizer側で詳しく結果を得られるメソッドを用意する(互換性維持のため、例えば Randomizer#roll_sum_with_details のような名前で追加する)のを検討してもいいかもしれません。

Repeatコマンドで同じRandomizerインスタンスが使いまわされるので、
`Randomizer#rand_results` から出目の情報を取得すると意図しない状況に
なるかもしれない。
@ysakasin ysakasin merged commit ab6d0c1 into master May 8, 2021
@ysakasin ysakasin deleted the battletech-improve_cd branch May 8, 2021 00:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhance dicebot ダイスボットへの機能追加
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants