SOLID что такие и с чем едят, серия статей части. S и O от SOLID

Как это ты не видишь? Там четко показано,

class NameView
{
     public function doOutput(Person $person)
     {
           if (Person instanceof Employee)
           {
                 echo ‘Hi employee ‘, $person->getName();
           }
           else
           {
                 Echo ‘Hi person ‘, $person->getName();
           }
     }
}

По сути тут если мы добавляем еще кучу всего, мы его должны изменять, но согласно открытия закрытия, мы не должны менять его, так как это не та причина для изменения, следовательно архитектура построена не верно, то есть требуется рефакторинг

В результате у нас получается

class NameView
{
     public function doOutput(Person $person)
     {
           echo ‘Hi ‘, $person->getTitle(), ‘ ‘, $person->getName();
     }
}

Что правильно, и теперь следуя принципа мы пишем только новый код, и не трогаем текущий, то есть тут уже архитектура верная

Вот нашел ещё пример
http://binary.freeperspective.net/countzer...-design-in-php/
Постарался сделать что-то похожее, чтобы было проще разобраться. Надеюсь получилось.

Проблемный класс. Кол-во типов ограничено двумя типами. Третий не проходит проверку. Чтобы добавить третий тип придется изменять базовый класс.

class Person
{
    private $type;
    public function __construct($type){
        $this->type = $type;
    }
    public function setName(){
        if ($this->type == 'employee')
            return $this->type;
        if ($this->type == 'manager')
            return $this->type;
    }

}

$person = new Person ('employee');
echo $person->setName();//employee
$person = new Person ('manager');
echo $person->setName();//manager

$person = new Person ('worker');
echo $person->setName();////

Следуя принципу Open/Close Principles, появляется возможность добавлять неограниченное кол-во типов при этом не изменяя базовый класс.

abstract class AnyPerson
{
    public function setName(){
        return $this->getPerson();
    }
    public abstract function getPerson ();
}

class employee extends AnyPerson
{
    private $employee = 'employee';
    public function getPerson(){
        return  $this->employee;
    }
}

$person = new employee();
echo $person->setName();//employee

class manager extends AnyPerson
{
    private $employee = 'manager';
    public function getPerson(){
        return  $this->employee;
    }
}

$person = new manager();
echo $person->setName();//manager

class worker extends AnyPerson
{
    private $employee = 'worker';
    public function getPerson(){
        return  $this->employee;
    }
}

$person = new worker();
echo $person->setName();//worker

Хотелось бы ещё обратить внимание на некоторые моменты.

1. Рабочий пример (на примрере ТС) несоответствующий OCP. Как видно, класс Manager расширяет класс Person. По условию все классы наследники Person - 'employee', что неправильно и требует изменений.

class Person
{
     public function getName(){}
     public function setName(){}
}

class Employee extends Person {}
class Manager extends Person{}

class NameView
{
    public function doOutput(Person $person)
    {
//        $employee = new Employee ;
        $employee = new Manager;

        if ($employee instanceof Person)
        {
                 echo 'Hi employee', $person->getName();
           }
        else
        {
                 Echo 'Hi person ', $person->getName();
           }
    }
}

class Controller
{
    public function personAction()
    {
        // Get a person from some data source
//           $person = ...;
           $person = new Person;
           $view = new NameView();
           $view->doOutput($person);
     }
}

$controller = new Controller ();
$controller->personAction();//Hi employee

2. Рабочий пример (на примрере ТС) соответствующий OCP (не полностью). Как видно, все классы расширяющие класс Person, возвращают требуемый результат, кроме одного - класс Worker. А если применить абстрактный класс и абстрактный метод, тогда класс Worker не останется без внимания.

class Person
{
     public function getName(){}
     public function setName(){}
     public function getTitle()
     {
           return 'person';
     }
}

class Employee extends Person
{
    public function getTitle()
    {
           return 'employee';
     }
}

class Manager extends Person
{
    public function getTitle()
    {
        return 'manager';
    }
}

class Worker extends Person
{

}

class NameView
{
    public function doOutput(Person $person)
    {
           echo 'Hi ', $person->getTitle(), ' ' , $person->getName();
     }
}

$output = new NameView();
$output->doOutput(new Employee);//Hi employee
$output->doOutput(new Manager);//Hi manager

$output->doOutput(new Worker);//Hi person

ты не прав тут

Worker это частный случай Person , и если он возвращает Person значит верно всё.