while（）を使用してランダムなファイル名を生成していますか？

Question

私はwhile()ループを使用してランダムなファイル名を作成し、それをデータベースと照合して、それがすでに存在する場合はループします。私は少し前にこれをやったことがあるので、私の文法/使用法はちょっと心配です。それ以来、このようにループはwhile()を使っていません。

は、ここに私のコードです：

$i = 0; 
while(++$i) { 
    $file_name = md5(mt_rand(0,9999999)) . ".php"; 
    $result = mysql_query("SELECT * FROM x WHERE file_name = '{$file_name}'"); 

    if(mysql_num_rows($result) == 0) { break; } else { continue;} 
} 

うこの作品、とされていない場合、それと間違って何ですか？

私が知っているのはちょっとした質問ですが、これがうまくいくかどうかをテストするのは、新しいテーブルを作成し、ファイル名を変更して1から3のものを選択し、メッセージを表示する代わりに、継続のなど常に、理解されるように

すべてのヘルプ、これは動作しますが、あなたが行を変更することができ

Answer 1

あなたの中に（++ $ I） $iは必要ありません。あなたはそれを使用していないので、ループにはcontinueは必要ありません。ループは自動的にループしますそれは何ですか。あなたがが少し壊れた方法でハッシュを使用している

while(true) { 
    $file_name = md5(mt_rand(0,9999999)) . ".php"; 
    $result = mysql_query("SELECT * FROM x WHERE file_name = '{$file_name}'"); 

    if (mysql_num_rows($result) == 0) break; 
} 

：あなたがループを終了するための条件に達した場合にのみ、breakする必要があります。 MD5ハッシュよりもランダムな文字列をより長く生成する必要があります。短くはありません。現在MD5が作成できる完全な2^128の代わりに、10,000,000の可能なファイル名（0..99999999）しかありません。宇宙空間でのハッシュ衝突を心配するのではなく、同じ数字を2回生成するだけで、データベースへの複数回のトリップが発生する可能性は非常に高くなります。

Answer 2

：！しばらく（真）

Answer 3

私はそれが些細な質問だと知っていますが、これがうまくいくかどうかをテストすることは、より多くの「タスク」（新しいテーブルを作成する必要がある、ファイル名を変更して1〜続行などの代わりにメッセージを表示する

コードが実行するアプローチを取ることは、必要以上に「タスク」です。実行可能ですが、データベースに「ユニークなファイルIDの作成」作業をさせてみましょう？

xのfile_name列を代わりにbigint(20) auto_incrementなどのfile_id列に定義します。次に、PHPコードで、新しい行を作成するクエリを発行し、保証された一意の数値IDを取得し、ファイル名に変換します。