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.

Sign up for GitHub

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

Jump to bottom

フィアスコダイスボット追加 #189

Merged
merged 9 commits into from
May 15, 2020
Merged

フィアスコダイスボット追加 #189

merged 9 commits into from
May 15, 2020

Conversation

FredGreenfield
Copy link
Contributor

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

キャラ作成と転落要素のためにダイスの目を分類するコマンドと、転落選択プレイヤーの決定、残響結果表の決定のための白黒ダイスの差分値を求めるコマンドを実装しました。

@ysakasin
Copy link
Member

@FredGreenfield 寄稿ありがとうございます。

コマンド

FSWxBx よりもDiscordでもともと提案されていた WxBx の方が入力しやすいと思いましたが、問題ありましたでしょうか？

出力

FSWxBxの時に、黒と白それぞれの合計も表示した方がわかりやすいと思うのですが、どうでしょうか。例えば、以下のような感じです。

Fiasco : 白ダイス6[1,2,3] 黒ダイス15[4,5,6] ＞ 黒9

あとは、結果中に「白ダイス」の「ダイス」は省略しても良い気もします。これはルールブックの書き方次第ですが。

Fiasco : 白6[1,2,3] 黒15[4,5,6] ＞ 黒9

@FredGreenfield
Copy link
Contributor Author

FredGreenfield commented May 14, 2020 •
edited
Loading

確認、ありがとうございました。

コマンド

FSWxBx よりもDiscordでもともと提案されていた WxBx の方が入力しやすいと思いましたが、問題ありましたでしょうか？

問題ありません。
システム略称的なものがプレフィックスになる慣習があるのかな、と勝手に思いこんでいました。申し訳ありません。
WxBxに修正いたしますが、FSSx(FiascoSetupぐらいのつもりです)の方もSxなどのようにした方がよろしいでしょうか？

出力

FSWxBxの時に、黒と白それぞれの合計も表示した方がわかりやすいと思うのですが、どうでしょうか。例えば、以下のような感じです。

了解です。こちらも修正いたします。

あとは、結果中に「白ダイス」の「ダイス」は省略しても良い気もします。これはルールブックの書き方次第ですが。

ルールでは「白のダイス5個」のような書き方ですが、合計は黒1などのように記載しています。
ダイスがついた方がわかりやすいかな、と思ったのですが合計としてみるなら仰る通りかと思いますので、修正いたします。

@ysakasin
Copy link
Member

@FredGreenfield

WxBxに修正いたしますが、FSSx(FiascoSetupぐらいのつもりです)の方もSxなどのようにした方がよろしいでしょうか？

FSx が良いと思います。 Sx だとシークレットコマンドとの衝突の恐れがあります。

@FredGreenfield
コマンドの見直し(FSSx→FSx,FSWxBx→WxBx)
04a74f2 
WxBxコマンド時のダイス表記を削除し、白の合計、黒の合計を表記するように修正
ysakasin
ysakasin approved these changes May 15, 2020
View reviewed changes
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.

よさそう！
Looks Good to Me!

@ysakasin ysakasin merged commit 8fde22e into bcdice:master May 15, 2020
@FredGreenfield FredGreenfield deleted the fiasco_dicebot branch May 15, 2020 21:38
ysakasin pushed a commit that referenced this pull request Aug 30, 2020
@FredGreenfield
フィアスコダイスボット追加 (#189)
1da010a 
* フィアスコダイスボットを作成。

* ・関数名修正

* 関数名修正

* フィアスコダイスボット追加

* コマンドの見直し(FSSx→FSx,FSWxBx→WxBx)
WxBxコマンド時のダイス表記を削除し、白の合計、黒の合計を表記するように修正
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ysakasin ysakasin ysakasin approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@FredGreenfield @ysakasin