[Système] comment éviter un warning

Version imprimable

17/02/2008, 14h29
laurentSc

[Système] comment éviter un warning

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 ?
17/02/2008, 14h31
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_*( ).
17/02/2008, 14h40
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...
17/02/2008, 14h42
Séb.

Montre-nous la portion de code incriminée.

Bonjour,
quelque chose comme ceci?

Code:

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.

Voici le code qui pose pb :

Code:

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.

Essayez avec celui-ci :

Code:

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.

17/02/2008, 15h38
Séb.

/!\ if ($result !== FALSE) ;)
17/02/2008, 17h06
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.
17/02/2008, 17h47
DaRiaN

Citation:

Envoyé par Séb.

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

Vous êtes sûr?
17/02/2008, 19h12
Séb.

Citation:

Envoyé par DaRiaN

Vous êtes sûr?

Ben pour un SELECT mysql_query( ) retourne soit une ressource soit FALSE si la requête échoue, donc oui.
17/02/2008, 19h22
DaRiaN

Citation:

Envoyé par http://fr.php.net/mysql_query

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.
17/02/2008, 19h25
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.
17/02/2008, 19h29
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.
17/02/2008, 21h37
Séb.

Citation:

Envoyé par DaRiaN

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 ;)
@+
17/02/2008, 22h25
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.
20/02/2008, 11h24
ThomasR

Code:

@$result = mysql_query($sql);

@ empêche l'affichage du warning