FormUtil::getPassedValue  Début

  • 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.

    Code

    // Argument check
        if (!isset($args['bid']) || !is_numeric($args['bid'])) {
            return LogUtil::registerError (_MODARGSERROR);
        }

  • 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



    Membre du PSR Project (Pagesetter replacement)
  • mumuri

    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
  • 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
  • 7 visiteurs

Données pour les 20 dernières minutes