PHPで作成するジェネレーターでのご質問

質問

以前も相談させていただいたのですが、
ツイッター診断メーカーに似たものを作成しています。
http://shindanmaker.com/

そこである程度は完成したのですが、1点だけどうしても作成方法が分からず、
その部分をご相談させてもらいたいです。

以下にソースコピペします。

//textAraryには50個ほどの文字列が入っています。
$txtNum = count($textArray);

//文字列を10進数に
$num = sprintf("%u",crc32($click));

10進数を配列の数で割り、余りを求める
$i= $num % $txtNum;

$finish = $textArray[$i];

抜粋なのですが、上記のソースで実装するとちゃんと意図した動作するのですが、
診断用の異なるニックネームを何回か入力すると、
かなりの確率でfinishに同じ文字列が抽出されてしまいます。

出来る限り、バラけさせたいのですが、
何か良いロジックはありますでしょうか。

当方頭が回らず、、お知恵を拝借できればと幸いです。。
また、参考になるサイトでも結構ですので、
ご教授の程、よろしくお願いいたします。

回答No.2

duron · ベストアンサー

sprintfの"%u"で得られる数値(正しくは数字の文字列)はマイナス符号のない値です。
つまりとりうる範囲は0～4294967295(0xFFFFFFFF)です。
しかしPHPで計算されるときは符号付の数値で計算されます。
つまり範囲は-2147483648～2147483647となります。

$num = sprintf("%u",crc32($click));
で得られる$numは文字列であり
$i= $num % $txtNum;
の計算の時点で$numは数値として評価されます。
このときに2147483647を超える値の場合にはすべて2147483647として評価されてしまいます。
つまり半分以上の確率で$numは2147483647として計算されるため同じ結果が頻発します。

ですので
$num = sprintf("%d",crc32($click));
としてマイナス符号付の数値として$numを受け取り
このままでは剰余の計算でマイナスが返ってくることもあるので
$i= abs($num % $txtNum);
で絶対値を取ればいいと思います。

・・・断定口調で書いてますが予想して書いたものですので厳密にあっているかは自信ないです。

お礼

ご回答有難うございます。

頂いた内容で無事解消いたしました！今までからいうとかなり改善されました。
これで問題なく取り組めそうです。

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

回答No.1

SAYKA

？　言っている意味がわからない。
普通に乱数を生成するのじゃだめなの？

補足

ご回答ありがとうございます。

すいません、言い忘れていましたが、同じ名前を入力すると同じ結果を表示させたいのです。
なので、乱数ですと結果がその都度変わってしまうので、ご提示したソースのように入力された文字列をベースに10進数に変換しています。

引き続きよろしくお願いします。