Архитектурная проблема, как лучше управлять доступностью публичных методов для классов?

Question

[tukreb]

Добрый день.

Есть одна проблема, первый раз я столкнулся с ней здесь Как отключить наследование в Doctrine? которая плавно перетекла в эту Насколько вредно джойнить таблицы где отсутствует информация?
В итоге от наследования пришлось отказаться, но теперь вопрос как редактировать свойства "родительского класса", но при этом запретив напрямую обращаться к этим методам класса (то есть аналог protected).

Я пришёл к такому решению.

Создать в классах "детях" приватное свойство $accessibleToParent и расширить эти классы абстрактным классом с одним методом:

<?php
declare(strict_types=1);
namespace App\Model\;

abstract class AccessibleToParent
{
    abstract public function isAccessibleToParent(): bool;
}

Теперь, перед тем как вызвать родительские методы в них передаётся класс AccessibleToParent и внутри метода проверяется, выставлен ли флаг на переменную $accessibleToParent:

public function edit(AccessibleToParent $accessibleToParent, string $name, string $notes, int $blabla): void
    {
        if(!$accessibleToParent->isAccessibleToParent ()){
            throw new \DomainException('You can not do that.');
        }
        $this->name = $name;
        $this->notes = $notes;
        //code
    }

Выглядит это прямо так дурно пахнущем :).
Вообще какие есть в данной ситуации решения? Я с радостью вернулся бы к наследованиям, но доктрина в своём нынешнем виде не даёт такой возможности, либо я не нашёл решения, которые удовлетворило бы меня.

Answer 1

[Антон Шаманов]

Вариантов масса, самые простые:
1. детали реализации не ясны, но возможен класс реализующий шаблон Factory для создания моделей, в фабрику передается объект AccessibleToParent, который в свою очередь передается в модель при её создании
по поводу проблем с наследованием:
2. шаблон Decorator: под капотом объект Parent
3. шаблон Composite: под капотом несколько объектов, пользователь работает с ними как с едиными Composite объектом

Comments

[tukreb]

Детали, ну довольно просто, есть так сказать "абстрактный" супер класс Parent, который имеет общие свойства для всех "детей". Некоторые из этих свойств или методы, не должны быть доступны напрямую из этого класса.
То есть в идеале, это должны быть protected методы у Parent, а дети иметь доступ к ним, но из-за того, что наследование использовать не получится, нужно придумать альтернативу.

Я рассмотрю ваши предложенные варианты чуть позже, и отмечу ответ как решением, ну или может вы ещё предложите что-то интересное.
Спасибо.

[Антон Шаманов]

tukreb, тогда не Composite, а Facade

[tukreb]

Антон Шаманов, спасибо. Вы дали много полезной информации :) Попробую реализовать в правильном ключе