xpjp / xpfiat-bot

XP JPのディスコード上で動作しているボットです。 XPをJPYに変換したりする機能があります。
MIT License
20 stars 5 forks source link

降雨量コマンド(?how_rain)の修正 #85

Closed serisia closed 6 years ago

serisia commented 6 years ago

何を解決するのか

?how_rainにおいて、空rainがカウントされている問題に対応。 伴って、未確定なrainを区別して出力するようにした。

・Botがまだ反応していない発言の区別 ・Botがエラーを返したコマンドの区別  ・min rain amount is ***  ・Insufficient Balance ・出力を「確定」「Bot待ち」の2種類に変更

レビューポイント

Xp-Botの発言に対しての反応を追加している形なので、 そのあたりで漏れなどがないか? (数量指定が低い、残高がない、という問題は対応した) 出力メッセージが適切かどうか?

serisia commented 6 years ago

自分で見てたらチェックがあまあまですね・・・ 150行目に配列の長さの、157行目にhas_key?のチェックを追加します

また、取得件数が有限であることにより、同じユーザーが短期にrainを繰り返した場合に別のrainを見てしまう可能性があります レアケース、また統計出力だけの機能なのでそこは割り切っていいと思っていますが、問題になるようであれば取り下げます

serisia commented 6 years ago

重大な問題が、これ今Brewing Stormじゃないですね 帰ってから実際のログで試します、申し訳ない 何やってんだか・・・

serisia commented 6 years ago

実装の方針について、BotのRainedの後の値をとるかどうか(->ユーザー辺りの降雨量になる)という案が出たため、一度Issueを立てて議論します。