Discussion :

[kate59]

bonjour,

Code :

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

1
2
3
4
5
6
7
public function setAuteur($auteur)
        {
            if (!is_string($auteur) || empty($auteur))
                $this->erreurs[] = self::AUTEUR_INVALIDE;
            else
                $this->auteur = $auteur;
        }

j'ai ce code et je voulais comprendre pk on ne met pas plutot:

Code :

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

1
2
3
4
5
6
7
public function setAuteur($auteur)
        {
            if (is_string($auteur) || empty($auteur))
                $this->erreurs[] = self::AUTEUR_INVALIDE;
            else
                $this->auteur = $auteur;
        }

qui serait plus logique,non?

[MaitrePylos]

Bonjour,

Dans le premier cas tu vérifies si la variables est différente du type string ou vide.
Dans le deuxième cas tu vérifies si il est du type string ou vide.


dans ton exemple d'amélioration si

Code :

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

$auteur = 12345

, alors c'est bon et il n'y a pas d'erreur

A toi de voir

[kate59]

MERCI je suis bête c'etait si simple, par contre si c est un booleen si elle 0 comme valeur ca se complique non pour le empty?

[guigo]

Salut,
non 0 est different de vide ! Ta condition va fonctionner

Guigo

[RunCodePhp]

Salut

Bon, je vais chipoter un peu, sinon beaucoup , mais est ce que cette condition is_string() est si importante que ça, est ce que ça vaut le coup de faire un tel traitement ?
Donc est ce qu'un simple empty() ne suffirait pas finalement ?

Je te donne mon avis, ça me semble malgré tout superflux (même si on sera d'accord que rajouter un is_string() doit représenter tout juste un micro de seconde).
La raison est simple, car d'un coté on refuse des valeurs comme 0, 12345, qui elles sont un peu farfelues, mais "TrucMuche" est tout aussi farfelue, et pourtant là ça passera.

Sans compter qu'il faudrait voir quelle type de donnée sera passée en argument, et si par exemple c'est un $_GET ou POST, toutes les valeur sont des chaines, même 12345.
Ca sous entend par là que lorsque la donnée est obtenue en GET ou POST qu'il faudra faire un traitement avant ta condition pour détecter si c'est une chaine ou pas, et typer/caster la donner si c'est un INT pour que ça génère une erreur et la gérer (avec $this->erreurs).
En finalité, c'est faire quelque part 2 fois la même chose, non ?


D'ailleurs, je ne sais pas si vous avez fais des essai, mais pour ma part, même coté Bdd le problème est le même, que ce soit avec PDO ou les fonctions musql_*, toutes les données sont des chaines de caractères, même venant d'un champ de type INT
C'est donc à soit même de se charger de "typer" ces données, ce qui n'est pas si pratique que ça, surtout si on automatise la récupération des données par exemple.

Les types de données, un long débat en faite, pas si bébête que ça.

[coach759]

Salut,
non 0 est different de vide ! Ta condition va fonctionner

Guigo

Surtout pas, si je peux me permettre,
Avec la fonction empty, zéro retourne vrai!!!

Ce qui suit est considéré comme étant vide : (et donc return true)
"" (une chaîne vide)
0 (0 en tant qu'entier)
"0" (0 en tant que chaîne de caractères)
NULL
FALSE
array() (un tableau vide)
var $var; (une variable déclarée, mais sans valeur dans une classe)