誰かがこの短いコードを見直してもらえますか？

Question

行はテーブルに挿入され、その行のIDはその特定のエントリの名前と姓の比較によって選択されます。

返されたIDは、その後、別のテーブルに挿入されています。重要なコメントの

$result = false; 

$result = mysql_query("INSERT INTO `reunie` (`naam`, `van`,`nooiensvan`, `selfoon`, `email`, `bywoon`, `metgesel_naam`, `spesifieke_onderwysers`, `spesifieke_musiek`, `bydrae`, `voorstelle`) VALUES ('$naam','$van','$nooiensvan','$selfoon','$email','$bywoon','$metgesel','$spesifiekeOnderwysers','$musiek','$bydrae','$voorstelle')") or die(mysql_error()); 

// sleep in order for query to finish 
while ($result === false) { 
    sleep(1); 
} 

$result = mysql_query("SELECT `id` FROM `reunie` WHERE `naam` = '$naam' AND `van` = '$van' ORDER BY `id` DESC LIMIT 1") or die(mysql_error()); 

//編集

みんなありがとうを。私にいくつかの重要な点を教えてくれてありがとう。

Answer 1

てみ使用しない：

$name = mysql_real_escape_string($name); 
$van = mysql_real_escape_string($van); 
$nooiensvan = mysql_real_escape_string($nooiensvan); 
$selfoon = mysql_real_escape_string($selfoon); 
$email = mysql_real_escape_string($email); 
$bywoon = mysql_real_escape_string($bywoon); 
$metgesel = mysql_real_escape_string($metgesel); 
$spesifiekeOnderwysers = mysql_real_escape_string($spesifiekeOnderwysers); 
$musiek = mysql_real_escape_string($musiek); 
$bydrae = mysql_real_escape_string($bydrae); 
$voorstelle = mysql_real_escape_string($voorstelle); 
$query = <<<END 
INSERT INTO reunie 
(naam, van, nooiensvan, selfoon, email, bywoon, metgesel_naam, 
spesifieke_onderwysers, spesifieke_musiek, bydrae, voorstelle) 
VALUES 
('$naam','$van','$nooiensvan','$selfoon','$email','$bywoon','$metgesel', 
'$spesifiekeOnderwysers','$musiek','$bydrae','$voorstelle') 
END; 
$result = mysql_query($query) or die(mysql_error() . ' - ' . $query); 
$id = mysql_insert_id(); 

そして、はい、これらすべてのmysql_real_escape_stringのは（）本当に重要です。 $結果は本当に偽の場合

Answer 2

IDを取得するためにselectを実行する代わりに、mysql_insert_idを使用する必要があります。

Answer 3

ここでホイールを再発明しようとしていませんか？

なぜ

$result = mysql_query("INSERT ... "); 
$new_id = mysql_insert_id(); 

Answer 4

// sleep in order for query to finish 

while ($result === false) { 
    sleep(1); 
} 

私はどのように愛、スクリプトはちょうどあなたがそこにスリープ機能を持っている理由私は正確にはわからない