utiliser mysql_query sans placement dans variable

**khamett** · 15/03/2011, 13h58

Bonjour,

tout est dans le titre.
J'utilisais le code suivant:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
$result=mysql_query("UPDATE matable SET Statut='".$_SESSION['status']."' WHERE id= ".$_SESSION['machine']) or die('Impossible de mettre à jour la base. Erreur relevée: ' . mysql_error());

et je test un code raccourci:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
mysql_query("UPDATE matable SET Statut='".$_SESSION['status']."' WHERE id= ".$_SESSION['machine']) or die('Impossible de mettre à jour la base. Erreur relevée: ' . mysql_error());

sachant que je ne réutilise pas la variable $result derrière est-il nécessaire de garder le 1er schéma ?

Merci

K

**Shikiryu** · 15/03/2011, 14h18

Dans le cadre d'un UPDATE, mysql_query retourne un booléen.

Le "mieux" dans ce cadre, c'est soit :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
$result=mysql_query("UPDATE matable SET Statut='".$_SESSION['status']."' WHERE id= ".$_SESSION['machine']);
if(!$result) echo "erreur";

ou

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if(!mysql_query("UPDATE matable SET Statut='".$_SESSION['status']."' WHERE id= ".$_SESSION['machine'])) echo "erreur";

La 1ère est plus lisible et réutilisable, la 2nde ne crée pas de variable et est plus courte.

Le résultat est le même, le tout est de garder le même système dans toute ton application pour être cohérent et débugger plus facilement par la suite.