
Sergey
29.07.2018
09:41:59
не то что у тебя нет dto-шек а то что ты делаешь $event->getEntity()
повторюсь - оно не dto ради dto - а что бы связанность уменьшить. есть масса ситуаций когда dto никак на связанность не повлияет и потому нет смысла. Важно понимать что тебе дает подход и какой ценой
в случае ивентов ты как бы закрыл себе дорогу прокидывать этот ивент куда-нибудь в какую-нибудь шину сообщений
ну и еще нюанс - все бы ничего если бы не глобальный UoW доктрины

Google

Roman
29.07.2018
09:44:02
а отсутствия DTO плохо потому что я имею доступ с одного слоя апликейшена в третий минуя второй?

Sergey
29.07.2018
09:44:10
и кто бы ее не менял все будет сохраняться, только теперь отследить это будет сложнее

Roman
29.07.2018
09:45:04
ясно

Sergey
29.07.2018
09:45:29
ну и еще - шутки ради можешь попробовать сериализовать сущность) из-за прокси классов и ленивой загрузки доктрины это не так просто как может показаться
и вот тебе уже надо обрабатывать эту ситуацию
или подключать какой symfony/jms serializer

Roman
29.07.2018
09:48:36
я както только пришол на раьботу и нашол публичний метод в интерфейсе setId() - на что мне тимлид сказал что все люди адекватные тут и нихто его вызивать не будет гне не нужно. и мне стало грусно. А изался он в одном месте которое можна переписать было.

Sergey
29.07.2018
09:49:01
"все люди адекватные тут" пока это не так
не надо оставлять методы которые никто не будет юзать и не надо "на будущее" делать методы
что до setId - если хочешь айдишку извне передавать (это нормальная практика) - пределавай в конструктор

Google

Sergey
29.07.2018
09:50:01
хотя зачем я это говорю, всеравно всем похеру
есть стэйт есть процедуры - живем

Roman
29.07.2018
09:51:12
нет это понятно. я ето к тому что сложно будет навязать правильною работу с даными в компании где все привикли сетери и гетери фигачить и прокидывать ентити потомучто так проще и быстрее

Sergey
29.07.2018
09:52:53
что бы внедрять какие-то решения надо проблему сначала найти)

Roman
29.07.2018
09:53:30
а плохой код это не проблема?)

Sergey
29.07.2018
09:53:46
дай определение плохому коду

Roman
29.07.2018
09:54:41
в моем понимании не соотвествия SOLID и Grasp и PSR

Sergey
29.07.2018
09:55:09
ну то есть солид ради солид, грасп ради грасп, psr ради psr
это опять же решения или рекомендации как избежать проблем. а проблемы то в чем?)
Короч. Помимо скорости написания кода существует масса других факторов которые в общей сложности влияют на скорость разработки в целом (то что для бизнеса важно).
Как часто возникают баги, как быстро баги находятся, какой процент времени разработчики тратят на отладку и фикс багов, регрессии - как часто возникают, причины возникновения и т.д. Бывает ли что приходят изменения в требованиях которые "плохо ложатся на код", было ли это известно заранее (что мы сча тут гвоздями прибьем потому что к дэмо надо быстро а потом хуй с ним) ну и куча других вещей.

Roman
29.07.2018
09:56:29
верно. соблюдение каждого из принцыпов принесут тебе менше боли в будущем при потдержке этого кода

Sergey
29.07.2018
09:56:44

Roman
29.07.2018
09:57:12
напоминает интервью)

Sergey
29.07.2018
09:57:48
просто хочу объяснить что SOLID это не решение проблемы, это способ проблему найти.

Roman
29.07.2018
09:57:51
знаю конешно

Sergey
29.07.2018
09:58:16
соблюсти SOLID невозможно, разве что ты знаешь будущее и можешь учесть все изменения требований, но тогда тебе солид не нужны
попробуй соблюсти open/close)) если что - 100% соблюдение этого принципа будет выражаться в том, что любая новая фича будет добавляться за счет добавления нового кода, никакой код не меняется (могут меняться только конфиги)
сможешь так?)

Artem
29.07.2018
09:59:40
всегда в вакансиях смущают фразы в требованиях типа "Знание ООП", сразу столько вопросов возникает =\

Google

Sergey
29.07.2018
09:59:44
имеет ли это практический смысл?) всего всеравно не учтешь

Roman
29.07.2018
09:59:51
да

Sergey
29.07.2018
09:59:59
как и в резюме стандартные "ответственный, целеустремленный" и прочий булшит. Если тебя смешат тексты вакансий - почитал бы резюмехи. вот там тотальный трэш и пиздец.
В одной из последних которые я читал у человека который говорит что у него 10+ лет опыта на последнем месте работы 2 из трех пунктов обязанностей (помимо код писать и фиксить баги) было 2 пункта - участие в дэйли митингах (скрам же), и логать время в джиру"

Roman
29.07.2018
10:02:20

Sergey
29.07.2018
10:02:40
вообще что бы код никогда не менялся

Roman
29.07.2018
10:02:50
)))

Sergey
29.07.2018
10:02:54
(багфикс и оптимизации не считаются)

Roman
29.07.2018
10:03:18
я потяол о чем ты. но это разве не крайности.

Sergey
29.07.2018
10:03:53
просто я подчеркиваю - целесообразность всего этого произрастает из понимания целей применения этих принципов. А цель у них простая - сделать разработку дешевле и предсказуемее

Artem
29.07.2018
10:04:17
это стандартные писульки на отъебись.
и вообще слово "знание" слишком нагруженная абстракция чтобы понять, чё там хотели-то =\
Ещё в каждой второй "знание PHP не ниже middle". И вот хз чё тут думать я ниже middle или где я? =\

Sergey
29.07.2018
10:04:21
и если соблюдая эти принципы ты делаешь разработку дороже и менее предсказуемой (за счет увеличения количества абстракций, зависимостей, связанности) - то может быть ты что-то делаешь не так?

Artem
29.07.2018
10:04:49

Sergey
29.07.2018
10:05:01
плохой пример - это уже на синьеров тянет)))))
(что очень и очень грустно мягко говоря)
я потяол о чем ты. но это разве не крайности.
без этой крайности сложно объяснить что соблюдать solid на 100% невозможно. Это вообще те принципы которые хорошо работают только в ретроспективе. когда ты написал чуть-чуть кода и посмотрел соблюдаются ли все еще эти пригнципы или нет

Roman
29.07.2018
10:06:47

Sergey
29.07.2018
10:06:52
не спроста solid в книге дяди Боба идет после рефакторинга а не до

Google

Denis
29.07.2018
10:07:01

Sergey
29.07.2018
10:07:12

Artem
29.07.2018
10:07:20

Roman
29.07.2018
10:07:30

Denis
29.07.2018
10:07:40

Sergey
29.07.2018
10:07:41
они не интересные. меня в такого рода резюме где все шаблонно интересуют ключевые слова и сколько где работал. Типа если ты раз в пол года меняешь рабочее место - у меня будут вопросы

Bohdan
29.07.2018
10:08:55

Sergey
29.07.2018
10:09:16
грустно - потому что yield не очень сложная штука, но среднестатистический php-ник не знает как это дело юзать
хорошо если для удобной работы с итераторами будет применять...

Denis
29.07.2018
10:09:36

Sergey
29.07.2018
10:09:48

Artem
29.07.2018
10:11:00

Sergey
29.07.2018
10:11:40

Artem
29.07.2018
10:11:47

Sergey
29.07.2018
10:11:52
а на твое скромное "ответственный, быстро учусь" им похеру вообще
проще просто это не писать а больше разукрасить именно свой опыт работы

Bohdan
29.07.2018
10:12:45

Sergey
29.07.2018
10:13:11

Google

Sergey
29.07.2018
10:13:21
мне на C++ пару раз присылали)

Bohdan
29.07.2018
10:13:29
ангулар и вью
надо пойти выбросить html/css из скиллов

Artem
29.07.2018
10:13:47

Denis
29.07.2018
10:14:01

Sergey
29.07.2018
10:14:15
ангулар и вью
в целом тебе до синьер фронтендера это только сделать хипстерскую прическу и разобраться с webpack)
(я шучу если что, фронтэнд нынче это так же сложно как и бэк)

Bohdan
29.07.2018
10:14:51

Sergey
29.07.2018
10:14:51
так же в том смысле что 2/3 пересекается ;)

Roman
29.07.2018
10:15:03

Bohdan
29.07.2018
10:15:39
аж жалею, что в компании я один на вью пишу - реактивщик, сосед по кабинету, еще недостаточно опытен, чтобы разбираться на ходу, хотя сам по себе парень неглупый

F01134H
29.07.2018
10:15:52

Bohdan
29.07.2018
10:16:04

Roman
29.07.2018
10:16:24
все завичет что тебе нравится

Denis
29.07.2018
10:16:29

F01134H
29.07.2018
10:16:45
Ну так программист тоже менеджер
занимается менеджментом кода

Roman
29.07.2018
10:16:53
руководящяя должность ето 20% кода и 80% менеджерских обязаностей

F01134H
29.07.2018
10:17:17
не всегда

Roman
29.07.2018
10:17:38
я лично знаю людей которые с тимлидов просилися назад в синеры

Artem
29.07.2018
10:17:51

F01134H
29.07.2018
10:18:01
да заебет после 10 лет то код писать