Discussion :

[andtwoturtles]

Bonjour,

J'ai une fonction getFileList qui me donne une liste de fichiers contenu dans la ma base de donnée dont un des paramètres est $id_folder.

Evidement pour vérifier que tout va bien (et que l'on ne me passe pas n'importe quel id), je vérifie si l'utilisateur à le droit de faire cette recherche, si le dossier existe,...
Dans les cas où ça ne va pas, je renvoi 0

Dans mon code j'ai donc :

Code :

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

1
2
3
4
5
6
7
<?php
...
 	$list_file = getFileList($id_folder);
 
	if ($list_file != 0) {
...
 ?>

qui me donne une erreur

Notice: Object of class PDOStatement could not be converted to int in /srv/http/file_tempo/proceed_files.php on line 290

Je suis assez surprit de ce message d'erreur.
Je ne lui ai pas demandé de me le convertir simplement de le comparer. Qu'il contienne 1, un objet PDO, NULL ou une image ça ne m'intéresse pas, je veux simplement qu'il me dise "non c'est autre chose que 0" point barre.
Mais apparemment php à besoin de convertir pour me dire cela...

Comment puis-je faire mon test autrement pour éviter cette conversion ?

merci

[Celira]

Tu peux contourner le problème et utiliser empty pour tester.

Code :

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

1
2
$list_file = getFileList($id_folder);
if (!empty($list_file)) {

[andtwoturtles]

ça m'a toujours un peu dérangé que la fonction empty réponde true lorsqu'une variable contient 0 (pas vraiment vide pour moi), mais je dois avouer que dans ce cas de figure ça rend service

merci ça fonctionne

[sabotage]

Je pense que le plus cohérent serait :

Code :

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

if (is_int($list_file) && $list_file != 0)

[Celira]

Sinon tu peux faire renvoyer null à ta fonction et tester avec is_null, ou bien renvoyer false.

[andtwoturtles]

la solution de sabotage fonctionne mais c'est vrai que modifier ma fonction pour renvoyer false me semble plus cohérent