疑問点: Diceクラスのrollメソッドの仕様について

[大山]

cw/dice.pyのDice.rollメソッドについて、1面ダイスを振ったとき(sided=1のとき)の動作について疑問があります。 現状、1面ダイスを何回振ろうが戻り値が必ず1になっていますが、この仕様で正しいのでしょうか？1面ダイスをt回振った時の戻り値は1*tになるのではないのかなと疑問に感じています。 現状で正しい場合は、恐縮ですがコード中にその旨のコメントがあると後進のものにとっては大変ありがたいです。

[k4nagatsuki]

いえ、これはバグです。仰る通り、これは1*timesを返すのが正しいはずです。おそらくたまたま問題になるような使い方をしている箇所が無かったので発覚しなかったのでしょう。

調べてみたところ実際にsidedに1を渡していた箇所は無く、sidedが変数になっている箇所も全てtimesが1だったので無害なはずですが、念のためにpull request #1902で修正しました。

追い追いマイナス値のチェックも入れておいた方がよさそうですね。

[大山]

k4nagatsuki様。早速のご対応ありがとうございました。