Échec de la requête PHP

Question

<?php 
if(isset($_POST['upload']) && $_FILES['userfile']['size'] > 0) 
{ 
$dbhost = 'localhost'; 
$dbuser = 'zuk1_boo'; 
$dbpass = 'lols'; 
$dbname = 'zuk1_boo'; 

$conn = mysql_connect($dbhost, $dbuser, $dbpass) or die ('Error connecting to mysql'); 

$name = $_POST['name']; 
$iq = $_POST['iq']; 

$nuname = str_replace(" ", "-", $name); 
$nuname = $nuname.".gif"; 
$path = "img/$nuname"; 

move_uploaded_file($_FILES['userfile']['tmp_name'],$path); 

$query = "INSERT INTO celebs (celeb,path1,iqq) VALUES ('$name','$path','$iq')"; 

mysql_query($query) or die('q fail'); 

mysql_close($conn); 

echo "<br>File $fileName uploaded<br>"; 
} 
?> 

<form method="post" enctype="multipart/form-data"> 
<table width="350" border="0" cellpadding="1" cellspacing="1" class="box"> 
<tr> 
<tr><td><input name="name" type="text" value="name"></td></tr> 
<tr><td><input name="iq" type="text" value="iq"></td></tr> 
<td width="246"> 
<input type="hidden" name="MAX_FILE_SIZE" value="2000000"> 
<input name="userfile" type="file" id="userfile"> 
</td> 
<td width="80"><input name="upload" type="submit" class="box" id="upload" value=" Upload "></td> 
</tr> 
</table> 
</form> 

Dans l'esprit, il existe une ligne d'ID avec incrémentation automatique, mais même si j'ajoute cela à la requête, cela ne fonctionnera toujours pas.

Peu importe ce que je fais avec cette requête, cela ne fonctionnera pas, j'ai triplé les détails de SQL et ils vont bien, même si cela semble bien se connecter de toute façon. J'ai joué avec les noms de champs dans la requête et ça devrait aller, mais ça ne marchera pas! 5 $ Paypal à tous ceux qui peuvent m'aider, honnêtement je suis tellement frustré que c'est faux.

Answer 1

mysql_query($query) or die('q fail'); 

remplacer par

mysql_query($query) or die(mysql_error()); 

et dites-nous ce que vous obtenez.

Answer 2

Eh bien, maintenant que vous avez résolu le problème réel, je pense qu'il est bon de souligner que vous avez des trous béants SQL injection.

La meilleure façon de fixer serait de passer à l'aide des PDO ou MySQLi fonctions et utiliser des requêtes paramétrées, mais le plus rapide solution serait de changer ces lignes:

$name = $_POST['name']; 
$iq = $_POST['iq']; 

à

$name = mysql_real_escape_string($_POST['name']); 
$iq = mysql_real_escape_string($_POST['iq']); 

Le code que vous avez maintenant serait extrêmement dangereux à rendre public.