search

bcdice / BCDice

The most popular TRPG dice command engine in Japan
https://bcdice.org
BSD 3-Clause "New" or "Revised" License
138 stars 183 forks source link

prefixesの挙動が名前と解離している #206

Closed ysakasin closed 4 years ago

ysakasin commented 4 years ago

ゲームシステム固有のコマンドを使えるようにするためにDiceBot.setPrefixesを用いるが、接頭辞として見るのではなく文字列全体がマッチするかチェックしている。

https://github.com/bcdice/BCDice/blob/1d33af43ba67560b4514056f553de97e8f37905e/src/diceBot/DiceBot.rb#L29-L41

そのため、CC<=10というコマンドを作りたい場合 setPrefixes("CC") だと認識しない。setPrefixes("CC.*")やsetPrefixes("CC<=\d+")のようにする必要がある。

これは名前に実態が促しているとは言えないので、修正する必要がある。上記例でsetPrefixes("CC") が認識されるように挙動を変更することで対応する。

ysakasin commented 4 years ago

a99ea0edbad9680c0c836a21994aa2707ff5f482