Недостатки переопределения функции isValid () в Zend_Form

Question

Я переопределил функцию isValid() для формы, и мне было интересно, есть ли какие-либо недостатки в добавлении логики там.

Пример кода:

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

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

    if ($isValid) {
        // additional validation
    }

    return $isValid;
}

Answer 1

Нет, нет. Но я бы поставил оператор if вокруг родителя:

$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;