我认为可以改进的功能,代码非常幼稚。
public function isValid($data)
{
$valid = true;
$valid = parent::isValid($data) && $valid;
$this->_errorsExist = !$valid;
return $valid;
}
parent :: isValid($ data)也将返回布尔值。
我通过删除&&中的$ valid来更正它条件。
public function isValid($data)
{
$valid = true;
$valid = parent::isValid($data);
$this->_errorsExist = !$valid;
return $valid;
}
但我认为这仍然可以改善。任何建议都将不胜感激。
答案 0 :(得分:2)
这段代码更为重要,但是它是否已经改进了#34;是完全主观的。
public function isValid($data) {
$this->_errorsExist = !parent::isValid($data);
return !$this->_errorsExist;
}
如果没有看到所有代码,很难为您提供更多帮助。