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

ゴブリンスレイヤーTRPGのダイスボット追加 #251

Merged
merged 10 commits into from
Jul 26, 2020

Conversation

KaoruKaneko
Copy link
Contributor

ゴブリンスレイヤーTRPG」(GoblinSlayer, https://ga.sbcr.jp/sp/goblin_slayer_trpg/ )のダイスボットを追加しました。

ルール

  • 判定
    達成値(=基準値+2d6)が目標値以上の場合は成功、未満の場合は失敗です。6ゾロと1ゾロはそれぞれ大成功と大失敗で、これらの場合は達成値と目標値の大小にかかわらず成功/失敗となります。

  • 因果点
    シナリオ開始時にGMが因果点を設定します。初期値はシナリオごとに異なります。
    PLは因果点を使用する「祈念」の成功で有利な効果を得ることができ、祈念は2d6(+ボーナス値)の出目が因果点以上で成功します。また祈念することで因果点が1点上昇します。
    祈念の内容は複数あり、2d6を実行せず因果点を3点上昇させることで効果を得るものもあります。

  • 命中判定の効力値によるボーナス
    武器攻撃は以下の手順で行われます。
     命中判定 → ダメージ決定ロール
    このとき効力値(命中判定の達成値+ボーナス値)に応じてダメージ決定ロールにボーナスがつきます。

ご確認、ご対応よろしくお願いします。

@codecov
Copy link

codecov bot commented Jul 26, 2020

Codecov Report

Merging #251 into master will increase coverage by 0.14%.
The diff coverage is 98.15%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #251      +/-   ##
==========================================
+ Coverage   87.86%   88.00%   +0.14%     
==========================================
  Files         220      224       +4     
  Lines       22942    23269     +327     
==========================================
+ Hits        20158    20478     +320     
- Misses       2784     2791       +7     
Impacted Files Coverage Δ
src/diceBot/DiceBotLoaderList.rb 100.00% <ø> (ø)
src/diceBot/Fiasco_Korean.rb 96.55% <96.55%> (ø)
src/diceBot/GoblinSlayer.rb 97.05% <97.05%> (ø)
src/diceBot/StratoShout_Korean.rb 97.72% <97.72%> (ø)
src/diceBot/DoubleCross_Korean.rb 99.08% <99.08%> (ø)
src/diceBot/Nechronica.rb 97.59% <100.00%> (+0.12%) ⬆️
src/diceBot/VampireTheMasquerade5th.rb 98.50% <100.00%> (ø)
... and 1 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 88deb43...9bd738d. Read the comment docs.

Copy link
Member

@ysakasin ysakasin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

要変更点があるので下記を確認してください。

Copy link
Member

@ysakasin ysakasin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

多くのセッションツールではインスタンスを使い回すようにはなっておらず、コマンド1回ごとにインスタンスが作られるようになっています。どどんとふ、ココフォリア 、BCDice-APIもそのようになっています。BCDiceとしても、永続的な状態を持つことは避けたいです。

ですから、セッションの共通リソースをインスタンス変数として共有するという方策は実施することができません。因果点はコマンドの末尾につけるなどで対応するようにしてください。

因果点はコマンドの末尾に $ で指定するのが妥当かなと思いますがどうでしょうか。たとえば MCPI+2$10 などです

@ysakasin
Copy link
Member

GS MCPI とですが、基準値と幸運は実質2D6への修正値として扱うことができるので、GS+2MCPI+2といった記述をサポートした方が良いと思います。

また、現状だと正規表現に文末の指定がされていないため、GS>=5+5などと入力されたときにGS>=5と認識されてしまいます。目標値の四則演算にまで対応する必要はないので、正規表現の末尾指定$を入れると良いと思います。

@ysakasin ysakasin added the new dicebot 新システムの対応 label Jul 26, 2020
@KaoruKaneko
Copy link
Contributor Author

レビューありがとうございます。
ご指摘頂いた箇所を修正しました。
改めてご確認をお願いします。

src/diceBot/GoblinSlayer.rb Outdated Show resolved Hide resolved
src/diceBot/GoblinSlayer.rb Outdated Show resolved Hide resolved
src/diceBot/GoblinSlayer.rb Outdated Show resolved Hide resolved
src/diceBot/GoblinSlayer.rb Outdated Show resolved Hide resolved
src/diceBot/GoblinSlayer.rb Outdated Show resolved Hide resolved
@KaoruKaneko
Copy link
Contributor Author

再度のご指摘ありがとうございます。
改めて考えたところ、2項目については不要だと感じましたので削除しました。

Copy link
Member

@ysakasin ysakasin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OKだと思います! 寄稿ありがとうございます!!

@ysakasin ysakasin merged commit 5ab4a30 into bcdice:master Jul 26, 2020
ysakasin added a commit that referenced this pull request Aug 30, 2020
* ゴブリンスレイヤーTRPGのダイスボット追加

* ゴブリンスレイヤーTRPGのダイスボット追加

* ゴブリンスレイヤーTRPGのダイスボット追加

* 改行コードを変更(CRLF→LF)

* レビュー指摘に対応

* Add files via upload

* レビュー指摘に対応

* レビュー指摘に対応

不要な関数を削除

* Add files via upload

* 正規表現を修正

Co-authored-by: SAKATA Sinji <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new dicebot 新システムの対応
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants