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

キルデスビジネス:難易度と補正値の区切り文字にカンマも使いたい #272

Closed
yuGit1985 opened this issue Sep 5, 2020 · 4 comments · Fixed by #346
Assignees
Labels
bug バグってる! enhance dicebot ダイスボットへの機能追加

Comments

@yuGit1985
Copy link

キルデスビジネスのJDコマンドにおける難易度と補正値の区切りは+か-になっているが、
これだと負数が展開された時に符号が重なってしまう場合があるため、使いづらい場合がある

それらを踏まえ、現状のコマンドに加えて下記のものも使えるようにしたい
JD5,3,1
JD7,-1,3

@ysakasin
Copy link
Member

ysakasin commented Sep 5, 2020

提案ありがとうございます

提案いただいた形式だと、単に数値が羅列されるため、どの数値がどんな意味を持つのか一見わかりづらい気もします。
負数を展開した時の挙動を直せば十分だったりはしますでしょうか?

ファンブルの指定は最近のBCDiceでは #[数字] とするのが標準になりつつあるので、 JD5#1 というような書き方ができるようにしようと思いますが、こちらについてはどうですか。

@yuGit1985
Copy link
Author

yuGit1985 commented Sep 5, 2020

最初の案だとコマンドから意味が分かりづらいのはその通りですね。
実装量、影響範囲も考えると、負数展開時の挙動を修正するのがベストかと思われます。

不勉強で申し訳ないのですが、#にするのが主流なのですね。
そうであれば合わせて実装すると直感的なコマンドになるので良いと思います。

@ysakasin
Copy link
Member

ysakasin commented Sep 5, 2020

了解です。

  • 負数が展開できるようにする
  • ファンブル指定は ,[数字]を残しつつ #[数字]でできるようにする

この二点を適用します。

現在BCDiceの次期バージョンに向けてプロジェクト全体で大幅な変更を行っていて、次回リリースは2021/01/01以降になる予定です。その点、ご了承ください。

@ysakasin
Copy link
Member

JD5+(1-2),4のようなコマンドが動作するように修正しました。次回リリースをお待ちください。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug バグってる! enhance dicebot ダイスボットへの機能追加
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants