PHP の mt_rand() は一貫して壊れている(consistently broken)らしい - 唯物是真 @Scaled_Wurm

PHPでMersenne Twister法で擬似乱数を生成する関数のmt_rand()にバグがあり出力がおかしい、という話が流れてきておもしろかったので簡単にまとめておく kusanoさんがmt_rand()の実装に9年以上前から1文字違いでバグがあったことを見つけて、数ヶ月後にマージされる(追記: 正確には、PHP版の実装が他と異なっているのは前から知られていたらしい*1 ) PHPに送った1文字修正するプルリクエストがマージされた🎉 mt_rand()の返す値が元のメルセンヌツイスタと異なっていた。https://t.co/Z5WJhHVyNd— kusanoさん@がんばらない (@kusano_k) February 17, 2016 その後、生成される擬似乱数列が変わってしまうので、後方互換性を壊す変更は議論してからmergeすべきということでrevertされるこの前マージされた

[sucrose]

プルリクエストを書かれたkusano_kさんが解説記事を書かれたので、そちらもご参照ください http://kusano-k.hatenablog.com/entry/2016/02/20/223229

[gabill]

perl「やり方はひとつじゃない」Python「たったひとつの冴えたやり方があるはずだ」Ruby「プログラミングを楽しく」PHP「確かに壊れてる、しかし一貫して壊れてる」

[VoQn]

悪いけど「実に」PHPらしいと思うよこの展開…

[psne]

一貫して壊れている物を直したら結果が変わってしまう(例えば再現性のある乱数の結果が欲しい条件でmt_srand(1)としたコードに影響する、ドルアーガの塔のMAP)、と言われると議論は必要

[bamch0h]

そうだ！mt_rand2()を作ろう！

[Mattyan]

これ、もしかしたらいま各所で騒がれているソシャゲのガチャのレアアイテムが出ない問題に関わっているかもしれない。なぜかってどのカードが出るかの抽選は妄想しているような複雑なことをせずmt_randで引いてるから

[sin20xx]

そもそもマニュアルにも厳密な乱数ではないから使う場面はちゃんと考えろとあり、且つ、先人たちからも現行版でもmt_randは非推奨なのでこの関数自体の現状の立ち位置を考えると最初の指摘はもっともだと思うよ？

[hnw]

これPHPがクソって話になるのは毎度皆さん欲しい結論があるんだなーとしか。まだPR出てから3日目で、対応方針決まってない段階ってことでしょ / 元のPRで議論再開してた。7.1から入るくらいが妥当な着地点かしら。

[hakaikosen]

7のリリース前に発見できてれば7で置き換えただろうねぇ。

[qtamaki]

そびえ立つく（ry

[t_trad]

PHPのどこを押せば後方互換性なんて言葉が出てくんだよ

[xll]

“(拙訳: 確かに壊れている。しかし、少なくとも一貫して壊れている)”

[deep_one]

「一貫して壊れている」事の意味はある意味分かる（笑）　普通はそのコマンドを非推奨にして凍結して、別コマンドをたてる。／コメントで実際に非推奨になっているらしいことを知ったので、対応は全くもって正しい。

[trashtoy]

後方互換性とか理由にしてるけど, 以前マイナーバージョンアップなのにデフォルトの挙動を変えてきた array_unique http://d.hatena.ne.jp/hnw/20090228 のことは忘れない

[KaeruHeika]

consistentlyは一貫して、堅実に、矛盾なく、というニュアンスのようなので。

[vanbraam]

もうmt_randをobsoletedにして,nmt_rand (not Mersenne twister)という従来動作版とrmt_rand (right MT)という正常版とを作って,2年くらい経ったらmt_randを廃止すればいいのでは

[hevohevo]

「一貫して壊れている」というより、「10年使い続けたらそれはもう正しい」という印象。PHPらしいエピソード。

[hirata_yasuyuki]

おおう… Browsing: “PHP の mt_rand() は一貫して壊れている(consistently broken)らしい - 唯物是真 @Scaled_Wurm”

[UDONCHAN]

『少なくとも一貫して壊れている』

[solidstatesociety]

クソだけど死なない！

[taruhachi]

phpだからこその問題があるとしたらそれはまだ使われているからという問題だけのような気が、、、。

[bouzuya]

互換性なら仕方ないな。

[AmaiSaeta]

「後方互換の問題が在るから議論要るよね」は尤もだと思うよ（ソフトウェアで生成できる乱数はあくまで疑似乱数な訳で）。

[nilab]

「Mersenne Twister法で擬似乱数を生成する関数のmt_rand()にバグ」「kusanoさんがmt_rand()の実装に9年以上前から1文字違いでバグがあったことを見つけて、数ヶ月後にマージされる」

[nnhrsk]

mt_rand()って最初に知った時、mtってわざわざ付いているのに違和感感じた

[you21979]

一貫して壊れているは新しい

[kanflu]

擬似乱数であって、生成結果が予測可能なのだからそれに依存した用例もありうる（均等分布することが重要なので、暗号に使うわけではない）ので、パッチバージョンでは修正できないという（あたりまえの）話っす。

[t2y-1979]

意味がわからないよ

[ngsw]

この対応方法実は嫌いじゃないし、むしろ好きかもしれない。「PHPは歯ブラシである」という割り切り方から言えば、言語としても筋の通ったかっこいい対応方法と思える。

[Peranikov]

PHPの情報は意図的に遮断しているのについ読んでしまった。くそ。

[bootJP]

わらえないけど。わらってしまう PHP の mt_rand() は一貫して壊れている(consistently broken)らしい - 唯物是真 @Scaled_Wurm

[jojo800]

もはやバグではなく仕様なのでは

[rokujyouhitoma]

このバグよく見つけたな

[t_yano]

MicrosoftはWindowsだかOfficeだかの、昔のバグを、互換性維持の為に互換モードでは正確に再現している、とかいう話を思い出した。

[abyssgate]

笑うしかねえｗ

[kazoo_oo]

buggy but can't fix な世界。つらい。

[butasan2007]

これだから、「これだからPHPは...」と言われちゃうんだよね...

[georgew]

適当に斜め読みしたところによると、後方互換性を損なう変更なのでよく議論してから大型のリリース中で修正することになるっぽい > 乱数のseed固定で書いてるコードは影響受けるからか...

[p_tan]

互換性は大事よなぁ…。

[peroon]

ガシャの排出もこれのせいにしたらいい

[teitei_tk]

いい話ですね。