Правильно ли я делаю с точки зрения ООП?

Question

[Crash]

Разрабатываю API. В экшенах я принимаю параметры, валидирую их, произвожу какие-либо действия, одно или несколько. Упрощенный пример кода экшена:

$validation = new Validation();
$validation->validate($this->request->post());

if ($validation->errors) {
    return 'error';
}

$helper = new Helper($validation);

if ($helper->doSomeThing()) {
    $res = 'ok';
} else {
    $res = 'error';
}

return $res;

Т.е. у меня есть специальный класс для валидации данных, где сможет быть сравнительно сложная логика проверки входных параметров. Далее, при успешном прохождении проверки, объект класса валидации я прокидываю в специальный хелпер, где провожу какое-либо действие. Все необходимые данные для этого действия у меня уже есть в объекте валидации, что весьма удобненько. Но для обеспечения работоспособности одного экшена, мне приходится создавать 2 дополнительных класса - один для валидации, другой для выполнения конкретных действий. Я не стал их объединять, т.к. получился бы перегруженный функционалом класс (в некоторых экшенах на обоих этапах есть достаточно сложная логика проверки, выборки итд). Как можно догадаться, классы нужны для того, чтобы вынести логику из контроллера и не дать разрастись экшену больше чем на 1 экран.

Соответственно у меня 2 вопроса:
1. Правильно ли я делаю в принципе?
2. В какую группу сущностей (сервисы, компоненты итд.) мне отнести хелпер из примера выше, как его лучше назвать.

Пишу на Фальконе.

Comments

[Crash]

Pardon Me! Where Do I Find 4giveness?, нет, пока за валидацию отвечает отдельный класс. Я хочу выяснить, насколько корректен такой подход. К тому же далеко не всегда можно сопоставить 1 экшен с 1-ой моделью.

[Антон Р.]

Каким образом у тебя Валидатор "содержит все что нужно" если реквест ты передаешь в метод а не в поле Валидатора? Он у тебя проверил данные и благополучно про них забыл.

[Crash]

Антон Р., ты по ходу не знаешь, что такое сеттеры)

Answer 1

[FanatPHP]

Ну в целом этот код выглядит чисто процедурным
Замени вызовы методов на вызовы функций - ничего не изменится.

ООПэ - это немного про другое.

Чисто если придираться по реализации - совершенно непонятно что делает класс Хелпер и почему он получает на вход не данные а валидатор.
Ну это я не знаю, как если для варки супа ты используешь не объект Кастрюля, а объект Посуда, кладешь в нее ножик для чистки овощей, а не сами овощи.

Comments

[Crash]

В валидаторе, помимо выходных данных, могут быть также необходимые для дальнейших действий сущности. Например, объект текущего пользователя ($valdation->user). Т.е. он также делает необходимые выборки. Если он не находит нужные сущности - выбрасывает соответствующую ошибку, которая передается в ответе клиенту. Все клиенты представляют собой авторизованные сервисы, о которых мы заранее знаем. Т.е. с улицы никого нет.

[FanatPHP]

Ну тогда это не то, что под валидатором понимает весь остальной мир.
Вообще непонятен принцип, по которому происходит разделение полномочий между валидатором и "хелпером".

[Crash]

FanatPHP, вижу это так:
1. Валидатор - принимает и проверяет параметры, выполняет выборку сущностей из БД. Т.е. проверка + выборка.
2. Хелпер (по идее он должен называться по-другому) - выполняет конкретные действия, используя заботливо приготовленные валидатором данные. Он знает, что данные верные и никаких проверок выполнять не надо. Например, хелпер может выполнить транзакцию, в которую обернуты несколько коротких действий. Т.е. это может быть пополнение баланса реферала и выплата вознаграждения рефереру.

[Crash]

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

[FanatPHP]

Сложно сказать. Зависит от выборок
И проблема "мы пишем процедурный код, называя функции методами" все равно остаётся

Answer 2

[dmitriy]

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

$validation = new Validation($this->request->post());
if(!$validation->validate()){
   //var_dump($validation->getErrors());
    return 'error';
}

про Helper молчу, вообще непонятно что делает