Discussion :

[Herve_be]

Bonjour,
Le code ci-dessous fonctionne très bien sauf quand il n'y a pas de row qui correspond au select ce qui donne un warning sur la dernière ligne.
Question de débutant : comment éviter le warning ?
if ($user !== null) ?

Code :

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

1
2
3
$users = mysqli_query($db,"SELECT * FROM RVRTusers where Userid = '".$Userid."' order by Dates desc");
$user = mysqli_fetch_array($users);
$Mail=$user['Email'];

Merci d'avance pour la réponse

[Séb.]

Il faut s'assurer que le retour de mysqli_fetch_array() ne vaut ni null et ni falseIl doit y avoir des exemples dans la doc : https://www.php.net/mysqli_fetch_array

Sinon il y a PDO https://www.php.net/pdo qui gère ça facilement avec une boucle :

Code :

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

1
2
3
4
$result = $pdo->query('SELECT ALL ...');
foreach ($result as $row) {
    ...
}

[escartefigue]

Et aussi : n'utilisez jamais SELECT * dans un traitement !

• SELECT * produira un résultat différent si la table ou la vue est modifiée, le traitement plantera ou produira un résultat erroné ;
• SELECT * transporte des colonnes inutiles pour le traitement, vous chargez donc le réseau au détriment des performances ;
• SELECT * compromet l'usage d'index couvrants si présents, là encore au détriment des performances ;
• SELECT * compromet les études d'impact, on ne sait pas quelles colonnes sont réellement utilisées.

Le seule moment où l'on peut utiliser SELECT *, c'est dans un test d'existence WHERE (NOT) EXISTS (SELECT *...)car dans ce cas, on ne transporte qu'un booléen.

[Herve_be]

Il faut s'assurer que le retour de mysqli_fetch_array() ne vaut ni null et ni falseIl doit y avoir des exemples dans la doc : https://www.php.net/mysqli_fetch_array

Sinon il y a PDO

php.net explique que le retour peut être null s'il n'y a plus de lignes dans le jeu de résultats, ou false si une erreur survient mais pas comment traiter ces cas alors que c'est ma question dont je n'avais justement pas trouvé réponse dans php.net.
PDO est inutilement compliqué pour un simple fetch_array.
Il faut savoit que dans cette table il y a une seule entrée où Userid = $Userid, j'ai fait comme ceci

Code :

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

1
2
3
4
5
$users = mysqli_query($db,"SELECT * FROM RVRTusers where Userid = '".$Userid."' order by Dates desc");
if ($user = mysqli_fetch_array($users))
	{
	$Mail=$user['Email'];
	...

[Séb.]

PDO est inutilement compliqué pour un simple fetch_array.
Il faut savoit que dans cette table il y a une seule entrée où Userid = $Userid

Code PHP :

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

1
2
3
4
5
6
$user = $pdo->query('SELECT ALL ...')->fetch();
if ($user !== false) {
    // J'ai retrouvé un utilisateur
} else {
   // Je n'ai pas retrouvé d'utilisateur
}

Il n'y a pas de difficulté particulière me semble-t-il.

Le principe est le même avec mysqli_fetch_array(), si ce n'est que 2 retours sont possibles.

[Herve_be]

Mon dernier code n'est pas correct ?

n'utilisez jamais SELECT * dans un traitement !

Même si on a besoin de toutes les colonnes ?

[SQLpro]

Mon dernier code n'est pas correct ?Même si on a besoin de toutes les colonnes ?

Même !!!

1) si tu le fais pas, qui le fera ? Le SGBDR => perte de temps donc perf de merde
2) si la table est modifié (ajout de colonne, suppression de colonne) => résultat faux, ou plantage

JAMAIS DE SELECT * EN PRODUCTION !

A +

[Herve_be]

Mon dernier code n'est pas correct ?

[escartefigue]

Non, comme je l'ai expliqué plus haut dans ma réponse n°3, il y a de nombreuses raisons à ne jamais utiliser SELECT * (à l'exception éventuellement du test d'existence).

C'est la raison pour laquelle de nombreuses entreprises interdisent purement et simplement les SELECT *.

[Herve_be]

Ma question ne concerne pas Select mais mysqli_fetch_array.
Je voudrais savoir comment faire pour savoir s'il y a au moins une entrée qui correspond au select, habituellement je fais

Code :

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

while($result = mysqli_fetch_array($results))

mais dans ce cas je suis certain qu'il n'y a qu'une entrée, inutile de faire une boucle, ceci est-il correct (simplement if au lieu de while) ?

Code :

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

1
2
$users = mysqli_query($db,"SELECT Email, Dates FROM RVRTusers where Userid = '".$Userid."' order by Dates desc");
if ($user = mysqli_fetch_array($users))

Sinon comment faire (sans PDO) ?

[Séb.]

Tu récupères le résultat de mysqli_fetch_array()
Si le résultat ne vaut ni false et ni null, alors tu as un utilisateur

[Herve_be]

Bien sur, c'est ce que je veux faire, je demande seulement si cette syntaxe est correcte

Code :

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

if ($user = mysqli_fetch_array($users))