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

[フルフェイス] (UPDATE) 一般判定の成否と戦闘判定のダメージ計算を実装しました #679

Merged
merged 5 commits into from
Feb 25, 2024

Conversation

Faceless192x
Copy link
Contributor

フルフェイス( https://booth.pm/ja/items/5302609 )のダイスに「6の目の数」「成否」と「ダメ―ジ計算」を入れて欲しいと要望があったので追加しました。

    ■判定 x+bFF<=[a[,t]]d   x:ヒート(省略時は3) b:判定修正 a:能力値 t:難易度(省略可) d:基本ダメージ(省略可)

    例)FF<=[2]:   能力値2で判定し、その結果(成功数,1の目の数,6の目の数,バースト)を表示。
       6FF<=[3]:  ヒート6,能力値3で戦闘判定し、その結果( 〃 )を表示。
       8+2FF<=[3]:ヒート8,判定修正+2,能力値3で戦闘判定し、その結果( 〃 )を表示。
       FF<=[2,1]: 能力値2,難易度1で判定し、その結果(成功数,1の目の数,6の目の数,成功・失敗,バースト)を表示。
       6FF<=[3]1:  ヒート6,能力値3で戦闘判定し、その結果(成功数,1の目の数,6の目の数,ダメージ,バースト)を表示。

と若干コマンドの説明がうるさくなりました。

お手隙の際に、ご確認ください。

Copy link

codecov bot commented Feb 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.73%. Comparing base (1e8e6b9) to head (dd74d7f).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #679   +/-   ##
=======================================
  Coverage   95.73%   95.73%           
=======================================
  Files         353      353           
  Lines       20549    20568   +19     
  Branches     5393     5400    +7     
=======================================
+ Hits        19672    19691   +19     
  Misses        877      877           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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.

Pull Requestありがとうございます。

FF コマンドの後方互換性がなくなってしまったので、互換性のある形式に変更してください。互換性のない変更が許容できるのは、以前の形式がゲームシステムのルールに則っていないなど、バグを修正する場合のみです。

@Faceless192x
Copy link
Contributor Author

Faceless192x commented Feb 17, 2024

了解しました。

ちょっと実装を試行錯誤してみます。

@Faceless192x
Copy link
Contributor Author

・FFコマンドを後方互換を維持する方向で決定しました。
・ダイスロールの結果をソートするようにして、結果を確認しやすくしました。

以上、お手隙の際にご確認ください。

@ysakasin ysakasin merged commit cc77c7e into bcdice:master Feb 25, 2024
8 checks passed
@ysakasin
Copy link
Member

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

@Faceless192x Faceless192x deleted the update/FullFace branch February 25, 2024 23:29
@Faceless192x
Copy link
Contributor Author

ありがとうございます!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants