и как обычно начнем с код стайла =)
ты уже потихоньку выходишь на тропу истины, и я вижу что приват/протектед свойства/методы ты называешь с префиксом _
это гут
но почему у тебя то addField() , то set_form_method() ?
или все через андерскор или все в кемелСтайле.
Спустя 112 сек.
public function validate($submit) {
require_once 'SFormValidator.php';???
сделай протектед метод типа _getValidator()
и в нем уже пиши require, и возвращай обьект SFormValidator
Спустя 173 сек.
private function getFormVariables() {
if ($this->_formMethod == 'get') {
$this->_incVars = $_GET;
}
else {
$this->_incVars = $_POST;
}
}мне кажется так будет по приятнее выглядеть
private function getFormVariables() {
$this->_incVars = ($this->_formMethod == 'get') ? $_GET : $_POST;
}ну и опять 25….
называй методы нормально.
если это у тебя геттер, то пусть он чтото возвращает, иначе назови его нормально
