Как оформить вызов методов один за другим?

Question

[IgorNoskov]

Здравствуйте! Реализую класс, у которого один публичный метод и остальные скрытые. Снизу приведёна примерная структура класса. То есть в нём публичный метод вызывает по цепочке один за другим приватные методы, но может также прекратиться на любом из методов. Такой подход нормальный или для такой логики есть более изящное решение? Подскажите пожалуйста.

class Test
{
    public function start()
    {
        try {
            // do something...
            return $this->one();
        } catch (Throwable $exception) {
            // do something...
        }

        return false;
    }

    private function one()
    {
        try {
            // do something...
            return $this->two();
        } catch (Throwable $exception) {
            // do something...
        }

        return false;
    }

    private function two()
    {
        try {
            // do something...
            return $this->three();
        } catch (Throwable $exception) {
            // do something...
        }

        return false;
    }

    private function three()
    {
        try {
            // do something...
            return true;
        } catch (Throwable $exception) {
            // do something...
        }

        return false;
    }
}

$test   = new Test();
$result = $test->start();

Answer 1

[Lander]

Не знаю что конкретно вы делаете, но, возможно, вам нужна цепочка обязанностей.

Comments

[IgorNoskov]

Спасибо, похоже то, что нужно!

[IgorNoskov]

Подскажите, пожалуйста. Если бы вы увидели в коде подход, который был описан изначально, стали бы вы бить разработчика палкой в таком случае?)

[Lander]

IgorNoskov, Тааааак... Вы решили заняться рефакторингом и сказали кому то "Твой код херня, я сейчас всё перепишу". Угадал? :)

[IgorNoskov]

Почти, только я свой код переписываю и через пол года он мне не нравится)

[Lander]

IgorNoskov, Тут два варианта:
1. У вас есть дедлайн, а вы занимаетесь хернёй - оставьте в покое код.
2. Вы работаете над своим проектом без всяких сроков - переписывайте, опыт лишним не будет.

[IgorNoskov]

Спасибо, хороший совет!

Answer 2

[FanatPHP]

Суть вопроса абсолютно непонятна.
Термин "вызов "по цепочке" имеет совершенно определенное значение, которое не имеет ничего общего с вызовом нескольких методов подряд. Это сильно сбивает с толку.

плюс абсолютно бессмысленный код, который вообще ничего не показывает, даже пресловутый "вызов по цепочке"

плюс непонятно, в чем конкретно вопрос.
можно ли в start() написать
$this->one()
$this->two()
$this->three()
?

Можно

Comments

[Lander]

Подобные вещи лучше в комментариях к вопросу писать.

[IgorNoskov]

Спасибо за ответ! Суть в том, что если метод two вернёт false, то метод three не нужно вызывать. Код можно было написать так:

class Test
{
    public function start()
    {
        try {
            // do something...
            $result_one = $this->one();

            if ($result_one === true) {
                $result_two = $this->two();

                if ($result_two === true) {
                    $result_three = $this->three();

                    return $result_three;
                }
            }
        } catch (Throwable $exception) {
            // do something...
        }

        return false;
    }

    private function one()
    {
        try {
            // do something...
            return true;
        } catch (Throwable $exception) {
            // do something...
        }

        return false;
    }

    private function two()
    {
        try {
            // do something...
            return true;
        } catch (Throwable $exception) {
            // do something...
        }

        return false;
    }

    private function three()
    {
        try {
            // do something...
            return true;
        } catch (Throwable $exception) {
            // do something...
        }

        return false;
    }
}

$test   = new Test();
$result = $test->start();

Но в этом случае мне не нравится порождение вложенностей.

[FanatPHP]

Какая-то попытка приладить костыль к изначально кривой кривая архитектуре.
Какие-то исключения, условия, все туманно и таинственно

В самом простом варианте не ловить исключение внутри метода, а завернуть все три в один трай

[IgorNoskov]

FanatPHP, здесь в каждом функции есть свой блок try catch, потому что в моём случае используется GuzzleHttp\Exception\RequestException, а не Throwable и нельзя, чтобы эти исключения вышли в другой слой. Потому что, например, в одном методе GuzzleHttp\Exception\RequestException с кодом 404 должна обработатьс одним образом, а в другом в таком случам абосолютно по другому.

Answer 3

[Антон Р.]

Только недавно осваивал подобное. Так как такие "каскады" функций часто встречаются во фреймворках.
Суть в том что каждый предыдущий метод должен возвращать объект $this (либо другой объект), общая суть - метод может вызывать только объект поэтому конструкция $x->metod()->metod2(); означает что первая функция $x->metod() возвращает некий объект и уже у него мы вызываем второй метод. Вот мой пример:

class Mailer {
	
	public $from;
	public $to;
	public $subject;
	public $message;

	public function setFrom(){
		$this->from = '[e-mail]';
		return $this;
	}
	
	public function setTo(){
		$this->to = '[e-mail]';
		return $this;
	}
	
	public function setSub(){
		$this->subject = 'Hello!';
		return $this;
	}
	
	public function setMess(){
		$this->message = 'Good morning friend.';
		return;
	}
	
	
}

$x = new Mailer();
$x->setFrom()->setTo()->setSub()->setMess();

// Смотрим что получилось:
echo $x->from . "<br>";
echo $x->to . "<br>";
echo $x->subject . "<br>";
echo $x->message . "<br>";

Comments

[Lander]

Вообще не то же самое!

[IgorNoskov]

Спасибо! Здесь я виноват, главный вопрос неправильно сформулировал, я бы если бы посмотрел на главный вопрос, про ту же самую цепочку подумал.

[Антон Р.]

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

[FanatPHP]

Как всегда, читаем только заголовок и тут же бежим отвечать, не удосужившись прочесть сам вопрос

[Lander]

Антон Р., Всё равно не то. :) У ТС методы приватные. Ваша реализация позволяет делать:
$object->one()->two()->three();
а ТСу нужно вызвать один публичный метод, а далее чтобы цепочка сама выполнилась. У него какое то подобие middleware.

[Антон Р.]

Lander,

а далее чтобы цепочка сама выполнилась

- в чем тогда проблема ВНУТРИ публичного метода вызвать приватный?

class Test {
	
	public $c;
	
	public function x(){
		$this->c = $this->z() + 5;
		return $this->c;
	}
	
	private function z(){
		return 5;
	}
}
$ccc = new Test();
echo $ccc->x();
// Выводит: 10.

[kafkiansky]

Антон Р., у вас просто по цепочке вызов. и, кстати, неясно, что делать, если вызвать setMess, т.к. он не возвращает $this. Надо ли говорить, что у вас еще и закон Деметры нарушается. В общем, как мне кажется, это совсем не то, о чем спрашивал ТС.

Answer 4

[Александр Скуснов]

Есть такой continuation-passing стиль

Comments

[IgorNoskov]

Спасибо!