ダンジョンズ＆ドラゴンス第５版、武器の両手持ちのダメージに対応

[Faceless192x]

D&Dの第５版、パラディンとファイターが武器を両手持ちした時のダメージのリロールに対応するコマンドを実装してみました。

・両手持ちのダメージ　2HnDx[m]
　n:ダイスの個数
　x:ダイスの面数
　m:+-修正。省略可。
　パラディンとファイターの武器の両手持ちによるダメージダイスの1,2の出目の振り直しを行います。

判り易く、通常のダイスコマンド(Dコマンド)に2Hを頭に足せば両手持ちになります。

ご確認の後、問題なければ取り込みをお願いします。

[ysakasin]

Suggested change

	if new_dice.!empty?
	if new_dice.empty?

記述ミスかと思いますので修正お願いします

[Faceless192x]

if じゃなくて unless を使った方が良いということでしょうか？
実は素では「if new_dice.length > 0」と書いていたらrubocop先生に「そこは .!empty? だろ」と訂正されたのでしたｗ

[ysakasin]

いえ、単に文法上 .!empty? という記法がないです。そのためテストも全て失敗しています。

if !new_dice.empty?が正しかったんですね。 unless と if ! のどちらでも問題ないです、

[Faceless192x]

なんと！文法上存在しないものがrubocopから提示されるとは考えてもみなかったですね。
そういうこともあるんだなぁ。

[codecov]

Codecov Report

Patch coverage: 97.29% and no project coverage change.

Comparison is base (c8c76b9) 95.43% compared to head (232e67d) 95.44%.

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #612   +/-   ##
=======================================
  Coverage   95.43%   95.44%           
=======================================
  Files         332      332           
  Lines       18589    18620   +31     
  Branches     4825     4833    +8     
=======================================
+ Hits        17741    17772   +31     
  Misses        848      848           

Impacted Files	Coverage Δ
lib/bcdice/game_system/DungeonsAndDragons5.rb	97.87% <97.29%> (+0.59%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[Faceless192x]

先日D&D5edをプレイしていて、ファイターの類型などによって武器攻撃時のクリティカル値が変動することを知りました。
慌ててATコマンドをクリティカル値を指定出来るように改修しました。

    ・攻撃ロール　AT[x][@c][>=t][y]
    　x：+-修正。省略可。
    　c:クリティカル値。省略可。
    　t:敵のアーマークラス。>=を含めて省略可。
    　y:有利(A), 不利(D)。省略可。
    　ファンブル／失敗／成功／クリティカル を自動判定。

内容をご確認ください。

[ysakasin]

@Faceless192x ありがとうございます。マージしました。

両手持ちコマンドの追加と攻撃ロールのクリティカルの件は、それぞれ独立して議論できることだと思います。次回このようなケースがあれば別々のPull Requestにしていただけると助かります。

[Faceless192x]

了解しました。
以後気をつけます。