Discussion :

[laurentSc]

Bonjour,

je voudrais savoir si je peux (et ds ce cas, comment) éviter un warning qui se produit lorsque le type d'une variable passée à une fct n'est pas celui attendu ? Ds ce cas, la fct n'est pas appelée, mais l'erreur est qd même détectée.
En fait, ayant un compte sur free avec une base de données, et trouvant que leur interface phpmyadmin est hyper lente, je me suis développé en php une interface me permettant d'envoyer des requêtes SQL. Ca marche, mais s'il n'y a aucun affichage, visiblement la variable retournée par la fct mysql_query n'est pas du type resource et l'appel à la fct mysql_fetch_array (qui n'est pas exécuté ds ce cas) génère qd même un warning. Y a-t-il moyen de l'éviter et, si oui, comment ?

[Séb.]

Oui avec un if( ) testant la valeur retournée par mysql_query( ).
=> Si différent de FALSE alors tu peux utiliser mysql_fetch_*( ).

[laurentSc]

J'ai déjà fait ça, mais visiblement, la seule présence de l'appel ds le code suffit à générer le warning...

[Séb.]

Montre-nous la portion de code incriminée.

[DaRiaN]

Bonjour,
quelque chose comme ceci?

Code :

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

1
2
3
4
  if (!$res = mysql_query($req)) exit(mysql_error());
 
  if ($res !== TRUE) echo mysql_num_rows($res).' ligne(s) retournee(s)';
  else echo 'La variable $res n\'est pas une ressource MySQL';

Cordialement,
DaRiaN.

[laurentSc]

Voici le code qui pose pb :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
if (!($result=mysql_query($sql))) echo "pb avec la requête"; else 
{echo "requête OK";echo"<br>";
echo $result; echo "<br>";
 
if (((is_resource($result))) or (!($result==""))or (!($result=="1")))
while ($ligne=mysql_fetch_array($result,MYSQL_NUM))
{
	$i=0;
	while (((!($ligne[$i]==""))&&($i==0)) or ($i <= $max))
	{
	if ($i==0) $max=$i;
	echo $ligne[$i++]." "; 
	}
}
}

J'y ai évidemment rajouté des tests inutiles.

[DaRiaN]

Essayez avec celui-ci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
<?php
 
  $result = mysql_query($sql) or die(mysql_error());
 
  if ($result !== TRUE)
  { 
    while ($ligne = mysql_fetch_array($result, MYSQL_NUM))
    {	
      $i=0;
 
      while (((!($ligne[$i]=="")) && ($i==0)) or ($i <= $max))
      {
        if ($i==0) $max=$i;
        echo $ligne[$i++]." "; 
      }
    }
  }
 
?>

PS: ce n'est pas pour critiquer, mais votre script est assez bizarre.

[Séb.]

/!\ if ($result !== FALSE)

[laurentSc]

Je viens de faire l'essai et ça marche. Je suis super content. Visiblement, ça vient de la fct die, mais que fait-elle exactement ? Et pourquoi ça marche maintenant ?
Sinon, mon code est peut-être un peu bizarre, mais en fait, ne connaissant pas la longueur d'une ligne affichée, et pensant qu'une ligne pouvait contenir des "" (donc c'est une mauvaise condition d'arrêt d'affichage), je mesure la longueur de la 1e ligne, et ça constitue ma condition d'arrêt d'affichage pour les suivantes.
En tout cas, merci.

[DaRiaN]

/!\ if ($result !== FALSE)

Vous êtes sûr?

[Séb.]

Vous êtes sûr?

Ben pour un SELECT mysql_query( ) retourne soit une ressource soit FALSE si la requête échoue, donc oui.

[DaRiaN]

Pour les requêtes du type SELECT, SHOW, DESCRIBE, EXPLAIN et les autres requêtes retournant un jeu de résultats, mysql_query() retournera une ressource en cas de succès, ou FALSE en cas d'erreur.

Pour les autres types de requêtes, UPDATE, DELETE, DROP, etc., mysql_query() retourne TRUE en cas de succès ou FALSE en cas d'erreur.

Si ce n'est pas FALSE ou TRUE alors c'est une ressource, d'où le !== TRUE.

[Yogui]

Salut

Si ton objectif est d'éviter un warning, tu as plusieurs options :

• Gérer les erreurs (pas toujours suffisant) ;
• Faire taire le warning en mettant '@' avant le nom de la fonction (très moche) ;
• Et la meilleure solution : modifier la config display_errors à Off.

[DaRiaN]

La cause du warning n'est plus, car !== TRUE vérifie que le résultat de mysql_query est une ressource, car si ce n'est pas FALSE ou strictement TRUE, alors c'est une ressource. Je n'aime pas masquer les warning, je préfère coder proprement mes scripts.

[Séb.]

Si ce n'est pas FALSE ou TRUE alors c'est une ressource, d'où le !== TRUE.

Bien sûr, mais je ne vois pas pourquoi tester TRUE car :
- il n'est jamais retourné dans ce cas là
- on cherche à exécuter le if( ) s'il n'y a pas d'erreurs, ce qui de prime abord n'est pas évident avec un test !== TRUE
Bon on va dire que c'est une question d'habitude
@+

[DaRiaN]

LaurentSc semble utiliser toutes sortes de requête avec ce script et pas seulement des SELECT, la preuve en est : le warning en question.

Voilà le pourquoi du !== TRUE maintenant si je me suis fourvoyé, cela ne gène en rien le script, par contre, dans le cas contraire, ça fait toute la différence.

[ThomasR]

Code :

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

@$result = mysql_query($sql);

@ empêche l'affichage du warning