Discussion :

[skins]

Bonjour,
je commence à me poser des questions sur l'efficacité du code que j'utilise pour mes requêtes, je me tourne vers vous pour me dire si c'est possible de développer la syntaxe des requêtes.
Par exemple pour chercher l’existence d'un enregistrement j'utilise :

Code :

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

1
2
3
	$result =mysql_query ("SELECT COUNT(*) AS nbr FROM Table WHERE id = $id  " );
	$donnees = mysql_fetch_array($result);
	$nbr = $donnees['nbr'] ;

et pour trouver un enregistrement :

Code :

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

1
2
3
	$reponse4 = mysql_query("SELECT Solde FROM Table WHERE id = $id  ");
	$donnees4 = mysql_fetch_array ($reponse4);
	$Solde = $donnees4['Solde'] ;

est-ce la façon la optimisée ou bien il existe d'autre syntaxe plus évolués.
Merci pour votre aide

[Dendrite]

Pourquoi tu fais une requête liminaire pour voir s'il existe ?
Quand un enregistrement ne retourne rien, ça ne provoque pas d'erreur, ce n'est pas un problème en soi, pas besoin de requête préalable.
Tu peux directement lancer la seconde requête et en PHP, simplement tester le sizeof($donnees4) pour l'affichage.

[skins]

Bonjour,
Merci Dendrite pour la remarque effectivement cela ne sert pas à grand chose dans mon cas.
J'ai une autre question concernant une requête : j'aimerais rendre mon système multi-langues en mettant les traductions dans une table, est ce que la requête ci-dessous est le meilleure façon pour le faire ?

Code :

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

1
2
3
$reponse4 = mysql_query("SELECT ENG FROM Table_langue WHERE id = 5  ");
$donnees4 = mysql_fetch_array ($reponse4);
$EXPRESSION5= $donnees4['ENG '] ;

Merci pour votre aide

[ABCIWEB]

Remarque sur tes exemples :

1/ Dans une requête, on doit toujours protéger ses variables avec mysql_real_escape_string. Dans ton cas s'agissant d'un entier ce n'est pas nécessaire mais uniquement si tu as auparavant casté la variable $id en entier, sinon c'est un trou de sécurité majeur (injection sql).

2/ Il est très pratique de séparer l'écriture de la requête de son exécution :

Code :

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

1
2
$query = "SELECT Solde FROM Table WHERE id = $id  ";
$reponse4 = mysql_query($query);

Cela permet de faire afficher echo $query pour débugage plus simplement.

3/ Normalement on fait un minimum des gestion des erreurs. Si pour une raison quelconque la requête échoue, les lignes suivantes ne devraient pas être exécutées sinon cela renverra des warning de tous côtés et une exécution aléatoire du reste du code. Donc par exemple :

Code :

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

1
2
3
4
5
6
7
8
$query = "SELECT Solde FROM Table WHERE id = $id  ";
$reponse4 = mysql_query($query);
if($reponse4 !== false)
{
    $donnees4 = mysql_fetch_array ($reponse4);
    $Solde = $donnees4['Solde'] ;
    //...
}

4/ Pendant que tu as du temps pour réfléchir à ton code tu devrais t'intéresser à mysqli ou pdo (pdo recommandé car offre plus de facilités et plus universel, mysqli plus facile en venant de mysql). Le message avec le gros triangle d'avertissement dans toutes les doc sur les fonctions de l'extension mysql est là pour le rappeler.

Enfin pour ta dernière question sur le multilingue il y a un sujet ressemblant ici