Или надо: $userService->createAdmin('name', 'email');

этот код в контроллере

с моделькой покрасивше

Да, но пугает немного создание в контроллере ентити (

и с порядком полей не ошибёшься

А что в этом такого? В базу из контроллера ты не лезешь и даже доктрину не дёргаешь

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

Хотелось как можно больше разгрузить контроллер

А почему вы вообще в контроллере не имеете объект? Я про форму, она отдаёт же объект. Т.е. в контоллере надо получить объект, тем или иным способом. Валидатор лучше вынести, чем тоньше класс , тем лучше, как по мне. Принцип единственной ответственности и все такое.

в контроллере должен быть объект иначе как записывать

а валидацию можно отдельно вынести

$form->isValid

А почему вы вообще в контроллере не имеете объект? Я про форму, она отдаёт же объект. Т.е. в контоллере надо получить объект, тем или иным способом. Валидатор лучше вынести, чем тоньше класс , тем лучше, как по мне. Принцип единственной ответственности и все такое.

Валидатор в смысле в контроллере $form->isValid сделать? У меня не получалось нормально уникальность валидировать модели. ValidatorInterface с этим справлялся

Нене, валидатор в общем смысле я имел ввиду

С телефона много не напишешь

Не совем понял про валидатор

Все что связано с валидатор интерфейс в отдельный класс, я про это говорил

Хм, ну у меня перед каждоый записью в базу вызывается: $errors = $this->validator->validate($entity);

это в сервисе

так норм? я к тому что это не в контроллере, а в сервисе

Я бы поднял вызов в контроллер

А в сервис пихнул бы валидную модель

Получается валидация формы и валидация энтити

Ну грубо говоря да. Вообще, я не делал сложные валидации с формами, но по хорошему бы через аннотация сделать валидацию

и форму исключить..

У меня так сейчас и работает

и форму исключить..

Эм. Не уверен, что это хорошая идея. Но тут вам виднее, я не знаю ваш кэйс

рест апи

т.е. по сути форма и не нужна (если валидация в модели)

рест апи

Аа, да, для апи я делал отдельные валидаторы

У меня там форм нет

Сейчас смотрю на graphql

Понятно ) Спасибо!

Есть еще одно но, что если сервис вызывается из другого места, не из контроллера

т.е. к примеру не передали пароль, а в сервисе это не проверяется

Есть еще одно но, что если сервис вызывается из другого места, не из контроллера

Эм, а это номальная практика? Вообще, какой-то уровень валидации из сервиса не уберешь, проверка критичных моментов должна быть. С другой стороны, можно говорить и о том, что если у вас не прошла аутентификация, то вы вообще не должны попадать в данный сервис. В общем, по разному можно делать и подходить.

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

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

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

Если использовать способ передачи в сервис готовой модели, получается что нет уверенности, например, что пароль не пустой.

Если использовать способ передачи в сервис готовой модели, получается что нет уверенности, например, что пароль не пустой.

вы наверно не так поняли, что я писал выше, или я написал сумбурно :) Чтобы быть уверенным, что модель валидна, у вас несколько путей решения этой задачи. Первый и достаточно очевидый - использовать валидатор внутри сервиса - инжектим/содаем инстанс и выполняем валидацию. Если не валидно, бросаем исключение/возвращаем ответ с состоянием. Другой вариант, мы подразумеваем, что наша модель идет в сервис уже валидной, что означает, что вызов валидатора должен быть на уровень (или несколько выше), в том же контроллере или команде, но он должен быть обязательно. Да, я согласен, что by design, он более подвержен коллизиям, когда, к примеру, кто-то сделал вызов и передал параметром не прошедший валидацию аргумент. Но, все это регулируется соглашениями дизайна системы. Почему я порекомендовал вынести валидатор выше на уровень, из сервиса - чтобы убрать выброс исключительной ситуации, к примеру, чтобы уменьшить количество затраченных шагов до проверки состояния объекта, чтобы упростить логику сервиса и т.п. Только ради этого.

Понял, да, именно для безопасности и делаю в сервисе валидацию. Возможно ещё зависит от ситуации. Спасибо)

Такой вопрос!

Допустим есть поля в бд firstname lastname, можно ли их в один input затолкнуть ? Может, кто сталкивался

по пробелу делил как-то

т.е. получается допустим делаю один input с Ф.И. , на выходе он мне пишет в два поля first и last

типо того, но это если не очень важны эти данные, потому что все равно буду писать и ФИ и ИФ и ИИФ

будут* писать

блин, если даже 2 инпута сделать, могут писать коряво

от этого не застрахован ни кто

стоит ли вообще это делать ?

объеденять в один? Большая форма?

да нет

сейчас вот как

Я бы объеденил, можно ещё валидацию сделать что ввели два слова

она у меня в два этапа

заполнив эти поля, добавляет в БД, и редирект на другую форму

может Учетку убрать, и сделать авторизацию по мылу только ?

Сделай необязательное поле просто

как сделать, проверку поля на тру или фолс в контроллере ?

взять из реквеста параметр и проверить?

не вариант

парни, а стоит ли разбивать телефон на префикс, код, номер ?

есть ли вообще смысл заморачиваться

?

да

код страны код региона номер

ну у меня так и есть

я думал, я мудак

там еще какой то код вроде есть или нет

не

все так делают

большинство

ок

смотри, тут вопрос в голове возник вынести мыла и телефоны в отдельные таблицы, во первых для подтверждения гуд, и для рассылок, правильно я думаю ?

чтобы их не хранить в кэше где нить, пока не авторизует

Норм

подскажите, каким образом мне передать данные определенному урлу из контроллера ? например, http://test.domain/file.php?user=test&pass=password

знаю, file_get_content("url"), а если мне тупо отправить, тогда как ?

@sheinan Подскажи плиз, можно ли использовать репозитории в контроллере, или только в сервисах?

можно

а как инжектить их? делать репозиторий сервисом?

указываешь репу эту и все

getRepository плохо же

правильнее же явно в параметрах инжектить

а зачем ты так делаешь ?

точнее верно

а ты что делаешь вообще ?

findById

в контроллере вызвать хочу

в репозитории написал функцию ?

да

а в контролере как делаешь?

пока никак. раньше было getRepository... Хочу от этого указаться

чтобы явно указывать зависимости

тогда сервисы твои друзья

ок, а как в сервис инжектить? делать репозиторий сервисом?

сервис как вызываешь ?