2010-07-07 11 views
2

J'ai surchargé la fonction isValid() pour un formulaire, et je me demandais s'il y avait des inconvénients à ajouter de la logique là-bas.Inconvénients de remplacer la fonction isValid() sur Zend_Form

code Exemple:

public function isValid($data) 
{ 
    // conditional requirement of fields 

    $isValid = parent::isValid($data); 

    if ($isValid) { 
     // additional validation 
    } 

    return $isValid; 
} 

Répondre

4

Non, aucun. Mais je mettrais une instruction if autour du parent:

$parentIsValid = parent::isValid($data); 
if($parentIsValid) { 
    //Extra stuff, you may as well only do this if the parent valid function returns true 
    //Set isValid var in here 
} else $isValid = false; 
return $isValid; 
+0

Bonne observation Ashley. J'utilise un conditionnel pour la validation supplémentaire. Y a-t-il une raison spécifique pour laquelle vous utilisez deux variables booléennes, ou était-ce pour illustrer l'exemple? – Sonny

+2

Oui juste pour illustrer l'exemple. Vous pouvez toujours utiliser la fonction parent :: isValid à l'intérieur de l'instruction if ou renommer parentIsValid juste pour isValid – Ashley

+0

J'accepterai votre réponse un peu demain si personne ne pèse avec un piège. Merci Ashley! – Sonny