Как правильно инициализировать рекурсивные зависимости?

Question

[runprogr]

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

Методом дедукции выяснил что проблема в следующем.

Class1
{
  private Class2 $class2;

    public function __construct()
    {
        $this->class2 = app(Class2::class);
    }
}
Class2
{
  private Class1 $class1;

    public function __construct()
    {
        $this->class1 = app(Class1::class);
    }
}

Класс1 в зависимости имеет Класс2
Класс2 в зависимости имеет Класс1
Что вызывает, видимо, бесконечную рекурсию и падение скрипта.

Вопрос - как правильно инициилизровать зависимости в таком случае ? (средствами laravel, или же в целом php)
На поверхности только вариант убирать зависимость из конструктора и создавать ее уже в конкретном методе. Таким образом пока метод запрошен не будет, зависимость создаваться не будет, рекурсия разорвется. Но есть ли вариант как то это сделать, не убирая из конструкторов классов ?

Comments

[tukreb]

А какие подробности надо, если вы пустили ваш код в бесконечную рекурсию? Конечно он будет падать :)
Любая рекурсия должна быть конечная. Вот и думайте как это сделать.

[runprogr]

Я предполагал что для этого сценария есть какое то готовое решение во фреймворках. Которое само следит чтобы не создавалась рекурсия зависимостей и разруливает этот момент) Или на крайняк кидает в этот момент понятный эксепшен. Если нет, жаль, придется самому следить за таким

[nokimaro]

runprogr, решение называется "статический анализатор"
https://habr.com/ru/company/badoo/blog/426605/

[nokimaro]

Просто что-может меня застраховать от написания

while(true) {
   //do nothing
}

или

function a() { b(); }
function b() { a(); }

b();

что накодили то и запустится

Answer 1

[Smoren Freelight]

Если получилась такая циклическая связь, скорее всего выбрана неудачная архитектура. Попробуйте решить задачу через третий класс как фасад над этими двумя.

class Class1
{
    public function doSomethingWithClass2Object(Class2 $obj2)
    {
    }
}
class Class2
{
    public function doSomethingWithClass1Object(Class1 $obj1)
    {
    }
}
class ClassFacade
{
    private Class1 $class1;
    private Class2 $class2;

    public function __construct()
    {
        $this->class1 = app(Class1::class);
        $this->class2 = app(Class2::class);
    }

    public function doSomething()
    {
    	$this->class1->doSomethingWithClass2Object($this->class2);
    	$this->class2->doSomethingWithClass1Object($this->class1);
    }
}