вполне в тему

да не особо) но, тем не менее, почему он там в тему?)

У меня IDE выкидывать ошибку будет в методе конструктора

если там более 3-х методов

вернее варнинг

это единственная причина по котором там такой объект?)

угу

а если этот объект можно разбить на несколько логических?)

а зачем пользователю сразу передавать activationToken?

он нужен в сущности?

в общем, хер меня Фессор убедит, что он там в тему) просто запихнули, что бы было одним параметром и всё)

в общем, хер меня Фессор убедит, что он там в тему) просто запихнули, что бы было одним параметром и всё)

+

в общем, хер меня Фессор убедит, что он там в тему) просто запихнули, что бы было одним параметром и всё)

нет, я про "в тему" про твое замечание. Это DTO там не нужно

нет, я про "в тему" про твое замечание. Это DTO там не нужно

а, прости, я дура

и activationToken должен в другом месте создаваться и отправлять пользователю, класс User с этим не должен быть связан почти никак (от User там только идентификатор и email нужен)

да и при регистрации changePassword чёт как-то не звучит

он инициализируется по дефолту как Null

поетому мы его change

у тебя при создании нет пароля) ты ничто меняешь на что-то)

а зачем ты его передаешь тогда в дто?

а зачем ты его передаешь тогда в дто?

даже так: отдельным VO с паролем, который не включает соль и метод криптования)

нет, при создании любой сущности надо инициализировать все параметры

ибо тогда будет невалидное состояние

Поле не должно быть null

а VO

и там уже null

ну так передавай сразу не нулл, а то, что передаешь в change

Это гениально)

а то пока херь какая-то получается: ты создаешь объект с невалидным состоянием и потом его меняешь)

ну так передавай сразу не нулл, а то, что передаешь в change

ему энкодер нужен) поэтому наоборот сделал)

угу

енкодер нужен

делал по гайдам симфони

и примерам с гитхаба

ну тут register реально на себя дохера берёт)

так для етого ето ж и Factory

public function register(array $data) : User { $salt = base64_encode(random_bytes(55)); $activationToken = preg_replace('/\//', '', base64_encode(random_bytes(55))); $slug = $this->stringHelper->transliterate($this->stringHelper->slug($data['nickname'])); $encodedPass = $this->encoderFactory->getEncoder($newUser)->encodePassword( $newUser->plainPassword()->plainPassword(), $newUser->salt()->salt() ); $newUser = new User( $data['nickname'], $slug, $data['email'], new UserPassword($encodedPass), $salt, $activationToken // TODO - add here default user UserRole ); //dump($newUser); //exit; $newUser->eraseCredentials(); // я не знаю че делает этот метод, потому оставлю его здесь return $newUser; }

?

ну и вот уже не нужно вызывать change

$newUser->eraseCredentials();

> // я не знаю че делает этот метод, потому оставлю его здесь удаляет plain данные

чистит пароль

доки симфы

да

причем он идет в интерфейсе из пакета Сукурити

я на симфони не пишу

я бы использоватл changePassword метод

и доки симфони не видел не разу

только компоненты отдельные

и доки симфони не видел не разу

футакимбыть)

ну, там много прилетаел лишнего с этим пакетом

футакимбыть)

да не, мне норм: в галере где работаю еще, симфонией не принято пользоваться

да не, мне норм: в галере где работаю еще, симфонией не принято пользоваться

такое себе оправдание) а как же спортивный интерес)

но возвращаясь к коду: вызывать changePassword при создании юзверя — изврат еще тот

такое себе оправдание) а как же спортивный интерес)

из спортивного интереса только DomCrawler юзаю

остально хз зачем мне нужно

public function register(array $data) : User { $salt = base64_encode(random_bytes(55)); $slug = $this->stringHelper->transliterate($this->stringHelper->slug($data['nickname'])); $encodedPass = $this->encoderFactory->getEncoder($newUser)->encodePassword( $data['password'], $salt ); $newUser = new User( $data['nickname'], $slug, $data['email'], new UserPassword($encodedPass), // TODO - add here default user UserRole ); //dump($newUser); //exit; $newUser->eraseCredentials(); // я не знаю и знать не хочу че делает этот метод, потому оставлю его здесь return $newUser; } скорее даже вот так: активацией должен другой сервис заниматься, который принимает нужные ему данные о пользователе, а не класс пользователя

а то тут какая-то мощная фабрика получается: она и экземпляр пользователя создает, и знает как токен активации придумать, жаль не отсасывает только

я бы использоватл changePassword метод

для того что бы его впервый раз проинициализировать? не, не надо быть таким ленивым

public function register(array $data) : User { $salt = base64_encode(random_bytes(55)); $slug = $this->stringHelper->transliterate($this->stringHelper->slug($data['nickname'])); $encodedPass = $this->encoderFactory->getEncoder($newUser)->encodePassword( $data['password'], $salt ); $newUser = new User( $data['nickname'], $slug, $data['email'], new UserPassword($encodedPass), // TODO - add here default user UserRole ); //dump($newUser); //exit; $newUser->eraseCredentials(); // я не знаю и знать не хочу че делает этот метод, потому оставлю его здесь return $newUser; } скорее даже вот так: активацией должен другой сервис заниматься, который принимает нужные ему данные о пользователе, а не класс пользователя

Мне кажется генерацию слуга надо в конструктор Юзера вынести, это как уникальный идентификатор, за него должна отвечать сама сущность (так же как и uuid например генерируют в конструкторах)

Мне кажется генерацию слуга надо в конструктор Юзера вынести, это как уникальный идентификатор, за него должна отвечать сама сущность (так же как и uuid например генерируют в конструкторах)

спорно: Uuid лучше тоже предоставлять уже готовый откуда-то извне, например, из сервиса. а уж откуда сервис его возьмет — другой вопрос

Мне кажется генерацию слуга надо в конструктор Юзера вынести, это как уникальный идентификатор, за него должна отвечать сама сущность (так же как и uuid например генерируют в конструкторах)

за "уникальный" должен отвечать репозиторий) тот кто может факт "уникальности" проверить

ну и слагу в целом нечего делать внутри пользователя

я бы это в отдельную вообще сущность выносил

хотя я это я, у меня вообще философия намного более радикальна в этом плане

суть остается той же: здесь много того, чем НЕ должна заниматься фабрика

суть остается той же: здесь много того, чем НЕ должна заниматься фабрика

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

фабрика инкапсулирует в себе все то что требуется для создания чего-то. Что бы предоставить максимально легкий фасад и спрятать все зависимости

ERROR: S client not available

тот вопрос что то что мы создаем это пачка данных связанных между собой только фактом того что "это про юзера" - это абсолютной другой вопрос

который может привести к тому что нам и фабрика отдельная не понадобится.

(хотя по хорошему все сущности должны пораждаться другими сущностями, то есть фабрика нужна всегда, просто это может быть не просто специализированный сервис-фабрика))

например "контроллеры" - так себе фабрика)

смотря насколько широко глянуть на понятие фабрики)

контроллер можно обозвать фабрикой для Response =D

ммм в целом можно, но смысла в этом мало

ну мой взгляд на фабрику такой: она должна взять набор готовых данных и вернуть объект, а заниматься созданием новых данных она не должна. всякие Uuid, Token, Slug и т.д. — это лишнее

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

Подскажите в PHPStorm горячие клавиши, для того чтобы, находясь в окне с содержимым файла, меня перекинуло в окно дерева файлов проекта, но уже в ту папку, в которой находится просматриваемый файл. Есть такое вообще? К примеру я смотрю файл /web/index.php. А в дереве файлов проекта у меня открыто /vendor/blabla/blablabla/. Я хочу чтобы я нажал кнопочку, и в окне с деревом файлов проекта, меня перекинуло на /web/

Спасибо добрый человек

или в настройках панели autoscroll from source

ну мой взгляд на фабрику такой: она должна взять набор готовых данных и вернуть объект, а заниматься созданием новых данных она не должна. всякие Uuid, Token, Slug и т.д. — это лишнее

мммм.... тогда это просто конструктор)

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

ты сейчас про abstract factory какой-то говоришь

короч ладно, делай что хочешь и как хочешь) не мне тебя учить)

ты сейчас про abstract factory какой-то говоришь

ну так-то да

Подскажите в PHPStorm горячие клавиши, для того чтобы, находясь в окне с содержимым файла, меня перекинуло в окно дерева файлов проекта, но уже в ту папку, в которой находится просматриваемый файл. Есть такое вообще? К примеру я смотрю файл /web/index.php. А в дереве файлов проекта у меня открыто /vendor/blabla/blablabla/. Я хочу чтобы я нажал кнопочку, и в окне с деревом файлов проекта, меня перекинуло на /web/

посмотри в KeyMap

у меня раскладка нетбинсовская, потому комбинации различаться будут

А как сделать ее нетбиновской?

спасибо

у меня просто синдром утенка, первой IDE был Netbeans, потому и клавиши оттуда использую, к клавишам от IDEA привыкнуть пока не могу

короч ладно, делай что хочешь и как хочешь) не мне тебя учить)

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

вроде бы и чатик как раз для подобных целей придуман)

посоветуйте где можно подробно почитать про принципы програмирования на пхп, ну и про самые распространненые практики

посоветуйте где можно подробно почитать про принципы програмирования на пхп, ну и про самые распространненые практики

https://github.com/getjump/ru-php-the-right-way

спасибо)

у меня просто синдром утенка, первой IDE был Netbeans, потому и клавиши оттуда использую, к клавишам от IDEA привыкнуть пока не могу

тож самое)

Парни, в php7 ввели default функционал callSOmeFunction(default) ? или мне приснилось

Парни, в php7 ввели default функционал callSOmeFunction(default) ? или мне приснилось

хотели, но нет