FormUtil::getPassedValue
Auteur Sujet

Post 
J'ai remarqué que dans les modules qui utilisent FormUtil::getPassedValue, ils attribuent toujours une valeur pas défaut(null) si la constante indiquée n'existe pas.

Et bien souvent après on appelle une api qui utilise cette constante, je me demande alors l'utilité d'avoir une vérification de l'existence des arguments au début de chaque api car si ils n'existaient pas, FormUtil::getPassedValue leur a donné une valeur : null.

Voici le genre de code que l'on retrouve au début de l'api, isset($args['bid']) sera toujours vrai car toujours égal à null au minimum.
  1.     // Argument check 
  2.     if (!isset($args['bid']) || !is_numeric($args['bid'])) { 
  3.         return LogUtil::registerError (_MODARGSERROR); 
  4.     } 



-------------------------------------------
...n'oubliez pas que l'outil Recherche est votre ami !
...bien souvent il répond à nos questions avant même que l'on ne se les pose ! ^^
Données personnelles Accueil

Post 
a priori cà doit être un pattern de codage par défaut basé sur des régles de base

quelques soit le paramètre (qu'il proviennent d'un form ou pas) , on vérifie les isset

si tu veux aller plus loin, is_numeric de null renvoie faux
Données personnelles Accueil

Post 
mumuri a écrit :
quelques soit le paramètre (qu'il proviennent d'un form ou pas) , on vérifie les isset


Ouaip mais bon ça perd son utilité de faire un isset sur une constant qui, quoi qu'il arrive, est définie. Cette condition ne sert strictement à rien à part à embrouiller celui qui essaye de comprendre le code. icon_wink

-------------------------------------------
...n'oubliez pas que l'outil Recherche est votre ami !
...bien souvent il répond à nos questions avant même que l'on ne se les pose ! ^^
Données personnelles Accueil
Post 
Ouaip pour pas changer j'ai parlé trop vite. icon_razz

Je pensais que si une constante valait "null", un isset sur cette constante allait répondre true mais ça répond false donc pas de problème dans le code. icon_wink

-------------------------------------------
...n'oubliez pas que l'outil Recherche est votre ami !
...bien souvent il répond à nos questions avant même que l'on ne se les pose ! ^^
Données personnelles Accueil