Вопрос по php, abstract, public, dry – Уровень доступа к определенному классу должен быть публичной ошибкой в PHP

6

Я создал этот класс

<?php
    abstract class Validator{
        public $_errors = array();
        abstract public function isValid($input);

        public function _addErrors($message){
            $this->_errors = $message;
        }

        public function getErrors(){
            return $this->_errors;
        }


        public function getMessage(){
            return $this->message;
        }
    }

    class Validator_NoSpaces extends Validator{

        public function __construct($value){
            $this->isValid($value);
        }
        public function isValid($value){
                if (preg_match('/\s/', $value)){
                $this->_addErrors("Spaces are not allowed");
                return false;
            }
            return true;
        }       
    }

    class Validator_MinimumLength extends Validator{

        protected $_minLength;
        protected $value;

        public function __construct($value ,$minLength=8){
            $this->_minLength = $minLength;
            $this->value = $value;
            $this->isValid($value);
        }

        public function isValid($input){
             if (strlen($input) > $this->_minLength) {
                 return true;
            }else{
                $this->_addErrors("Input must be at least {$this_minLength}");
                return false;
          }
        }
    }

    class Form_Element_Validators extends Validator{

        protected $_validators = array();

    public function addValidator(Validator $validator)
    {
        $this->_validators[] = $validator;
    }

    public function getValidators()
    {
        return $this->_validators;
    }

    protected function _addErrors(array $errors)
    {
        foreach ($errors as $error) {
            $this->_addErrors($error);
        }
    }

    public function hasErrors()
    {
        return (count($this->getErrors()) !== 0);
    }

    public function isValid($input)
    {
        foreach ($this->_validators as $validator) {
            if (!$validator->isValid($input)) {
                $this->_addErrors($validator->getErrors());
            }
        }
        return !$this->hasErrors();
    }

    }

    class Form_Element extends  Form_Element_Validators{

        public function __construct($value){
              $this->addValidator(new Validator_NoSpaces($value));
              $this->addValidator(new Validator_MinimumLength($value));
        }
    }

для целей проверки, но он продолжал давать мне эту ошибку

Fatal error: Access level to Form_Element_Validators::_addErrors() must be public (as in class Validator) in C:\xampp\htdocs\beatbeast\includes\Db\Validators.php on line 91

Но переменная экземпляра в этом классе $ _errors объявлена общедоступной, я не понимаю, почему я получаю эту ошибку.

Ваш Ответ

3   ответа
13

потому что видимость метода должна быть такой же или менее ограничивающей, чем у его определения в родительском классе. В этом случае у вас естьaddErrors какpublic на вашем абстрактном классе и пытаемся сделать этоprotected на детском уроке.

8

вы не можете сделать метод подкласса более строгим, чем родительский; это потому, что подклассы должны быть допустимой заменой их родительского класса.

В вашем конкретном случае я бы изменил видимость всех методов и свойств, которые начинаются с подчеркивания:protected.

@sumid нет, если вы не хотите, чтобы вы просто выдавали исключение.
Почему так? Рассмотрим древовидную структуру. У него будут занятияNode а такжеLeaf, гдеLeaf это особый случайNodeтаким образомclass Leaf extends Node, СейчасNode имеетfunction addChild($child), Очевидно, я не хочу разрешатьfunction addChild($child) вLeaf, Естественный способ состоит в том, чтобы установить его частным (тогда как в родительском классе он является общедоступным), чтобы к нему нельзя было получить доступ. Или я буду наследоватьNode отLeaf ?? : -o
1

protected доступ кprotected function _addErrors(array $errors) методForm_Element_Validators учебный класс. Так что поменяй его на публичный.

Редактировать:

Вы заметили? Метод подкласса (переопределенный метод) определяется с помощьюТип Хинтинг, Пожалуйста, сохраните один и тот же тип параметра для обоих; метод суперкласса и подкласса.

 abstract class Validator{
        public $_errors = array();
        abstract public function isValid($input);

        public function _addErrors(array $message){
            $this->_errors = $message;
        }
        ....
@user: Это потому, что ваше определениеmessage в качестве аргумента, а затем в вашемForm_Element_Validator класс у вас типа намекнул, что аргумент являетсяarray либо удалитеarray введите подсказку из метода потомка или добавьте его к родителю.
Я уже изменил его тип доступа на публичный, но он дал мне еще одну ошибку: "Строгие стандарты: объявление Form_Element_Validators :: _ addErrors () должно быть совместимо с таковым из Validator :: _ addErrors () в C: \ xampp \ htdocs \ beatbeast \ включает в себя \ Db \ Validator.php в строке 91 & quot; user962206

Похожие вопросы